Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Mon, 3 Apr 2023 09:09:47 -0500
From:   Bryan Brattlof <bb@ti.com>
To:     Christian Gmeiner <christian.gmeiner@gmail.com>
CC:     "Raghavendra, Vignesh" <vigneshr@ti.com>,
        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: <20230403140947.kj7t5lq2puketgkt@bryanbrattlof.com>
References: <20230120003051.9100-1-bb@ti.com>
 <20230120003051.9100-3-bb@ti.com>
 <0172f045-4bd0-ec5e-35a0-efc80da5ace0@ti.com>
 <20230131015728.347i3cttyxtitcwi@bryanbrattlof.com>
 <CAH9NwWeDSFrF61UC=NENTvQpUU_erGtKrhUzBVQWvA3KjbH_Fg@mail.gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Disposition: inline
In-Reply-To: <CAH9NwWeDSFrF61UC=NENTvQpUU_erGtKrhUzBVQWvA3KjbH_Fg@mail.gmail.com>
Precedence: bulk

Hi Christian!

On April  3, 2023 thus sayeth Christian Gmeiner:
> > 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
> 
> Any update on this? Would be great if these changes could make it into
> the next merge window.
> 

You got me :) I let this fall off my radar... I'll send out v3 by today 
or tomorrow to restart the review process.

Thank you for reviewing and for the reminder Christian
~Bryan