On Fri, Jun 14, 2024 at 12:53:37PM +0300, Roger Quadros wrote: > Hi Siddharth, [...] > > --- a/arch/arm64/boot/dts/ti/k3-j722s.dtsi > > +++ b/arch/arm64/boot/dts/ti/k3-j722s.dtsi > > @@ -232,14 +232,3 @@ cbass_wakeup: bus@b00000 { > > > > /* Include J722S specific peripherals */ > > #include "k3-j722s-main.dtsi" > > - > > -/* Main domain overrides */ > > - > > -&inta_main_dmss { > > - ti,interrupt-ranges = <7 71 21>; > > -}; > > - > > -&oc_sram { > > - reg = <0x00 0x70000000 0x00 0x40000>; > > - ranges = <0x00 0x00 0x70000000 0x40000>; > > -}; > These are originally set in k3-am62p-j722s-common-main.dtsi > which cannot be as they are not common to both SoCs. > > Should we be moving the AM62 specific portion into k3-am62p-main.dtsi. > > So they are no longer overrides but SoC specific changes? Roger, Thank you for reviewing the patch. I will remove the above properties within the respective nodes of k3-am62p-j722s-common-main.dtsi and add them to match the respective SoC in k3-{soc}-main.dtsi. Regards, Siddharth.