Re: [PATCH v2 2/3] arm64: dts: ti: Introduce J742S2 SoC family

Nishanth Menon
2024-07-31T15:36:10Z

On 20:48-20240731, Manorit Chawdhry wrote:
[...]
> Yeah, was gonna go with that file only right now, but now would I have
>
> - k3-j784s4-mcu-wakeup-common.dtsi ( this is not required at this stage,
> but ig for consistency better to now itself )
> - k3-j784s4-main-common.dtsi ( all dsps excluding c7x_3 )
> - k3-j784s4-thermal-common.dtsi ( not required again but consistency )
> - k3-j784s4-common.dtsi ( all this won't have the cpu but will have all
> other ranges including for the last dsp and all )

We already use k3-am62p-j722s-common-main.dtsi so that people don't
ask what is this common to and track down via grep who is including
what. I'd rather us follow that convention for now that debate is done
with and integrated in master. I don't see a strong reason yet to change
the same. Again, everyone will have an subjective opinion, I prefer to
follow existing convention that is established on topics that are
cosmetic.

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