Date:   Mon, 30 Jan 2023 19:57:28 -0600
From:   Bryan Brattlof <bb@ti.com>
To:     "Raghavendra, Vignesh" <vigneshr@ti.com>
CC:     Nishanth Menon <nm@ti.com>, Tero Kristo <kristo@kernel.org>,
        Rob Herring <robh+dt@kernel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        ARM Linux Mailing List <linux-arm-kernel@lists.infradead.org>,
        Device Tree Mailing List <devicetree@vger.kernel.org>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        Keerthy <j-keerthy@ti.com>
Subject: Re: [PATCH v2 2/6] arm64: dts: ti: k3-am62-wakeup: add VTM node
Message-ID: <20230131015728.347i3cttyxtitcwi@bryanbrattlof.com>
References: <20230120003051.9100-1-bb@ti.com>
 <20230120003051.9100-3-bb@ti.com>
 <0172f045-4bd0-ec5e-35a0-efc80da5ace0@ti.com>
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Disposition: inline
In-Reply-To: <0172f045-4bd0-ec5e-35a0-efc80da5ace0@ti.com>
Precedence: bulk

Hi Vignesh!

On January 28, 2023 thus sayeth Raghavendra, Vignesh:
> Hi Bryan,
> 
> On 1/20/2023 6:00 AM, Bryan Brattlof wrote:
> > @@ -94,10 +95,13 @@ cbass_wakeup: bus@2b000000 {
> >  			compatible = "simple-bus";
> >  			#address-cells = <2>;
> >  			#size-cells = <2>;
> > -			ranges = <0x00 0x2b000000 0x00 0x2b000000 0x00 0x00300400>, /* Peripheral Window */
> > +			ranges = <0x00 0x00b00000 0x00 0x00b00000 0x00 0x00002400>, /* VTM */
> 
> You would need to update unit address in node name as well:
> 			cbass_wakeup: bus@b00000 {

Ah! I missed that. I'll get this fixed

~Bryan