Received-SPF: pass (google.com: domain of linux-kernel+bounces-215065-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) client-ip=2604:1380:45e3:2400::1;
Date: Fri, 14 Jun 2024 20:29:46 +0530
From: Siddharth Vadapalli <s-vadapalli@ti.com>
To: Roger Quadros <rogerq@kernel.org>
CC: Siddharth Vadapalli <s-vadapalli@ti.com>, <nm@ti.com>, <vigneshr@ti.com>,
        <afd@ti.com>, <kristo@kernel.org>, <robh@kernel.org>,
        <krzk+dt@kernel.org>, <conor+dt@kernel.org>,
        <devicetree@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
        <linux-arm-kernel@lists.infradead.org>, <u-kumar1@ti.com>,
        <danishanwar@ti.com>, <srk@ti.com>
Subject: Re: [PATCH v6 5/8] arm64: dts: ti: k3-j722s: Move MAIN domain
 overrides to k3-j722s-main.dtsi
Message-ID: <8b06ba3e-f4b2-4654-aaaf-2f15816fff07@ti.com>
References: <20240612132409.2477888-1-s-vadapalli@ti.com>
 <20240612132409.2477888-6-s-vadapalli@ti.com>
 <d4c33ee4-1e91-40e5-90c8-bd793bcf34ff@kernel.org>
Precedence: bulk
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline
In-Reply-To: <d4c33ee4-1e91-40e5-90c8-bd793bcf34ff@kernel.org>

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.