linux-kernel - Re: [PATCH v4 0/5] media: imx8qxp: add parallel camera support

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <aTClH5RSpXpXfxXx@lizhi-Precision-Tower-5810>
Date: Wed, 3 Dec 2025 16:01:19 -0500
From: Frank Li <Frank.li@....com>
To: Rui Miguel Silva <rui.silva@...aro.org>
Cc: Rui Miguel Silva <rmfrfs@...il.com>,
	Laurent Pinchart <laurent.pinchart@...asonboard.com>,
	Mauro Carvalho Chehab <mchehab@...nel.org>,
	Hans Verkuil <hverkuil+cisco@...nel.org>,
	Shawn Guo <shawnguo@...nel.org>,
	Sascha Hauer <s.hauer@...gutronix.de>,
	Pengutronix Kernel Team <kernel@...gutronix.de>,
	Fabio Estevam <festevam@...il.com>,
	Martin Kepplinger <martink@...teo.de>,
	Purism Kernel Team <kernel@...i.sm>, Rob Herring <robh@...nel.org>,
	Krzysztof Kozlowski <krzk+dt@...nel.org>,
	Conor Dooley <conor+dt@...nel.org>,
	Philipp Zabel <p.zabel@...gutronix.de>, linux-media@...r.kernel.org,
	imx@...ts.linux.dev, linux-arm-kernel@...ts.infradead.org,
	linux-kernel@...r.kernel.org, devicetree@...r.kernel.org,
	Alice Yuan <alice.yuan@....com>,
	Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>,
	Robert Chiras <robert.chiras@....com>,
	Zhipeng Wang <zhipeng.wang_1@....com>,
	Hans Verkuil <hans@...erkuil.nl>,
	Sakari Ailus <sakari.ailus@....fi>,
	Kieran Bingham <kieran.bingham@...asonboard.com>
Subject: Re: [PATCH v4 0/5] media: imx8qxp: add parallel camera support

On Wed, Dec 03, 2025 at 12:19:04PM +0000, Rui Miguel Silva wrote:
> Hey Frank,
> On Tue Dec 2, 2025 at 8:36 PM WET, Frank Li wrote:
>
> > On Thu, Nov 06, 2025 at 10:58:00AM -0500, Frank Li wrote:
> >> On Thu, Nov 06, 2025 at 10:34:48AM +0000, Rui Miguel Silva wrote:
> >> > On Wed Nov 5, 2025 at 7:31 PM WET, Frank Li wrote:
> >> >
> >> > > On Wed, Nov 05, 2025 at 07:19:28PM +0200, Laurent Pinchart wrote:
> >> > >> Hi Frank,
> >> > >>
> >> > >> On Wed, Nov 05, 2025 at 12:03:05PM -0500, Frank Li wrote:
> >> > >> > On Wed, Sep 03, 2025 at 09:56:29AM -0400, Frank Li wrote:
> >> > >> > > On Tue, Sep 02, 2025 at 02:39:20PM +0200, Laurent Pinchart wrote:
> >> > >> > > > On Tue, Sep 02, 2025 at 08:01:40AM -0400, Frank Li wrote:
> >> > >> > > > > On Tue, Aug 05, 2025 at 04:08:22AM +0300, Laurent Pinchart wrote:
> >> > >> > > > > > Hi Frank,
> >> > >> > > > > >
> >> > >> > > > > > Thank you for the patches.
> >> > >> > > > > >
> >> > >> > > > > > I've quite busy these days, and I don't believe I will have time to
> >> > >> > > > > > review this series before coming back from OSS Europe at the beginning
> >> > >> > > > > > of September. Let's see if anyone on CC could volunteer.
> >> > >> > > > >
> >> > >> > > > > Laurent Pincha
> >> > >> > > > > 	I hope you have good time at OSS.
> >> > >> > > > >
> >> > >> > > > > 	Do you have chance to review this patch?
> >> > >> > > >
> >> > >> > > > I'm going through my mail backlog, which is really big at the moment.
> >> > >> > >
> >> > >> > > Understand.
> >> > >> > >
> >> > >> > > > I'd like someone else to volunteer to review this series. It won't scale
> >> > >> > > > if I have to review all NXP media patches in my spare time :-/
> >> > >> > >
> >> > >> > > Yes, but none volunteer review this in passed months. Expecially key
> >> > >> > > reviewer. I am reviewing i3c patches. but Not familiar v4l system yet. It
> >> > >> > > need scalable solution. I can help filter some common and simple problem
> >> > >> > > from beginning.
> >> > >> >
> >> > >> > Laurent Pinchart:
> >> > >> >
> >> > >> > 	Do you have chance to check this serise? this one should be related simple.
> >> > >> > 	This one sent at 7/29. Still not any volunteer to review it.
> >> > >>
> >> > >> I'm afraid I won't have time to review this for the time being. My spare
> >> > >> time is already exhausted by all the other drivers I maintain upstream.
> >> > >>
> >> > >> > 	How do we move forward?
> >> > >>
> >> > >> I think this is a question for the subsystem maintainers. Hans, Mauro ?
> >> > >
> >> > > Mauro Carvalho Chehab and Hans Verkuil:
> >> > >
> >> > > 	Laurent provided great help about review and land i.MX related
> >> > > patches in past, who are quite famillar with i.MX chips. But he is quite
> >> > > busy. So the whole reviews cycles takes quite long time and offten cross
> >> > > some merge windows.
> >> > >
> >> > > 	In pull requests for 6.19:
> >> > > https://lore.kernel.org/all/4989c563-47f4-478c-80c4-41f7e98597e4@kernel.org/
> >> > > only 10 patches, and 4 patches is trivial clean up.
> >> > >
> >> > > 	In reviewing patch queue, there are
> >> > > 	1: media: nxp: imx8-isi: Add ISI support for i.MX95
> >> > > 	   https://lore.kernel.org/imx/20251105-isi_imx95-v3-0-3987533cca1c@nxp.com/T/#t
> >> > > 	   This one already review, but I am not sure if it capture 6.19 cycle because
> >> > > PULL-request already sent.
> >> > >
> >> > > 	2: Add MIPI CSI-2 support for i.MX8ULP
> >> > > 	   https://lore.kernel.org/imx/20251023-csi2_imx8ulp-v7-0-5ecb081ce79b@nxp.com/
> >> > >
> >> > > 	3: media: add imx93 mipi/controller csi support
> >> > > 	   https://lore.kernel.org/imx/20250821-95_cam-v3-0-c9286fbb34b9@nxp.com/
> >> > > 	   This one is quite big, but first 10 patches is simple trivial cleanup patches.
> >> > > 	   I post at 8/27, but get first feedback around 10/27, I am not
> >> > > 	   sure if missing somethings.
> >> > >
> >> > > 	4: This series, laurent already said no time review it.
> >> > >
> >> > > 	5: ap1302 sensor patches
> >> > >            https://lore.kernel.org/imx/20250811-ap1302-v4-0-80cc41b91662@nxp.com/
> >> > >            binding already ACK, most maintainer want to pick binding with
> >> > > 	   driver together, but not an feedback since 8/11.
> >> > >
> >> > > 	I jump into and help do some review.
> >> > >
> >> > > 	The questions is how to move forward pending patches, like [3], [4],
> >> > > [5]. How to keep good community channel to avoid long time pending?
> >> >
> >> > Sorry, but like Laurent I am really without spare cycles to go over the
> >> > patch series that have arrived in media (also I do not have any hw anymore).
> >> > So, Frank or maybe Laurent knows someone that would like to be add also as
> >> > maintainer of this drivers for me would be great, that would also help to
> >> > avoid such bottlenecks.
> >>
> >> If there are not other candidate, I can help maintain it although I am more
> >> familar with dt binding, i3c and dmaengine. I can start from simple patches
> >> and we have test team to help cover testing on the real hardware.
> >
> > Rui:
> > 	Any update? How to move forward? It was already passed more than 3
> > month!
>
> Sorry, I did not realise something was blocked because of me, specially imx8
> related. I thought you have sent the update to MAINTAINERS.

I just marked me as reviewer in prevous's patch.

>
> For me it is fine, it makes perfect sense since you have access to hardware
> and documentation, for you to be the maintainer and steer the features and
> improvements in the imx media. you can move me to Reviewer, since there are
> already other 2 persons (and Laurent already handle the pulls, thanks for that)
> much more active in the IMX media subsystem.

Laruent Pinchart:

	If I will remove common file change in patch
"media: v4l2-common: Add helper function v4l_get_required_align_by_bpp()"

	Are you okay to pick up this patches? Or what's way to get it
merged? Or anyone else can pickup and send pull if you have not time.

Frank

>
> Thanks again and Cheers,
>     Rui