mbox series

[GIT,PULL,FOR,v6.6] media: NXP miscellaneous changes

Message ID 20230801221112.GB31500@pendragon.ideasonboard.com
State New
Headers show
Series [GIT,PULL,FOR,v6.6] media: NXP miscellaneous changes | expand

Pull-request

git://git.kernel.org/pub/scm/linux/kernel/git/pinchartl/linux.git tags/media-next-imx-20230802

Message

Laurent Pinchart Aug. 1, 2023, 10:11 p.m. UTC
Hi Mauro,

The following changes since commit 60a25e001759c90f35e2e7704f3e8b0d4fb10f96:

  media: atomisp: Fix me->stages error checking in sh_css_sp_init_pipeline() (2023-07-31 11:39:28 +0200)

are available in the Git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/pinchartl/linux.git tags/media-next-imx-20230802

for you to fetch changes up to 0e217f58c7c9a4a881c69c56ec4a85c623a54682:

  media: imx: imx7-media-csi: Fix frame sizes enumeration (2023-08-02 01:08:19 +0300)

----------------------------------------------------------------
Miscellaneous changes for NXP media drivers

----------------------------------------------------------------
Guoniu.zhou (3):
      media: dt-bindings: nxp,imx8-isi: Add i.MX93 ISI compatible string
      media: nxp: imx8-isi: Move i.MX8 gasket configuration to an ops structure
      media: nxp: imx8-isi: Add ISI support for i.MX93

Laurent Pinchart (3):
      staging: media: imx: Merge VIDEO_IMX_CSI into VIDEO_IMX_MEDIA
      arm64: defconfig: Drop CONFIG_VIDEO_IMX_MEDIA
      media: imx: imx7-media-csi: Fix frame sizes enumeration

Lukas Bulwahn (1):
      MAINTAINERS: correct file entry in MEDIA DRIVERS FOR FREESCALE IMX7/8

Ruan Jinjie (1):
      media: nxp: Remove redundant dev_err()

Yang Yingliang (1):
      media: nxp: Fix wrong return pointer check in mxc_isi_crossbar_init()

 .../devicetree/bindings/media/nxp,imx8-isi.yaml    |  5 +-
 MAINTAINERS                                        |  2 +-
 arch/arm64/configs/defconfig                       |  1 -
 drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.c     |  1 -
 drivers/media/platform/nxp/imx7-media-csi.c        | 14 ++--
 drivers/media/platform/nxp/imx8-isi/Makefile       |  4 +-
 .../media/platform/nxp/imx8-isi/imx8-isi-core.c    | 21 +++++-
 .../media/platform/nxp/imx8-isi/imx8-isi-core.h    | 14 +++-
 .../platform/nxp/imx8-isi/imx8-isi-crossbar.c      | 38 ++--------
 .../media/platform/nxp/imx8-isi/imx8-isi-gasket.c  | 85 ++++++++++++++++++++++
 .../media/platform/nxp/imx8-isi/imx8-isi-pipe.c    |  1 -
 drivers/staging/media/imx/Kconfig                  | 17 +----
 drivers/staging/media/imx/Makefile                 |  7 +-
 13 files changed, 144 insertions(+), 66 deletions(-)
 create mode 100644 drivers/media/platform/nxp/imx8-isi/imx8-isi-gasket.c

Comments

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

Pull request: https://patchwork.linuxtv.org/project/linux-media/patch/20230801221112.GB31500@pendragon.ideasonboard.com/
Build log: https://builder.linuxtv.org/job/patchwork/327172/
Build time: 00:26:57
Link: https://lore.kernel.org/linux-media/20230801221112.GB31500@pendragon.ideasonboard.com

gpg: Signature made Tue 01 Aug 2023 10:09:10 PM UTC
gpg:                using EDDSA key C09EF871B3827B413F971CA9CC3F2D800327DE64
gpg:                issuer "laurent.pinchart@ideasonboard.com"
gpg: Can't check signature: No public key

Summary: got 3/9 patches with issues, being 2 at build time

Error/warnings:

patches/0001-staging-media-imx-Merge-VIDEO_IMX_CSI-into-VIDEO_IMX.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/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/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/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/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/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2558 dvb_register() parse error: OOM: 3000008Kb sm_state_count = 1971907
	../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 54 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
	../drivers/media/usb/em28xx/em28xx-video.c: ../drivers/media/usb/em28xx/em28xx-video.c:2891 em28xx_v4l2_init() parse error: turning off implications after 60 seconds

patches/0004-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?

patches/0006-media-nxp-imx8-isi-Move-i.MX8-gasket-configuration-t.patch:

   checkpatch.pl:
	$ cat patches/0006-media-nxp-imx8-isi-Move-i.MX8-gasket-configuration-t.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:189: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?