[PATCH v2 10/10] arch: arm: dts: k3-am62-sk-u-boot: Add missing "bootph-all" property to phy_gmii_sel node
Chintan Vankar
c-vankar at ti.com
Mon May 20 08:04:58 CEST 2024
On 25/04/24 18:06, Chintan Vankar wrote:
>
>
> 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/
>
Since I have posted patch which adds bootph-all property to
"k3-am62x-sk-common.dtsi" at here:
https://lore.kernel.org/r/20240430085048.3143665-1-c-vankar@ti.com/
and it has no open comments and this series
also does not have any open comments, so can it be merged ?
>>>
>>> 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;
>>> +};
>>
More information about the U-Boot
mailing list