diff mbox series

[v3,8/8] arm64: dts: qcom: Update sleep_clk frequency to 32000 on SA8775P

Message ID 20240715-sa8775p-mm-v3-v1-8-badaf35ed670@quicinc.com (mailing list archive)
State Not Applicable, archived
Headers show
Series Add support for SA8775P Multimedia clock controllers | expand

Commit Message

Taniya Das July 15, 2024, 8:23 a.m. UTC
The HW supported sleep_clk frequency on SA8775P is 32000, hence
update the sleep_clk frequency with the correct value on SA8775P.

Fixes: 603f96d4c9d0 ("arm64: dts: qcom: add initial support for qcom sa8775p-ride")
Signed-off-by: Taniya Das <quic_tdas@quicinc.com>
---
 arch/arm64/boot/dts/qcom/sa8775p-ride.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Krzysztof Kozlowski July 16, 2024, 7:45 a.m. UTC | #1
On 15/07/2024 10:23, Taniya Das wrote:
> The HW supported sleep_clk frequency on SA8775P is 32000, hence
> update the sleep_clk frequency with the correct value on SA8775P.
> 
> Fixes: 603f96d4c9d0 ("arm64: dts: qcom: add initial support for qcom sa8775p-ride")
> Signed-off-by: Taniya Das <quic_tdas@quicinc.com>
> ---

Avoid combining fixes with features, but if you do, then fixes are never
the last patch, but first.

Best regards,
Krzysztof
Konrad Dybcio July 16, 2024, 12:08 p.m. UTC | #2
On 15.07.2024 10:23 AM, Taniya Das wrote:
> The HW supported sleep_clk frequency on SA8775P is 32000, hence
> update the sleep_clk frequency with the correct value on SA8775P.
> 
> Fixes: 603f96d4c9d0 ("arm64: dts: qcom: add initial support for qcom sa8775p-ride")
> Signed-off-by: Taniya Das <quic_tdas@quicinc.com>
> ---

Reviewed-by: Konrad Dybcio <konrad.dybcio@linaro.org>

Konrad
Taniya Das Aug. 16, 2024, 5:16 a.m. UTC | #3
On 7/16/2024 1:15 PM, Krzysztof Kozlowski wrote:
> On 15/07/2024 10:23, Taniya Das wrote:
>> The HW supported sleep_clk frequency on SA8775P is 32000, hence
>> update the sleep_clk frequency with the correct value on SA8775P.
>>
>> Fixes: 603f96d4c9d0 ("arm64: dts: qcom: add initial support for qcom sa8775p-ride")
>> Signed-off-by: Taniya Das <quic_tdas@quicinc.com>
>> ---
> 
> Avoid combining fixes with features, but if you do, then fixes are never
> the last patch, but first.
> 
> Best regards,
> Krzysztof
> 
Thanks Krzysztof, I will change the order of the patch, putting it 
before the MM clock nodes.
diff mbox series

Patch

diff --git a/arch/arm64/boot/dts/qcom/sa8775p-ride.dtsi b/arch/arm64/boot/dts/qcom/sa8775p-ride.dtsi
index 2a6170623ea9..864ad109371c 100644
--- a/arch/arm64/boot/dts/qcom/sa8775p-ride.dtsi
+++ b/arch/arm64/boot/dts/qcom/sa8775p-ride.dtsi
@@ -517,7 +517,7 @@  &serdes1 {
 };
 
 &sleep_clk {
-	clock-frequency = <32764>;
+	clock-frequency = <32000>;
 };
 
 &spi16 {