Re: [GIT PULL FOR 6.8] V4L2 sub-device API changes plus new drivers

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Sakari,

On 28/11/2023 12:56, Sakari Ailus wrote:
> Hi Mauro, Hans,
> 
> Here are a bunch of new drivers (Thine ISP, gc2145 sensor driver, dcmipp
> camera interface driver) and an init_cfg pad op change to init_state
> internal op. Included is also the media tree portion of the DisCo for
> Imaging support and a new fwnode property API function, needed by the Thine
> ISP driver. Also a number of fixes and improvements to the mipid02 and
> vgxy61 drivers are included --- they also depend on the sub-device API
> changes.
> 
> Please pull.
> 
> 
> The following changes since commit 3b8551e73271fc375b15c887db54ad31686eb2ea:
> 
>   media: wave5: add OF and V4L_MEM2MEM_DRIVERS dependencies (2023-11-24 14:11:22 +0100)
> 
> are available in the Git repository at:
> 
>   git://linuxtv.org/sailus/media_tree.git tags/for-6.8-4-signed
> 
> for you to fetch changes up to 710fd99e818be5dc946f689a05971c2a9a1c3b8d:
> 
>   media: stm32-dcmipp: STM32 DCMIPP camera interface driver (2023-11-28 12:31:47 +0200)
> 
> ----------------------------------------------------------------
> V4L2 patches for 6.8
> 
> ----------------------------------------------------------------
> Alain Volmat (15):
>       dt-bindings: vendor-prefixes: Add prefix for GalaxyCore Inc.
>       dt-bindings: media: i2c: add galaxycore,gc2145 dt-bindings
>       media: i2c: gc2145: Galaxy Core GC2145 sensor support
>       media: i2c: st-mipid02: correct format propagation
>       media: i2c: st-mipid02: add usage of v4l2_get_link_freq
>       media: i2c: st-mipid02: don't keep track of streaming status
>       media: i2c: st-mipid02: use cci_* helpers for register access.
>       media: i2c: st-mipid02: use active state to store pad formats
>       media: i2c: st-mipid02: use mipi-csi macro for data-type
>       media: i2c: st-mipid02: removal of unused link_frequency variable
>       media: i2c: st-mipid02: add Y8 format support
>       media: i2c: st-vgxy61: Add V4L2_SUBDEV_FL_HAS_EVENTS and subscribe hooks
>       media: i2c: st-vgxy61: add v4l2_fwnode ctrls parse and addition
>       dt-bindings: media: add bindings for stm32 dcmipp
>       media: MAINTAINERS: add entry for STM32 DCMIPP driver
> 
> Hugues Fruchet (1):
>       media: stm32-dcmipp: STM32 DCMIPP camera interface driver

I'm getting these kerneldoc warnings/errors in dcmipp-common:

drivers/media/platform/st/stm32/stm32-dcmipp/dcmipp-common.h:59: error: Cannot parse struct or union!
drivers/media/platform/st/stm32/stm32-dcmipp/dcmipp-common.h:106: warning: Function parameter or member 'pads_flag' not described in 'dcmipp_pads_init'
drivers/media/platform/st/stm32/stm32-dcmipp/dcmipp-common.h:106: warning: Excess function parameter 'pads_flags' description in 'dcmipp_pads_init'
drivers/media/platform/st/stm32/stm32-dcmipp/dcmipp-common.h:147: warning: Function parameter or member 'handler' not described in 'dcmipp_ent_sd_register'
drivers/media/platform/st/stm32/stm32-dcmipp/dcmipp-common.h:147: warning: Function parameter or member 'thread_fn' not described in 'dcmipp_ent_sd_register'

It is likely best to resolve this in a follow-up patch. Hugues, can you take
a look at that?

In addition, this PR conflicts with the "[PATCH] media: vsp1: Remove unbalanced
.s_stream(0) calls" vsp1 v6.7 fix: that was merged in mainline yesterday, but this
PR was created without that patch.

I plan to merge v6.7-rc4 into staging on Monday, so perhaps the best way forward
is if you can apply that fix to your tree, then add the patches from this PR on top
of that, resolving the conflict. Then I can merge the updated PR on Monday. There
are no other issues with this PR, it passed all other tests.

Regards,

	Hans

> 
> Laurent Pinchart (3):
>       media: renesas: vsp1: Fix references to pad config
>       media: uapi: Add controls for the THP7312 ISP
>       media: v4l2-subdev: Rename .init_cfg() operation to .init_state()
> 
> Paul Elder (2):
>       dt-bindings: media: Add bindings for THine THP7312 ISP
>       media: i2c: Add driver for THine THP7312
> 
> Sakari Ailus (2):
>       device property: Add fwnode_name_eq()
>       media: v4l: fwnode: Parse MIPI DisCo for Imaging properties
> 
>  .../bindings/media/i2c/galaxycore,gc2145.yaml      |  113 +
>  .../bindings/media/i2c/thine,thp7312.yaml          |  224 ++
>  .../devicetree/bindings/media/st,stm32-dcmipp.yaml |   89 +
>  .../devicetree/bindings/vendor-prefixes.yaml       |    2 +
>  .../userspace-api/media/drivers/index.rst          |    1 +
>  .../userspace-api/media/drivers/thp7312.rst        |   39 +
>  MAINTAINERS                                        |   24 +-
>  drivers/base/property.c                            |   28 +
>  drivers/media/i2c/Kconfig                          |   27 +
>  drivers/media/i2c/Makefile                         |    2 +
>  drivers/media/i2c/adv7180.c                        |   10 +-
>  drivers/media/i2c/ccs/ccs-core.c                   |    6 +-
>  drivers/media/i2c/ds90ub913.c                      |   10 +-
>  drivers/media/i2c/ds90ub953.c                      |   10 +-
>  drivers/media/i2c/ds90ub960.c                      |   11 +-
>  drivers/media/i2c/gc2145.c                         | 1450 +++++++++++++
>  drivers/media/i2c/hi846.c                          |   10 +-
>  drivers/media/i2c/imx214.c                         |   12 +-
>  drivers/media/i2c/imx219.c                         |    9 +-
>  drivers/media/i2c/imx290.c                         |   10 +-
>  drivers/media/i2c/imx296.c                         |   10 +-
>  drivers/media/i2c/imx334.c                         |   12 +-
>  drivers/media/i2c/imx335.c                         |   12 +-
>  drivers/media/i2c/imx412.c                         |   12 +-
>  drivers/media/i2c/imx415.c                         |   10 +-
>  drivers/media/i2c/mt9m001.c                        |   10 +-
>  drivers/media/i2c/mt9m111.c                        |   10 +-
>  drivers/media/i2c/mt9m114.c                        |   16 +-
>  drivers/media/i2c/mt9p031.c                        |    8 +-
>  drivers/media/i2c/mt9v111.c                        |   10 +-
>  drivers/media/i2c/ov01a10.c                        |   10 +-
>  drivers/media/i2c/ov02a10.c                        |   10 +-
>  drivers/media/i2c/ov2640.c                         |   10 +-
>  drivers/media/i2c/ov2680.c                         |   10 +-
>  drivers/media/i2c/ov2740.c                         |   10 +-
>  drivers/media/i2c/ov5640.c                         |   10 +-
>  drivers/media/i2c/ov5645.c                         |   12 +-
>  drivers/media/i2c/ov5670.c                         |   10 +-
>  drivers/media/i2c/ov7251.c                         |   12 +-
>  drivers/media/i2c/ov8858.c                         |   10 +-
>  drivers/media/i2c/ov9282.c                         |   12 +-
>  drivers/media/i2c/st-mipid02.c                     |  480 ++---
>  drivers/media/i2c/st-vgxy61.c                      |   29 +-
>  drivers/media/i2c/tc358746.c                       |   10 +-
>  drivers/media/i2c/tda1997x.c                       |   10 +-
>  drivers/media/i2c/thp7312.c                        | 2244 ++++++++++++++++++++
>  drivers/media/i2c/tvp5150.c                        |    6 +-
>  drivers/media/pci/intel/ivsc/mei_csi.c             |   10 +-
>  drivers/media/platform/cadence/cdns-csi2rx.c       |   10 +-
>  .../media/platform/microchip/microchip-csi2dc.c    |   10 +-
>  .../platform/microchip/microchip-isc-scaler.c      |   10 +-
>  drivers/media/platform/nxp/imx-mipi-csis.c         |   10 +-
>  drivers/media/platform/nxp/imx7-media-csi.c        |    6 +-
>  .../platform/nxp/imx8-isi/imx8-isi-crossbar.c      |   10 +-
>  .../media/platform/nxp/imx8-isi/imx8-isi-pipe.c    |   10 +-
>  drivers/media/platform/nxp/imx8mq-mipi-csi2.c      |   10 +-
>  .../media/platform/renesas/rzg2l-cru/rzg2l-csi2.c  |   10 +-
>  .../media/platform/renesas/rzg2l-cru/rzg2l-ip.c    |   10 +-
>  drivers/media/platform/renesas/vsp1/vsp1_brx.c     |   41 +-
>  drivers/media/platform/renesas/vsp1/vsp1_clu.c     |    4 +-
>  drivers/media/platform/renesas/vsp1/vsp1_entity.c  |  130 +-
>  drivers/media/platform/renesas/vsp1/vsp1_entity.h  |   12 +-
>  drivers/media/platform/renesas/vsp1/vsp1_hgo.c     |    4 +-
>  drivers/media/platform/renesas/vsp1/vsp1_hgt.c     |    4 +-
>  drivers/media/platform/renesas/vsp1/vsp1_histo.c   |   24 +-
>  drivers/media/platform/renesas/vsp1/vsp1_hsit.c    |   12 +-
>  drivers/media/platform/renesas/vsp1/vsp1_lif.c     |    3 +-
>  drivers/media/platform/renesas/vsp1/vsp1_lut.c     |    1 -
>  drivers/media/platform/renesas/vsp1/vsp1_rpf.c     |    8 +-
>  drivers/media/platform/renesas/vsp1/vsp1_rwpf.c    |   38 +-
>  drivers/media/platform/renesas/vsp1/vsp1_sru.c     |   37 +-
>  drivers/media/platform/renesas/vsp1/vsp1_uds.c     |   40 +-
>  drivers/media/platform/renesas/vsp1/vsp1_uif.c     |   25 +-
>  drivers/media/platform/renesas/vsp1/vsp1_video.c   |    4 +-
>  drivers/media/platform/renesas/vsp1/vsp1_wpf.c     |   10 +-
>  .../media/platform/rockchip/rkisp1/rkisp1-csi.c    |   10 +-
>  .../media/platform/rockchip/rkisp1/rkisp1-isp.c    |   10 +-
>  .../platform/rockchip/rkisp1/rkisp1-resizer.c      |   10 +-
>  drivers/media/platform/st/stm32/Kconfig            |   16 +
>  drivers/media/platform/st/stm32/Makefile           |    1 +
>  .../media/platform/st/stm32/stm32-dcmipp/Makefile  |    4 +
>  .../st/stm32/stm32-dcmipp/dcmipp-bytecap.c         |  956 +++++++++
>  .../st/stm32/stm32-dcmipp/dcmipp-byteproc.c        |  565 +++++
>  .../platform/st/stm32/stm32-dcmipp/dcmipp-common.c |  111 +
>  .../platform/st/stm32/stm32-dcmipp/dcmipp-common.h |  216 ++
>  .../platform/st/stm32/stm32-dcmipp/dcmipp-core.c   |  604 ++++++
>  .../st/stm32/stm32-dcmipp/dcmipp-parallel.c        |  440 ++++
>  drivers/media/platform/sunxi/sun4i-csi/sun4i_csi.c |    1 +
>  drivers/media/platform/sunxi/sun4i-csi/sun4i_csi.h |    1 +
>  .../media/platform/sunxi/sun4i-csi/sun4i_v4l2.c    |    9 +-
>  .../platform/sunxi/sun6i-csi/sun6i_csi_bridge.c    |   10 +-
>  .../sunxi/sun6i-mipi-csi2/sun6i_mipi_csi2.c        |   10 +-
>  .../sun8i-a83t-mipi-csi2/sun8i_a83t_mipi_csi2.c    |   10 +-
>  drivers/media/platform/ti/cal/cal-camerarx.c       |   10 +-
>  drivers/media/platform/video-mux.c                 |   10 +-
>  drivers/media/platform/xilinx/xilinx-csi2rxss.c    |   10 +-
>  drivers/media/test-drivers/vimc/vimc-debayer.c     |   11 +-
>  drivers/media/test-drivers/vimc/vimc-scaler.c      |   11 +-
>  drivers/media/test-drivers/vimc/vimc-sensor.c      |   11 +-
>  drivers/media/v4l2-core/v4l2-fwnode.c              |    4 +-
>  drivers/media/v4l2-core/v4l2-subdev.c              |   20 +-
>  drivers/staging/media/imx/imx-ic-prp.c             |    2 +-
>  drivers/staging/media/imx/imx-ic-prpencvf.c        |    2 +-
>  drivers/staging/media/imx/imx-media-csi.c          |    2 +-
>  drivers/staging/media/imx/imx-media-utils.c        |    8 +-
>  drivers/staging/media/imx/imx-media-vdic.c         |    2 +-
>  drivers/staging/media/imx/imx-media.h              |    4 +-
>  drivers/staging/media/imx/imx6-mipi-csi2.c         |    2 +-
>  drivers/staging/media/starfive/camss/stf-isp.c     |    6 +-
>  .../staging/media/sunxi/sun6i-isp/sun6i_isp_proc.c |   10 +-
>  include/linux/property.h                           |    1 +
>  include/media/v4l2-subdev.h                        |    7 +-
>  include/uapi/linux/thp7312.h                       |   19 +
>  include/uapi/linux/v4l2-controls.h                 |    6 +
>  114 files changed, 7983 insertions(+), 754 deletions(-)
>  create mode 100644 Documentation/devicetree/bindings/media/i2c/galaxycore,gc2145.yaml
>  create mode 100644 Documentation/devicetree/bindings/media/i2c/thine,thp7312.yaml
>  create mode 100644 Documentation/devicetree/bindings/media/st,stm32-dcmipp.yaml
>  create mode 100644 Documentation/userspace-api/media/drivers/thp7312.rst
>  create mode 100644 drivers/media/i2c/gc2145.c
>  create mode 100644 drivers/media/i2c/thp7312.c
>  create mode 100644 drivers/media/platform/st/stm32/stm32-dcmipp/Makefile
>  create mode 100644 drivers/media/platform/st/stm32/stm32-dcmipp/dcmipp-bytecap.c
>  create mode 100644 drivers/media/platform/st/stm32/stm32-dcmipp/dcmipp-byteproc.c
>  create mode 100644 drivers/media/platform/st/stm32/stm32-dcmipp/dcmipp-common.c
>  create mode 100644 drivers/media/platform/st/stm32/stm32-dcmipp/dcmipp-common.h
>  create mode 100644 drivers/media/platform/st/stm32/stm32-dcmipp/dcmipp-core.c
>  create mode 100644 drivers/media/platform/st/stm32/stm32-dcmipp/dcmipp-parallel.c
>  create mode 100644 include/uapi/linux/thp7312.h
> 





[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux