Re: [PATCH v4 0/5] media: imx8qxp: add parallel camera support
From: Frank Li <Frank.li@nxp.com>
Date: 2025-12-03 21:01:36
Also in:
imx, linux-devicetree, linux-media, lkml
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:quoted
On Thu, Nov 06, 2025 at 10:58:00AM -0500, Frank Li wrote:quoted
On Thu, Nov 06, 2025 at 10:34:48AM +0000, Rui Miguel Silva wrote:quoted
On Wed Nov 5, 2025 at 7:31 PM WET, Frank Li wrote:quoted
On Wed, Nov 05, 2025 at 07:19:28PM +0200, Laurent Pinchart wrote:quoted
Hi Frank, On Wed, Nov 05, 2025 at 12:03:05PM -0500, Frank Li wrote:quoted
On Wed, Sep 03, 2025 at 09:56:29AM -0400, Frank Li wrote:quoted
On Tue, Sep 02, 2025 at 02:39:20PM +0200, Laurent Pinchart wrote:quoted
On Tue, Sep 02, 2025 at 08:01:40AM -0400, Frank Li wrote:quoted
On Tue, Aug 05, 2025 at 04:08:22AM +0300, Laurent Pinchart wrote:quoted
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.quoted
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.quoted
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/ (local) 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 (local) 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/ (local) 3: media: add imx93 mipi/controller csi support https://lore.kernel.org/imx/20250821-95_cam-v3-0-c9286fbb34b9@nxp.com/ (local) 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/ (local) 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