From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 4F411C3DA45 for ; Sat, 13 Jul 2024 03:53:08 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Transfer-Encoding: Content-Type:Message-ID:References:In-Reply-To:Subject:Cc:To:From:Date: MIME-Version:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=l3cOgi5Y3zeJt1VWYzdIVf8SSuAoPfH0fWAeIPZGLq4=; b=E9fS5v/n6UZRNEfhHB/B4aN/hm QHa1E/Yz+0sWbWt5/roP/15k8usOUbX/Ur6geX41b+jInAHBEETG8Dc4Tvwk1gmgudDaf8ZMTc6vk JpUGesQ0M7FpfpeHJsbSIJXV/LkqPNDpZ2rdW+gwvWDsu0ingjjSZ883fLNXfZPl+N4MGrjJOgjnS TsYBZQ1dzwt+Zr7u9iVditKophQs5aYohyo0uHH+hNT5RI9wL6ZQmeQmQbRs/IqCFTfE8FRDemyPZ Atle8KzSpjR/Ea398tz6hbejAOdcAYDnuIGEltGxegHOm2fEB0STxIB9HFjvQn1sIS7m50c5YGdU4 Via502sA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1sSTod-00000001pLi-0IXd; Sat, 13 Jul 2024 03:52:55 +0000 Received: from mail.manjaro.org ([116.203.91.91]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1sSToK-00000001pJ6-0jhp; Sat, 13 Jul 2024 03:52:38 +0000 MIME-Version: 1.0 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=manjaro.org; s=2021; t=1720842751; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=l3cOgi5Y3zeJt1VWYzdIVf8SSuAoPfH0fWAeIPZGLq4=; b=Of43tAHhTd2+9w4q7DXLn9iyAenDxH6Z8xtCFETibo/4cwhDyEhXc8zTIfREBIjOTLF7Sg +we/LH3zbjuCNGn8kPb8colY5yayiTvYspW1JH4geCw88XY7Jzxl7XpY8MQ0wchgWr6Suv hQkKSi9iDDV3h1wCqO2ANxjXPTtS15EOV1X2TKu/oV7lC+YOojWhINui4CWy+fRV2nByJO tVOb7zcxpNEgOuY3QROLSefqoAkAr/wGOTKVynJGWo/YBwqRhTZBdtFTxHrmw03rrdjml+ uomOmFztRj8qWOTL+CFKKXrrEOYb6twUe9ieFSOx+HFq6xyPXxj8Uk/xBybEIA== Date: Sat, 13 Jul 2024 05:51:42 +0200 From: Dragan Simic To: Alexander Shiyan Cc: linux-arm-kernel@lists.infradead.org, Michael Turquette , Stephen Boyd , Heiko Stuebner , linux-clk@vger.kernel.org, linux-rockchip@lists.infradead.org Subject: Re: [PATCH v2] clk: rockchip: clk-rk3588: Fix 32k clock name for pmu_24m_32k_100m_src_p In-Reply-To: <20240710165354.1338287-1-eagle.alexander923@gmail.com> References: <20240710165354.1338287-1-eagle.alexander923@gmail.com> Message-ID: X-Sender: dsimic@manjaro.org Content-Type: text/plain; charset=US-ASCII; format=flowed Content-Transfer-Encoding: 7bit Authentication-Results: ORIGINATING; auth=pass smtp.auth=dsimic@manjaro.org smtp.mailfrom=dsimic@manjaro.org X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240712_205236_851443_A3AFC32B X-CRM114-Status: GOOD ( 15.38 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hello Alexander, On 2024-07-10 18:53, Alexander Shiyan wrote: > The 32kHz input clock is named "xin32k" in the driver, > so the name "32k" appears to be a typo in this case. Lets fix this. > > Signed-off-by: Alexander Shiyan Makes sense to me, and it seems to be a typo inherited from the downstream code, [1] which the base RK3588 dtsi confirms, [2] as well as the RK3588 Hardware Design Guide, version 1.0. Thus, please include: Reviewed-by: Dragan Simic I'd also suggest that this patch receives a Fixes tag, and gets submitted for inclusion into stable kernels. Thus: Fixes: f1c506d152ff ("clk: rockchip: add clock controller for the RK3588") Cc: stable@vger.kernel.org ... but you should actually submit the v2 with these tags, if you choose to agree with this suggestion. Furthermore, it seems that the board dts for Radxa ROCK 5B needs some related fixes, because it deviates from the RK3588 EVB design, but I still need to dig deeper into that and actually do some testing to confirm it. Perhaps that will apply to other similar RK3588-based boards as well. [1] https://raw.githubusercontent.com/rockchip-linux/kernel/develop-5.10/drivers/clk/rockchip/clk-rk3588.c [2] https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/tree/arch/arm64/boot/dts/rockchip/rk3588-base.dtsi#n423 > --- > drivers/clk/rockchip/clk-rk3588.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/clk/rockchip/clk-rk3588.c > b/drivers/clk/rockchip/clk-rk3588.c > index b30279a96dc8..3027379f2fdd 100644 > --- a/drivers/clk/rockchip/clk-rk3588.c > +++ b/drivers/clk/rockchip/clk-rk3588.c > @@ -526,7 +526,7 @@ PNAME(pmu_200m_100m_p) = { "clk_pmu1_200m_src", > "clk_pmu1_100m_src" }; > PNAME(pmu_300m_24m_p) = { "clk_300m_src", "xin24m" }; > PNAME(pmu_400m_24m_p) = { "clk_400m_src", "xin24m" }; > PNAME(pmu_100m_50m_24m_src_p) = { "clk_pmu1_100m_src", > "clk_pmu1_50m_src", "xin24m" }; > -PNAME(pmu_24m_32k_100m_src_p) = { "xin24m", "32k", > "clk_pmu1_100m_src" }; > +PNAME(pmu_24m_32k_100m_src_p) = { "xin24m", "xin32k", > "clk_pmu1_100m_src" }; > PNAME(hclk_pmu1_root_p) = { "clk_pmu1_200m_src", > "clk_pmu1_100m_src", "clk_pmu1_50m_src", "xin24m" }; > PNAME(hclk_pmu_cm0_root_p) = { "clk_pmu1_400m_src", > "clk_pmu1_200m_src", "clk_pmu1_100m_src", "xin24m" }; > PNAME(mclk_pdm0_p) = { "clk_pmu1_300m_src", "clk_pmu1_200m_src" };