Re: [PATCH v2 4/4] arm64: dts: qcom: talos-evk-camera: Add DT overlay

[Vladimir Zapolskiy]

On 1/7/26 05:05, Wenmeng Liu wrote:
> On 1/7/2026 2:23 AM, Dmitry Baryshkov wrote:
> > On Tue, Jan 06, 2026 at 05:39:56PM +0800, Wenmeng Liu wrote:
> > > Enable IMX577 via CCI on Taloss EVK Core Kit.
> > >
> > > The Talos EVK board does not include a camera sensor
> > > by default, this overlay reflects the possibility of
> > > attaching an optional camera sensor.
> > > For this reason, the camera sensor configuration is
> > > placed in talos-evk-camera.dtso, rather than
> > > modifying the base talos-evk.dts.
> > >
> > > Signed-off-by: Wenmeng Liu <wenmeng.liu@xxxxxxxxxxxxxxxx>
> > > ---
> > >    arch/arm64/boot/dts/qcom/Makefile              |  2 +
> > >    arch/arm64/boot/dts/qcom/talos-evk-camera.dtso | 63 ++++++++++++++++++++++++++
> >
> > Is it possible to attach other sensors? If so, overlay name should
> > depicit which sensors are attached (compare this to the RBn boards where
> > specifying "vision kit" defines all sensors attached to the device).
>
> Okay, we previously had a discussion on this. I will rename the file to
> talos-evk-camera-imx577.dtso.

Other camera or display .dtso names commonly repeat the name given by
the vendor, and the bare minimum is to name it the commit message or in
the code. Is it Arducam 12.3MP IMX577 Mini Camera Module or something else?

--
Best wishes,
Vladimir