lkml.org 
[lkml]   [2022]   [Jun]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH] arm64: dts: rockchip: Assign RK3399 VDU clock rate
From
Date
Le mardi 07 juin 2022 à 14:15 -0700, Brian Norris a écrit :
> Before commit 9998943f6dfc ("media: rkvdec: Stop overclocking the
> decoder"), the rkvdec driver was forcing the VDU clock rate. After that
> commit, we rely on the default clock rate. That rate works OK on many
> boards, with the default PLL settings (CPLL is 800MHz, VDU dividers
> leave it at 400MHz); but some boards change PLL settings.
>
> Assign the expected default clock rate explicitly, so that the rate is
> consistent, regardless of PLL configuration.
>
> This was particularly broken on RK3399 Gru Scarlet systems, where the
> rk3399-gru-scarlet.dtsi assigns PLL_CPLL to 1.6 GHz, and so the VDU
> clock ends up at 800 MHz (twice the expected rate), and causes video
> artifacts and other issues.
>
> Note: I assign the clock rate in the clock controller instead of the
> vdec node, because there are multiple nodes that use this clock, and per
> the clock.yaml specification:
>
> Configuring a clock's parent and rate through the device node that
> consumes the clock can be done only for clocks that have a single
> user. Specifying conflicting parent or rate configuration in multiple
> consumer nodes for a shared clock is forbidden.
>
> Configuration of common clocks, which affect multiple consumer devices
> can be similarly specified in the clock provider node.
>
> Fixes: 9998943f6dfc ("media: rkvdec: Stop overclocking the decoder")
> Cc: <stable@vger.kernel.org>
> Signed-off-by: Brian Norris <briannorris@chromium.org>

Reviewed-by: Nicolas Dufresne <nicolas.dufresne@collabora.com>

My only doubt was if you really needed to duplicate that setting into gru-
scarlet.dtsi, but I've simply assumed the answer is yes, and that you already
checked that.

> ---
> This is a candidate for 5.19 IMO, since commit 9998943f6dfc landed in
> 5.19-rc1 and is being queued up for -stable as we speak.
>
> arch/arm64/boot/dts/rockchip/rk3399-gru-scarlet.dtsi | 4 +++-
> arch/arm64/boot/dts/rockchip/rk3399.dtsi | 6 ++++--
> 2 files changed, 7 insertions(+), 3 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/rockchip/rk3399-gru-scarlet.dtsi b/arch/arm64/boot/dts/rockchip/rk3399-gru-scarlet.dtsi
> index 913d845eb51a..1977103a5ef4 100644
> --- a/arch/arm64/boot/dts/rockchip/rk3399-gru-scarlet.dtsi
> +++ b/arch/arm64/boot/dts/rockchip/rk3399-gru-scarlet.dtsi
> @@ -376,7 +376,8 @@ &cru {
> <&cru ACLK_VIO>,
> <&cru ACLK_GIC_PRE>,
> <&cru PCLK_DDR>,
> - <&cru ACLK_HDCP>;
> + <&cru ACLK_HDCP>,
> + <&cru ACLK_VDU>;
> assigned-clock-rates =
> <600000000>, <1600000000>,
> <1000000000>,
> @@ -388,6 +389,7 @@ &cru {
> <400000000>,
> <200000000>,
> <200000000>,
> + <400000000>,
> <400000000>;
> };
>
> diff --git a/arch/arm64/boot/dts/rockchip/rk3399.dtsi b/arch/arm64/boot/dts/rockchip/rk3399.dtsi
> index fbd0346624e6..9d5b0e8c9cca 100644
> --- a/arch/arm64/boot/dts/rockchip/rk3399.dtsi
> +++ b/arch/arm64/boot/dts/rockchip/rk3399.dtsi
> @@ -1462,7 +1462,8 @@ cru: clock-controller@ff760000 {
> <&cru HCLK_PERILP1>, <&cru PCLK_PERILP1>,
> <&cru ACLK_VIO>, <&cru ACLK_HDCP>,
> <&cru ACLK_GIC_PRE>,
> - <&cru PCLK_DDR>;
> + <&cru PCLK_DDR>,
> + <&cru ACLK_VDU>;
> assigned-clock-rates =
> <594000000>, <800000000>,
> <1000000000>,
> @@ -1473,7 +1474,8 @@ cru: clock-controller@ff760000 {
> <100000000>, <50000000>,
> <400000000>, <400000000>,
> <200000000>,
> - <200000000>;
> + <200000000>,
> + <400000000>;
> };
>
> grf: syscon@ff770000 {

\
 
 \ /
  Last update: 2022-06-08 03:39    [W:0.057 / U:0.588 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site