[v2] arm64: dts: ti: k3-am62x-sk-common: Add bootph-all property in cpsw_mac_syscon node

[PATCH v2] arm64: dts: ti: k3-am62x-sk-common: Add bootph-all property in cpsw_mac_syscon node

Posted by Chintan Vankar 1 month, 2 weeks ago

Add bootph-all property in CPSW MAC's eFuse node cpsw_mac_syscon.

Signed-off-by: Chintan Vankar <c-vankar@ti.com>
---

This patch is based on linux-next tagged next-20241011.

 arch/arm64/boot/dts/ti/k3-am62x-sk-common.dtsi | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/arch/arm64/boot/dts/ti/k3-am62x-sk-common.dtsi b/arch/arm64/boot/dts/ti/k3-am62x-sk-common.dtsi
index 44ff67b6bf1e..82d34dfb91ed 100644
--- a/arch/arm64/boot/dts/ti/k3-am62x-sk-common.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am62x-sk-common.dtsi
@@ -303,6 +303,10 @@ AM62X_MCU_IOPAD(0x028, PIN_OUTPUT, 0) /* (C5/C6) WKUP_UART0_TXD */
 	};
 };
 
+&cpsw_mac_syscon {
+	bootph-all;
+};
+
 &wkup_uart0 {
 	/* WKUP UART0 is used by DM firmware */
 	bootph-pre-ram;
-- 
2.34.1

Re: [PATCH v2] arm64: dts: ti: k3-am62x-sk-common: Add bootph-all property in cpsw_mac_syscon node

Posted by Vignesh Raghavendra 4 weeks, 1 day ago


On 11/10/24 16:32, Chintan Vankar wrote:
> Add bootph-all property in CPSW MAC's eFuse node cpsw_mac_syscon.
> 

Why?

Please make sure commit message is verbose enough to say why the change
is needed vs what that change is (latter is obvious lookng at the diff)

> Signed-off-by: Chintan Vankar <c-vankar@ti.com>
> ---
> 
> This patch is based on linux-next tagged next-20241011.
> 
>  arch/arm64/boot/dts/ti/k3-am62x-sk-common.dtsi | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/ti/k3-am62x-sk-common.dtsi b/arch/arm64/boot/dts/ti/k3-am62x-sk-common.dtsi
> index 44ff67b6bf1e..82d34dfb91ed 100644
> --- a/arch/arm64/boot/dts/ti/k3-am62x-sk-common.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-am62x-sk-common.dtsi
> @@ -303,6 +303,10 @@ AM62X_MCU_IOPAD(0x028, PIN_OUTPUT, 0) /* (C5/C6) WKUP_UART0_TXD */
>  	};
>  };
>  
> +&cpsw_mac_syscon {
> +	bootph-all;
> +};
> +
>  &wkup_uart0 {
>  	/* WKUP UART0 is used by DM firmware */
>  	bootph-pre-ram;

-- 
Regards
Vignesh

Re: [PATCH v2] arm64: dts: ti: k3-am62x-sk-common: Add bootph-all property in cpsw_mac_syscon node

Posted by Chintan Vankar 1 week, 6 days ago


On 28/10/24 22:06, Vignesh Raghavendra wrote:
> 
> 
> On 11/10/24 16:32, Chintan Vankar wrote:
>> Add bootph-all property in CPSW MAC's eFuse node cpsw_mac_syscon.
>>
> 
> Why?
> 
> Please make sure commit message is verbose enough to say why the change
> is needed vs what that change is (latter is obvious lookng at the diff)
> 

Sure Vignesh. I will update the commit message accordingly in next
version.

>> Signed-off-by: Chintan Vankar <c-vankar@ti.com>
>> ---
>>
>> This patch is based on linux-next tagged next-20241011.
>>
>>   arch/arm64/boot/dts/ti/k3-am62x-sk-common.dtsi | 4 ++++
>>   1 file changed, 4 insertions(+)
>>
>> diff --git a/arch/arm64/boot/dts/ti/k3-am62x-sk-common.dtsi b/arch/arm64/boot/dts/ti/k3-am62x-sk-common.dtsi
>> index 44ff67b6bf1e..82d34dfb91ed 100644
>> --- a/arch/arm64/boot/dts/ti/k3-am62x-sk-common.dtsi
>> +++ b/arch/arm64/boot/dts/ti/k3-am62x-sk-common.dtsi
>> @@ -303,6 +303,10 @@ AM62X_MCU_IOPAD(0x028, PIN_OUTPUT, 0) /* (C5/C6) WKUP_UART0_TXD */
>>   	};
>>   };
>>   
>> +&cpsw_mac_syscon {
>> +	bootph-all;
>> +};
>> +
>>   &wkup_uart0 {
>>   	/* WKUP UART0 is used by DM firmware */
>>   	bootph-pre-ram;
>

Re: [PATCH v2] arm64: dts: ti: k3-am62x-sk-common: Add bootph-all property in cpsw_mac_syscon node

Posted by Siddharth Vadapalli 1 month, 1 week ago

On Fri, Oct 11, 2024 at 04:32:07PM +0530, Chintan Vankar wrote:
> Add bootph-all property in CPSW MAC's eFuse node cpsw_mac_syscon.
> 
> Signed-off-by: Chintan Vankar <c-vankar@ti.com>

Reviewed-by: Siddharth Vadapalli <s-vadapalli@ti.com>

Regards,
Siddharth.