mbox series

[GIT,PULL,FOR,v5.15] Various fixes

Message ID 4d9b08ca-ec5e-b75f-514c-950d2c2abd64@xs4all.nl (mailing list archive)
State New, archived
Headers show
Series [GIT,PULL,FOR,v5.15] Various fixes | expand

Pull-request

git://linuxtv.org/hverkuil/media_tree.git tags/br-v5.15a

Message

Hans Verkuil July 20, 2021, 1:34 p.m. UTC
The following changes since commit 379e205dab9d7f9761984728e7d6f5f8305cc424:

  media: usb: dvb-usb-v2: af9035: let subdrv autoselect enable si2168 and si2157 (2021-07-12 14:28:49 +0200)

are available in the Git repository at:

  git://linuxtv.org/hverkuil/media_tree.git tags/br-v5.15a

for you to fetch changes up to 8e8ee611c5aa4e1ddb6efd1395f1dd16e13eca98:

  media: coda: fix frame_mem_ctrl for YUV420 and YVU420 formats (2021-07-20 15:17:45 +0200)

----------------------------------------------------------------
Tag branch

----------------------------------------------------------------
Dafna Hirschfeld (3):
      media: mtk-jpeg: fix setting plane paylod
      media: rkisp1: remove field 'vaddr' from 'rkisp1_buffer'
      media: rkisp1: cap: initialize dma buf address in 'buf_init' cb

Dan Carpenter (2):
      media: v4l2-subdev: fix some NULL vs IS_ERR() checks
      media: rockchip/rga: fix error handling in probe

Evgeny Novikov (1):
      media: platform: stm32: unprepare clocks at handling errors in probe

Fabio Estevam (3):
      dt-bindings: adv7180: Introduce the 'reset-gpios' property
      media: i2c: adv7180: Print the chip ID on probe
      dt-bindings: adv7180: Introduce 'adv,force-bt656-4' property

Frieder Schrempf (1):
      media: adv7180: Add optional reset GPIO

Hans Verkuil (1):
      media/cec-core.rst: update adap_enable doc

Matthew Michilot (1):
      media: i2c: adv7180: fix adv7280 BT.656-4 compatibility

Pavel Skripkin (3):
      media: go7007: fix memory leak in go7007_usb_probe
      media: go7007: remove redundant initialization
      media: stkwebcam: fix memory leak in stk_camera_probe

Philipp Zabel (1):
      media: coda: fix frame_mem_ctrl for YUV420 and YVU420 formats

Tom Rix (1):
      media: ti-vpe: cal: fix indexing of cal->ctx[] in cal_probe()

 Documentation/devicetree/bindings/media/i2c/adv7180.yaml |  8 ++++++
 Documentation/driver-api/media/cec-core.rst              |  9 ++++---
 drivers/media/cec/platform/stm32/stm32-cec.c             | 26 +++++++++++++------
 drivers/media/i2c/adv7180.c                              | 66 ++++++++++++++++++++++++++++++++++++++++++-----
 drivers/media/platform/coda/coda-bit.c                   | 18 +++++++++----
 drivers/media/platform/mtk-jpeg/mtk_jpeg_core.c          | 20 +++++++-------
 drivers/media/platform/rcar-vin/rcar-v4l2.c              |  4 +--
 drivers/media/platform/rockchip/rga/rga.c                | 27 +++++++++++++++----
 drivers/media/platform/rockchip/rkisp1/rkisp1-capture.c  | 12 ++++++++-
 drivers/media/platform/rockchip/rkisp1/rkisp1-common.h   |  6 +----
 drivers/media/platform/rockchip/rkisp1/rkisp1-params.c   |  3 +--
 drivers/media/platform/rockchip/rkisp1/rkisp1-stats.c    |  6 ++---
 drivers/media/platform/ti-vpe/cal.c                      |  6 ++---
 drivers/media/platform/vsp1/vsp1_entity.c                |  4 +--
 drivers/media/usb/go7007/go7007-driver.c                 | 26 -------------------
 drivers/media/usb/go7007/go7007-usb.c                    |  2 +-
 drivers/media/usb/stkwebcam/stk-webcam.c                 |  6 +++--
 drivers/staging/media/tegra-video/vi.c                   |  4 +--
 18 files changed, 167 insertions(+), 86 deletions(-)

Comments

Jenkins July 20, 2021, 2:12 p.m. UTC | #1
From: builder@linuxtv.org

Pull request: https://patchwork.linuxtv.org/project/linux-media/patch/4d9b08ca-ec5e-b75f-514c-950d2c2abd64@xs4all.nl/
Build log: https://builder.linuxtv.org/job/patchwork/125580/
Build time: 00:21:58
Link: https://lore.kernel.org/linux-media/4d9b08ca-ec5e-b75f-514c-950d2c2abd64@xs4all.nl

gpg: Signature made Tue 20 Jul 2021 01:30:59 PM UTC
gpg:                using RSA key AAA7FFBA4D2D77EF4CAEA1421326E0CD23ABDCE5
gpg: Good signature from "Hans Verkuil <hverkuil-cisco@xs4all.nl>" [unknown]
gpg:                 aka "Hans Verkuil <hverkuil@xs4all.nl>" [full]
gpg: Note: This key has expired!
Primary key fingerprint: 052C DE7B C215 053B 689F  1BCA BD2D 6148 6614 3B4C
     Subkey fingerprint: AAA7 FFBA 4D2D 77EF 4CAE  A142 1326 E0CD 23AB DCE5

Summary: got 1/17 patches with issues, being 1 at build time

Error/warnings:

patches/0001-dt-bindings-adv7180-Introduce-the-reset-gpios-proper.patch:

    allyesconfig: return code #0:
	../scripts/genksyms/parse.y: warning: 9 shift/reduce conflicts [-Wconflicts-sr]
	../scripts/genksyms/parse.y: warning: 5 reduce/reduce conflicts [-Wconflicts-rr]

    allyesconfig: return code #0:
	SPARSE:../drivers/media/cec/core/cec-core.c ../include/asm-generic/bitops/find.h:90:32:  warning: shift count is negative (-192)
	SPARSE:../drivers/media/mc/mc-devnode.c ../include/asm-generic/bitops/find.h:90:32:  warning: shift count is negative (-192)
	SPARSE:../drivers/media/v4l2-core/v4l2-dev.c ../include/asm-generic/bitops/find.h:132:46:  warning: shift count is negative (-192)
	../drivers/media/v4l2-core/v4l2-ioctl.c: ../drivers/media/v4l2-core/v4l2-ioctl.c:268 v4l_print_fmtdesc() error: unrecognized %p extension '4', treated as normal %p
	../drivers/media/v4l2-core/v4l2-ioctl.c: ../drivers/media/v4l2-core/v4l2-ioctl.c:292 v4l_print_format() error: unrecognized %p extension '4', treated as normal %p
	../drivers/media/v4l2-core/v4l2-ioctl.c: ../drivers/media/v4l2-core/v4l2-ioctl.c:302 v4l_print_format() error: unrecognized %p extension '4', treated as normal %p
	../drivers/media/v4l2-core/v4l2-ioctl.c: ../drivers/media/v4l2-core/v4l2-ioctl.c:328 v4l_print_format() error: unrecognized %p extension '4', treated as normal %p
	../drivers/media/v4l2-core/v4l2-ioctl.c: ../drivers/media/v4l2-core/v4l2-ioctl.c:347 v4l_print_format() error: unrecognized %p extension '4', treated as normal %p
	../drivers/media/v4l2-core/v4l2-ioctl.c: ../drivers/media/v4l2-core/v4l2-ioctl.c:352 v4l_print_format() error: unrecognized %p extension '4', treated as normal %p
	../drivers/media/v4l2-core/v4l2-ioctl.c: ../drivers/media/v4l2-core/v4l2-ioctl.c:362 v4l_print_framebuffer() error: unrecognized %p extension '4', treated as normal %p
	../drivers/media/v4l2-core/v4l2-ioctl.c: ../drivers/media/v4l2-core/v4l2-ioctl.c:735 v4l_print_frmsizeenum() error: unrecognized %p extension '4', treated as normal %p
	../drivers/media/v4l2-core/v4l2-ioctl.c: ../drivers/media/v4l2-core/v4l2-ioctl.c:762 v4l_print_frmivalenum() error: unrecognized %p extension '4', treated as normal %p
	../drivers/media/v4l2-core/v4l2-ioctl.c: ../drivers/media/v4l2-core/v4l2-ioctl.c:1424 v4l_fill_fmtdesc() error: unrecognized %p extension '4', treated as normal %p
	../drivers/media/test-drivers/vivid/vivid-core.c: ../drivers/media/test-drivers/vivid/vivid-core.c:1969 vivid_create_instance() parse error: turning off implications after 60 seconds
	../drivers/media/usb/em28xx/em28xx-video.c: ../drivers/media/usb/em28xx/em28xx-video.c:2841 em28xx_v4l2_init() parse error: turning off implications after 60 seconds
Mauro Carvalho Chehab July 22, 2021, 6:34 a.m. UTC | #2
Hi Hans,

Em Tue, 20 Jul 2021 15:34:22 +0200
Hans Verkuil <hverkuil@xs4all.nl> escreveu:

> The following changes since commit 379e205dab9d7f9761984728e7d6f5f8305cc424:
> 
>   media: usb: dvb-usb-v2: af9035: let subdrv autoselect enable si2168 and si2157 (2021-07-12 14:28:49 +0200)
> 
> are available in the Git repository at:
> 
>   git://linuxtv.org/hverkuil/media_tree.git tags/br-v5.15a
> 
> for you to fetch changes up to 8e8ee611c5aa4e1ddb6efd1395f1dd16e13eca98:
> 
>   media: coda: fix frame_mem_ctrl for YUV420 and YVU420 formats (2021-07-20 15:17:45 +0200)

It seems you forgot to add your own SOB at the patches on this series!

> 
> ----------------------------------------------------------------
> Tag branch
> 
> ----------------------------------------------------------------
> Dafna Hirschfeld (3):
>       media: mtk-jpeg: fix setting plane paylod
>       media: rkisp1: remove field 'vaddr' from 'rkisp1_buffer'
>       media: rkisp1: cap: initialize dma buf address in 'buf_init' cb
> 
> Dan Carpenter (2):
>       media: v4l2-subdev: fix some NULL vs IS_ERR() checks
>       media: rockchip/rga: fix error handling in probe
> 
> Evgeny Novikov (1):
>       media: platform: stm32: unprepare clocks at handling errors in probe
> 
> Fabio Estevam (3):
>       dt-bindings: adv7180: Introduce the 'reset-gpios' property
>       media: i2c: adv7180: Print the chip ID on probe
>       dt-bindings: adv7180: Introduce 'adv,force-bt656-4' property
> 
> Frieder Schrempf (1):
>       media: adv7180: Add optional reset GPIO
> 
> Hans Verkuil (1):
>       media/cec-core.rst: update adap_enable doc
> 
> Matthew Michilot (1):
>       media: i2c: adv7180: fix adv7280 BT.656-4 compatibility
> 
> Pavel Skripkin (3):
>       media: go7007: fix memory leak in go7007_usb_probe
>       media: go7007: remove redundant initialization
>       media: stkwebcam: fix memory leak in stk_camera_probe
> 
> Philipp Zabel (1):
>       media: coda: fix frame_mem_ctrl for YUV420 and YVU420 formats
> 
> Tom Rix (1):
>       media: ti-vpe: cal: fix indexing of cal->ctx[] in cal_probe()
> 
>  Documentation/devicetree/bindings/media/i2c/adv7180.yaml |  8 ++++++
>  Documentation/driver-api/media/cec-core.rst              |  9 ++++---
>  drivers/media/cec/platform/stm32/stm32-cec.c             | 26 +++++++++++++------
>  drivers/media/i2c/adv7180.c                              | 66 ++++++++++++++++++++++++++++++++++++++++++-----
>  drivers/media/platform/coda/coda-bit.c                   | 18 +++++++++----
>  drivers/media/platform/mtk-jpeg/mtk_jpeg_core.c          | 20 +++++++-------
>  drivers/media/platform/rcar-vin/rcar-v4l2.c              |  4 +--
>  drivers/media/platform/rockchip/rga/rga.c                | 27 +++++++++++++++----
>  drivers/media/platform/rockchip/rkisp1/rkisp1-capture.c  | 12 ++++++++-
>  drivers/media/platform/rockchip/rkisp1/rkisp1-common.h   |  6 +----
>  drivers/media/platform/rockchip/rkisp1/rkisp1-params.c   |  3 +--
>  drivers/media/platform/rockchip/rkisp1/rkisp1-stats.c    |  6 ++---
>  drivers/media/platform/ti-vpe/cal.c                      |  6 ++---
>  drivers/media/platform/vsp1/vsp1_entity.c                |  4 +--
>  drivers/media/usb/go7007/go7007-driver.c                 | 26 -------------------
>  drivers/media/usb/go7007/go7007-usb.c                    |  2 +-
>  drivers/media/usb/stkwebcam/stk-webcam.c                 |  6 +++--
>  drivers/staging/media/tegra-video/vi.c                   |  4 +--
>  18 files changed, 167 insertions(+), 86 deletions(-)



Thanks,
Mauro
Dafna Hirschfeld July 22, 2021, 11:43 a.m. UTC | #3
On 22.07.21 08:34, Mauro Carvalho Chehab wrote:
> Hi Hans,
> 
> Em Tue, 20 Jul 2021 15:34:22 +0200
> Hans Verkuil <hverkuil@xs4all.nl> escreveu:
> 
>> The following changes since commit 379e205dab9d7f9761984728e7d6f5f8305cc424:
>>
>>    media: usb: dvb-usb-v2: af9035: let subdrv autoselect enable si2168 and si2157 (2021-07-12 14:28:49 +0200)
>>
>> are available in the Git repository at:
>>
>>    git://linuxtv.org/hverkuil/media_tree.git tags/br-v5.15a
>>
>> for you to fetch changes up to 8e8ee611c5aa4e1ddb6efd1395f1dd16e13eca98:
>>
>>    media: coda: fix frame_mem_ctrl for YUV420 and YVU420 formats (2021-07-20 15:17:45 +0200)
> 
> It seems you forgot to add your own SOB at the patches on this series!
> 
>>
>> ----------------------------------------------------------------
>> Tag branch
>>
>> ----------------------------------------------------------------
>> Dafna Hirschfeld (3):
>>        media: mtk-jpeg: fix setting plane paylod

I am actually not 100% sure about this patch. I think it is better if the driver maintainers review it

>>        media: rkisp1: remove field 'vaddr' from 'rkisp1_buffer'
>>        media: rkisp1: cap: initialize dma buf address in 'buf_init' cb
>>
>> Dan Carpenter (2):
>>        media: v4l2-subdev: fix some NULL vs IS_ERR() checks
>>        media: rockchip/rga: fix error handling in probe
>>
>> Evgeny Novikov (1):
>>        media: platform: stm32: unprepare clocks at handling errors in probe
>>
>> Fabio Estevam (3):
>>        dt-bindings: adv7180: Introduce the 'reset-gpios' property
>>        media: i2c: adv7180: Print the chip ID on probe
>>        dt-bindings: adv7180: Introduce 'adv,force-bt656-4' property
>>
>> Frieder Schrempf (1):
>>        media: adv7180: Add optional reset GPIO
>>
>> Hans Verkuil (1):
>>        media/cec-core.rst: update adap_enable doc
>>
>> Matthew Michilot (1):
>>        media: i2c: adv7180: fix adv7280 BT.656-4 compatibility
>>
>> Pavel Skripkin (3):
>>        media: go7007: fix memory leak in go7007_usb_probe
>>        media: go7007: remove redundant initialization
>>        media: stkwebcam: fix memory leak in stk_camera_probe
>>
>> Philipp Zabel (1):
>>        media: coda: fix frame_mem_ctrl for YUV420 and YVU420 formats
>>
>> Tom Rix (1):
>>        media: ti-vpe: cal: fix indexing of cal->ctx[] in cal_probe()
>>
>>   Documentation/devicetree/bindings/media/i2c/adv7180.yaml |  8 ++++++
>>   Documentation/driver-api/media/cec-core.rst              |  9 ++++---
>>   drivers/media/cec/platform/stm32/stm32-cec.c             | 26 +++++++++++++------
>>   drivers/media/i2c/adv7180.c                              | 66 ++++++++++++++++++++++++++++++++++++++++++-----
>>   drivers/media/platform/coda/coda-bit.c                   | 18 +++++++++----
>>   drivers/media/platform/mtk-jpeg/mtk_jpeg_core.c          | 20 +++++++-------
>>   drivers/media/platform/rcar-vin/rcar-v4l2.c              |  4 +--
>>   drivers/media/platform/rockchip/rga/rga.c                | 27 +++++++++++++++----
>>   drivers/media/platform/rockchip/rkisp1/rkisp1-capture.c  | 12 ++++++++-
>>   drivers/media/platform/rockchip/rkisp1/rkisp1-common.h   |  6 +----
>>   drivers/media/platform/rockchip/rkisp1/rkisp1-params.c   |  3 +--
>>   drivers/media/platform/rockchip/rkisp1/rkisp1-stats.c    |  6 ++---
>>   drivers/media/platform/ti-vpe/cal.c                      |  6 ++---
>>   drivers/media/platform/vsp1/vsp1_entity.c                |  4 +--
>>   drivers/media/usb/go7007/go7007-driver.c                 | 26 -------------------
>>   drivers/media/usb/go7007/go7007-usb.c                    |  2 +-
>>   drivers/media/usb/stkwebcam/stk-webcam.c                 |  6 +++--
>>   drivers/staging/media/tegra-video/vi.c                   |  4 +--
>>   18 files changed, 167 insertions(+), 86 deletions(-)
> 
> 
> 
> Thanks,
> Mauro
>
Hans Verkuil July 22, 2021, 12:14 p.m. UTC | #4
Hi Dafna,

On 22/07/2021 13:43, Dafna Hirschfeld wrote:
> 
> 
> On 22.07.21 08:34, Mauro Carvalho Chehab wrote:
>> Hi Hans,
>>
>> Em Tue, 20 Jul 2021 15:34:22 +0200
>> Hans Verkuil <hverkuil@xs4all.nl> escreveu:
>>
>>> The following changes since commit 379e205dab9d7f9761984728e7d6f5f8305cc424:
>>>
>>>    media: usb: dvb-usb-v2: af9035: let subdrv autoselect enable si2168 and si2157 (2021-07-12 14:28:49 +0200)
>>>
>>> are available in the Git repository at:
>>>
>>>    git://linuxtv.org/hverkuil/media_tree.git tags/br-v5.15a
>>>
>>> for you to fetch changes up to 8e8ee611c5aa4e1ddb6efd1395f1dd16e13eca98:
>>>
>>>    media: coda: fix frame_mem_ctrl for YUV420 and YVU420 formats (2021-07-20 15:17:45 +0200)
>>
>> It seems you forgot to add your own SOB at the patches on this series!
>>
>>>
>>> ----------------------------------------------------------------
>>> Tag branch
>>>
>>> ----------------------------------------------------------------
>>> Dafna Hirschfeld (3):
>>>        media: mtk-jpeg: fix setting plane paylod
> 
> I am actually not 100% sure about this patch. I think it is better if the driver maintainers review it

Thanks for the heads-up, it's been dropped and the state in patchwork is now again 'New'.

Regards,

	Hans

> 
>>>        media: rkisp1: remove field 'vaddr' from 'rkisp1_buffer'
>>>        media: rkisp1: cap: initialize dma buf address in 'buf_init' cb
>>>
>>> Dan Carpenter (2):
>>>        media: v4l2-subdev: fix some NULL vs IS_ERR() checks
>>>        media: rockchip/rga: fix error handling in probe
>>>
>>> Evgeny Novikov (1):
>>>        media: platform: stm32: unprepare clocks at handling errors in probe
>>>
>>> Fabio Estevam (3):
>>>        dt-bindings: adv7180: Introduce the 'reset-gpios' property
>>>        media: i2c: adv7180: Print the chip ID on probe
>>>        dt-bindings: adv7180: Introduce 'adv,force-bt656-4' property
>>>
>>> Frieder Schrempf (1):
>>>        media: adv7180: Add optional reset GPIO
>>>
>>> Hans Verkuil (1):
>>>        media/cec-core.rst: update adap_enable doc
>>>
>>> Matthew Michilot (1):
>>>        media: i2c: adv7180: fix adv7280 BT.656-4 compatibility
>>>
>>> Pavel Skripkin (3):
>>>        media: go7007: fix memory leak in go7007_usb_probe
>>>        media: go7007: remove redundant initialization
>>>        media: stkwebcam: fix memory leak in stk_camera_probe
>>>
>>> Philipp Zabel (1):
>>>        media: coda: fix frame_mem_ctrl for YUV420 and YVU420 formats
>>>
>>> Tom Rix (1):
>>>        media: ti-vpe: cal: fix indexing of cal->ctx[] in cal_probe()
>>>
>>>   Documentation/devicetree/bindings/media/i2c/adv7180.yaml |  8 ++++++
>>>   Documentation/driver-api/media/cec-core.rst              |  9 ++++---
>>>   drivers/media/cec/platform/stm32/stm32-cec.c             | 26 +++++++++++++------
>>>   drivers/media/i2c/adv7180.c                              | 66 ++++++++++++++++++++++++++++++++++++++++++-----
>>>   drivers/media/platform/coda/coda-bit.c                   | 18 +++++++++----
>>>   drivers/media/platform/mtk-jpeg/mtk_jpeg_core.c          | 20 +++++++-------
>>>   drivers/media/platform/rcar-vin/rcar-v4l2.c              |  4 +--
>>>   drivers/media/platform/rockchip/rga/rga.c                | 27 +++++++++++++++----
>>>   drivers/media/platform/rockchip/rkisp1/rkisp1-capture.c  | 12 ++++++++-
>>>   drivers/media/platform/rockchip/rkisp1/rkisp1-common.h   |  6 +----
>>>   drivers/media/platform/rockchip/rkisp1/rkisp1-params.c   |  3 +--
>>>   drivers/media/platform/rockchip/rkisp1/rkisp1-stats.c    |  6 ++---
>>>   drivers/media/platform/ti-vpe/cal.c                      |  6 ++---
>>>   drivers/media/platform/vsp1/vsp1_entity.c                |  4 +--
>>>   drivers/media/usb/go7007/go7007-driver.c                 | 26 -------------------
>>>   drivers/media/usb/go7007/go7007-usb.c                    |  2 +-
>>>   drivers/media/usb/stkwebcam/stk-webcam.c                 |  6 +++--
>>>   drivers/staging/media/tegra-video/vi.c                   |  4 +--
>>>   18 files changed, 167 insertions(+), 86 deletions(-)
>>
>>
>>
>> Thanks,
>> Mauro
>>