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

[Vignesh Raghavendra <vigneshr@ti.com>]

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@ti.com>
> Signed-off-by: Kendall Willis <k-willis@ti.com>
> ---
>  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/

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

-- 
Regards
Vignesh
https://ti.com/opensource