Re: [PATCH 6/8] arm64: dts: freescale: imx8mm-verdin: Add Toradex OV5640 CSI Cameras

[Kieran Bingham <kieran.bingham@ideasonboard.com>]

Hi Francesco,

Quoting Francesco Dolcini (2026-07-22 11:59:14)
> Hello Kieran,
> 
> On Wed, Jul 22, 2026 at 11:30:29AM +0100, Kieran Bingham wrote:
> > Quoting Ernest Van Hoecke (2026-07-22 10:27:32)
> > > On Mon, Jul 20, 2026 at 02:16:46PM -0400, Frank Li wrote:
> > > > On Mon, Jul 13, 2026 at 05:06:27PM +0200, Ernest Van Hoecke wrote:
> > > > > From: Ernest Van Hoecke <ernest.vanhoecke@toradex.com>
> > > > >
> > > > > Add device tree overlays for the Toradex OV5640 CSI Camera on Verdin CSI_1.
> > > > >
> > > > > The default overlay describes the current CSI Camera Set 5MP OV5640 with a
> > > > > 27 MHz on-board oscillator. Add a separate 24 MHz overlay for the legacy
> > > > > camera module.
> > > > >
> > > > > Link: https://developer.toradex.com/hardware/accessories/cameras/csi-camera-module-5mp-ov5640-arducam
> > > > > Link: https://www.toradex.com/accessories/csi-camera-ov5640
> > > > > Link: https://developer.toradex.com/hardware/legacy-products/other/csi-camera-module-5mp-ov5640/
> > > > > Signed-off-by: Ernest Van Hoecke <ernest.vanhoecke@toradex.com>
> > > > > ---
> > > > >  arch/arm64/boot/dts/freescale/Makefile             |  6 ++
> > > > >  .../dts/freescale/imx8mm-verdin-ov5640-24mhz.dtso  | 17 +++++
> > > > >  .../boot/dts/freescale/imx8mm-verdin-ov5640.dtsi   | 78 ++++++++++++++++++++++
> > > > >  .../boot/dts/freescale/imx8mm-verdin-ov5640.dtso   | 18 +++++
> > > > >  4 files changed, 119 insertions(+)
> ...
> > I think with the Toradex ecosystem there would be some value in
> > supporting or helping with the ongoing dt-connectors or dt-addons topics
> > so that we can abstract the hardware which is being 'added'.
> > 
> > I think it's important that we tackle the problem of combinatorial
> > explosions of overlays when we can add a component to multiple
> > platforms.
> > 
> > For example, your OV5640 camera could be added to many different boards
> > - and each board could have many different cameras - in different ports.
> > 
> > We should not be copy/pasting overlays for each combination, or we'll
> > have 'thousands' of identical overlays.
> 
> I see your point, and I agree that it would be valuable to move this
> topic forward. I will raise it internally at Toradex and see how we can
> contribute.
> 
> At the same time, quoting Marex (https://lore.kernel.org/all/dec2a7f6-80fd-4692-8936-969f8837a555@nabladev.com/):
> 
>  | DT connectors have been discussed for the last 10 or so years and three 
>  | is still no real progress.
>  | 
>  | I would be happy to send a follow up patchset which would convert the 
>  | DTOs to whatever connector implementation format lands in the future, 
>  | but I am concerned that waiting for DT connectors will block this 
>  | patchset from landing for a long time.

Indeed, it's been around for a long time - and yet the problem persists,
and that's why it worries me here. It won't be fixed until someone works
on it or supports the development.

Marex is an independent community developer - without someone buying products
to support his development. It wouldn't be fair to push back on him
personnally. He has no mechanism to support the development required.

--
Kieran


> 
> 
> Francesco
>