Re: [PATCH 2/2] arm64: dts: rk3568-u-boot: Drop combphy0 assigned-clocks/rates

[Kever Yang <kever.yang@rock-chips.com>]

Hi Vasily,

On 2023/3/8 05:34, Vasily Khoruzhick wrote:
> On Tue, Mar 7, 2023 at 1:04 PM Mark Kettenis <mark.kettenis@xs4all.nl> wrote:
>
>> And implement support for the CLK_PCIEPHYn_REF clocks in
>> drivers/clk/rockchip/clk_rk3568.c:rk3568_pmuclk_set_rate()?
>>
>> Yes, I'd say so.
>>
>> If U-Boot doesn't actually need these clocks to run at the frequency
>> provided by assigned-clock-rates, that could be as simple as returning
>> 0 for these clocks.
> Sounds good. I'll send v2 that reverts 5bec4b0de785 and implements
> stubs for CLK_PCIEPHYn_REF clocks.
>
> FWIW, I tried booting linux-6.2.2 with u-boot dtb and these clocks
> dropped, and USB works just fine.

The assigned-clock is a software set default value, the hardware can 
works fine because

these clocks have a correct default hardware value.

So for the PHY,  it works in U-Boot without assigned clock in most case.

Returing 0 for those clocks is also OK.


Thanks,

- Kever

>
>>>>> commit 5bec4b0de785 ("arm64: dts: rk356x-u-boot: Drop combphy1 assigned-clocks/rates")
>>>>> dropped these properties for combphy1, so let's drop them for combphy0 as well.
>>>>>
>>>>> Signed-off-by: Vasily Khoruzhick <anarsoul@gmail.com>
>>>>> ---
>>>>>   arch/arm/dts/rk3568-evb-u-boot.dtsi     |  1 +
>>>>>   arch/arm/dts/rk3568-rock-3a-u-boot.dtsi |  1 +
>>>>>   arch/arm/dts/rk3568-u-boot.dtsi         | 11 +++++++++++
>>>>>   3 files changed, 13 insertions(+)
>>>>>   create mode 100644 arch/arm/dts/rk3568-u-boot.dtsi
>>>>>
>>>>> diff --git a/arch/arm/dts/rk3568-evb-u-boot.dtsi b/arch/arm/dts/rk3568-evb-u-boot.dtsi
>>>>> index 17503d3d27..77430da3ba 100644
>>>>> --- a/arch/arm/dts/rk3568-evb-u-boot.dtsi
>>>>> +++ b/arch/arm/dts/rk3568-evb-u-boot.dtsi
>>>>> @@ -4,6 +4,7 @@
>>>>>    */
>>>>>
>>>>>   #include "rk356x-u-boot.dtsi"
>>>>> +#include "rk3568-u-boot.dtsi"
>>>>>
>>>>>   / {
>>>>>        chosen {
>>>>> diff --git a/arch/arm/dts/rk3568-rock-3a-u-boot.dtsi b/arch/arm/dts/rk3568-rock-3a-u-boot.dtsi
>>>>> index ed47efa44b..44cf33ed4b 100644
>>>>> --- a/arch/arm/dts/rk3568-rock-3a-u-boot.dtsi
>>>>> +++ b/arch/arm/dts/rk3568-rock-3a-u-boot.dtsi
>>>>> @@ -5,6 +5,7 @@
>>>>>    */
>>>>>
>>>>>   #include "rk356x-u-boot.dtsi"
>>>>> +#include "rk3568-u-boot.dtsi"
>>>>>
>>>>>   / {
>>>>>        chosen {
>>>>> diff --git a/arch/arm/dts/rk3568-u-boot.dtsi b/arch/arm/dts/rk3568-u-boot.dtsi
>>>>> new file mode 100644
>>>>> index 0000000000..c6776666df
>>>>> --- /dev/null
>>>>> +++ b/arch/arm/dts/rk3568-u-boot.dtsi
>>>>> @@ -0,0 +1,11 @@
>>>>> +// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
>>>>> +/*
>>>>> + * Copyright (c) 2023 Vasily Khoruzhick <anarsoul@gmail.com>
>>>>> + */
>>>>> +
>>>>> +#include "rockchip-u-boot.dtsi"
>>>>> +
>>>>> +&combphy0 {
>>>>> +     /delete-property/ assigned-clocks;
>>>>> +     /delete-property/ assigned-clock-rates;
>>>>> +};
>>>>> --
>>>>> 2.39.2
>>>>>
>>>>>