mbox series

[0/2] arm: dts: qcom: rename HDMI PHY nodes

Message ID 20220924094347.178666-1-dmitry.baryshkov@linaro.org
Headers show
Series arm: dts: qcom: rename HDMI PHY nodes | expand

Message

Dmitry Baryshkov Sept. 24, 2022, 9:43 a.m. UTC 
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(-)

Comments

Konrad Dybcio Sept. 24, 2022, 2:48 p.m. UTC | #1 
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>;
Bjorn Andersson Nov. 7, 2022, 3:12 a.m. UTC | #2 
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,