[PATCH 2/2] arm64: dts: rk3568-u-boot: Drop combphy0 assigned-clocks/rates
Mark Kettenis
mark.kettenis at xs4all.nl
Tue Mar 7 22:04:27 CET 2023
> From: Vasily Khoruzhick <anarsoul at gmail.com>
> Date: Tue, 7 Mar 2023 12:34:48 -0800
>
> On Tue, Mar 7, 2023 at 11:53 AM Mark Kettenis <mark.kettenis at xs4all.nl> wrote:
>
> > That is probably the wrong approach. It should be possible to boot an
> > OS with the device tree provided by U-Boot. Removing these properties
> > means the OS doesn't see them either. But if the assigned-clocks
> > property isn't needed it wouldn't be there would it?
>
> Well, should we revert 5bec4b0de785 in this case?
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.
> > > 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 at 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 at gmail.com>
> > > + */
> > > +
> > > +#include "rockchip-u-boot.dtsi"
> > > +
> > > +&combphy0 {
> > > + /delete-property/ assigned-clocks;
> > > + /delete-property/ assigned-clock-rates;
> > > +};
> > > --
> > > 2.39.2
> > >
> > >
>
More information about the U-Boot
mailing list