dri-devel.lists.freedesktop.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 0/2] arm: dts: qcom: rename HDMI PHY nodes
@ 2022-09-24  9:43 Dmitry Baryshkov
  2022-09-24  9:43 ` [PATCH 1/2] ARM: dts: qcom-apq8064: change HDMI PHY node name to generic one Dmitry Baryshkov
                   ` (2 more replies)
  0 siblings, 3 replies; 8+ messages in thread
From: Dmitry Baryshkov @ 2022-09-24  9:43 UTC (permalink / raw)
  To: Andy Gross, Bjorn Andersson, Konrad Dybcio, Rob Clark, Sean Paul,
	Abhinav Kumar, Rob Herring, Krzysztof Kozlowski
  Cc: devicetree, David Airlie, linux-arm-msm, dri-devel, Stephen Boyd,
	freedreno

Historically HDMI PHY device tree nodes used the hdmi-phy@ names.
Replace them with generic phy@ names.

While there is no such requirement in the DT schema, it's worth doing
that because:

1) The recent qcom DT files already use just phy@ for most of PHY nodes

2) The recommended list from devicetree specs mentions
usb-phy/ethernet-phy, but not <anything>-phy, so I'd think that those
two are mostly for backwards compatibility.

3) I liked the example of sc7280 which switched all MDSS PHYs to just
phy@ (this includes DSI PHY, eDP PHY and, by extension, the HDMI PHY).

Dmitry Baryshkov (2):
  ARM: dts: qcom-apq8064: change HDMI PHY node name to generic one
  arm64: dts: qcom: msm8996: change HDMI PHY node name to generic one

 arch/arm/boot/dts/qcom-apq8064-ifc6410.dts | 2 +-
 arch/arm/boot/dts/qcom-apq8064.dtsi        | 2 +-
 arch/arm64/boot/dts/qcom/msm8996.dtsi      | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

-- 
2.35.1


^ permalink raw reply	[flat|nested] 8+ messages in thread

* [PATCH 1/2] ARM: dts: qcom-apq8064: change HDMI PHY node name to generic one
  2022-09-24  9:43 [PATCH 0/2] arm: dts: qcom: rename HDMI PHY nodes Dmitry Baryshkov
@ 2022-09-24  9:43 ` Dmitry Baryshkov
  2022-09-24 14:48   ` Konrad Dybcio
  2022-09-29  9:36   ` Krzysztof Kozlowski
  2022-09-24  9:43 ` [PATCH 2/2] arm64: dts: qcom: msm8996: " Dmitry Baryshkov
  2022-11-07  3:12 ` (subset) [PATCH 0/2] arm: dts: qcom: rename HDMI PHY nodes Bjorn Andersson
  2 siblings, 2 replies; 8+ messages in thread
From: Dmitry Baryshkov @ 2022-09-24  9:43 UTC (permalink / raw)
  To: Andy Gross, Bjorn Andersson, Konrad Dybcio, Rob Clark, Sean Paul,
	Abhinav Kumar, Rob Herring, Krzysztof Kozlowski
  Cc: devicetree, David Airlie, linux-arm-msm, dri-devel, Stephen Boyd,
	freedreno

Change HDMI PHY node names from custom 'hdmi-phy' to the generic 'phy'.

Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
---
 arch/arm/boot/dts/qcom-apq8064-ifc6410.dts | 2 +-
 arch/arm/boot/dts/qcom-apq8064.dtsi        | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm/boot/dts/qcom-apq8064-ifc6410.dts b/arch/arm/boot/dts/qcom-apq8064-ifc6410.dts
index 0322cb88d448..497c4012a65b 100644
--- a/arch/arm/boot/dts/qcom-apq8064-ifc6410.dts
+++ b/arch/arm/boot/dts/qcom-apq8064-ifc6410.dts
@@ -361,7 +361,7 @@ endpoint {
 			};
 		};
 
-		hdmi-phy@4a00400 {
+		phy@4a00400 {
 			status = "okay";
 
 			core-vdda-supply = <&pm8921_hdmi_switch>;
diff --git a/arch/arm/boot/dts/qcom-apq8064.dtsi b/arch/arm/boot/dts/qcom-apq8064.dtsi
index 1b704c7ea890..58f239218639 100644
--- a/arch/arm/boot/dts/qcom-apq8064.dtsi
+++ b/arch/arm/boot/dts/qcom-apq8064.dtsi
@@ -1453,7 +1453,7 @@ hdmi_out: endpoint {
 			};
 		};
 
-		hdmi_phy: hdmi-phy@4a00400 {
+		hdmi_phy: phy@4a00400 {
 			compatible = "qcom,hdmi-phy-8960";
 			reg = <0x4a00400 0x60>,
 			      <0x4a00500 0x100>;
-- 
2.35.1


^ permalink raw reply related	[flat|nested] 8+ messages in thread

* [PATCH 2/2] arm64: dts: qcom: msm8996: change HDMI PHY node name to generic one
  2022-09-24  9:43 [PATCH 0/2] arm: dts: qcom: rename HDMI PHY nodes Dmitry Baryshkov
  2022-09-24  9:43 ` [PATCH 1/2] ARM: dts: qcom-apq8064: change HDMI PHY node name to generic one Dmitry Baryshkov
@ 2022-09-24  9:43 ` Dmitry Baryshkov
  2022-09-24 14:48   ` Konrad Dybcio
  2022-09-29  9:36   ` Krzysztof Kozlowski
  2022-11-07  3:12 ` (subset) [PATCH 0/2] arm: dts: qcom: rename HDMI PHY nodes Bjorn Andersson
  2 siblings, 2 replies; 8+ messages in thread
From: Dmitry Baryshkov @ 2022-09-24  9:43 UTC (permalink / raw)
  To: Andy Gross, Bjorn Andersson, Konrad Dybcio, Rob Clark, Sean Paul,
	Abhinav Kumar, Rob Herring, Krzysztof Kozlowski
  Cc: devicetree, David Airlie, linux-arm-msm, dri-devel, Stephen Boyd,
	freedreno

Change HDMI PHY node name from custom 'hdmi-phy' to the generic 'phy'.

Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
---
 arch/arm64/boot/dts/qcom/msm8996.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/qcom/msm8996.dtsi b/arch/arm64/boot/dts/qcom/msm8996.dtsi
index a7d0e5d68141..ed9863854904 100644
--- a/arch/arm64/boot/dts/qcom/msm8996.dtsi
+++ b/arch/arm64/boot/dts/qcom/msm8996.dtsi
@@ -1144,7 +1144,7 @@ hdmi_in: endpoint {
 				};
 			};
 
-			hdmi_phy: hdmi-phy@9a0600 {
+			hdmi_phy: phy@9a0600 {
 				#phy-cells = <0>;
 				compatible = "qcom,hdmi-phy-8996";
 				reg = <0x009a0600 0x1c4>,
-- 
2.35.1


^ permalink raw reply related	[flat|nested] 8+ messages in thread

* Re: [PATCH 1/2] ARM: dts: qcom-apq8064: change HDMI PHY node name to generic one
  2022-09-24  9:43 ` [PATCH 1/2] ARM: dts: qcom-apq8064: change HDMI PHY node name to generic one Dmitry Baryshkov
@ 2022-09-24 14:48   ` Konrad Dybcio
  2022-09-29  9:36   ` Krzysztof Kozlowski
  1 sibling, 0 replies; 8+ messages in thread
From: Konrad Dybcio @ 2022-09-24 14:48 UTC (permalink / raw)
  To: Dmitry Baryshkov, Andy Gross, Bjorn Andersson, Rob Clark,
	Sean Paul, Abhinav Kumar, Rob Herring, Krzysztof Kozlowski
  Cc: devicetree, David Airlie, linux-arm-msm, dri-devel, Stephen Boyd,
	freedreno



On 24.09.2022 11:43, Dmitry Baryshkov wrote:
> Change HDMI PHY node names from custom 'hdmi-phy' to the generic 'phy'.
> 
> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> ---
Reviewed-by: Konrad Dybcio <konrad.dybcio@somainline.org>

Konrad
>  arch/arm/boot/dts/qcom-apq8064-ifc6410.dts | 2 +-
>  arch/arm/boot/dts/qcom-apq8064.dtsi        | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/qcom-apq8064-ifc6410.dts b/arch/arm/boot/dts/qcom-apq8064-ifc6410.dts
> index 0322cb88d448..497c4012a65b 100644
> --- a/arch/arm/boot/dts/qcom-apq8064-ifc6410.dts
> +++ b/arch/arm/boot/dts/qcom-apq8064-ifc6410.dts
> @@ -361,7 +361,7 @@ endpoint {
>  			};
>  		};
>  
> -		hdmi-phy@4a00400 {
> +		phy@4a00400 {
>  			status = "okay";
>  
>  			core-vdda-supply = <&pm8921_hdmi_switch>;
> diff --git a/arch/arm/boot/dts/qcom-apq8064.dtsi b/arch/arm/boot/dts/qcom-apq8064.dtsi
> index 1b704c7ea890..58f239218639 100644
> --- a/arch/arm/boot/dts/qcom-apq8064.dtsi
> +++ b/arch/arm/boot/dts/qcom-apq8064.dtsi
> @@ -1453,7 +1453,7 @@ hdmi_out: endpoint {
>  			};
>  		};
>  
> -		hdmi_phy: hdmi-phy@4a00400 {
> +		hdmi_phy: phy@4a00400 {
>  			compatible = "qcom,hdmi-phy-8960";
>  			reg = <0x4a00400 0x60>,
>  			      <0x4a00500 0x100>;

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH 2/2] arm64: dts: qcom: msm8996: change HDMI PHY node name to generic one
  2022-09-24  9:43 ` [PATCH 2/2] arm64: dts: qcom: msm8996: " Dmitry Baryshkov
@ 2022-09-24 14:48   ` Konrad Dybcio
  2022-09-29  9:36   ` Krzysztof Kozlowski
  1 sibling, 0 replies; 8+ messages in thread
From: Konrad Dybcio @ 2022-09-24 14:48 UTC (permalink / raw)
  To: Dmitry Baryshkov, Andy Gross, Bjorn Andersson, Rob Clark,
	Sean Paul, Abhinav Kumar, Rob Herring, Krzysztof Kozlowski
  Cc: devicetree, David Airlie, linux-arm-msm, dri-devel, Stephen Boyd,
	freedreno



On 24.09.2022 11:43, Dmitry Baryshkov wrote:
> Change HDMI PHY node name from custom 'hdmi-phy' to the generic 'phy'.
> 
> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> ---
Reviewed-by: Konrad Dybcio <konrad.dybcio@somainline.org>

Konrad
>  arch/arm64/boot/dts/qcom/msm8996.dtsi | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/arch/arm64/boot/dts/qcom/msm8996.dtsi b/arch/arm64/boot/dts/qcom/msm8996.dtsi
> index a7d0e5d68141..ed9863854904 100644
> --- a/arch/arm64/boot/dts/qcom/msm8996.dtsi
> +++ b/arch/arm64/boot/dts/qcom/msm8996.dtsi
> @@ -1144,7 +1144,7 @@ hdmi_in: endpoint {
>  				};
>  			};
>  
> -			hdmi_phy: hdmi-phy@9a0600 {
> +			hdmi_phy: phy@9a0600 {
>  				#phy-cells = <0>;
>  				compatible = "qcom,hdmi-phy-8996";
>  				reg = <0x009a0600 0x1c4>,

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH 1/2] ARM: dts: qcom-apq8064: change HDMI PHY node name to generic one
  2022-09-24  9:43 ` [PATCH 1/2] ARM: dts: qcom-apq8064: change HDMI PHY node name to generic one Dmitry Baryshkov
  2022-09-24 14:48   ` Konrad Dybcio
@ 2022-09-29  9:36   ` Krzysztof Kozlowski
  1 sibling, 0 replies; 8+ messages in thread
From: Krzysztof Kozlowski @ 2022-09-29  9:36 UTC (permalink / raw)
  To: Dmitry Baryshkov, Andy Gross, Bjorn Andersson, Konrad Dybcio,
	Rob Clark, Sean Paul, Abhinav Kumar, Rob Herring,
	Krzysztof Kozlowski
  Cc: devicetree, David Airlie, linux-arm-msm, dri-devel, Stephen Boyd,
	freedreno

On 24/09/2022 11:43, Dmitry Baryshkov wrote:
> Change HDMI PHY node names from custom 'hdmi-phy' to the generic 'phy'.
> 
> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>


Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Best regards,
Krzysztof


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH 2/2] arm64: dts: qcom: msm8996: change HDMI PHY node name to generic one
  2022-09-24  9:43 ` [PATCH 2/2] arm64: dts: qcom: msm8996: " Dmitry Baryshkov
  2022-09-24 14:48   ` Konrad Dybcio
@ 2022-09-29  9:36   ` Krzysztof Kozlowski
  1 sibling, 0 replies; 8+ messages in thread
From: Krzysztof Kozlowski @ 2022-09-29  9:36 UTC (permalink / raw)
  To: Dmitry Baryshkov, Andy Gross, Bjorn Andersson, Konrad Dybcio,
	Rob Clark, Sean Paul, Abhinav Kumar, Rob Herring,
	Krzysztof Kozlowski
  Cc: devicetree, David Airlie, linux-arm-msm, dri-devel, Stephen Boyd,
	freedreno

On 24/09/2022 11:43, Dmitry Baryshkov wrote:
> Change HDMI PHY node name from custom 'hdmi-phy' to the generic 'phy'.
> 
> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>


Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Best regards,
Krzysztof


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: (subset) [PATCH 0/2] arm: dts: qcom: rename HDMI PHY nodes
  2022-09-24  9:43 [PATCH 0/2] arm: dts: qcom: rename HDMI PHY nodes Dmitry Baryshkov
  2022-09-24  9:43 ` [PATCH 1/2] ARM: dts: qcom-apq8064: change HDMI PHY node name to generic one Dmitry Baryshkov
  2022-09-24  9:43 ` [PATCH 2/2] arm64: dts: qcom: msm8996: " Dmitry Baryshkov
@ 2022-11-07  3:12 ` Bjorn Andersson
  2 siblings, 0 replies; 8+ messages in thread
From: Bjorn Andersson @ 2022-11-07  3:12 UTC (permalink / raw)
  To: krzysztof.kozlowski+dt, dmitry.baryshkov, sean, Rob Herring,
	quic_abhinavk, konrad.dybcio, robdclark, Andy Gross
  Cc: devicetree, airlied, linux-arm-msm, dri-devel, swboyd, freedreno

On Sat, 24 Sep 2022 12:43:45 +0300, Dmitry Baryshkov wrote:
> Historically HDMI PHY device tree nodes used the hdmi-phy@ names.
> Replace them with generic phy@ names.
> 
> While there is no such requirement in the DT schema, it's worth doing
> that because:
> 
> 1) The recent qcom DT files already use just phy@ for most of PHY nodes
> 
> [...]

Applied, thanks!

[1/2] ARM: dts: qcom-apq8064: change HDMI PHY node name to generic one
      commit: 5743efe0e73e4e1c8d042e982e31bb8145e35baf

Best regards,
-- 
Bjorn Andersson <andersson@kernel.org>

^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2022-11-07  3:12 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-09-24  9:43 [PATCH 0/2] arm: dts: qcom: rename HDMI PHY nodes Dmitry Baryshkov
2022-09-24  9:43 ` [PATCH 1/2] ARM: dts: qcom-apq8064: change HDMI PHY node name to generic one Dmitry Baryshkov
2022-09-24 14:48   ` Konrad Dybcio
2022-09-29  9:36   ` Krzysztof Kozlowski
2022-09-24  9:43 ` [PATCH 2/2] arm64: dts: qcom: msm8996: " Dmitry Baryshkov
2022-09-24 14:48   ` Konrad Dybcio
2022-09-29  9:36   ` Krzysztof Kozlowski
2022-11-07  3:12 ` (subset) [PATCH 0/2] arm: dts: qcom: rename HDMI PHY nodes Bjorn Andersson

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).