All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] ARM: dts: qcom-apq8064: Fix dsi and hdmi phy cells
@ 2017-08-16  4:28 ` Andy Gross
  0 siblings, 0 replies; 4+ messages in thread
From: Andy Gross @ 2017-08-16  4:28 UTC (permalink / raw)
  To: linux-arm-msm-u79uwXL29TY76Z2rM5mHXA
  Cc: devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Rob Herring,
	Andy Gross

This patch adds the necessary #phy-cells property to the DSI and HDMI
phys.

Signed-off-by: Andy Gross <andy.gross-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
---
 arch/arm/boot/dts/qcom-apq8064.dtsi | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/qcom-apq8064.dtsi b/arch/arm/boot/dts/qcom-apq8064.dtsi
index f3db185..d08f3db 100644
--- a/arch/arm/boot/dts/qcom-apq8064.dtsi
+++ b/arch/arm/boot/dts/qcom-apq8064.dtsi
@@ -906,11 +906,11 @@
 				usb_hs1_phy: phy {
 					compatible = "qcom,usb-hs-phy-apq8064",
 						     "qcom,usb-hs-phy";
-					#phy-cells = <0>;
 					clocks = <&sleep_clk>, <&cxo_board>;
 					clock-names = "sleep", "ref";
 					resets = <&usb1 0>;
 					reset-names = "por";
+					#phy-cells = <0>;
 				};
 			};
 		};
@@ -1263,6 +1263,7 @@
 		dsi0_phy: dsi-phy@4700200 {
 			compatible = "qcom,dsi-phy-28nm-8960";
 			#clock-cells = <1>;
+			#phy-cells = <0>;
 
 			reg = <0x04700200 0x100>,
 				<0x04700300 0x200>,
@@ -1417,6 +1418,7 @@
 
 			clocks = <&mmcc HDMI_S_AHB_CLK>;
 			clock-names = "slave_iface_clk";
+			#phy-cells = <0>;
 		};
 
 		mdp: mdp@5100000 {
-- 
2.7.4

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply related	[flat|nested] 4+ messages in thread
* [PATCH] ARM: dts: qcom-apq8064: Fix dsi and hdmi phy cells
@ 2017-08-16  4:28 ` Andy Gross
  0 siblings, 0 replies; 4+ messages in thread
From: Andy Gross @ 2017-08-16  4:28 UTC (permalink / raw)
  To: linux-arm-kernel

This patch adds the necessary #phy-cells property to the DSI and HDMI
phys.

Signed-off-by: Andy Gross <andy.gross@linaro.org>
---
 arch/arm/boot/dts/qcom-apq8064.dtsi | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/qcom-apq8064.dtsi b/arch/arm/boot/dts/qcom-apq8064.dtsi
index f3db185..d08f3db 100644
--- a/arch/arm/boot/dts/qcom-apq8064.dtsi
+++ b/arch/arm/boot/dts/qcom-apq8064.dtsi
@@ -906,11 +906,11 @@
 				usb_hs1_phy: phy {
 					compatible = "qcom,usb-hs-phy-apq8064",
 						     "qcom,usb-hs-phy";
-					#phy-cells = <0>;
 					clocks = <&sleep_clk>, <&cxo_board>;
 					clock-names = "sleep", "ref";
 					resets = <&usb1 0>;
 					reset-names = "por";
+					#phy-cells = <0>;
 				};
 			};
 		};
@@ -1263,6 +1263,7 @@
 		dsi0_phy: dsi-phy at 4700200 {
 			compatible = "qcom,dsi-phy-28nm-8960";
 			#clock-cells = <1>;
+			#phy-cells = <0>;
 
 			reg = <0x04700200 0x100>,
 				<0x04700300 0x200>,
@@ -1417,6 +1418,7 @@
 
 			clocks = <&mmcc HDMI_S_AHB_CLK>;
 			clock-names = "slave_iface_clk";
+			#phy-cells = <0>;
 		};
 
 		mdp: mdp at 5100000 {
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 4+ messages in thread
* Re: [PATCH] ARM: dts: qcom-apq8064: Fix dsi and hdmi phy cells
  2017-08-16  4:28 ` Andy Gross
@ 2017-08-18  4:21   ` Archit Taneja
  -1 siblings, 0 replies; 4+ messages in thread
From: Archit Taneja @ 2017-08-18  4:21 UTC (permalink / raw)
  To: Andy Gross, linux-arm-msm; +Cc: devicetree, linux-arm-kernel, Rob Herring



On 08/16/2017 09:58 AM, Andy Gross wrote:
> This patch adds the necessary #phy-cells property to the DSI and HDMI
> phys.
The DSI and HDMI drivers don't use the common PHY framework, so we end up
being okay without #phy-cells. But we do follow the PHY DT bindings, so it's
necessary in that sense.

Reviewed-by: Archit Taneja <architt@codeaurora.org>

> 
> Signed-off-by: Andy Gross <andy.gross@linaro.org>
> ---
>   arch/arm/boot/dts/qcom-apq8064.dtsi | 4 +++-
>   1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/arch/arm/boot/dts/qcom-apq8064.dtsi b/arch/arm/boot/dts/qcom-apq8064.dtsi
> index f3db185..d08f3db 100644
> --- a/arch/arm/boot/dts/qcom-apq8064.dtsi
> +++ b/arch/arm/boot/dts/qcom-apq8064.dtsi
> @@ -906,11 +906,11 @@
>   				usb_hs1_phy: phy {
>   					compatible = "qcom,usb-hs-phy-apq8064",
>   						     "qcom,usb-hs-phy";
> -					#phy-cells = <0>;
>   					clocks = <&sleep_clk>, <&cxo_board>;
>   					clock-names = "sleep", "ref";
>   					resets = <&usb1 0>;
>   					reset-names = "por";
> +					#phy-cells = <0>;
>   				};
>   			};
>   		};
> @@ -1263,6 +1263,7 @@
>   		dsi0_phy: dsi-phy@4700200 {
>   			compatible = "qcom,dsi-phy-28nm-8960";
>   			#clock-cells = <1>;
> +			#phy-cells = <0>;
>   
>   			reg = <0x04700200 0x100>,
>   				<0x04700300 0x200>,
> @@ -1417,6 +1418,7 @@
>   
>   			clocks = <&mmcc HDMI_S_AHB_CLK>;
>   			clock-names = "slave_iface_clk";
> +			#phy-cells = <0>;
>   		};
>   
>   		mdp: mdp@5100000 {
> 

-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project

^ permalink raw reply	[flat|nested] 4+ messages in thread
* [PATCH] ARM: dts: qcom-apq8064: Fix dsi and hdmi phy cells
@ 2017-08-18  4:21   ` Archit Taneja
  0 siblings, 0 replies; 4+ messages in thread
From: Archit Taneja @ 2017-08-18  4:21 UTC (permalink / raw)
  To: linux-arm-kernel



On 08/16/2017 09:58 AM, Andy Gross wrote:
> This patch adds the necessary #phy-cells property to the DSI and HDMI
> phys.
The DSI and HDMI drivers don't use the common PHY framework, so we end up
being okay without #phy-cells. But we do follow the PHY DT bindings, so it's
necessary in that sense.

Reviewed-by: Archit Taneja <architt@codeaurora.org>

> 
> Signed-off-by: Andy Gross <andy.gross@linaro.org>
> ---
>   arch/arm/boot/dts/qcom-apq8064.dtsi | 4 +++-
>   1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/arch/arm/boot/dts/qcom-apq8064.dtsi b/arch/arm/boot/dts/qcom-apq8064.dtsi
> index f3db185..d08f3db 100644
> --- a/arch/arm/boot/dts/qcom-apq8064.dtsi
> +++ b/arch/arm/boot/dts/qcom-apq8064.dtsi
> @@ -906,11 +906,11 @@
>   				usb_hs1_phy: phy {
>   					compatible = "qcom,usb-hs-phy-apq8064",
>   						     "qcom,usb-hs-phy";
> -					#phy-cells = <0>;
>   					clocks = <&sleep_clk>, <&cxo_board>;
>   					clock-names = "sleep", "ref";
>   					resets = <&usb1 0>;
>   					reset-names = "por";
> +					#phy-cells = <0>;
>   				};
>   			};
>   		};
> @@ -1263,6 +1263,7 @@
>   		dsi0_phy: dsi-phy at 4700200 {
>   			compatible = "qcom,dsi-phy-28nm-8960";
>   			#clock-cells = <1>;
> +			#phy-cells = <0>;
>   
>   			reg = <0x04700200 0x100>,
>   				<0x04700300 0x200>,
> @@ -1417,6 +1418,7 @@
>   
>   			clocks = <&mmcc HDMI_S_AHB_CLK>;
>   			clock-names = "slave_iface_clk";
> +			#phy-cells = <0>;
>   		};
>   
>   		mdp: mdp at 5100000 {
> 

-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project

^ permalink raw reply	[flat|nested] 4+ messages in thread
end of thread, other threads:[~2017-08-18  4:21 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-08-16  4:28 [PATCH] ARM: dts: qcom-apq8064: Fix dsi and hdmi phy cells Andy Gross
2017-08-16  4:28 ` Andy Gross
2017-08-18  4:21 ` Archit Taneja
2017-08-18  4:21   ` Archit Taneja

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.