[PATCH 0/9] arm: dts: qcom: rename DSI PHY nodes

[PATCH 0/9] arm: dts: qcom: rename DSI PHY nodes

[Dmitry Baryshkov]

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

Dmitry Baryshkov (9):
  ARM: dts: qcom-apq8064: change DSI PHY node name to generic one
  ARM: dts: qcom-msm8974: change DSI PHY node name to generic one
  arm64: dts: qcom: msm8916: change DSI PHY node name to generic one
  arm64: dts: qcom: msm8996: change DSI PHY node name to generic one
  arm64: dts: qcom: sc7180: change DSI PHY node name to generic one
  arm64: dts: qcom: sdm630: change DSI PHY node name to generic one
  arm64: dts: qcom: sdm660: change DSI PHY node name to generic one
  arm64: dts: qcom: sdm845: change DSI PHY node name to generic one
  arm64: dts: qcom: sm8250: change DSI PHY node name to generic one

 arch/arm/boot/dts/qcom-apq8064-asus-nexus7-flo.dts | 2 +-
 arch/arm/boot/dts/qcom-apq8064.dtsi                | 2 +-
 arch/arm/boot/dts/qcom-msm8974.dtsi                | 2 +-
 arch/arm64/boot/dts/qcom/msm8916.dtsi              | 2 +-
 arch/arm64/boot/dts/qcom/msm8996.dtsi              | 4 ++--
 arch/arm64/boot/dts/qcom/sc7180.dtsi               | 2 +-
 arch/arm64/boot/dts/qcom/sdm630.dtsi               | 2 +-
 arch/arm64/boot/dts/qcom/sdm660.dtsi               | 2 +-
 arch/arm64/boot/dts/qcom/sdm845.dtsi               | 4 ++--
 arch/arm64/boot/dts/qcom/sm8250.dtsi               | 4 ++--
 10 files changed, 13 insertions(+), 13 deletions(-)

-- 
2.35.1

[PATCH 1/9] ARM: dts: qcom-apq8064: change DSI PHY node name to generic one

[Dmitry Baryshkov]

Change DSI PHY node names from custom 'dsi-phy' to the generic 'phy'.

Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
---
 arch/arm/boot/dts/qcom-apq8064-asus-nexus7-flo.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-asus-nexus7-flo.dts b/arch/arm/boot/dts/qcom-apq8064-asus-nexus7-flo.dts
index fee278e32cb6..2c5ee2e98fe2 100644
--- a/arch/arm/boot/dts/qcom-apq8064-asus-nexus7-flo.dts
+++ b/arch/arm/boot/dts/qcom-apq8064-asus-nexus7-flo.dts
@@ -252,7 +252,7 @@ dsi0_out: endpoint {
 			};
 		};
 
-		dsi-phy@4700200 {
+		phy@4700200 {
 			status = "okay";
 			vddio-supply = <&pm8921_lvs7>;/*VDD_PLL2_1 to 7*/
 		};
diff --git a/arch/arm/boot/dts/qcom-apq8064.dtsi b/arch/arm/boot/dts/qcom-apq8064.dtsi
index ada4c828bf2f..1b704c7ea890 100644
--- a/arch/arm/boot/dts/qcom-apq8064.dtsi
+++ b/arch/arm/boot/dts/qcom-apq8064.dtsi
@@ -1303,7 +1303,7 @@ dsi0_out: endpoint {
 		};
 
 
-		dsi0_phy: dsi-phy@4700200 {
+		dsi0_phy: phy@4700200 {
 			compatible = "qcom,dsi-phy-28nm-8960";
 			#clock-cells = <1>;
 			#phy-cells = <0>;
-- 
2.35.1

Re: [PATCH 1/9] ARM: dts: qcom-apq8064: change DSI PHY node name to generic one

[Krzysztof Kozlowski]

On 24/09/2022 11:01, Dmitry Baryshkov wrote:
> Change DSI PHY node names from custom 'dsi-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

[PATCH 2/9] ARM: dts: qcom-msm8974: change DSI PHY node name to generic one

[Dmitry Baryshkov]

Change DSI PHY node names from custom 'dsi-phy' to the generic 'phy'.

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

diff --git a/arch/arm/boot/dts/qcom-msm8974.dtsi b/arch/arm/boot/dts/qcom-msm8974.dtsi
index 8baca2a77717..d3a7de42f693 100644
--- a/arch/arm/boot/dts/qcom-msm8974.dtsi
+++ b/arch/arm/boot/dts/qcom-msm8974.dtsi
@@ -1592,7 +1592,7 @@ dsi0_out: endpoint {
 				};
 			};
 
-			dsi0_phy: dsi-phy@fd922a00 {
+			dsi0_phy: phy@fd922a00 {
 				compatible = "qcom,dsi-phy-28nm-hpm";
 				reg = <0xfd922a00 0xd4>,
 				      <0xfd922b00 0x280>,
-- 
2.35.1

Re: [PATCH 2/9] ARM: dts: qcom-msm8974: change DSI PHY node name to generic one

[Krzysztof Kozlowski]

On 24/09/2022 11:01, Dmitry Baryshkov wrote:
> Change DSI PHY node names from custom 'dsi-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

[PATCH 3/9] arm64: dts: qcom: msm8916: change DSI PHY node name to generic one

[Dmitry Baryshkov]

Change DSI PHY node names from custom 'dsi-phy' to the generic 'phy'.

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

diff --git a/arch/arm64/boot/dts/qcom/msm8916.dtsi b/arch/arm64/boot/dts/qcom/msm8916.dtsi
index 48bc2e09128d..9e0a5cb469c8 100644
--- a/arch/arm64/boot/dts/qcom/msm8916.dtsi
+++ b/arch/arm64/boot/dts/qcom/msm8916.dtsi
@@ -1056,7 +1056,7 @@ dsi0_out: endpoint {
 				};
 			};
 
-			dsi_phy0: dsi-phy@1a98300 {
+			dsi_phy0: phy@1a98300 {
 				compatible = "qcom,dsi-phy-28nm-lp";
 				reg = <0x01a98300 0xd4>,
 				      <0x01a98500 0x280>,
-- 
2.35.1

Re: [PATCH 3/9] arm64: dts: qcom: msm8916: change DSI PHY node name to generic one

[Krzysztof Kozlowski]

On 24/09/2022 11:01, Dmitry Baryshkov wrote:
> Change DSI PHY node names from custom 'dsi-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

[PATCH 4/9] arm64: dts: qcom: msm8996: change DSI PHY node name to generic one

[Dmitry Baryshkov]

Change DSI PHY node names from custom 'dsi-phy' to the generic 'phy'.

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

diff --git a/arch/arm64/boot/dts/qcom/msm8996.dtsi b/arch/arm64/boot/dts/qcom/msm8996.dtsi
index 742eac4ce9b3..a7d0e5d68141 100644
--- a/arch/arm64/boot/dts/qcom/msm8996.dtsi
+++ b/arch/arm64/boot/dts/qcom/msm8996.dtsi
@@ -1017,7 +1017,7 @@ dsi0_out: endpoint {
 				};
 			};
 
-			dsi0_phy: dsi-phy@994400 {
+			dsi0_phy: phy@994400 {
 				compatible = "qcom,dsi-phy-14nm";
 				reg = <0x00994400 0x100>,
 				      <0x00994500 0x300>,
@@ -1085,7 +1085,7 @@ dsi1_out: endpoint {
 				};
 			};
 
-			dsi1_phy: dsi-phy@996400 {
+			dsi1_phy: phy@996400 {
 				compatible = "qcom,dsi-phy-14nm";
 				reg = <0x00996400 0x100>,
 				      <0x00996500 0x300>,
-- 
2.35.1

Re: [PATCH 4/9] arm64: dts: qcom: msm8996: change DSI PHY node name to generic one

[Krzysztof Kozlowski]

On 24/09/2022 11:01, Dmitry Baryshkov wrote:
> Change DSI PHY node names from custom 'dsi-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

[PATCH 5/9] arm64: dts: qcom: sc7180: change DSI PHY node name to generic one

[Dmitry Baryshkov]

Change DSI PHY node names from custom 'dsi-phy' to the generic 'phy'.

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

diff --git a/arch/arm64/boot/dts/qcom/sc7180.dtsi b/arch/arm64/boot/dts/qcom/sc7180.dtsi
index b82c335c25af..0002d92eb29b 100644
--- a/arch/arm64/boot/dts/qcom/sc7180.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc7180.dtsi
@@ -3057,7 +3057,7 @@ opp-358000000 {
 				};
 			};
 
-			dsi_phy: dsi-phy@ae94400 {
+			dsi_phy: phy@ae94400 {
 				compatible = "qcom,dsi-phy-10nm";
 				reg = <0 0x0ae94400 0 0x200>,
 				      <0 0x0ae94600 0 0x280>,
-- 
2.35.1

Re: [PATCH 5/9] arm64: dts: qcom: sc7180: change DSI PHY node name to generic one

[Krzysztof Kozlowski]

On 24/09/2022 11:01, Dmitry Baryshkov wrote:
> Change DSI PHY node names from custom 'dsi-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

[PATCH 6/9] arm64: dts: qcom: sdm630: change DSI PHY node name to generic one

[Dmitry Baryshkov]

Change DSI PHY node names from custom 'dsi-phy' to the generic 'phy'.

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

diff --git a/arch/arm64/boot/dts/qcom/sdm630.dtsi b/arch/arm64/boot/dts/qcom/sdm630.dtsi
index 1bc9091cad2a..dc83329689a1 100644
--- a/arch/arm64/boot/dts/qcom/sdm630.dtsi
+++ b/arch/arm64/boot/dts/qcom/sdm630.dtsi
@@ -1633,7 +1633,7 @@ dsi0_out: endpoint {
 				};
 			};
 
-			dsi0_phy: dsi-phy@c994400 {
+			dsi0_phy: phy@c994400 {
 				compatible = "qcom,dsi-phy-14nm-660";
 				reg = <0x0c994400 0x100>,
 				      <0x0c994500 0x300>,
-- 
2.35.1

Re: [PATCH 6/9] arm64: dts: qcom: sdm630: change DSI PHY node name to generic one

[Krzysztof Kozlowski]

On 24/09/2022 11:01, Dmitry Baryshkov wrote:
> Change DSI PHY node names from custom 'dsi-phy' to the generic 'phy'.
> 
> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> ---
>  arch/arm64/boot/dts/qcom/sdm630.dtsi | 2 +-


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

Best regards,
Krzysztof

[PATCH 7/9] arm64: dts: qcom: sdm660: change DSI PHY node name to generic one

[Dmitry Baryshkov]

Change DSI PHY node names from custom 'dsi-phy' to the generic 'phy'.

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

diff --git a/arch/arm64/boot/dts/qcom/sdm660.dtsi b/arch/arm64/boot/dts/qcom/sdm660.dtsi
index 43220af1b685..d102c7e25b79 100644
--- a/arch/arm64/boot/dts/qcom/sdm660.dtsi
+++ b/arch/arm64/boot/dts/qcom/sdm660.dtsi
@@ -213,7 +213,7 @@ dsi1_out: endpoint {
 		};
 	};
 
-	dsi1_phy: dsi-phy@c996400 {
+	dsi1_phy: phy@c996400 {
 		compatible = "qcom,dsi-phy-14nm-660";
 		reg = <0x0c996400 0x100>,
 				<0x0c996500 0x300>,
-- 
2.35.1

Re: [PATCH 7/9] arm64: dts: qcom: sdm660: change DSI PHY node name to generic one

[Krzysztof Kozlowski]

On 24/09/2022 11:01, Dmitry Baryshkov wrote:
> Change DSI PHY node names from custom 'dsi-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

[PATCH 8/9] arm64: dts: qcom: sdm845: change DSI PHY node name to generic one

[Dmitry Baryshkov]

Change DSI PHY node names from custom 'dsi-phy' to the generic 'phy'.

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

diff --git a/arch/arm64/boot/dts/qcom/sdm845.dtsi b/arch/arm64/boot/dts/qcom/sdm845.dtsi
index f0e286715d1b..df6427ba2284 100644
--- a/arch/arm64/boot/dts/qcom/sdm845.dtsi
+++ b/arch/arm64/boot/dts/qcom/sdm845.dtsi
@@ -4537,7 +4537,7 @@ dsi0_out: endpoint {
 				};
 			};
 
-			dsi0_phy: dsi-phy@ae94400 {
+			dsi0_phy: phy@ae94400 {
 				compatible = "qcom,dsi-phy-10nm";
 				reg = <0 0x0ae94400 0 0x200>,
 				      <0 0x0ae94600 0 0x280>,
@@ -4609,7 +4609,7 @@ dsi1_out: endpoint {
 				};
 			};
 
-			dsi1_phy: dsi-phy@ae96400 {
+			dsi1_phy: phy@ae96400 {
 				compatible = "qcom,dsi-phy-10nm";
 				reg = <0 0x0ae96400 0 0x200>,
 				      <0 0x0ae96600 0 0x280>,
-- 
2.35.1

Re: [PATCH 8/9] arm64: dts: qcom: sdm845: change DSI PHY node name to generic one

[Krzysztof Kozlowski]

On 24/09/2022 11:01, Dmitry Baryshkov wrote:
> Change DSI PHY node names from custom 'dsi-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

Re: [PATCH 8/9] arm64: dts: qcom: sdm845: change DSI PHY node name to generic one

[Steev Klimaszewski]

On 9/24/22 4:01 AM, Dmitry Baryshkov wrote:
> Change DSI PHY node names from custom 'dsi-phy' to the generic 'phy'.
>
> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> ---
>   arch/arm64/boot/dts/qcom/sdm845.dtsi | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/qcom/sdm845.dtsi b/arch/arm64/boot/dts/qcom/sdm845.dtsi
> index f0e286715d1b..df6427ba2284 100644
> --- a/arch/arm64/boot/dts/qcom/sdm845.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sdm845.dtsi
> @@ -4537,7 +4537,7 @@ dsi0_out: endpoint {
>   				};
>   			};
>   
> -			dsi0_phy: dsi-phy@ae94400 {
> +			dsi0_phy: phy@ae94400 {
>   				compatible = "qcom,dsi-phy-10nm";
>   				reg = <0 0x0ae94400 0 0x200>,
>   				      <0 0x0ae94600 0 0x280>,
> @@ -4609,7 +4609,7 @@ dsi1_out: endpoint {
>   				};
>   			};
>   
> -			dsi1_phy: dsi-phy@ae96400 {
> +			dsi1_phy: phy@ae96400 {
>   				compatible = "qcom,dsi-phy-10nm";
>   				reg = <0 0x0ae96400 0 0x200>,
>   				      <0 0x0ae96600 0 0x280>,

Tested on Lenovo Yoga C630

Tested-by: Steev Klimaszewski <steev@kali.org>

[PATCH 9/9] arm64: dts: qcom: sm8250: change DSI PHY node name to generic one

[Dmitry Baryshkov]

Change DSI PHY node names from custom 'dsi-phy' to the generic 'phy'.

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

diff --git a/arch/arm64/boot/dts/qcom/sm8250.dtsi b/arch/arm64/boot/dts/qcom/sm8250.dtsi
index 5843e46a3164..e3ea39fcec13 100644
--- a/arch/arm64/boot/dts/qcom/sm8250.dtsi
+++ b/arch/arm64/boot/dts/qcom/sm8250.dtsi
@@ -3592,7 +3592,7 @@ opp-358000000 {
 				};
 			};
 
-			dsi0_phy: dsi-phy@ae94400 {
+			dsi0_phy: phy@ae94400 {
 				compatible = "qcom,dsi-phy-7nm";
 				reg = <0 0x0ae94400 0 0x200>,
 				      <0 0x0ae94600 0 0x280>,
@@ -3665,7 +3665,7 @@ dsi1_out: endpoint {
 				};
 			};
 
-			dsi1_phy: dsi-phy@ae96400 {
+			dsi1_phy: phy@ae96400 {
 				compatible = "qcom,dsi-phy-7nm";
 				reg = <0 0x0ae96400 0 0x200>,
 				      <0 0x0ae96600 0 0x280>,
-- 
2.35.1

Re: [PATCH 9/9] arm64: dts: qcom: sm8250: change DSI PHY node name to generic one

[Krzysztof Kozlowski]

On 24/09/2022 11:01, Dmitry Baryshkov wrote:
> Change DSI PHY node names from custom 'dsi-phy' to the generic 'phy'.
> 


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

Best regards,
Krzysztof

Re: [PATCH 0/9] arm: dts: qcom: rename DSI PHY nodes

[Krzysztof Kozlowski]

On 24/09/2022 11:00, Dmitry Baryshkov wrote:
> Historically DSI PHY device tree nodes used the dsi-phy@ names. Replace
> them with generic phy@ names.
> 
> Dmitry Baryshkov (9):
>   ARM: dts: qcom-apq8064: change DSI PHY node name to generic one

dsi-phy is similar to other generic names (usb-phy, ethernet-phy) [1] so
it could stay. Is there particular need for this, like coming from DT
schema?

[1]
https://devicetree-specification.readthedocs.io/en/latest/chapter2-devicetree-basics.html#generic-names-recommendation


Best regards,
Krzysztof

Re: [PATCH 0/9] arm: dts: qcom: rename DSI PHY nodes

[Dmitry Baryshkov]

On 24/09/2022 12:11, Krzysztof Kozlowski wrote:
> On 24/09/2022 11:00, Dmitry Baryshkov wrote:
>> Historically DSI PHY device tree nodes used the dsi-phy@ names. Replace
>> them with generic phy@ names.
>>
>> Dmitry Baryshkov (9):
>>    ARM: dts: qcom-apq8064: change DSI PHY node name to generic one
> 
> dsi-phy is similar to other generic names (usb-phy, ethernet-phy) [1] so
> it could stay. Is there particular need for this, like coming from DT
> schema?

No, there is no requirement from the DT schema. However:
1) The resent qcom DT files already use just phy@ for most of PHY nodes
2) The recommended list 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).

> 
> [1]
> https://devicetree-specification.readthedocs.io/en/latest/chapter2-devicetree-basics.html#generic-names-recommendation
> 
> 
> Best regards,
> Krzysztof
> 

-- 
With best wishes
Dmitry

Re: [PATCH 0/9] arm: dts: qcom: rename DSI PHY nodes

[Krzysztof Kozlowski]

On 24/09/2022 11:20, Dmitry Baryshkov wrote:
> On 24/09/2022 12:11, Krzysztof Kozlowski wrote:
>> On 24/09/2022 11:00, Dmitry Baryshkov wrote:
>>> Historically DSI PHY device tree nodes used the dsi-phy@ names. Replace
>>> them with generic phy@ names.
>>>
>>> Dmitry Baryshkov (9):
>>>    ARM: dts: qcom-apq8064: change DSI PHY node name to generic one
>>
>> dsi-phy is similar to other generic names (usb-phy, ethernet-phy) [1] so
>> it could stay. Is there particular need for this, like coming from DT
>> schema?
> 
> No, there is no requirement from the DT schema. However:
> 1) The resent qcom DT files already use just phy@ for most of PHY nodes
> 2) The recommended list 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).

Good explanation. If there is going to be resent/submit, please add it
to cover letter. :)

Best regards,
Krzysztof

Re: [PATCH 0/9] arm: dts: qcom: rename DSI PHY nodes

[Dmitry Baryshkov]

On 24/09/2022 12:29, Krzysztof Kozlowski wrote:
> On 24/09/2022 11:20, Dmitry Baryshkov wrote:
>> On 24/09/2022 12:11, Krzysztof Kozlowski wrote:
>>> On 24/09/2022 11:00, Dmitry Baryshkov wrote:
>>>> Historically DSI PHY device tree nodes used the dsi-phy@ names. Replace
>>>> them with generic phy@ names.
>>>>
>>>> Dmitry Baryshkov (9):
>>>>     ARM: dts: qcom-apq8064: change DSI PHY node name to generic one
>>>
>>> dsi-phy is similar to other generic names (usb-phy, ethernet-phy) [1] so
>>> it could stay. Is there particular need for this, like coming from DT
>>> schema?
>>
>> No, there is no requirement from the DT schema. However:
>> 1) The resent qcom DT files already use just phy@ for most of PHY nodes
>> 2) The recommended list 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).
> 
> Good explanation. If there is going to be resent/submit, please add it
> to cover letter. :)

I hope, there will be no v2. For now I have included this explanation 
into the HDMI PHY counterpart.

-- 
With best wishes
Dmitry

Re: [PATCH 0/9] arm: dts: qcom: rename DSI PHY nodes

[Konrad Dybcio]

On 24.09.2022 11:00, Dmitry Baryshkov wrote:
> Historically DSI PHY device tree nodes used the dsi-phy@ names. Replace
> them with generic phy@ names.
> 
> Dmitry Baryshkov (9):
>   ARM: dts: qcom-apq8064: change DSI PHY node name to generic one
>   ARM: dts: qcom-msm8974: change DSI PHY node name to generic one
>   arm64: dts: qcom: msm8916: change DSI PHY node name to generic one
>   arm64: dts: qcom: msm8996: change DSI PHY node name to generic one
>   arm64: dts: qcom: sc7180: change DSI PHY node name to generic one
>   arm64: dts: qcom: sdm630: change DSI PHY node name to generic one
>   arm64: dts: qcom: sdm660: change DSI PHY node name to generic one
>   arm64: dts: qcom: sdm845: change DSI PHY node name to generic one
>   arm64: dts: qcom: sm8250: change DSI PHY node name to generic one
Reviewed-by: Konrad Dybcio <konrad.dybcio@somainline.org>

for the entire series

Konrad
> 
>  arch/arm/boot/dts/qcom-apq8064-asus-nexus7-flo.dts | 2 +-
>  arch/arm/boot/dts/qcom-apq8064.dtsi                | 2 +-
>  arch/arm/boot/dts/qcom-msm8974.dtsi                | 2 +-
>  arch/arm64/boot/dts/qcom/msm8916.dtsi              | 2 +-
>  arch/arm64/boot/dts/qcom/msm8996.dtsi              | 4 ++--
>  arch/arm64/boot/dts/qcom/sc7180.dtsi               | 2 +-
>  arch/arm64/boot/dts/qcom/sdm630.dtsi               | 2 +-
>  arch/arm64/boot/dts/qcom/sdm660.dtsi               | 2 +-
>  arch/arm64/boot/dts/qcom/sdm845.dtsi               | 4 ++--
>  arch/arm64/boot/dts/qcom/sm8250.dtsi               | 4 ++--
>  10 files changed, 13 insertions(+), 13 deletions(-)
> 

Re: (subset) [PATCH 0/9] arm: dts: qcom: rename DSI PHY nodes

[Bjorn Andersson]

On Sat, 24 Sep 2022 12:00:59 +0300, Dmitry Baryshkov wrote:
> Historically DSI PHY device tree nodes used the dsi-phy@ names. Replace
> them with generic phy@ names.
> 
> Dmitry Baryshkov (9):
>   ARM: dts: qcom-apq8064: change DSI PHY node name to generic one
>   ARM: dts: qcom-msm8974: change DSI PHY node name to generic one
>   arm64: dts: qcom: msm8916: change DSI PHY node name to generic one
>   arm64: dts: qcom: msm8996: change DSI PHY node name to generic one
>   arm64: dts: qcom: sc7180: change DSI PHY node name to generic one
>   arm64: dts: qcom: sdm630: change DSI PHY node name to generic one
>   arm64: dts: qcom: sdm660: change DSI PHY node name to generic one
>   arm64: dts: qcom: sdm845: change DSI PHY node name to generic one
>   arm64: dts: qcom: sm8250: change DSI PHY node name to generic one
> 
> [...]

Applied, thanks!

[1/9] ARM: dts: qcom-apq8064: change DSI PHY node name to generic one
      commit: cf6cea98662dc9ecf7707076b10499785870ff23
[2/9] ARM: dts: qcom-msm8974: change DSI PHY node name to generic one
      commit: 798e65cdd910a59a34de365ff9e00c186fb568b4

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