[PATCH v2 3/6] arm64: dts: qcom: apq8016-sbc: Set ov5640 assigned-clock

Bryan O'Donoghue posted 6 patches 2 years, 6 months ago
There is a newer version of this series
[PATCH v2 3/6] arm64: dts: qcom: apq8016-sbc: Set ov5640 assigned-clock
Posted by Bryan O'Donoghue 2 years, 6 months ago
The driver for the ov5640 doesn't do a set-rate, instead it expects the
clock to already be set at an appropriate rate.

Similarly the yaml for ov5640 doesn't understand clock-frequency. Convert
from clock-rate to assigned-clock and assigned-clock-rate to remediate.

Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
---
 arch/arm64/boot/dts/qcom/apq8016-sbc.dts | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/qcom/apq8016-sbc.dts b/arch/arm64/boot/dts/qcom/apq8016-sbc.dts
index 0481a4a82090a..ada0777567623 100644
--- a/arch/arm64/boot/dts/qcom/apq8016-sbc.dts
+++ b/arch/arm64/boot/dts/qcom/apq8016-sbc.dts
@@ -280,7 +280,8 @@ camera_rear@3b {
 
 		clocks = <&gcc GCC_CAMSS_MCLK0_CLK>;
 		clock-names = "xclk";
-		clock-frequency = <23880000>;
+		assigned-clocks = <&gcc GCC_CAMSS_MCLK0_CLK>;
+		assigned-clock-rates = <23880000>;
 
 		DOVDD-supply = <&camera_vdddo_1v8>;
 		AVDD-supply = <&camera_vdda_2v8>;
-- 
2.39.2
Re: [PATCH v2 3/6] arm64: dts: qcom: apq8016-sbc: Set ov5640 assigned-clock
Posted by Konrad Dybcio 2 years, 6 months ago
On 9.08.2023 22:23, Bryan O'Donoghue wrote:
> The driver for the ov5640 doesn't do a set-rate, instead it expects the
> clock to already be set at an appropriate rate.
> 
> Similarly the yaml for ov5640 doesn't understand clock-frequency. Convert
> from clock-rate to assigned-clock and assigned-clock-rate to remediate.
> 
> Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
> ---
Reviewed-by: Konrad Dybcio <konrad.dybcio@linaro.org>

Konrad