mbox series

[GIT,PULL,FOR,v6.4] Various fixes and enhancements

Message ID a5d6312b-1129-2369-6acf-4d1cce4ee403@xs4all.nl (mailing list archive)
State New, archived
Headers show
Series [GIT,PULL,FOR,v6.4] Various fixes and enhancements | expand

Pull-request

git://linuxtv.org/hverkuil/media_tree.git tags/br-v6.4a

Message

Hans Verkuil March 13, 2023, 3:12 p.m. UTC 
The following changes since commit 3e62aba8284de0994a669d07983299242e68fe72:

  media: imx-mipi-csis: Check csis_fmt validity before use (2023-02-26 11:21:33 +0100)

are available in the Git repository at:

  git://linuxtv.org/hverkuil/media_tree.git tags/br-v6.4a

for you to fetch changes up to ef1a0b98517d9968f06280bafa12080315f76d96:

  media: docs: vidioc-g-ext-ctrls.rst: Document p_s32 and p_s64 fields (2023-03-13 16:10:20 +0100)

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

----------------------------------------------------------------
Bryan O'Donoghue (1):
      MAINTAINERS: Add myself as CAMSS maintainer

Colin Ian King (1):
      media: platform: exynos4-is: Fix spelling mistake "palne" -> "plane"

Dan Carpenter (1):
      media: av7110: prevent underflow in write_ts_to_decoder()

Daniel Lundberg Pedersen (1):
      media: docs: vidioc-g-ext-ctrls.rst: Document p_s32 and p_s64 fields

Jakub Kicinski (1):
      media: drop unnecessary networking includes

Jiasheng Jiang (2):
      media: bdisp: Add missing check for create_workqueue
      media: platform: mtk-mdp3: Add missing check and free for ida_alloc

Jonathan Neuschäfer (1):
      media: dvb-frontends: Fix a typo ("Unknow sleep mode")

Kees Cook (2):
      media: mxl5005s: Bounds check size used for max array index
      media: imx-jpeg: Bounds check sizeimage access

Kevin Chiu (1):
      media: platform: cros-ec: Add Gladios/Lisbon to the match table

Krzysztof Kozlowski (1):
      media: dt-bindings: silabs,si470x: Convert to DT schema

Marek Vasut (1):
      media: stm32-dcmi: Enable incoherent buffer allocation

Ming Qian (1):
      media: amphion: decoder implement display delay enable

Tom Rix (1):
      media: pvrusb2: VIDEO_PVRUSB2 depends on DVB_CORE to use dvb_* symbols

Yang Li (1):
      media: tc358746: Remove unneeded semicolon

Ye Xingchen (3):
      media: dw100: use devm_platform_get_and_ioremap_resource()
      media: platform: renesas: use devm_platform_get_and_ioremap_resource()
      media: platform: stm32: use devm_platform_get_and_ioremap_resource()

Zoey Wu (1):
      media: platform: cros-ec: Add aurash to the match table

harperchen (3):
      media: cx23885: Fix a null-ptr-deref bug in buffer_prepare() and buffer_finish()
      media: platform: via: Handle error for dma_set_mask
      media: pci: tw68: Fix null-ptr-deref bug in buf prepare and finish

oushixiong (1):
      mtk-jpegenc: Fix a compilation issue

 Documentation/devicetree/bindings/media/si470x.txt           | 26 --------------------------
 Documentation/devicetree/bindings/media/silabs,si470x.yaml   | 48 ++++++++++++++++++++++++++++++++++++++++++++++++
 Documentation/userspace-api/media/v4l/vidioc-g-ext-ctrls.rst | 10 ++++++++++
 MAINTAINERS                                                  |  2 ++
 drivers/media/cec/platform/cros-ec/cros-ec-cec.c             |  6 ++++++
 drivers/media/dvb-frontends/drx39xyj/drxj.c                  |  2 +-
 drivers/media/dvb-frontends/drxk_hard.c                      |  2 +-
 drivers/media/i2c/tc358746.c                                 |  4 ++--
 drivers/media/pci/cx23885/cx23885-core.c                     |  4 +++-
 drivers/media/pci/cx23885/cx23885-video.c                    | 13 +++++++------
 drivers/media/pci/ttpci/budget-av.c                          |  1 +
 drivers/media/pci/tw68/tw68-video.c                          | 16 +++++++++-------
 drivers/media/platform/amphion/vdec.c                        | 32 ++++++++++++++++++++++++++++++++
 drivers/media/platform/amphion/vpu_codec.h                   |  3 ++-
 drivers/media/platform/amphion/vpu_malone.c                  |  4 +++-
 drivers/media/platform/mediatek/jpeg/mtk_jpeg_core.c         |  2 +-
 drivers/media/platform/mediatek/mdp3/mtk-mdp3-m2m.c          |  8 +++++++-
 drivers/media/platform/nxp/dw100/dw100.c                     |  4 +---
 drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.c               |  5 +++++
 drivers/media/platform/renesas/rcar-isp.c                    |  5 +----
 drivers/media/platform/samsung/exynos4-is/fimc-is-errno.c    |  2 +-
 drivers/media/platform/samsung/exynos4-is/fimc-is-errno.h    |  2 +-
 drivers/media/platform/st/sti/bdisp/bdisp-v4l2.c             |  2 ++
 drivers/media/platform/st/stm32/dma2d/dma2d.c                |  5 +----
 drivers/media/platform/st/stm32/stm32-dcmi.c                 |  1 +
 drivers/media/platform/via/via-camera.c                      |  4 +++-
 drivers/media/tuners/mxl5005s.c                              | 12 ++++++++----
 drivers/media/usb/dvb-usb/pctv452e.c                         |  2 ++
 drivers/media/usb/pvrusb2/Kconfig                            |  2 +-
 drivers/staging/media/av7110/av7110_av.c                     |  4 ++--
 include/media/dvb_net.h                                      |  6 ++----
 include/media/tveeprom.h                                     |  2 +-
 32 files changed, 167 insertions(+), 74 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/media/si470x.txt
 create mode 100644 Documentation/devicetree/bindings/media/silabs,si470x.yaml

Comments

Jenkins March 19, 2023, 12:47 p.m. UTC | #1 
From: builder@linuxtv.org

Pull request: https://patchwork.linuxtv.org/project/linux-media/patch/a5d6312b-1129-2369-6acf-4d1cce4ee403@xs4all.nl/
Build log: https://builder.linuxtv.org/job/patchwork/290890/
Build time: 00:30:51
Link: https://lore.kernel.org/linux-media/a5d6312b-1129-2369-6acf-4d1cce4ee403@xs4all.nl

gpg: Signature made Mon 13 Mar 2023 03:11:17 PM UTC
gpg:                using EDDSA key 52ADCAAE8A4F70B99ACD8D726B425DF79B1C1E76
gpg: Good signature from "Hans Verkuil <hverkuil-cisco@xs4all.nl>" [unknown]
gpg:                 aka "Hans Verkuil <hverkuil@xs4all.nl>" [full]

Summary: got 9/24 patches with issues, being 3 at build time, plus one error when buinding PDF document

Error/warnings:

patches/0001-media-tc358746-Remove-unneeded-semicolon.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]
	../scripts/genksyms/parse.y: note: rerun with option '-Wcounterexamples' to generate conflict counterexamples
	../drivers/staging/media/atomisp/i2c/atomisp-ov2680.c:415 ov2680_s_stream() warn: missing error code 'ret'
	../drivers/staging/media/atomisp/pci/atomisp_cmd.c: ../drivers/staging/media/atomisp/pci/atomisp_cmd.c:3357 atomisp_cp_dvs_6axis_config() warn: missing unwind goto?
	../drivers/staging/media/atomisp/pci/atomisp_cmd.c: ../drivers/staging/media/atomisp/pci/atomisp_cmd.c:3456 atomisp_cp_morph_table() warn: missing unwind goto?

    allyesconfig: return code #0:
	../drivers/media/i2c/adp1653.c: ../drivers/media/i2c/adp1653.c:444 adp1653_of_init() warn: missing unwind goto?
	SMATCH:../drivers/media/usb/siano/smsusb.c ../drivers/media/usb/siano/smsusb.c:53:38: :warning: array of flexible structures
	../drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.c: ../drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.c:2485 mxc_jpeg_probe() warn: missing unwind goto?
	../drivers/media/i2c/ov5645.c: ../drivers/media/i2c/ov5645.c:687 ov5645_set_power_on() warn: 'ov5645->xclk' from clk_prepare_enable() not released on lines: 687.
	../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2570 dvb_register() parse error: OOM: 3000024Kb sm_state_count = 1939314
	../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2570 dvb_register() warn: Function too hairy.  No more merges.
	../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2570 dvb_register() parse error: __split_smt: function too hairy.  Giving up after 53 seconds
	../drivers/media/i2c/ov5670.c: ../drivers/media/i2c/ov5670.c:2670 ov5670_probe() warn: passing zero to 'PTR_ERR'
	../drivers/media/usb/em28xx/em28xx-video.c: ../drivers/media/usb/em28xx/em28xx-video.c:2878 em28xx_v4l2_init() parse error: turning off implications after 60 seconds
	../drivers/media/usb/pvrusb2/pvrusb2-hdw.c: ../drivers/media/usb/pvrusb2/pvrusb2-hdw.c:3293 pvr2_hdw_get_tuner_status() warn: inconsistent indenting

   checkpatch.pl:
	$ cat patches/0001-media-tc358746-Remove-unneeded-semicolon.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:10: WARNING: Reported-by: should be immediately followed by Link: with a URL to the report

patches/0003-media-dt-bindings-silabs-si470x-Convert-to-DT-schema.patch:

   checkpatch.pl:
	$ cat patches/0003-media-dt-bindings-silabs-si470x-Convert-to-DT-schema.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:23: WARNING: DT binding docs and includes should be a separate patch. See: Documentation/devicetree/bindings/submitting-patches.rst
	-:51: WARNING: DT binding docs and includes should be a separate patch. See: Documentation/devicetree/bindings/submitting-patches.rst

patches/0004-media-dvb-frontends-Fix-a-typo-Unknow-sleep-mode.patch:

   checkpatch.pl:
	$ cat patches/0004-media-dvb-frontends-Fix-a-typo-Unknow-sleep-mode.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:4: WARNING: 'Unknow' may be misspelled - perhaps 'Unknown'?

patches/0005-media-drop-unnecessary-networking-includes.patch:

    allyesconfig: return code #0:
	../drivers/media/usb/pvrusb2/pvrusb2-hdw.c: ../drivers/media/usb/pvrusb2/pvrusb2-hdw.c:3293 pvr2_hdw_get_tuner_status() warn: inconsistent indenting
	../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2570 dvb_register() parse error: OOM: 3000016Kb sm_state_count = 1973361
	../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2570 dvb_register() warn: Function too hairy.  No more merges.
	../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2570 dvb_register() parse error: __split_smt: function too hairy.  Giving up after 54 seconds

patches/0006-media-mxl5005s-Bounds-check-size-used-for-max-array-.patch:

   checkpatch.pl:
	$ cat patches/0006-media-mxl5005s-Bounds-check-size-used-for-max-array-.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:38: CHECK: Avoid CamelCase: <Init_Ctrl>
	-:52: CHECK: Avoid CamelCase: <CH_Ctrl>

patches/0007-media-imx-jpeg-Bounds-check-sizeimage-access.patch:

    allyesconfig: return code #0:
	../drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.c: ../drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.c:2490 mxc_jpeg_probe() warn: missing unwind goto?

   checkpatch.pl:
	$ cat patches/0007-media-imx-jpeg-Bounds-check-sizeimage-access.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:14: WARNING: Possible unwrapped commit description (prefer a maximum 75 chars per line)

patches/0015-media-cx23885-Fix-a-null-ptr-deref-bug-in-buffer_pre.patch:

   checkpatch.pl:
	$ cat patches/0015-media-cx23885-Fix-a-null-ptr-deref-bug-in-buffer_pre.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:12: WARNING: Possible unwrapped commit description (prefer a maximum 75 chars per line)
	-:58: CHECK: Alignment should match open parenthesis
	-:64: CHECK: Alignment should match open parenthesis
	-:73: CHECK: Alignment should match open parenthesis
	-:81: CHECK: Alignment should match open parenthesis
	-:89: CHECK: Alignment should match open parenthesis

patches/0018-media-pci-tw68-Fix-null-ptr-deref-bug-in-buf-prepare.patch:

   checkpatch.pl:
	$ cat patches/0018-media-pci-tw68-Fix-null-ptr-deref-bug-in-buf-prepare.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:12: WARNING: Possible unwrapped commit description (prefer a maximum 75 chars per line)
	-:42: CHECK: Alignment should match open parenthesis
	-:47: CHECK: Alignment should match open parenthesis
	-:52: CHECK: Alignment should match open parenthesis
	-:58: CHECK: Alignment should match open parenthesis
	-:65: CHECK: Alignment should match open parenthesis

patches/0020-mtk-jpegenc-Fix-a-compilation-issue.patch:

   checkpatch.pl:
	$ cat patches/0020-mtk-jpegenc-Fix-a-compilation-issue.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:9: WARNING: Possible unwrapped commit description (prefer a maximum 75 chars per line)


Error #512 when building PDF docs