Re: [PATCH v3 2/8] arm64: dts: qcom: sdm630: disable dsi1/dsi1_phy by default

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

 



On 14/05/2022 12:37, Marijn Suijten wrote:
Title copy-paste error: it should read dsi0/dsi0_phy.  Can you also
reorder this patch together with the dsi1/dsi1_phy one, and perhaps they
should just be squashed?  Otherwise, does a Suggested-by: make sense
here?

Yes, it does. Please excuse me.


- Marijn

On 2022-05-14 02:45:12, Dmitry Baryshkov wrote:
Follow the typical practice and keep DSI0/DSI0 PHY disabled by default.
They should be enabled in the board DT files. No existing boards use
them at this moment.

Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx>
---
  arch/arm64/boot/dts/qcom/sdm630.dtsi | 3 +++
  1 file changed, 3 insertions(+)

diff --git a/arch/arm64/boot/dts/qcom/sdm630.dtsi b/arch/arm64/boot/dts/qcom/sdm630.dtsi
index 240293592ef9..8697d40e9b74 100644
--- a/arch/arm64/boot/dts/qcom/sdm630.dtsi
+++ b/arch/arm64/boot/dts/qcom/sdm630.dtsi
@@ -1559,6 +1559,8 @@ dsi0: dsi@c994000 {
  				phys = <&dsi0_phy>;
  				phy-names = "dsi";
+ status = "disabled";
+
  				ports {
  					#address-cells = <1>;
  					#size-cells = <0>;
@@ -1592,6 +1594,7 @@ dsi0_phy: dsi-phy@c994400 {
clocks = <&mmcc MDSS_AHB_CLK>, <&xo_board>;
  				clock-names = "iface", "ref";
+				status = "disabled";
  			};
  		};
--
2.35.1



--
With best wishes
Dmitry



[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