[PATCH v2 10/10] arch: arm: dts: k3-am62-sk-u-boot: Add missing "bootph-all" property to phy_gmii_sel node

[Chintan Vankar]

On 25/04/24 18:01, Roger Quadros wrote:
> 
> 
> On 25/04/2024 15:08, Chintan Vankar wrote:
>> Add "bootph-all" property to CPSW MAC's PHY node phy_gmii_sel.
>>
>> Signed-off-by: Chintan Vankar <c-vankar at ti.com>
>> ---
>>
>> Changes from v1 to v2:
>> - This patch is newly added in this series to enable CPSW MAC's PHY
>>    node "phy_gmii_sel". As per discussion at here:
>>    https://lore.kernel.org/r/20240112130127.rvvrhz7p4vmlyalh@smother/
>>    changes made by this patch can be dropped in the future when the
>>    DT-Sync is performed with am62-main.dtsi containing this change in
>>    the Linux DT which will match U-Boot's DT.
> 
> I don't think bootph-all exists in am62-main.dtsi. It should come from
> board.dts
> 

Yes, I am having the same discussion at here:
https://lore.kernel.org/all/c13ac165-7cbd-4e53-914e-8c6bc28250e3@ti.com/

>>
>>   arch/arm/dts/k3-am625-sk-u-boot.dtsi | 4 ++++
>>   1 file changed, 4 insertions(+)
>>
>> diff --git a/arch/arm/dts/k3-am625-sk-u-boot.dtsi b/arch/arm/dts/k3-am625-sk-u-boot.dtsi
>> index fa778b0ff4..e9a1afde95 100644
>> --- a/arch/arm/dts/k3-am625-sk-u-boot.dtsi
>> +++ b/arch/arm/dts/k3-am625-sk-u-boot.dtsi
>> @@ -46,3 +46,7 @@
>>   &cpsw_port2 {
>>   	status = "disabled";
>>   };
>> +
>> +&phy_gmii_sel {
>> +	bootph-all;
>> +};
>