Message-ID: <b7cccc39-612d-4736-8689-890b4077e730@wolfvision.net>
Date: Mon, 13 Oct 2025 09:00:58 +0200
From: Gerald Loacker <gerald.loacker@...fvision.net>
To: Michael Riesch <michael.riesch@...labora.com>,
 Bryan O'Donoghue <bod@...nel.org>, Mehdi Djait
 <mehdi.djait@...ux.intel.com>,
 Maxime Chevallier <maxime.chevallier@...tlin.com>,
 Théo Lebrun <theo.lebrun@...tlin.com>,
 Thomas Petazzoni <thomas.petazzoni@...tlin.com>,
 Bryan O'Donoghue <bryan.odonoghue@...aro.org>,
 Markus Elfring <Markus.Elfring@....de>,
 Laurent Pinchart <laurent.pinchart@...asonboard.com>,
 Mauro Carvalho Chehab <mchehab@...nel.org>, Rob Herring
 <robh+dt@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>,
 Conor Dooley <conor+dt@...nel.org>, Heiko Stuebner <heiko@...ech.de>,
 Kever Yang <kever.yang@...k-chips.com>,
 Nicolas Dufresne <nicolas.dufresne@...labora.com>,
 Sebastian Reichel <sebastian.reichel@...labora.com>,
 Collabora Kernel Team <kernel@...labora.com>,
 Paul Kocialkowski <paulk@...-base.io>,
 Alexander Shiyan <eagle.alexander923@...il.com>,
 Val Packett <val@...kett.cool>, Rob Herring <robh@...nel.org>,
 Philipp Zabel <p.zabel@...gutronix.de>,
 Sakari Ailus <sakari.ailus@...ux.intel.com>
Cc: linux-media@...r.kernel.org, devicetree@...r.kernel.org,
 linux-kernel@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
 linux-rockchip@...ts.infradead.org, Mehdi Djait <mehdi.djait@...tlin.com>,
 Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
Subject: Re: [PATCH v11 00/17] media: rockchip: add a driver for the rockchip
 camera interface

Hi Michael, Bryan,

Am 10.10.2025 um 15:46 schrieb Michael Riesch:
> Hi Bryan,
> 
> On 10/10/25 14:15, Bryan O'Donoghue wrote:
>> 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@...ux.intel.com>
>>> To: Maxime Chevallier<maxime.chevallier@...tlin.com>
>>> To: Théo Lebrun<theo.lebrun@...tlin.com>
>>> To: Thomas Petazzoni<thomas.petazzoni@...tlin.com>
>>> To: Gerald Loacker<gerald.loacker@...fvision.net>
>>> To: Bryan O'Donoghue<bryan.odonoghue@...aro.org>
>>> To: Markus Elfring<Markus.Elfring@....de>
>>> To: Sakari Ailus<sakari.ailus@....fi>
>>> To: Laurent Pinchart<laurent.pinchart@...asonboard.com>
>>> To: Mauro Carvalho Chehab<mchehab@...nel.org>
>>> To: Rob Herring<robh+dt@...nel.org>
>>> To: Krzysztof Kozlowski<krzk+dt@...nel.org>
>>> To: Conor Dooley<conor+dt@...nel.org>
>>> To: Heiko Stuebner<heiko@...ech.de>
>>> To: Kever Yang<kever.yang@...k-chips.com>
>>> To: Nicolas Dufresne<nicolas.dufresne@...labora.com>
>>> To: Sebastian Reichel<sebastian.reichel@...labora.com>
>>> To: Collabora Kernel Team<kernel@...labora.com>
>>> To: Paul Kocialkowski<paulk@...-base.io>
>>> To: Alexander Shiyan<eagle.alexander923@...il.com>
>>> To: Val Packett<val@...kett.cool>
>>> To: Rob Herring<robh@...nel.org>
>>> To: Philipp Zabel<p.zabel@...gutronix.de>
>>> Cc:linux-media@...r.kernel.org
>>> Cc:devicetree@...r.kernel.org
>>> Cc:linux-kernel@...r.kernel.org
>>> Cc:linux-arm-kernel@...ts.infradead.org
>>> Cc:linux-rockchip@...ts.infradead.org
>>> Signed-off-by: Michael Riesch<michael.riesch@...fvision.net>
>>> Signed-off-by: Michael Riesch<michael.riesch@...labora.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@...labora.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.
> 
> Correct. The tag list used to be quite complicated [0] due to the
> complicated history of this driver. Maxime worked on the driver up to
> v5, Mehdi up to v14, I added RK3568 support on top of Mehdi's v14 (with
> some valuable contributions from Gerald) and submitted that as v1. v2 of
> my work incorporated Mehdi's v14, and here we are, after several rounds
> of feedback, at soon-to-be v12 (or, in fact, v25).
> 
>> On question on that, are these people aware of the change and cc'd on
>> the list of recipients/contactable/agreeable to the change ?
> 
> I announced this plan as a response to your comment to v10 [1] with
> Maxime, Mehdi and Gerald on Cc: and did not hear any objections.
> 
> Hope that this is still OK for everybody!
> 

I agree with that!

Regards,
Gerald

> Best regards,
> Michael
> 
> [0]
> https://lore.kernel.org/linux-media/20240220-rk3568-vicap-v10-6-62d8a7b209b4@collabora.com/
> [1]
> https://lore.kernel.org/linux-media/23ccc744-745d-4a31-a79c-2d64bf1ed43d@collabora.com/
> 

Powered by blists - more mailing lists