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

[Kendall Willis <k-willis@ti.com>]

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@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/
> 

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>,
>>
>