[PATCH] ARM64: dts: rockchip: assign default rates for core rk3399 clocks
Doug Anderson
dianders at chromium.org
Fri May 13 14:20:42 PDT 2016
Hi,
On Fri, May 13, 2016 at 1:50 PM, Brian Norris <briannorris at chromium.org> wrote:
> From: Xing Zheng <zhengxing at rock-chips.com>
>
> These clocks are all core clocks used by many blocks/peripherals, many
> of whose drivers don't set their clock rates at all. Let's assign
> reasonable default clock rates for these core clocks, so that these
> peripherals get something reasonable by default, and also so that if
> child devices want to select a clock rate themselves, their muxes have
> some reasonable parent clock rates to branch off of (rather than just
> the boot-time defaults).
>
> This helps the eMMC PHY, for one, to get a reasonable ACLK rate.
>
> Signed-off-by: Xing Zheng <zhengxing at rock-chips.com>
> Signed-off-by: Brian Norris <briannorris at chromium.org>
> ---
> arch/arm64/boot/dts/rockchip/rk3399.dtsi | 16 ++++++++++++++++
> 1 file changed, 16 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/rockchip/rk3399.dtsi b/arch/arm64/boot/dts/rockchip/rk3399.dtsi
> index 46f325a143b0..6fa9cc332482 100644
> --- a/arch/arm64/boot/dts/rockchip/rk3399.dtsi
> +++ b/arch/arm64/boot/dts/rockchip/rk3399.dtsi
> @@ -478,6 +478,22 @@
> reg = <0x0 0xff760000 0x0 0x1000>;
> #clock-cells = <1>;
> #reset-cells = <1>;
> + assigned-clocks =
> + <&cru PLL_GPLL>, <&cru PLL_CPLL>,
> + <&cru PLL_NPLL>,
> + <&cru ACLK_PERIHP>, <&cru HCLK_PERIHP>,
> + <&cru PCLK_PERIHP>,
> + <&cru ACLK_PERILP0>, <&cru HCLK_PERILP0>,
> + <&cru PCLK_PERILP0>,
> + <&cru HCLK_PERILP1>, <&cru PCLK_PERILP1>;
> + assigned-clock-rates =
> + <594000000>, <800000000>,
> + <1000000000>,
> + <150000000>, <75000000>,
> + <37500000>,
> + <100000000>, <100000000>,
> + <50000000>,
> + <100000000>, <50000000>;
I agree that
* this is sane information to have in this node (like in rk3288)
* the rates look sane (similar to rk3288 but two are double)
* these rates match what I see in Rockchip's kernel
So:
Reviewed-by: Douglas Anderson <dianders at chromium.org>
More information about the Linux-rockchip
mailing list