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

Message ID 20230809202343.1098425-4-bryan.odonoghue@linaro.org
State New
Headers
Series apq8016: camss: Update dts with various fixes |

Commit Message

Bryan O'Donoghue Aug. 9, 2023, 8:23 p.m. UTC
  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(-)
  

Comments

Konrad Dybcio Aug. 10, 2023, 1:31 p.m. UTC | #1
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
  

Patch

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>;