.../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(+)
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 <Frank.Li@nxp.com>
---
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
Best regards,
--
Frank Li <Frank.Li@nxp.com>
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. 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 <Frank.Li@nxp.com> > --- > 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
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? Frank > > 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 <Frank.Li@nxp.com> > > --- > > 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
Hi Frank, 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. 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 :-/ > > 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 <Frank.Li@nxp.com> > > > --- > > > 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
On Tue, Sep 02, 2025 at 02:39:20PM +0200, Laurent Pinchart wrote: > Hi Frank, > > 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. Frank > > > > 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 <Frank.Li@nxp.com> > > > > --- > > > > 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
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: > > Hi Frank, > > > > 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. How do we move forward? Frank > > Frank > > > > > > > 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 <Frank.Li@nxp.com> > > > > > --- > > > > > 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
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 <Frank.Li@nxp.com> > > > > > > --- > > > > > > 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
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?
Best regards
Frank Li
>
> > > > > > 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 <Frank.Li@nxp.com>
> > > > > > > ---
> > > > > > > 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
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.
Thanks in advance,
Cheers,
Rui
>
> Best regards
> Frank Li
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. Frank > > Thanks in advance, > Cheers, > Rui > > > > Best regards > > Frank Li
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! Frank > > Frank > > > > Thanks in advance, > > Cheers, > > Rui > > > > > > Best regards > > > Frank Li
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.
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.
Thanks again and Cheers,
Rui
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
© 2016 - 2026 Red Hat, Inc.