mbox series

[GIT,PULL,FOR,v6.6] Various fixes

Message ID 7e90c37e-f11d-90b6-e51d-daade4c15f4b@xs4all.nl (mailing list archive)
State New, archived
Headers show
Series [GIT,PULL,FOR,v6.6] Various fixes | expand

Pull-request

git://linuxtv.org/hverkuil/media_tree.git tags/br-v6.6j

Message

Hans Verkuil Aug. 7, 2023, 1:16 p.m. UTC
The following changes since commit a0e657a03ffbd26332f316f13c3e5dbc98cb1fca:

  media: mediatek: vcodec: Fix possible invalid memory access for encoder (2023-08-03 08:45:26 +0200)

are available in the Git repository at:

  git://linuxtv.org/hverkuil/media_tree.git tags/br-v6.6j

for you to fetch changes up to a6b6575d4e520bcfb3c7073c0da73a8a86d7f2ed:

  media: firewire: firedtv-avc.c: replace BUG with proper, error return (2023-08-07 15:07:04 +0200)

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

----------------------------------------------------------------
Alain Volmat (2):
      media: i2c: st_mipid02: cascade s_stream call to the source subdev
      media: stm32: dcmi: only call s_stream on the source subdev

Hans Verkuil (2):
      media: usb: pvrusb2: fix inconsistent indenting
      media: firewire: firedtv-avc.c: replace BUG with proper, error return

Krzysztof Kozlowski (1):
      media: dt-bindings: drop unneeded status from examples

Mariusz Bialonczyk (1):
      media: cx23885: debug cosmetics

Paul Kocialkowski (3):
      media: doc: dev-encoder: Fixup whitespace before bold asterisks
      media: doc: codec: Rename "Return fields" to "Returned fields" for consistency
      media: doc: dev-encoder: Fixup type for ENUM_FRAMEINTERVALS

Peng Fan (1):
      dt-bindings: media: amphion,vpu: correct node name

Ruan Jinjie (4):
      media: jpeg-core: Remove redundant dev_err()
      media: rcar_jpu: Remove redundant dev_err()
      media: nxp: Remove redundant dev_err()
      media: tegra_cec: fix the return value handle for platform_get_irq()

Yuanjun Gong (1):
      media: dvb: mb86a16: check the return value of mb86a16_read()

 Documentation/devicetree/bindings/media/amphion,vpu.yaml             |  8 ++++----
 Documentation/devicetree/bindings/media/cec/nvidia,tegra114-cec.yaml |  1 -
 Documentation/devicetree/bindings/media/i2c/st,st-mipid02.yaml       |  1 -
 Documentation/userspace-api/media/v4l/dev-decoder.rst                | 16 +++++++--------
 Documentation/userspace-api/media/v4l/dev-encoder.rst                | 24 +++++++++++-----------
 Documentation/userspace-api/media/v4l/dev-stateless-decoder.rst      |  4 ++--
 drivers/media/cec/platform/tegra/tegra_cec.c                         |  4 ++--
 drivers/media/dvb-frontends/mb86a16.c                                | 10 +++++----
 drivers/media/firewire/firedtv-avc.c                                 |  4 +++-
 drivers/media/i2c/st-mipid02.c                                       | 13 +++++++++++-
 drivers/media/pci/cx23885/cx23885-core.c                             |  6 +++---
 drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.c                       |  1 -
 drivers/media/platform/nxp/imx8-isi/imx8-isi-pipe.c                  |  1 -
 drivers/media/platform/renesas/rcar_jpu.c                            |  4 +---
 drivers/media/platform/samsung/s5p-jpeg/jpeg-core.c                  |  4 +---
 drivers/media/platform/st/stm32/stm32-dcmi.c                         | 63 +++++++++++++--------------------------------------------
 drivers/media/usb/pvrusb2/pvrusb2-hdw.c                              |  6 ++++--
 17 files changed, 72 insertions(+), 98 deletions(-)

Comments

Jenkins Aug. 7, 2023, 1:52 p.m. UTC | #1
From: builder@linuxtv.org

Pull request: https://patchwork.linuxtv.org/project/linux-media/patch/7e90c37e-f11d-90b6-e51d-daade4c15f4b@xs4all.nl/
Build log: https://builder.linuxtv.org/job/patchwork/328743/
Build time: 00:28:04
Link: https://lore.kernel.org/linux-media/7e90c37e-f11d-90b6-e51d-daade4c15f4b@xs4all.nl

gpg: Signature made Mon 07 Aug 2023 01:15:01 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 2/15 patches with issues, being 2 at build time

Error/warnings:

patches/0001-media-doc-dev-encoder-Fixup-whitespace-before-bold-a.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: ../drivers/staging/media/atomisp/i2c/atomisp-ov2680.c:476 ov2680_init_cfg() error: we previously assumed 'sd_state' could be null (see line 468)
	../drivers/staging/media/atomisp/i2c/atomisp-ov2680.c:524 ov2680_s_stream() warn: missing error code 'ret'
	../drivers/staging/media/atomisp/i2c/atomisp-gc0310.c:447 gc0310_s_stream() warn: missing error code 'ret'
	../drivers/staging/media/atomisp/pci/atomisp_fops.c: ../drivers/staging/media/atomisp/pci/atomisp_fops.c:517 atomisp_open() warn: missing unwind goto?
	../drivers/staging/media/atomisp/pci/atomisp_cmd.c: ../drivers/staging/media/atomisp/pci/atomisp_cmd.c:2801 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:2900 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?
	../drivers/media/i2c/ds90ub913.c: ../drivers/media/i2c/ds90ub913.c:479 ub913_log_status() error: uninitialized symbol 'v1'.
	../drivers/media/i2c/ds90ub913.c: ../drivers/media/i2c/ds90ub913.c:479 ub913_log_status() error: uninitialized symbol 'v2'.
	../drivers/media/i2c/ds90ub953.c: ../drivers/media/i2c/ds90ub953.c:642 ub953_log_status() error: uninitialized symbol 'gpio_local_data'.
	../drivers/media/i2c/ds90ub953.c: ../drivers/media/i2c/ds90ub953.c:642 ub953_log_status() error: uninitialized symbol 'gpio_input_ctrl'.
	../drivers/media/i2c/ds90ub953.c: ../drivers/media/i2c/ds90ub953.c:642 ub953_log_status() error: uninitialized symbol 'gpio_pin_sts'.
	../drivers/media/i2c/ds90ub960.c: ../drivers/media/i2c/ds90ub960.c:1780 ub960_init_tx_ports() error: uninitialized symbol 'pll_div'.
	SMATCH:../drivers/media/usb/siano/smsusb.c ../drivers/media/usb/siano/smsusb.c:53:38: :warning: array of flexible structures
	../drivers/media/usb/dvb-usb-v2/af9035.c: ../drivers/media/usb/dvb-usb-v2/af9035.c:467 af9035_i2c_master_xfer() warn: inconsistent returns '&d->i2c_mutex'.
	  Locked on  : 326,387
	  Unlocked on: 465,467
	../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/usb/uvc/uvc_v4l2.c: note: in included file (through ../arch/x86/include/asm/uaccess.h, ../include/linux/uaccess.h, ../include/linux/sched/task.h, ../include/linux/sched/signal.h, ../include/linux/rcuwait.h, ...):
	SPARSE:../drivers/media/usb/uvc/uvc_v4l2.c ../arch/x86/include/asm/uaccess_64.h:88:24: warning: cast removes address space '__user' of expression
	../drivers/media/usb/em28xx/em28xx-video.c: ../drivers/media/usb/em28xx/em28xx-video.c:2890 em28xx_v4l2_init() parse error: turning off implications after 60 seconds
	../drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.c: ../drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.c:2772 mxc_jpeg_probe() warn: missing unwind goto?
	../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2558 dvb_register() parse error: OOM: 3000004Kb sm_state_count = 1970928
	../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2558 dvb_register() warn: Function too hairy.  No more merges.
	../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2558 dvb_register() parse error: __split_smt: function too hairy.  Giving up after 53 seconds
	../drivers/media/pci/ivtv/ivtvfb.c: note: in included file (through ../arch/x86/include/asm/uaccess.h, ../include/linux/uaccess.h, ../include/linux/sched/task.h, ../include/linux/sched/signal.h, ../drivers/media/pci/ivtv/ivtv-driver.h):
	SPARSE:../drivers/media/pci/ivtv/ivtvfb.c ../arch/x86/include/asm/uaccess_64.h:88:24: warning: cast removes address space '__user' of expression
	../drivers/media/usb/pvrusb2/pvrusb2-hdw.c: ../drivers/media/usb/pvrusb2/pvrusb2-hdw.c:3293 pvr2_hdw_get_tuner_status() warn: inconsistent indenting

patches/0011-media-nxp-Remove-redundant-dev_err.patch:

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