mbox series

[GIT,PULL,FOR,v6.4] NXP i.MX8 ISI driver

Message ID 20230415163404.GA31708@pendragon.ideasonboard.com (mailing list archive)
State New, archived
Headers show
Series [GIT,PULL,FOR,v6.4] NXP i.MX8 ISI driver | expand

Pull-request

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

Message

Laurent Pinchart April 15, 2023, 4:34 p.m. UTC
Hi Mauro,

The following changes since commit 671397d7c6a5d2b923aaf1b4414e3ac9333674a7:

  media: atomisp: gmin_platform: Add Lenovo Ideapad Miix 310 gmin_vars (2023-04-15 10:43:00 +0100)

are available in the Git repository at:

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

for you to fetch changes up to 833e2bcf678cb2b06db31eb821007d21a5a7934e:

  media: nxp: imx8-isi: Remove 300ms sleep after enabling channel (2023-04-15 19:10:43 +0300)

----------------------------------------------------------------
NXP i.MX8 ISI driver

----------------------------------------------------------------
Jacopo Mondi (1):
      media: nxp: imx8-isi: Remove 300ms sleep after enabling channel

Laurent Pinchart (4):
      dt-bindings: media: Add i.MX8 ISI DT bindings
      media: nxp: Add i.MX8 ISI driver
      media: nxp: imx8-isi: Drop partial support for i.MX8QM and i.MX8QXP
      media: nxp: imx8-isi: Replace udelay() with fsleep()

 .../devicetree/bindings/media/nxp,imx8-isi.yaml    |  173 +++
 MAINTAINERS                                        |    7 +
 drivers/media/platform/nxp/Kconfig                 |    2 +
 drivers/media/platform/nxp/Makefile                |    1 +
 drivers/media/platform/nxp/imx8-isi/Kconfig        |   22 +
 drivers/media/platform/nxp/imx8-isi/Makefile       |    8 +
 .../media/platform/nxp/imx8-isi/imx8-isi-core.c    |  539 +++++++
 .../media/platform/nxp/imx8-isi/imx8-isi-core.h    |  394 +++++
 .../platform/nxp/imx8-isi/imx8-isi-crossbar.c      |  529 +++++++
 .../media/platform/nxp/imx8-isi/imx8-isi-debug.c   |  109 ++
 drivers/media/platform/nxp/imx8-isi/imx8-isi-hw.c  |  650 +++++++++
 drivers/media/platform/nxp/imx8-isi/imx8-isi-m2m.c |  858 +++++++++++
 .../media/platform/nxp/imx8-isi/imx8-isi-pipe.c    |  867 +++++++++++
 .../media/platform/nxp/imx8-isi/imx8-isi-regs.h    |  418 ++++++
 .../media/platform/nxp/imx8-isi/imx8-isi-video.c   | 1512 ++++++++++++++++++++
 15 files changed, 6089 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/media/nxp,imx8-isi.yaml
 create mode 100644 drivers/media/platform/nxp/imx8-isi/Kconfig
 create mode 100644 drivers/media/platform/nxp/imx8-isi/Makefile
 create mode 100644 drivers/media/platform/nxp/imx8-isi/imx8-isi-core.c
 create mode 100644 drivers/media/platform/nxp/imx8-isi/imx8-isi-core.h
 create mode 100644 drivers/media/platform/nxp/imx8-isi/imx8-isi-crossbar.c
 create mode 100644 drivers/media/platform/nxp/imx8-isi/imx8-isi-debug.c
 create mode 100644 drivers/media/platform/nxp/imx8-isi/imx8-isi-hw.c
 create mode 100644 drivers/media/platform/nxp/imx8-isi/imx8-isi-m2m.c
 create mode 100644 drivers/media/platform/nxp/imx8-isi/imx8-isi-pipe.c
 create mode 100644 drivers/media/platform/nxp/imx8-isi/imx8-isi-regs.h
 create mode 100644 drivers/media/platform/nxp/imx8-isi/imx8-isi-video.c

Comments

Jenkins April 15, 2023, 5 p.m. UTC | #1
From: builder@linuxtv.org

Pull request: https://patchwork.linuxtv.org/project/linux-media/patch/20230415163404.GA31708@pendragon.ideasonboard.com/
Build log: https://builder.linuxtv.org/job/patchwork/297965/
Build time: 00:21:42
Link: https://lore.kernel.org/linux-media/20230415163404.GA31708@pendragon.ideasonboard.com

gpg: Signature made Sat 15 Apr 2023 04:10:56 PM UTC
gpg:                using EDDSA key C09EF871B3827B413F971CA9CC3F2D800327DE64
gpg:                issuer "laurent.pinchart@ideasonboard.com"
gpg: Can't check signature: No public key

Summary: got 2/5 patches with issues, being 1 at build time, plus one error when buinding PDF document

Error/warnings:

patches/0001-dt-bindings-media-Add-i.MX8-ISI-DT-bindings.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:416 ov2680_s_stream() warn: missing error code 'ret'
	../drivers/staging/media/atomisp/i2c/atomisp-gc0310.c:212 gc0310_s_stream() warn: missing error code 'ret'
	../drivers/staging/media/atomisp/pci/atomisp_cmd.c: ../drivers/staging/media/atomisp/pci/atomisp_cmd.c:3013 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:3112 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/platform/nxp/imx-jpeg/mxc-jpeg.c: ../drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.c:2775 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.
	SMATCH:../drivers/media/usb/siano/smsusb.c ../drivers/media/usb/siano/smsusb.c:53:38: :warning: array of flexible structures
	../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2570 dvb_register() parse error: OOM: 3000020Kb sm_state_count = 1974589
	../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 56 seconds
	../drivers/media/usb/em28xx/em28xx-video.c: ../drivers/media/usb/em28xx/em28xx-video.c:2884 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-dt-bindings-media-Add-i.MX8-ISI-DT-bindings.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:20: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?

patches/0002-media-nxp-Add-i.MX8-ISI-driver.patch:

   checkpatch.pl:
	$ cat patches/0002-media-nxp-Add-i.MX8-ISI-driver.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:98: WARNING: please write a help paragraph that fully describes the config symbol
	-:112: WARNING: please write a help paragraph that fully describes the config symbol
	-:761: WARNING: DT compatible string "fsl,imx8-isi" appears un-documented -- check ./Documentation/devicetree/bindings/
	-:1128: CHECK: Please use a blank line after function/struct/union/enum declarations
	-:1178: CHECK: Please use a blank line after function/struct/union/enum declarations
	-:1928: CHECK: usleep_range is preferred over udelay; see Documentation/timers/timers-howto.rst


Error #512 when building PDF docs