Re: [PATCH v2 3/4] arm64: dts: ti: k3-am67a-beagley-ai: Add overlay for IMX219 on CSI0

[Nishanth Menon]

On 06:46-20260515, Jai Luthra wrote:
> RPi v2 Camera (IMX219) is an 8MP camera that can be used with BeagleY AI
> through the 22-pin CSI-RX connectors. Add a DT overlay to enable use of
> this camera sensor through the CSI0 connector.
> 
> Signed-off-by: Jai Luthra <jai.luthra@xxxxxxxxxxxxxxxx>
> ---
>  arch/arm64/boot/dts/ti/Makefile                    |   4 +
>  .../dts/ti/k3-am67a-beagley-ai-csi0-imx219.dtso    | 121 +++++++++++++++++++++

Oh man..
https://lore.kernel.org/linux-arm-kernel/20240702164403.29067-1-afd@xxxxxx/
comes to memory.. Afd - what ever happened to that?

other than the dsi mux, painful to see so much duplication..

Other than that, there is the previous comment (ordering etc, repeated
here). I also suggest people put the url of the board/part that the
overlay is supposed to refer to when creating overlays.. at least, some
sort of canonical link so folks are'nt confused in a future date.


-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D
https://ti.com/opensource