Thread (18 messages) 18 messages, 4 authors, 2025-12-03

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
  
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help