Re: [PATCH v3 2/4] arm64: boot: dts: ti: k3-am62l-wakeup: create label for target-module

[Kendall Willis]

Hi Vignesh,
On 2/5/26 22:41, Vignesh Raghavendra wrote:
> Hi
>
> On 28/01/26 04:09, Kendall Willis wrote:
> > Add label to the target-module node so that it can easily be referenced.
> > The node specifically configures the SYSCONFIG related registers for WKUP
> > UART.
> >
> > Reviewed-by: Dhruva Gole <d-gole@xxxxxx>
> > Signed-off-by: Kendall Willis <k-willis@xxxxxx>
> > ---
> >   arch/arm64/boot/dts/ti/k3-am62l-wakeup.dtsi | 2 +-
> >   1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/arch/arm64/boot/dts/ti/k3-am62l-wakeup.dtsi b/arch/arm64/boot/dts/ti/k3-am62l-wakeup.dtsi
> > index 61bfcdcfc66ea8d802a36ed43cd01fbbf3decc70..e9d638d9ffd3a52aa6e0df70f6003879bc292358 100644
> > --- a/arch/arm64/boot/dts/ti/k3-am62l-wakeup.dtsi
> > +++ b/arch/arm64/boot/dts/ti/k3-am62l-wakeup.dtsi
> > @@ -81,7 +81,7 @@ wkup_i2c0: i2c@2b200000 {
> >   		status = "disabled";
> >   	};
> >   
> > -	target-module@2b300050 {
> > +	wkup_uart0_interconnect: target-module@2b300050 {
>
> This is not an interconnect node, so the label is a bit misleading, just
> use something like wkup_uart_target or something similar to how these
> are labeled in arch/arm/boot/dts/ti/

Will fix this in v4, thanks for pointing this out.

Best,
Kendall

> >   		compatible = "ti,sysc-omap2", "ti,sysc";
> >   		reg = <0x00 0x2b300050 0x00 0x4>,
> >   		      <0x00 0x2b300054 0x00 0x4>,