Re: [PATCH 1/1] arm64: dts: qcom: qrb5165-rb5: Enable the IMX577 on cam1

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 18/05/2022 14:55, Konrad Dybcio wrote:
Hi!


On 18/05/2022 15:30, Bryan O'Donoghue wrote:
The IMX577 is on CCI1/CSI2 providing four lanes of camera data.

Commit says IMX577, code says IMX412.



The silicon enabling code for imx412 from Sony is the same as is used on imx577.

We have an imx577. I'll explain the difference in the V2 commit though.


An example media-ctl pipeline is:

media-ctl --reset
media-ctl -v -d /dev/media0 -V '"imx412 '20-001a'":0[fmt:SRGGB10/4056x3040 field:none]'
media-ctl -V '"msm_csiphy2":0[fmt:SRGGB10/4056x3040]'
media-ctl -V '"msm_csid0":0[fmt:SRGGB10/4056x3040]'
media-ctl -V '"msm_vfe0_rdi0":0[fmt:SRGGB10/4056x3040]'
media-ctl -l '"msm_csiphy2":1->"msm_csid0":0[1]'
media-ctl -l '"msm_csid0":1->"msm_vfe0_rdi0":0[1]'

yavta -B capture-mplane -c -I -n 5 -f SRGGB10P -s 4056x3040 -F /dev/video0

Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@xxxxxxxxxx>
---
  arch/arm64/boot/dts/qcom/qrb5165-rb5.dts | 98 ++++++++++++++++++++++++
  1 file changed, 98 insertions(+)

diff --git a/arch/arm64/boot/dts/qcom/qrb5165-rb5.dts b/arch/arm64/boot/dts/qcom/qrb5165-rb5.dts
index 0e63f707b911..48b31790c434 100644
--- a/arch/arm64/boot/dts/qcom/qrb5165-rb5.dts
+++ b/arch/arm64/boot/dts/qcom/qrb5165-rb5.dts
@@ -1203,6 +1203,43 @@ sdc2_card_det_n: sd-card-det-n {
          function = "gpio";
          bias-pull-up;
      };
+
+    cam2_default: cam2-default {
+        rst {
+            pins = "gpio78";
+            function = "gpio";
+
+            drive-strength = <2>;
+            bias-disable;

Other pins in this DT don't have a newline between function and drive-strength, please remove it for consistency.


+        };
+
+        mclk {
+            pins = "gpio96";
+            function = "cam_mclk";
+
+            drive-strength = <16>;
+            bias-disable;
+        };
+    };
+
+    cam2_suspend: cam2-suspend {
+        rst {
+            pins = "gpio78";
+            function = "gpio";
+
+            drive-strength = <2>;
+            bias-pull-down;
+            output-low;
+        };
+
+        mclk {
+            pins = "gpio96";
+            function = "cam_mclk";
+
+            drive-strength = <2>;
+            bias-disable;
+        };
+    };
  };
  &uart12 {
@@ -1294,3 +1331,64 @@ &qup_spi0_data_clk {
      drive-strength = <6>;
      bias-disable;
  };
+
+&camcc {
+    status = "okay";
+};

It's enabled by default.

I'm assuming Vladimir's patch to disable by default goes in.
I'll include his patch as #1 in V2 so its clear on this point.


+
+&camss {
+    status = "okay";
+    vdda-phy-supply = <&vreg_l5a_0p88>;
+    vdda-pll-supply = <&vreg_l9a_1p2>;
+
+    ports {

Maybe the port definitions along with #-cells here and on camss could be moved to the SoC DTSI?

Makes sense.

---
bod



[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [Linux for Sparc]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux