[PATCH] arm64: dts: rockchip: Correct clock-output-names for rk817

Heiko Stübner heiko at sntech.de
Wed Feb 16 12:42:09 PST 2022


Hi,

Am Mittwoch, 16. Februar 2022, 17:58:37 CET schrieb Chris Morgan:
> From: Chris Morgan <macromorgan at hotmail.com>
> 
> According to the datasheet for the rk817 there is only a single output
> clock for the PMIC. I don't believe this is causing any harm though
> and testing this change doesn't appear to have any noticeable effects.

hmm, looking at the go2's schematics, I'd disagree:

- yes, the rk817 has only one clock output
- but no, it is used for the clkout_32k signal, which gets piped back into
  the SoC. As that clock dynamically appears at some unspecified point
  when the rk817 probes, the clock controller expects it to be named
  xin32k to bind into its clock-tree.

The driver (drivers/clk/clk-rk808.c) really puts that rk817-clk32k at
position 2 but I'm not sure if that simply papers over some implementation


Heiko

> Signed-off-by: Chris Morgan <macromorgan at hotmail.com>
> ---
>  arch/arm64/boot/dts/rockchip/rk3326-odroid-go2.dts | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/arch/arm64/boot/dts/rockchip/rk3326-odroid-go2.dts b/arch/arm64/boot/dts/rockchip/rk3326-odroid-go2.dts
> index ea0695b51ecd..5b23e607876e 100644
> --- a/arch/arm64/boot/dts/rockchip/rk3326-odroid-go2.dts
> +++ b/arch/arm64/boot/dts/rockchip/rk3326-odroid-go2.dts
> @@ -296,7 +296,7 @@ rk817: pmic at 20 {
>  		reg = <0x20>;
>  		interrupt-parent = <&gpio0>;
>  		interrupts = <RK_PB2 IRQ_TYPE_LEVEL_LOW>;
> -		clock-output-names = "rk808-clkout1", "xin32k";
> +		clock-output-names = "rk808-clkout1";
>  		clock-names = "mclk";
>  		clocks = <&cru SCLK_I2S1_OUT>;
>  		pinctrl-names = "default";
> 







More information about the Linux-rockchip mailing list