Re: [PATCH v11 00/17] media: rockchip: add a driver for the rockchip camera interface

[Bryan O'Donoghue <bod@kernel.org>]

On 17/09/2025 16:38, Michael Riesch via B4 Relay wrote:
> Habidere,
> 
> This series introduces support for the Rockchip Camera Interface (CIF),
> which is featured in many Rockchip SoCs in different variations.
> For example, the PX30 Video Input Processor (VIP) is able to receive
> video data via the Digital Video Port (DVP, a parallel data interface)
> and transfer it into system memory using a double-buffering mechanism
> called ping-pong mode.
> The RK3568 Video Capture (VICAP) unit, on the other hand, features a
> DVP and a MIPI CSI-2 receiver that can receive video data independently
> (both using the ping-pong scheme).
> The different variants may have additional features, such as scaling
> and/or cropping.
> Finally, the RK3588 VICAP unit constitutes an essential piece of the
> camera interface with one DVP, six MIPI CSI-2 receivers, scale/crop
> units, and a data path multiplexer (to scaler units, to ISP, ...).
> 
> The v11 of the series adds a media controller centric V4L2 device driver
> for the Rockchip CIF with
>   - support for the PX30 VIP (not tested, though, due to the lack of HW)
>   - support for the RK3568 VICAP, including
>      - capturing frames from the DVP
>      - capturing frames from the MIPI CSI-2 receiver
>   - abstraction for the ping-pong scheme to allow for future extensions
>   - abstraction for the INTERFACE and CROP parts to allow for future
>     extensions
>   - initial support for different virtual channels (not tested, though,
>     due to the lack of HW)
> and a V4L2 subdevice driver for the Rockchip MIPI CSI-2 Receiver.
> 
> The driver can be readily extended to provide support for the RK3588
> VICAP variant. In order to keep things simple, however, this extension
> shall be submitted separately.
> 
> Looking forward to your comments!
> 
> To: Mehdi Djait<mehdi.djait@linux.intel.com>
> To: Maxime Chevallier<maxime.chevallier@bootlin.com>
> To: Théo Lebrun<theo.lebrun@bootlin.com>
> To: Thomas Petazzoni<thomas.petazzoni@bootlin.com>
> To: Gerald Loacker<gerald.loacker@wolfvision.net>
> To: Bryan O'Donoghue<bryan.odonoghue@linaro.org>
> To: Markus Elfring<Markus.Elfring@web.de>
> To: Sakari Ailus<sakari.ailus@iki.fi>
> To: Laurent Pinchart<laurent.pinchart@ideasonboard.com>
> To: Mauro Carvalho Chehab<mchehab@kernel.org>
> To: Rob Herring<robh+dt@kernel.org>
> To: Krzysztof Kozlowski<krzk+dt@kernel.org>
> To: Conor Dooley<conor+dt@kernel.org>
> To: Heiko Stuebner<heiko@sntech.de>
> To: Kever Yang<kever.yang@rock-chips.com>
> To: Nicolas Dufresne<nicolas.dufresne@collabora.com>
> To: Sebastian Reichel<sebastian.reichel@collabora.com>
> To: Collabora Kernel Team<kernel@collabora.com>
> To: Paul Kocialkowski<paulk@sys-base.io>
> To: Alexander Shiyan<eagle.alexander923@gmail.com>
> To: Val Packett<val@packett.cool>
> To: Rob Herring<robh@kernel.org>
> To: Philipp Zabel<p.zabel@pengutronix.de>
> Cc:linux-media@vger.kernel.org
> Cc:devicetree@vger.kernel.org
> Cc:linux-kernel@vger.kernel.org
> Cc:linux-arm-kernel@lists.infradead.org
> Cc:linux-rockchip@lists.infradead.org
> Signed-off-by: Michael Riesch<michael.riesch@wolfvision.net>
> Signed-off-by: Michael Riesch<michael.riesch@collabora.com>
> 
> Changes in v11:
> - rkcif: split large driver patch (6/13 of v10) into smaller
>    patches (6-11/17 of v11) (Bryan)
> - rkcsi: replaced devm_reset_control_array_get_exclusive with
>    devm_reset_control_get_exclusive (Philipp)
> - Link to v10:https://lore.kernel.org/r/20240220-rk3568-vicap- 
> v10-0-62d8a7b209b4@collabora.com

I believe it is the case and please feel free to correct me if I'm wrong 
that you've dropped a long list of sob/co-develop-by, I think we 
discussed that too, because of the level of change, it seems reasonable too.

On question on that, are these people aware of the change and cc'd on 
the list of recipients/contactable/agreeable to the change ?

---
bod