arm64: dts: sc7180: add dsi controller and phy entries for idp dts

subject: arm64: dts: sc7180: add dsi controller and phy entries for idp dts

Adding dsi controller and phy entries for idp dt.

Signed-off-by Harigovindan P <[email protected]>

---

Changes in v1:
- Added dsi controller and dsi phy entries for idp dts

Changes in v2:
- Adding dependency patchwork series
- Removing suspend configuration
- Adding blank before curly brace

This patch depends on following patchwork series:

https://patchwork.kernel.org/patch/11364687/
https://patchwork.kernel.org/patch/11366303/

 arch/arm64/boot/dts/qcom/sc7180-idp.dts | 55 +++++++++++++++++++++++++
 1 file changed, 55 insertions(+)

arch/arm64/boot/dts/qcom/sc7180-idp.dts +48 -0

index 388f50ad4fde..6ccf8c3603ab 100644
@@ -7,6 +7,7 @@
 
 /dts-v1/;
 
+#include <dt-bindings/gpio/gpio.h>
 #include <dt-bindings/regulator/qcom,rpmh-regulator.h>
 #include "sc7180.dtsi"
 #include "pm6150.dtsi"

@@ -232,6 +233,49 @@ vreg_bob: bob {
 	};
 };
 
+&dsi0 {
+	status = "okay";
+
+	vdda-supply = <&vreg_l3c_1p2>;
+
+	panel@0 {
+		compatible = "visionox,rm69299-1080p-display";
+		reg = <0>;
+
+		vdda-supply = <&vreg_l8c_1p8>;
+		vdd3p3-supply = <&vreg_l18a_2p8>;
+
+		pinctrl-names = "default";
+		pinctrl-0 = <&disp_pins>;
+
+		reset-gpios = <&pm6150l_gpio 3 GPIO_ACTIVE_HIGH>;
+
+		ports {
+			#address-cells = <1>;
+			#size-cells = <0>;
+			port@0 {
+				reg = <0>;
+				panel0_in: endpoint {
+					remote-endpoint = <&dsi0_out>;
+				};
+			};
+		};
+	};
+
+	ports {
+		port@1 {
+			endpoint {
+				remote-endpoint = <&panel0_in>;
+				data-lanes = <0 1 2 3>;
+			};
+		};
+	};
+};
+
+&dsi_phy {
+	status = "okay";
+};
+
 &qspi {
 	status = "okay";
 	pinctrl-names = "default";

@@ -289,6 +333,17 @@ &usb_1_qmpphy {
 
 /* PINCTRL - additions to nodes defined in sc7180.dtsi */
 
+&pm6150l_gpio {
+	disp_pins: disp-pins {
+		pins = "gpio3";
+		function = "func1";

Quoted 1 line · Harigovindan P

subject: arm64: dts: sc7180: add dsi controller and phy entries for idp dts

nit: 'dts' at the end is redundant, the prefixes make it clear that this is about DT entries.

Also the message isn't really concise. The main entries for the DSI controller and the PHY are in sc7180.dtsi. I would suggest to drop any mentions of DSI controller and PHYs, and just say something like 'Add nodes for IDP display'. In the body you could mention that the display is the Visionox RM69299.

Quoted 7 lines

Adding dsi controller and phy entries for idp dt.

Signed-off-by Harigovindan P <[email protected]>

---

Changes in v1:
- Added dsi controller and dsi phy entries for idp dts

Changes in v1 is pointless, it's the first patch

Quoted 83 lines

Changes in v2:
- Adding dependency patchwork series
- Removing suspend configuration
- Adding blank before curly brace

This patch depends on following patchwork series:

https://patchwork.kernel.org/patch/11364687/
https://patchwork.kernel.org/patch/11366303/

 arch/arm64/boot/dts/qcom/sc7180-idp.dts | 55 +++++++++++++++++++++++++
 1 file changed, 55 insertions(+)

arch/arm64/boot/dts/qcom/sc7180-idp.dts +48 -0

index 388f50ad4fde..6ccf8c3603ab 100644
@@ -7,6 +7,7 @@
 
 /dts-v1/;
 
+#include <dt-bindings/gpio/gpio.h>
 #include <dt-bindings/regulator/qcom,rpmh-regulator.h>
 #include "sc7180.dtsi"
 #include "pm6150.dtsi"

@@ -232,6 +233,49 @@ vreg_bob: bob {
 	};
 };
 
+&dsi0 {
+	status = "okay";
+
+	vdda-supply = <&vreg_l3c_1p2>;
+
+	panel@0 {
+		compatible = "visionox,rm69299-1080p-display";
+		reg = <0>;
+
+		vdda-supply = <&vreg_l8c_1p8>;
+		vdd3p3-supply = <&vreg_l18a_2p8>;
+
+		pinctrl-names = "default";
+		pinctrl-0 = <&disp_pins>;
+
+		reset-gpios = <&pm6150l_gpio 3 GPIO_ACTIVE_HIGH>;
+
+		ports {
+			#address-cells = <1>;
+			#size-cells = <0>;
+			port@0 {
+				reg = <0>;
+				panel0_in: endpoint {
+					remote-endpoint = <&dsi0_out>;
+				};
+			};
+		};
+	};
+
+	ports {
+		port@1 {
+			endpoint {
+				remote-endpoint = <&panel0_in>;
+				data-lanes = <0 1 2 3>;
+			};
+		};
+	};
+};
+
+&dsi_phy {
+	status = "okay";
+};
+
 &qspi {
 	status = "okay";
 	pinctrl-names = "default";

@@ -289,6 +333,17 @@ &usb_1_qmpphy {
 
 /* PINCTRL - additions to nodes defined in sc7180.dtsi */
 
+&pm6150l_gpio {
+	disp_pins: disp-pins {
+		pins = "gpio3";
+		function = "func1";

arch/arm64/boot/dts/qcom/sc7180-idp.dts &usb_1_qmpphy { · line 349

Quoted 10 lines

arch/arm64/boot/dts/qcom/sc7180-idp.dts +7 -0

+		qcom,drive-strength = <2>;
+		power-source = <0>;
+		bias-disable;
+		output-low;
+	};
+};
+
 &qspi_clk {
 	pinconf {
 		pins = "gpio63";

To get the display actually to work you also need this:

arch/arm64/boot/dts/qcom/sc7180-idp.dts +8 -0

index 88919da1510b03..fdbcb56dfa81f9 100644
@@ -276,6 +276,14 @@
        status = "okay";
 };
 
+&mdp {
+       status = "okay";
+};
+
+&mdss {
+       status = "okay";
+};
+
 &qspi {
        status = "okay";
        pinctrl-names = "default";

Maybe just add this to this patch?