Re: [PATCH v3 4/4] arm64: dts: freescale: imx8mp-verdin: Add mezzanine OV5640 overlays

[Ernest Van Hoecke]

Hi Frank,

On Fri, Oct 23, 2026 at 03:23:12PM -0500, Frank Li wrote:
> On Fri, Oct 23, 2026 at 03:51:17PM +0200, Ernest Van Hoecke wrote:
> > On Thu, Oct 22, 2026 at 05:20:48PM -0400, Frank Li wrote:
> > > On Thu, Oct 22, 2026 at 04:31:28PM +0200, Ernest Van Hoecke wrote:
> > > > From: Ernest Van Hoecke <ernest.vanhoecke@xxxxxxxxxxx>
> > > >
> > > > The Verdin iMX8M Plus exposes its second MIPI CSI-2 interface through the
> > > > Verdin iMX8M Plus Mezzanine. Add overlays for using a Toradex OV5640 camera
> > > > on this interface.
> > > >
> > > > The mezzanine can route the camera control signals through two jumper
> > > > configurations. Describe each configuration as a GPIO nexus mapping so the
> > > > camera description remains independent of the physical routing.
> > > >
> > > > The default positions share the native CSI_1 GPIOs and are intended for a
> > > > single camera. The alternate positions permit simultaneous native and
> > > > mezzanine cameras, but conflict with the Atmel MXT touchscreen GPIO.
> > > >
> > > > The current camera set uses a 27 MHz on-board oscillator. The legacy camera
> > > > module uses 24 MHz, so provide variants for both.
> > > >
> > > > Link: https://developer.toradex.com/hardware/verdin-som-family/add-ons/verdin-imx8m-plus-mezzanine/
> > > > Link: https://developer.toradex.com/hardware/accessories/cameras/csi-camera-module-5mp-ov5640-arducam
> > > > Link: https://developer.toradex.com/hardware/legacy-products/other/csi-camera-module-5mp-ov5640/
> > > > Signed-off-by: Ernest Van Hoecke <ernest.vanhoecke@xxxxxxxxxxx>
> > > > ---
> > > >  arch/arm64/boot/dts/freescale/Makefile             | 19 ++++++
> > > >  ...-verdin-mezzanine-ov5640-24mhz-alt-jumpers.dtso | 19 ++++++
> > > >  ...din-mezzanine-ov5640-24mhz-default-jumpers.dtso | 18 +++++
> > > >  ...imx8mp-verdin-mezzanine-ov5640-alt-jumpers.dtso | 38 +++++++++++
> > > >  ...mp-verdin-mezzanine-ov5640-default-jumpers.dtso | 42 ++++++++++++
> > > >  .../freescale/imx8mp-verdin-mezzanine-ov5640.dtsi  | 78 ++++++++++++++++++++++
> > >
> > > why not share with imx8m-verdin-ov5640.dtsi, which is almost the same.
> > >
> > > Frank
> >
> > Hi Frank,
> >
> > Both camera modules have their own oscillator, and the native and
> > Mezzanine connectors have their own supply, so each overlay describes its
> > own clock and regulator nodes. They also use a different I2C bus, CSI
> > receiver and connector. Both overlays are applied together for two
> > cameras (imx8mp-verdin-wifi-dev-dual-ov5640),
> 
> Okay, I see share one imx8m-verdin-ov5640.dtso
> 
> > so their node names and
> > labels must be unique, and a single .dtsi cannot be included twice.
> 
> mezzanine actually connect by another connectors ? so it need new one files
> because current nexus node's limitation.
>

Yes, the Mezzanine has its own camera connector, wired to the second
MIPI CSI-2 receiver of the i.MX8M Plus. Its GPIOs already go through a
connector nexus of their own, defined in the Mezzanine overlays.
 
> Can you make imx8mp-verdin-mezzanine-ov5640.dtsi also standardize, like
> use unified camera_i2c_2, camera_connector2, compre to imx8m-verdin-ov5640.dtso
>

This is possible, but the labels used by imx8m-verdin-ov5640.dtso exist
because that overlay is shared by Verdin iMX8M Mini and Verdin iMX8M Plus.
The Mezzanine is specific to the Verdin iMX8M Plus and connects to its
module-specific pins, which another module could wire differently. New
labels in the SoM dtsi would become ABI with these overlays as their only
user, so I would rather not add them. If a future Verdin module shares
this connector, I will add the labels then.
 
> And avoid dtso include dtso file.  Now perfer only include dtsi file.
> 
> Frank

Okay I will go back to only including dtsi in v4 of this series, and
also for the next SMARC series. Thanks.

Kind regards,
Ernest