mbox series

[GIT,PULL,FOR,v6.7] Miscellaneous changes to the rkisp1 driver

Message ID 20231009214219.GA25507@pendragon.ideasonboard.com (mailing list archive)
State New, archived
Headers show
Series [GIT,PULL,FOR,v6.7] Miscellaneous changes to the rkisp1 driver | expand

Pull-request

git://git.kernel.org/pub/scm/linux/kernel/git/pinchartl/linux.git tags/media-rkisp1-next-20231010

Message

Laurent Pinchart Oct. 9, 2023, 9:42 p.m. UTC
Hello Mauro, Hans,

The following changes since commit a1766a4fd83befa0b34d932d532e7ebb7fab1fa7:

  media: imon: fix access to invalid resource for the second interface (2023-10-07 10:55:48 +0200)

are available in the Git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/pinchartl/linux.git tags/media-rkisp1-next-20231010

for you to fetch changes up to 12c5b2a0154c4217e0e27465cda309851f188434:

  media: rkisp1: resizer: Fix resizer disable check when starting stream (2023-10-10 00:15:20 +0300)

----------------------------------------------------------------
Miscellaneous cleanups to the rkisp1 driver

----------------------------------------------------------------
Laurent Pinchart (14):
      MAINTAINERS: Add co-maintainer for the rkisp1 driver
      media: rkisp1: resizer: Use V4L2 subdev active state
      media: rkisp1: isp: Use V4L2 subdev active state
      media: rkisp1: csi: Use V4L2 subdev active state
      media: rkisp1: Convert hex constants to lowercase
      media: rkisp1: Constify rkisp1_v12_params_ops
      media: rkisp1: Fix line stride calculation
      media: rkisp1: Remove dual crop control register from config structure
      media: rkisp1: Program RKISP1_CIF_MI_SP_Y_PIC_SIZE register
      media: rkisp1: resizer: Constify argument and local variables
      media: rkisp1: resizer: Use v4l2_area instead of v4l2_rect to store size
      media: rkisp1: resizer: Drop unneeded local variable
      media: rkisp1: resizer: Improve debug message when configuring resizer
      media: rkisp1: resizer: Fix resizer disable check when starting stream

 MAINTAINERS                                        |   1 +
 .../platform/rockchip/rkisp1/rkisp1-capture.c      |  24 +-
 .../media/platform/rockchip/rkisp1/rkisp1-common.h |  18 -
 .../media/platform/rockchip/rkisp1/rkisp1-csi.c    | 107 ++--
 .../media/platform/rockchip/rkisp1/rkisp1-isp.c    | 263 ++++-----
 .../media/platform/rockchip/rkisp1/rkisp1-params.c |   4 +-
 .../media/platform/rockchip/rkisp1/rkisp1-regs.h   | 618 ++++++++++-----------
 .../platform/rockchip/rkisp1/rkisp1-resizer.c      | 255 ++++-----
 8 files changed, 564 insertions(+), 726 deletions(-)

Comments

Jenkins Oct. 9, 2023, 10:15 p.m. UTC | #1
From: builder@linuxtv.org

Pull request: https://patchwork.linuxtv.org/project/linux-media/patch/20231009214219.GA25507@pendragon.ideasonboard.com/
Build log: https://builder.linuxtv.org/job/patchwork/346853/
Build time: 00:27:05
Link: https://lore.kernel.org/linux-media/20231009214219.GA25507@pendragon.ideasonboard.com

gpg: Signature made Mon 09 Oct 2023 09:39:59 PM UTC
gpg:                using EDDSA key C09EF871B3827B413F971CA9CC3F2D800327DE64
gpg:                issuer "laurent.pinchart@ideasonboard.com"
gpg: Can't check signature: No public key

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

Error/warnings:

patches/0001-MAINTAINERS-Add-co-maintainer-for-the-rkisp1-driver.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-gc0310.c:446 gc0310_s_stream() warn: missing error code 'ret'
	../drivers/staging/media/atomisp/pci/atomisp_cmd.c: ../drivers/staging/media/atomisp/pci/atomisp_cmd.c:2779 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:2878 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:2792 mxc_jpeg_probe() warn: missing unwind goto?
	../drivers/media/i2c/mt9m114.c: ../drivers/media/i2c/mt9m114.c:2381 mt9m114_probe() warn: missing unwind goto?
	../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.
	SMATCH:../drivers/media/usb/siano/smsusb.c ../drivers/media/usb/siano/smsusb.c:53:38: :warning: array of flexible structures
	../drivers/media/pci/cx25821/cx25821-medusa-video.c: ../drivers/media/pci/cx25821/cx25821-medusa-video.c:399 medusa_set_videostandard() parse error: OOM: 3003988Kb sm_state_count = 2158
	../drivers/media/pci/cx25821/cx25821-medusa-video.c: ../drivers/media/pci/cx25821/cx25821-medusa-video.c:399 medusa_set_videostandard() parse error: __split_smt: function too hairy.  Giving up after 5 seconds
	../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2517 dvb_register() parse error: OOM: 3014564Kb sm_state_count = 1757213
	../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2517 dvb_register() parse error: __split_smt: function too hairy.  Giving up after 59 seconds
	../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/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/pci/mgb4/mgb4_sysfs_out.c: ../drivers/media/pci/mgb4/mgb4_sysfs_out.c:118 video_source_store() warn: potential spectre issue 'mgbdev->vin' [r] (local cap)
	../drivers/media/pci/mgb4/mgb4_sysfs_out.c: ../drivers/media/pci/mgb4/mgb4_sysfs_out.c:122 video_source_store() warn: possible spectre second half.  'loopin_new'
	../drivers/media/usb/em28xx/em28xx-video.c: ../drivers/media/usb/em28xx/em28xx-video.c:2836 em28xx_v4l2_init() parse error: turning off implications after 60 seconds