From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id EC8FDCCF9F8 for ; Wed, 5 Nov 2025 17:19:49 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:In-Reply-To:Content-Type: MIME-Version:References:Message-ID:Subject:Cc:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=hbDLaMQc43zRsZzj0vQ448TEWOOV8AZWqwQlSVqJlLc=; b=nTjKZzxp//z3aW8BB+NYQaz55j f6gx/WfTGQ2rs7z/oZcBPy0KEbFc4gN+rnKwLuP55k3DEoyAkLjD+d05cXk/G34Nqf3BACeKpc5mM EjdQoYywpTvOmWszyWGtf49FL3AGO4u6uMGiFhRbyiihR7sfm+nIZs0tGhWCojCNT3dw40rGuthuh +nGwDetaIniyA9YxIvUSsRIj7uiB8+FQkVIaR3tJPO8kDg2god9mKI6RZVkAtOtPqqLKfc/ypixpq 9GohVXc4U4NMd1/C69I7qfNpLK2WBf235HkdWrXrYyT2ODCOHTxgIsoJkNMDCPLdPa/rQsBnQT2Vh 5jzE3w5Q==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux)) id 1vGhAa-0000000E9JC-3a28; Wed, 05 Nov 2025 17:19:40 +0000 Received: from perceval.ideasonboard.com ([213.167.242.64]) by bombadil.infradead.org with esmtps (Exim 4.98.2 #2 (Red Hat Linux)) id 1vGhAX-0000000E9Io-3KY8 for linux-arm-kernel@lists.infradead.org; Wed, 05 Nov 2025 17:19:39 +0000 Received: from pendragon.ideasonboard.com (82-203-161-95.bb.dnainternet.fi [82.203.161.95]) by perceval.ideasonboard.com (Postfix) with UTF8SMTPSA id 810CA7E0; Wed, 5 Nov 2025 18:17:37 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com; s=mail; t=1762363057; bh=nUw06SpualPBrUCbreEZukSxYJFldjGiyXLqJA/+W/A=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=GYE5iFNCFro4v8Mks7/CnxXPWRfIDY59DAKVU5nZwniLh25QUunrqQI754SbS/+vw jvFGZY9x0IhNlWsrVWlR7qqo3oo34033+O32uKPTlENQT/f8sN0OUsVZ+nZgMnEbfL i+VY0epv5jVCDJRTGD+us3Z7UAercYn7Wa+lfrko= Date: Wed, 5 Nov 2025 19:19:28 +0200 From: Laurent Pinchart To: Frank Li Cc: Mauro Carvalho Chehab , Shawn Guo , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam , Rui Miguel Silva , Martin Kepplinger , Purism Kernel Team , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Philipp Zabel , linux-media@vger.kernel.org, imx@lists.linux.dev, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, Alice Yuan , Krzysztof Kozlowski , Robert Chiras , Zhipeng Wang , Hans Verkuil , Sakari Ailus , Kieran Bingham Subject: Re: [PATCH v4 0/5] media: imx8qxp: add parallel camera support Message-ID: <20251105171928.GB6046@pendragon.ideasonboard.com> References: <20250729-imx8qxp_pcam-v4-0-4dfca4ed2f87@nxp.com> <20250805010822.GC24627@pendragon.ideasonboard.com> <20250902123920.GM13448@pendragon.ideasonboard.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20251105_091938_298193_1F72FFDD X-CRM114-Status: GOOD ( 38.62 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org 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 ? > > > > > On Tue, Jul 29, 2025 at 12:06:21PM -0400, Frank Li wrote: > > > > > > Add parallel camera support for i.MX8 chips. > > > > > > > > > > > > The below patch to add new format support to test ov5640 sensor > > > > > > media: nxp: isi: add support for UYVY8_2X8 and YUYV8_2X8 bus codes > > > > > > > > > > > > The bindings and driver for parallel CSI > > > > > > dt-bindings: media: add i.MX parallel csi support > > > > > > media: nxp: add V4L2 subdev driver for parallel CSI > > > > > > > > > > > > DTS part need depend on previous MIPI CSI patches. > > > > > > https://lore.kernel.org/imx/20250522-8qxp_camera-v5-13-d4be869fdb7e@nxp.com/ > > > > > > > > > > > > arm64: dts: imx8: add parellel csi nodes > > > > > > arm64: dts: imx8qxp-mek: add parallel ov5640 camera support > > > > > > > > > > > > Signed-off-by: Frank Li > > > > > > --- > > > > > > Changes in v4: > > > > > > - remove imx93 driver support since have not camera sensor module to do test now. > > > > > > Add it later > > > > > > - Add new patch > > > > > > media: v4l2-common: Add helper function v4l_get_required_align_by_bpp() > > > > > > - See each patche's change log for detail. > > > > > > - Link to v3: https://lore.kernel.org/r/20250708-imx8qxp_pcam-v3-0-c8533e405df1@nxp.com > > > > > > > > > > > > Changes in v3: > > > > > > - replace CSI with CPI. > > > > > > - detail change see each patch's change logs > > > > > > - Link to v2: https://lore.kernel.org/r/20250703-imx8qxp_pcam-v2-0-188be85f06f1@nxp.com > > > > > > > > > > > > Changes in v2: > > > > > > - remove patch media: nxp: isi: add support for UYVY8_2X8 and YUYV8_2X8 bus codes > > > > > > because pcif controller convert 2x8 to 1x16 to match isi's input > > > > > > - rename comaptible string to fsl,imx8qxp-pcif > > > > > > - See each patches's change log for detail > > > > > > - Link to v1: https://lore.kernel.org/r/20250630-imx8qxp_pcam-v1-0-eccd38d99201@nxp.com > > > > > > > > > > > > --- > > > > > > Alice Yuan (2): > > > > > > dt-bindings: media: add i.MX parallel CPI support > > > > > > media: nxp: add V4L2 subdev driver for camera parallel interface (CPI) > > > > > > > > > > > > Frank Li (3): > > > > > > media: v4l2-common: Add helper function v4l_get_required_align_by_bpp() > > > > > > arm64: dts: imx8: add camera parallel interface (CPI) node > > > > > > arm64: dts: imx8qxp-mek: add parallel ov5640 camera support > > > > > > > > > > > > .../devicetree/bindings/media/fsl,imx93-pcif.yaml | 126 ++++ > > > > > > MAINTAINERS | 2 + > > > > > > arch/arm64/boot/dts/freescale/Makefile | 3 + > > > > > > arch/arm64/boot/dts/freescale/imx8-ss-img.dtsi | 13 + > > > > > > .../boot/dts/freescale/imx8qxp-mek-ov5640-cpi.dtso | 83 +++ > > > > > > arch/arm64/boot/dts/freescale/imx8qxp-ss-img.dtsi | 27 + > > > > > > drivers/media/platform/nxp/Kconfig | 11 + > > > > > > drivers/media/platform/nxp/Makefile | 1 + > > > > > > drivers/media/platform/nxp/imx-parallel-cpi.c | 728 +++++++++++++++++++++ > > > > > > include/media/v4l2-common.h | 30 + > > > > > > 10 files changed, 1024 insertions(+) > > > > > > --- > > > > > > base-commit: 37a294c6211bea9deb14bedd2dcce498935cbd4e > > > > > > change-id: 20250626-imx8qxp_pcam-d851238343c3 -- Regards, Laurent Pinchart