* [PATCH 00/11] qcom,mdss-dsi-ctrl: Remove redundant phy-names from yaml and dtsi
@ 2022-09-07 0:00 Bryan O'Donoghue
2022-09-07 0:00 ` [PATCH 01/11] dt-bindings: msm: dsi-controller-main: Drop redundant phy-names Bryan O'Donoghue
` (13 more replies)
0 siblings, 14 replies; 28+ messages in thread
From: Bryan O'Donoghue @ 2022-09-07 0:00 UTC (permalink / raw)
To: robdclark, quic_abhinavk, dmitry.baryshkov, sean, robh+dt,
krzysztof.kozlowski+dt, quic_mkrishn, agross, andersson,
konrad.dybcio, linux-arm-msm, devicetree
Cc: bryan.odonoghue
phy-names is both inconsistently used and not actually required by the driver.
Previous discussion with Krzysztof and Rob suggested to remove the property.
https://www.spinics.net/lists/linux-arm-msm/msg116525.html
- Mark as deprecated and not required in yaml
- Drop from associated dtsi files => grep -r mdss-dsi arch/arm* | grep qcom
Bryan O'Donoghue (11):
dt-bindings: msm: dsi-controller-main: Drop redundant phy-names
ARM: dts: qcom: apq8064: Drop redundant phy-names from DSI controller
ARM: dts: qcom: msm8974: Drop redundant phy-names from DSI controller
arm64: dts: qcom: msm8916: Drop redundant phy-names from DSI
controller
arm64: dts: qcom: msm8996: Drop redundant phy-names from DSI
controller
arm64: dts: qcom: sc7180: Drop redundant phy-names from DSI controller
arm64: dts: qcom: sc7280: Drop redundant phy-names from DSI controller
arm64: dts: qcom: sdm660: Drop redundant phy-names from DSI controller
arm64: dts: qcom: sdm630: Drop redundant phy-names from DSI controller
arm64: dts: qcom: sdm845: Drop redundant phy-names from DSI controller
arm64: dts: qcom: sm8250: Drop redundant phy-names from DSI controller
.../devicetree/bindings/display/msm/dsi-controller-main.yaml | 2 +-
arch/arm/boot/dts/qcom-apq8064.dtsi | 1 -
arch/arm/boot/dts/qcom-msm8974.dtsi | 1 -
arch/arm64/boot/dts/qcom/msm8916.dtsi | 1 -
arch/arm64/boot/dts/qcom/msm8996.dtsi | 2 --
arch/arm64/boot/dts/qcom/sc7180.dtsi | 1 -
arch/arm64/boot/dts/qcom/sc7280.dtsi | 1 -
arch/arm64/boot/dts/qcom/sdm630.dtsi | 1 -
arch/arm64/boot/dts/qcom/sdm660.dtsi | 1 -
arch/arm64/boot/dts/qcom/sdm845.dtsi | 2 --
arch/arm64/boot/dts/qcom/sm8250.dtsi | 2 --
11 files changed, 1 insertion(+), 14 deletions(-)
--
2.37.3
^ permalink raw reply [flat|nested] 28+ messages in thread
* [PATCH 01/11] dt-bindings: msm: dsi-controller-main: Drop redundant phy-names
2022-09-07 0:00 [PATCH 00/11] qcom,mdss-dsi-ctrl: Remove redundant phy-names from yaml and dtsi Bryan O'Donoghue
@ 2022-09-07 0:00 ` Bryan O'Donoghue
2022-09-08 14:11 ` Krzysztof Kozlowski
2022-09-08 22:31 ` Dmitry Baryshkov
2022-09-07 0:00 ` [PATCH 02/11] ARM: dts: qcom: apq8064: Drop redundant phy-names from DSI controller Bryan O'Donoghue
` (12 subsequent siblings)
13 siblings, 2 replies; 28+ messages in thread
From: Bryan O'Donoghue @ 2022-09-07 0:00 UTC (permalink / raw)
To: robdclark, quic_abhinavk, dmitry.baryshkov, sean, robh+dt,
krzysztof.kozlowski+dt, quic_mkrishn, agross, andersson,
konrad.dybcio, linux-arm-msm, devicetree
Cc: bryan.odonoghue
Adding in msm8939 which is based msm8916 dtsi I stumbled across a binding
check complaining about the phy name for msm8916 which we were reusing for
msm8939.
The currently inconsistent upstream dtsi naming of "dsi" and "dsi-phy" is
not captured in the yaml for this driver.
The driver however doesn't care about the name of DSI phy, hence the yaml
check is redundant.
Both Krzysztof and Rob suggested we could drop the phy-names entirely if it
really isn't a dependency.
So, drop the inconsistent and unnecessary phy-names field from the yaml.
Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
---
.../devicetree/bindings/display/msm/dsi-controller-main.yaml | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
index 880bfe930830c..3742e2ab4fb1d 100644
--- a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
+++ b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
@@ -49,6 +49,7 @@ properties:
maxItems: 1
phy-names:
+ deprecated: true
const: dsi
"#address-cells": true
@@ -133,7 +134,6 @@ required:
- clocks
- clock-names
- phys
- - phy-names
- assigned-clocks
- assigned-clock-parents
- power-domains
--
2.37.3
^ permalink raw reply related [flat|nested] 28+ messages in thread
* [PATCH 02/11] ARM: dts: qcom: apq8064: Drop redundant phy-names from DSI controller
2022-09-07 0:00 [PATCH 00/11] qcom,mdss-dsi-ctrl: Remove redundant phy-names from yaml and dtsi Bryan O'Donoghue
2022-09-07 0:00 ` [PATCH 01/11] dt-bindings: msm: dsi-controller-main: Drop redundant phy-names Bryan O'Donoghue
@ 2022-09-07 0:00 ` Bryan O'Donoghue
2022-09-08 14:12 ` Krzysztof Kozlowski
2022-09-07 0:00 ` [PATCH 03/11] ARM: dts: qcom: msm8974: " Bryan O'Donoghue
` (11 subsequent siblings)
13 siblings, 1 reply; 28+ messages in thread
From: Bryan O'Donoghue @ 2022-09-07 0:00 UTC (permalink / raw)
To: robdclark, quic_abhinavk, dmitry.baryshkov, sean, robh+dt,
krzysztof.kozlowski+dt, quic_mkrishn, agross, andersson,
konrad.dybcio, linux-arm-msm, devicetree
Cc: bryan.odonoghue
phy-names has been marked deprecated. Remove it from the apq8064 DSI
controller block.
Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
---
arch/arm/boot/dts/qcom-apq8064.dtsi | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/arm/boot/dts/qcom-apq8064.dtsi b/arch/arm/boot/dts/qcom-apq8064.dtsi
index ada4c828bf2f4..698764e862716 100644
--- a/arch/arm/boot/dts/qcom-apq8064.dtsi
+++ b/arch/arm/boot/dts/qcom-apq8064.dtsi
@@ -1281,7 +1281,6 @@ dsi0: dsi@4700000 {
<&dsi0_phy 1>;
syscon-sfpb = <&mmss_sfpb>;
phys = <&dsi0_phy>;
- phy-names = "dsi";
status = "disabled";
ports {
--
2.37.3
^ permalink raw reply related [flat|nested] 28+ messages in thread
* [PATCH 03/11] ARM: dts: qcom: msm8974: Drop redundant phy-names from DSI controller
2022-09-07 0:00 [PATCH 00/11] qcom,mdss-dsi-ctrl: Remove redundant phy-names from yaml and dtsi Bryan O'Donoghue
2022-09-07 0:00 ` [PATCH 01/11] dt-bindings: msm: dsi-controller-main: Drop redundant phy-names Bryan O'Donoghue
2022-09-07 0:00 ` [PATCH 02/11] ARM: dts: qcom: apq8064: Drop redundant phy-names from DSI controller Bryan O'Donoghue
@ 2022-09-07 0:00 ` Bryan O'Donoghue
2022-09-08 14:12 ` Krzysztof Kozlowski
2022-09-07 0:00 ` [PATCH 04/11] arm64: dts: qcom: msm8916: " Bryan O'Donoghue
` (10 subsequent siblings)
13 siblings, 1 reply; 28+ messages in thread
From: Bryan O'Donoghue @ 2022-09-07 0:00 UTC (permalink / raw)
To: robdclark, quic_abhinavk, dmitry.baryshkov, sean, robh+dt,
krzysztof.kozlowski+dt, quic_mkrishn, agross, andersson,
konrad.dybcio, linux-arm-msm, devicetree
Cc: bryan.odonoghue
phy-names has been marked deprecated. Remove it from the msm8974 DSI
controller block.
Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
---
arch/arm/boot/dts/qcom-msm8974.dtsi | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/arm/boot/dts/qcom-msm8974.dtsi b/arch/arm/boot/dts/qcom-msm8974.dtsi
index 8baca2a777176..979e88a064cca 100644
--- a/arch/arm/boot/dts/qcom-msm8974.dtsi
+++ b/arch/arm/boot/dts/qcom-msm8974.dtsi
@@ -1566,7 +1566,6 @@ dsi0: dsi@fd922800 {
"core_mmss";
phys = <&dsi0_phy>;
- phy-names = "dsi-phy";
status = "disabled";
--
2.37.3
^ permalink raw reply related [flat|nested] 28+ messages in thread
* [PATCH 04/11] arm64: dts: qcom: msm8916: Drop redundant phy-names from DSI controller
2022-09-07 0:00 [PATCH 00/11] qcom,mdss-dsi-ctrl: Remove redundant phy-names from yaml and dtsi Bryan O'Donoghue
` (2 preceding siblings ...)
2022-09-07 0:00 ` [PATCH 03/11] ARM: dts: qcom: msm8974: " Bryan O'Donoghue
@ 2022-09-07 0:00 ` Bryan O'Donoghue
2022-09-08 14:12 ` Krzysztof Kozlowski
2022-09-07 0:00 ` [PATCH 05/11] arm64: dts: qcom: msm8996: " Bryan O'Donoghue
` (9 subsequent siblings)
13 siblings, 1 reply; 28+ messages in thread
From: Bryan O'Donoghue @ 2022-09-07 0:00 UTC (permalink / raw)
To: robdclark, quic_abhinavk, dmitry.baryshkov, sean, robh+dt,
krzysztof.kozlowski+dt, quic_mkrishn, agross, andersson,
konrad.dybcio, linux-arm-msm, devicetree
Cc: bryan.odonoghue
phy-names has been marked deprecated. Remove it from the msm8916 DSI
controller block.
Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
---
arch/arm64/boot/dts/qcom/msm8916.dtsi | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/arm64/boot/dts/qcom/msm8916.dtsi b/arch/arm64/boot/dts/qcom/msm8916.dtsi
index 7a9882e39e905..f0f457688f3a4 100644
--- a/arch/arm64/boot/dts/qcom/msm8916.dtsi
+++ b/arch/arm64/boot/dts/qcom/msm8916.dtsi
@@ -1032,7 +1032,6 @@ dsi0: dsi@1a98000 {
"pixel",
"core";
phys = <&dsi_phy0>;
- phy-names = "dsi-phy";
#address-cells = <1>;
#size-cells = <0>;
--
2.37.3
^ permalink raw reply related [flat|nested] 28+ messages in thread
* [PATCH 05/11] arm64: dts: qcom: msm8996: Drop redundant phy-names from DSI controller
2022-09-07 0:00 [PATCH 00/11] qcom,mdss-dsi-ctrl: Remove redundant phy-names from yaml and dtsi Bryan O'Donoghue
` (3 preceding siblings ...)
2022-09-07 0:00 ` [PATCH 04/11] arm64: dts: qcom: msm8916: " Bryan O'Donoghue
@ 2022-09-07 0:00 ` Bryan O'Donoghue
2022-09-08 14:12 ` Krzysztof Kozlowski
2022-09-07 0:01 ` [PATCH 06/11] arm64: dts: qcom: sc7180: " Bryan O'Donoghue
` (8 subsequent siblings)
13 siblings, 1 reply; 28+ messages in thread
From: Bryan O'Donoghue @ 2022-09-07 0:00 UTC (permalink / raw)
To: robdclark, quic_abhinavk, dmitry.baryshkov, sean, robh+dt,
krzysztof.kozlowski+dt, quic_mkrishn, agross, andersson,
konrad.dybcio, linux-arm-msm, devicetree
Cc: bryan.odonoghue
phy-names has been marked deprecated. Remove it from the msm8996 DSI
controller block.
Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
---
arch/arm64/boot/dts/qcom/msm8996.dtsi | 2 --
1 file changed, 2 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/msm8996.dtsi b/arch/arm64/boot/dts/qcom/msm8996.dtsi
index 742eac4ce9b35..cb15dfce76efb 100644
--- a/arch/arm64/boot/dts/qcom/msm8996.dtsi
+++ b/arch/arm64/boot/dts/qcom/msm8996.dtsi
@@ -992,7 +992,6 @@ dsi0: dsi@994000 {
assigned-clock-parents = <&dsi0_phy 0>, <&dsi0_phy 1>;
phys = <&dsi0_phy>;
- phy-names = "dsi";
status = "disabled";
#address-cells = <1>;
@@ -1060,7 +1059,6 @@ dsi1: dsi@996000 {
assigned-clock-parents = <&dsi1_phy 0>, <&dsi1_phy 1>;
phys = <&dsi1_phy>;
- phy-names = "dsi";
status = "disabled";
#address-cells = <1>;
--
2.37.3
^ permalink raw reply related [flat|nested] 28+ messages in thread
* [PATCH 06/11] arm64: dts: qcom: sc7180: Drop redundant phy-names from DSI controller
2022-09-07 0:00 [PATCH 00/11] qcom,mdss-dsi-ctrl: Remove redundant phy-names from yaml and dtsi Bryan O'Donoghue
` (4 preceding siblings ...)
2022-09-07 0:00 ` [PATCH 05/11] arm64: dts: qcom: msm8996: " Bryan O'Donoghue
@ 2022-09-07 0:01 ` Bryan O'Donoghue
2022-09-08 14:12 ` Krzysztof Kozlowski
2022-09-07 0:01 ` [PATCH 07/11] arm64: dts: qcom: sc7280: " Bryan O'Donoghue
` (7 subsequent siblings)
13 siblings, 1 reply; 28+ messages in thread
From: Bryan O'Donoghue @ 2022-09-07 0:01 UTC (permalink / raw)
To: robdclark, quic_abhinavk, dmitry.baryshkov, sean, robh+dt,
krzysztof.kozlowski+dt, quic_mkrishn, agross, andersson,
konrad.dybcio, linux-arm-msm, devicetree
Cc: bryan.odonoghue
phy-names has been marked deprecated. Remove it from the sc7180 DSI
controller block.
Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
---
arch/arm64/boot/dts/qcom/sc7180.dtsi | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/arm64/boot/dts/qcom/sc7180.dtsi b/arch/arm64/boot/dts/qcom/sc7180.dtsi
index b82c335c25aff..932e9cabe48e5 100644
--- a/arch/arm64/boot/dts/qcom/sc7180.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc7180.dtsi
@@ -3012,7 +3012,6 @@ dsi0: dsi@ae94000 {
power-domains = <&rpmhpd SC7180_CX>;
phys = <&dsi_phy>;
- phy-names = "dsi";
#address-cells = <1>;
#size-cells = <0>;
--
2.37.3
^ permalink raw reply related [flat|nested] 28+ messages in thread
* [PATCH 07/11] arm64: dts: qcom: sc7280: Drop redundant phy-names from DSI controller
2022-09-07 0:00 [PATCH 00/11] qcom,mdss-dsi-ctrl: Remove redundant phy-names from yaml and dtsi Bryan O'Donoghue
` (5 preceding siblings ...)
2022-09-07 0:01 ` [PATCH 06/11] arm64: dts: qcom: sc7180: " Bryan O'Donoghue
@ 2022-09-07 0:01 ` Bryan O'Donoghue
2022-09-08 14:12 ` Krzysztof Kozlowski
2022-09-07 0:01 ` [PATCH 08/11] arm64: dts: qcom: sdm660: " Bryan O'Donoghue
` (6 subsequent siblings)
13 siblings, 1 reply; 28+ messages in thread
From: Bryan O'Donoghue @ 2022-09-07 0:01 UTC (permalink / raw)
To: robdclark, quic_abhinavk, dmitry.baryshkov, sean, robh+dt,
krzysztof.kozlowski+dt, quic_mkrishn, agross, andersson,
konrad.dybcio, linux-arm-msm, devicetree
Cc: bryan.odonoghue
phy-names has been marked deprecated. Remove it from the sc7280 DSI
controller block.
Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
---
arch/arm64/boot/dts/qcom/sc7280.dtsi | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/arm64/boot/dts/qcom/sc7280.dtsi b/arch/arm64/boot/dts/qcom/sc7280.dtsi
index 13d7f267b2891..7dc94e84243ab 100644
--- a/arch/arm64/boot/dts/qcom/sc7280.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc7280.dtsi
@@ -3640,7 +3640,6 @@ mdss_dsi: dsi@ae94000 {
power-domains = <&rpmhpd SC7280_CX>;
phys = <&mdss_dsi_phy>;
- phy-names = "dsi";
#address-cells = <1>;
#size-cells = <0>;
--
2.37.3
^ permalink raw reply related [flat|nested] 28+ messages in thread
* [PATCH 08/11] arm64: dts: qcom: sdm660: Drop redundant phy-names from DSI controller
2022-09-07 0:00 [PATCH 00/11] qcom,mdss-dsi-ctrl: Remove redundant phy-names from yaml and dtsi Bryan O'Donoghue
` (6 preceding siblings ...)
2022-09-07 0:01 ` [PATCH 07/11] arm64: dts: qcom: sc7280: " Bryan O'Donoghue
@ 2022-09-07 0:01 ` Bryan O'Donoghue
2022-09-08 14:12 ` Krzysztof Kozlowski
2022-09-07 0:01 ` [PATCH 09/11] arm64: dts: qcom: sdm630: " Bryan O'Donoghue
` (5 subsequent siblings)
13 siblings, 1 reply; 28+ messages in thread
From: Bryan O'Donoghue @ 2022-09-07 0:01 UTC (permalink / raw)
To: robdclark, quic_abhinavk, dmitry.baryshkov, sean, robh+dt,
krzysztof.kozlowski+dt, quic_mkrishn, agross, andersson,
konrad.dybcio, linux-arm-msm, devicetree
Cc: bryan.odonoghue
phy-names has been marked deprecated. Remove it from the sdm660 DSI
controller block.
Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
---
arch/arm64/boot/dts/qcom/sdm660.dtsi | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/arm64/boot/dts/qcom/sdm660.dtsi b/arch/arm64/boot/dts/qcom/sdm660.dtsi
index 43220af1b685e..10bf1c45cf6ec 100644
--- a/arch/arm64/boot/dts/qcom/sdm660.dtsi
+++ b/arch/arm64/boot/dts/qcom/sdm660.dtsi
@@ -190,7 +190,6 @@ dsi1: dsi@c996000 {
"core";
phys = <&dsi1_phy>;
- phy-names = "dsi";
status = "disabled";
--
2.37.3
^ permalink raw reply related [flat|nested] 28+ messages in thread
* [PATCH 09/11] arm64: dts: qcom: sdm630: Drop redundant phy-names from DSI controller
2022-09-07 0:00 [PATCH 00/11] qcom,mdss-dsi-ctrl: Remove redundant phy-names from yaml and dtsi Bryan O'Donoghue
` (7 preceding siblings ...)
2022-09-07 0:01 ` [PATCH 08/11] arm64: dts: qcom: sdm660: " Bryan O'Donoghue
@ 2022-09-07 0:01 ` Bryan O'Donoghue
2022-09-08 14:13 ` Krzysztof Kozlowski
2022-09-07 0:01 ` [PATCH 10/11] arm64: dts: qcom: sdm845: " Bryan O'Donoghue
` (4 subsequent siblings)
13 siblings, 1 reply; 28+ messages in thread
From: Bryan O'Donoghue @ 2022-09-07 0:01 UTC (permalink / raw)
To: robdclark, quic_abhinavk, dmitry.baryshkov, sean, robh+dt,
krzysztof.kozlowski+dt, quic_mkrishn, agross, andersson,
konrad.dybcio, linux-arm-msm, devicetree
Cc: bryan.odonoghue
phy-names has been marked deprecated. Remove it from the sdm630 DSI
controller block.
Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
---
arch/arm64/boot/dts/qcom/sdm630.dtsi | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/arm64/boot/dts/qcom/sdm630.dtsi b/arch/arm64/boot/dts/qcom/sdm630.dtsi
index 1bc9091cad2a8..caf424a449be1 100644
--- a/arch/arm64/boot/dts/qcom/sdm630.dtsi
+++ b/arch/arm64/boot/dts/qcom/sdm630.dtsi
@@ -1610,7 +1610,6 @@ dsi0: dsi@c994000 {
"core";
phys = <&dsi0_phy>;
- phy-names = "dsi";
status = "disabled";
--
2.37.3
^ permalink raw reply related [flat|nested] 28+ messages in thread
* [PATCH 10/11] arm64: dts: qcom: sdm845: Drop redundant phy-names from DSI controller
2022-09-07 0:00 [PATCH 00/11] qcom,mdss-dsi-ctrl: Remove redundant phy-names from yaml and dtsi Bryan O'Donoghue
` (8 preceding siblings ...)
2022-09-07 0:01 ` [PATCH 09/11] arm64: dts: qcom: sdm630: " Bryan O'Donoghue
@ 2022-09-07 0:01 ` Bryan O'Donoghue
2022-09-07 3:13 ` Steev Klimaszewski
2022-09-08 14:13 ` Krzysztof Kozlowski
2022-09-07 0:01 ` [PATCH 11/11] arm64: dts: qcom: sm8250: " Bryan O'Donoghue
` (3 subsequent siblings)
13 siblings, 2 replies; 28+ messages in thread
From: Bryan O'Donoghue @ 2022-09-07 0:01 UTC (permalink / raw)
To: robdclark, quic_abhinavk, dmitry.baryshkov, sean, robh+dt,
krzysztof.kozlowski+dt, quic_mkrishn, agross, andersson,
konrad.dybcio, linux-arm-msm, devicetree
Cc: bryan.odonoghue
phy-names has been marked deprecated. Remove it from the sdm845 DSI
controller block.
Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
---
arch/arm64/boot/dts/qcom/sdm845.dtsi | 2 --
1 file changed, 2 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/sdm845.dtsi b/arch/arm64/boot/dts/qcom/sdm845.dtsi
index f0e286715d1bd..813af058fdec4 100644
--- a/arch/arm64/boot/dts/qcom/sdm845.dtsi
+++ b/arch/arm64/boot/dts/qcom/sdm845.dtsi
@@ -4511,7 +4511,6 @@ dsi0: dsi@ae94000 {
power-domains = <&rpmhpd SDM845_CX>;
phys = <&dsi0_phy>;
- phy-names = "dsi";
status = "disabled";
@@ -4583,7 +4582,6 @@ dsi1: dsi@ae96000 {
power-domains = <&rpmhpd SDM845_CX>;
phys = <&dsi1_phy>;
- phy-names = "dsi";
status = "disabled";
--
2.37.3
^ permalink raw reply related [flat|nested] 28+ messages in thread
* [PATCH 11/11] arm64: dts: qcom: sm8250: Drop redundant phy-names from DSI controller
2022-09-07 0:00 [PATCH 00/11] qcom,mdss-dsi-ctrl: Remove redundant phy-names from yaml and dtsi Bryan O'Donoghue
` (9 preceding siblings ...)
2022-09-07 0:01 ` [PATCH 10/11] arm64: dts: qcom: sdm845: " Bryan O'Donoghue
@ 2022-09-07 0:01 ` Bryan O'Donoghue
2022-09-08 14:13 ` Krzysztof Kozlowski
2022-09-08 22:30 ` [PATCH 00/11] qcom,mdss-dsi-ctrl: Remove redundant phy-names from yaml and dtsi Dmitry Baryshkov
` (2 subsequent siblings)
13 siblings, 1 reply; 28+ messages in thread
From: Bryan O'Donoghue @ 2022-09-07 0:01 UTC (permalink / raw)
To: robdclark, quic_abhinavk, dmitry.baryshkov, sean, robh+dt,
krzysztof.kozlowski+dt, quic_mkrishn, agross, andersson,
konrad.dybcio, linux-arm-msm, devicetree
Cc: bryan.odonoghue
phy-names has been marked deprecated. Remove it from the sm8250 DSI
controller block.
Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
---
arch/arm64/boot/dts/qcom/sm8250.dtsi | 2 --
1 file changed, 2 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/sm8250.dtsi b/arch/arm64/boot/dts/qcom/sm8250.dtsi
index bc773e210023c..10e9b76e2aa69 100644
--- a/arch/arm64/boot/dts/qcom/sm8250.dtsi
+++ b/arch/arm64/boot/dts/qcom/sm8250.dtsi
@@ -3547,7 +3547,6 @@ dsi0: dsi@ae94000 {
power-domains = <&rpmhpd SM8250_MMCX>;
phys = <&dsi0_phy>;
- phy-names = "dsi";
status = "disabled";
@@ -3620,7 +3619,6 @@ dsi1: dsi@ae96000 {
power-domains = <&rpmhpd SM8250_MMCX>;
phys = <&dsi1_phy>;
- phy-names = "dsi";
status = "disabled";
--
2.37.3
^ permalink raw reply related [flat|nested] 28+ messages in thread
* Re: [PATCH 10/11] arm64: dts: qcom: sdm845: Drop redundant phy-names from DSI controller
2022-09-07 0:01 ` [PATCH 10/11] arm64: dts: qcom: sdm845: " Bryan O'Donoghue
@ 2022-09-07 3:13 ` Steev Klimaszewski
2022-09-08 14:13 ` Krzysztof Kozlowski
1 sibling, 0 replies; 28+ messages in thread
From: Steev Klimaszewski @ 2022-09-07 3:13 UTC (permalink / raw)
To: Bryan O'Donoghue, robdclark, quic_abhinavk, dmitry.baryshkov,
sean, robh+dt, krzysztof.kozlowski+dt, quic_mkrishn, agross,
andersson, konrad.dybcio, linux-arm-msm, devicetree
On 9/6/22 7:01 PM, Bryan O'Donoghue wrote:
> phy-names has been marked deprecated. Remove it from the sdm845 DSI
> controller block.
>
> Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
> ---
> arch/arm64/boot/dts/qcom/sdm845.dtsi | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/qcom/sdm845.dtsi b/arch/arm64/boot/dts/qcom/sdm845.dtsi
> index f0e286715d1bd..813af058fdec4 100644
> --- a/arch/arm64/boot/dts/qcom/sdm845.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sdm845.dtsi
> @@ -4511,7 +4511,6 @@ dsi0: dsi@ae94000 {
> power-domains = <&rpmhpd SDM845_CX>;
>
> phys = <&dsi0_phy>;
> - phy-names = "dsi";
>
> status = "disabled";
>
> @@ -4583,7 +4582,6 @@ dsi1: dsi@ae96000 {
> power-domains = <&rpmhpd SDM845_CX>;
>
> phys = <&dsi1_phy>;
> - phy-names = "dsi";
>
> status = "disabled";
>
Tested on the Lenovo Yoga C630
Tested-by: Steev Klimaszewski <steev@kali.org>
^ permalink raw reply [flat|nested] 28+ messages in thread
* Re: [PATCH 01/11] dt-bindings: msm: dsi-controller-main: Drop redundant phy-names
2022-09-07 0:00 ` [PATCH 01/11] dt-bindings: msm: dsi-controller-main: Drop redundant phy-names Bryan O'Donoghue
@ 2022-09-08 14:11 ` Krzysztof Kozlowski
2022-09-08 22:31 ` Dmitry Baryshkov
1 sibling, 0 replies; 28+ messages in thread
From: Krzysztof Kozlowski @ 2022-09-08 14:11 UTC (permalink / raw)
To: Bryan O'Donoghue, robdclark, quic_abhinavk, dmitry.baryshkov,
sean, robh+dt, krzysztof.kozlowski+dt, quic_mkrishn, agross,
andersson, konrad.dybcio, linux-arm-msm, devicetree
On 07/09/2022 02:00, Bryan O'Donoghue wrote:
> Adding in msm8939 which is based msm8916 dtsi I stumbled across a binding
> check complaining about the phy name for msm8916 which we were reusing for
> msm8939.
>
> The currently inconsistent upstream dtsi naming of "dsi" and "dsi-phy" is
> not captured in the yaml for this driver.
>
> The driver however doesn't care about the name of DSI phy, hence the yaml
> check is redundant.
>
> Both Krzysztof and Rob suggested we could drop the phy-names entirely if it
> really isn't a dependency.
>
> So, drop the inconsistent and unnecessary phy-names field from the yaml.
>
> Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 28+ messages in thread
* Re: [PATCH 02/11] ARM: dts: qcom: apq8064: Drop redundant phy-names from DSI controller
2022-09-07 0:00 ` [PATCH 02/11] ARM: dts: qcom: apq8064: Drop redundant phy-names from DSI controller Bryan O'Donoghue
@ 2022-09-08 14:12 ` Krzysztof Kozlowski
0 siblings, 0 replies; 28+ messages in thread
From: Krzysztof Kozlowski @ 2022-09-08 14:12 UTC (permalink / raw)
To: Bryan O'Donoghue, robdclark, quic_abhinavk, dmitry.baryshkov,
sean, robh+dt, krzysztof.kozlowski+dt, quic_mkrishn, agross,
andersson, konrad.dybcio, linux-arm-msm, devicetree
On 07/09/2022 02:00, Bryan O'Donoghue wrote:
> phy-names has been marked deprecated. Remove it from the apq8064 DSI
> controller block.
>
> Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
> ---
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 28+ messages in thread
* Re: [PATCH 03/11] ARM: dts: qcom: msm8974: Drop redundant phy-names from DSI controller
2022-09-07 0:00 ` [PATCH 03/11] ARM: dts: qcom: msm8974: " Bryan O'Donoghue
@ 2022-09-08 14:12 ` Krzysztof Kozlowski
0 siblings, 0 replies; 28+ messages in thread
From: Krzysztof Kozlowski @ 2022-09-08 14:12 UTC (permalink / raw)
To: Bryan O'Donoghue, robdclark, quic_abhinavk, dmitry.baryshkov,
sean, robh+dt, krzysztof.kozlowski+dt, quic_mkrishn, agross,
andersson, konrad.dybcio, linux-arm-msm, devicetree
On 07/09/2022 02:00, Bryan O'Donoghue wrote:
> phy-names has been marked deprecated. Remove it from the msm8974 DSI
> controller block.
>
> Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
> ---
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 28+ messages in thread
* Re: [PATCH 04/11] arm64: dts: qcom: msm8916: Drop redundant phy-names from DSI controller
2022-09-07 0:00 ` [PATCH 04/11] arm64: dts: qcom: msm8916: " Bryan O'Donoghue
@ 2022-09-08 14:12 ` Krzysztof Kozlowski
0 siblings, 0 replies; 28+ messages in thread
From: Krzysztof Kozlowski @ 2022-09-08 14:12 UTC (permalink / raw)
To: Bryan O'Donoghue, robdclark, quic_abhinavk, dmitry.baryshkov,
sean, robh+dt, krzysztof.kozlowski+dt, quic_mkrishn, agross,
andersson, konrad.dybcio, linux-arm-msm, devicetree
On 07/09/2022 02:00, Bryan O'Donoghue wrote:
> phy-names has been marked deprecated. Remove it from the msm8916 DSI
> controller block.
>
> Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 28+ messages in thread
* Re: [PATCH 05/11] arm64: dts: qcom: msm8996: Drop redundant phy-names from DSI controller
2022-09-07 0:00 ` [PATCH 05/11] arm64: dts: qcom: msm8996: " Bryan O'Donoghue
@ 2022-09-08 14:12 ` Krzysztof Kozlowski
0 siblings, 0 replies; 28+ messages in thread
From: Krzysztof Kozlowski @ 2022-09-08 14:12 UTC (permalink / raw)
To: Bryan O'Donoghue, robdclark, quic_abhinavk, dmitry.baryshkov,
sean, robh+dt, krzysztof.kozlowski+dt, quic_mkrishn, agross,
andersson, konrad.dybcio, linux-arm-msm, devicetree
On 07/09/2022 02:00, Bryan O'Donoghue wrote:
> phy-names has been marked deprecated. Remove it from the msm8996 DSI
> controller block.
>
> Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 28+ messages in thread
* Re: [PATCH 06/11] arm64: dts: qcom: sc7180: Drop redundant phy-names from DSI controller
2022-09-07 0:01 ` [PATCH 06/11] arm64: dts: qcom: sc7180: " Bryan O'Donoghue
@ 2022-09-08 14:12 ` Krzysztof Kozlowski
0 siblings, 0 replies; 28+ messages in thread
From: Krzysztof Kozlowski @ 2022-09-08 14:12 UTC (permalink / raw)
To: Bryan O'Donoghue, robdclark, quic_abhinavk, dmitry.baryshkov,
sean, robh+dt, krzysztof.kozlowski+dt, quic_mkrishn, agross,
andersson, konrad.dybcio, linux-arm-msm, devicetree
On 07/09/2022 02:01, Bryan O'Donoghue wrote:
> phy-names has been marked deprecated. Remove it from the sc7180 DSI
> controller block.
>
> Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
> ---
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 28+ messages in thread
* Re: [PATCH 07/11] arm64: dts: qcom: sc7280: Drop redundant phy-names from DSI controller
2022-09-07 0:01 ` [PATCH 07/11] arm64: dts: qcom: sc7280: " Bryan O'Donoghue
@ 2022-09-08 14:12 ` Krzysztof Kozlowski
0 siblings, 0 replies; 28+ messages in thread
From: Krzysztof Kozlowski @ 2022-09-08 14:12 UTC (permalink / raw)
To: Bryan O'Donoghue, robdclark, quic_abhinavk, dmitry.baryshkov,
sean, robh+dt, krzysztof.kozlowski+dt, quic_mkrishn, agross,
andersson, konrad.dybcio, linux-arm-msm, devicetree
On 07/09/2022 02:01, Bryan O'Donoghue wrote:
> phy-names has been marked deprecated. Remove it from the sc7280 DSI
> controller block.
>
> Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
> ---
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 28+ messages in thread
* Re: [PATCH 08/11] arm64: dts: qcom: sdm660: Drop redundant phy-names from DSI controller
2022-09-07 0:01 ` [PATCH 08/11] arm64: dts: qcom: sdm660: " Bryan O'Donoghue
@ 2022-09-08 14:12 ` Krzysztof Kozlowski
0 siblings, 0 replies; 28+ messages in thread
From: Krzysztof Kozlowski @ 2022-09-08 14:12 UTC (permalink / raw)
To: Bryan O'Donoghue, robdclark, quic_abhinavk, dmitry.baryshkov,
sean, robh+dt, krzysztof.kozlowski+dt, quic_mkrishn, agross,
andersson, konrad.dybcio, linux-arm-msm, devicetree
On 07/09/2022 02:01, Bryan O'Donoghue wrote:
> phy-names has been marked deprecated. Remove it from the sdm660 DSI
> controller block.
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 28+ messages in thread
* Re: [PATCH 09/11] arm64: dts: qcom: sdm630: Drop redundant phy-names from DSI controller
2022-09-07 0:01 ` [PATCH 09/11] arm64: dts: qcom: sdm630: " Bryan O'Donoghue
@ 2022-09-08 14:13 ` Krzysztof Kozlowski
0 siblings, 0 replies; 28+ messages in thread
From: Krzysztof Kozlowski @ 2022-09-08 14:13 UTC (permalink / raw)
To: Bryan O'Donoghue, robdclark, quic_abhinavk, dmitry.baryshkov,
sean, robh+dt, krzysztof.kozlowski+dt, quic_mkrishn, agross,
andersson, konrad.dybcio, linux-arm-msm, devicetree
On 07/09/2022 02:01, Bryan O'Donoghue wrote:
> phy-names has been marked deprecated. Remove it from the sdm630 DSI
> controller block.
>
> Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 28+ messages in thread
* Re: [PATCH 10/11] arm64: dts: qcom: sdm845: Drop redundant phy-names from DSI controller
2022-09-07 0:01 ` [PATCH 10/11] arm64: dts: qcom: sdm845: " Bryan O'Donoghue
2022-09-07 3:13 ` Steev Klimaszewski
@ 2022-09-08 14:13 ` Krzysztof Kozlowski
1 sibling, 0 replies; 28+ messages in thread
From: Krzysztof Kozlowski @ 2022-09-08 14:13 UTC (permalink / raw)
To: Bryan O'Donoghue, robdclark, quic_abhinavk, dmitry.baryshkov,
sean, robh+dt, krzysztof.kozlowski+dt, quic_mkrishn, agross,
andersson, konrad.dybcio, linux-arm-msm, devicetree
On 07/09/2022 02:01, Bryan O'Donoghue wrote:
> phy-names has been marked deprecated. Remove it from the sdm845 DSI
> controller block.
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 28+ messages in thread
* Re: [PATCH 11/11] arm64: dts: qcom: sm8250: Drop redundant phy-names from DSI controller
2022-09-07 0:01 ` [PATCH 11/11] arm64: dts: qcom: sm8250: " Bryan O'Donoghue
@ 2022-09-08 14:13 ` Krzysztof Kozlowski
0 siblings, 0 replies; 28+ messages in thread
From: Krzysztof Kozlowski @ 2022-09-08 14:13 UTC (permalink / raw)
To: Bryan O'Donoghue, robdclark, quic_abhinavk, dmitry.baryshkov,
sean, robh+dt, krzysztof.kozlowski+dt, quic_mkrishn, agross,
andersson, konrad.dybcio, linux-arm-msm, devicetree
On 07/09/2022 02:01, Bryan O'Donoghue wrote:
> phy-names has been marked deprecated. Remove it from the sm8250 DSI
> controller block.
>
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 28+ messages in thread
* Re: [PATCH 00/11] qcom,mdss-dsi-ctrl: Remove redundant phy-names from yaml and dtsi
2022-09-07 0:00 [PATCH 00/11] qcom,mdss-dsi-ctrl: Remove redundant phy-names from yaml and dtsi Bryan O'Donoghue
` (10 preceding siblings ...)
2022-09-07 0:01 ` [PATCH 11/11] arm64: dts: qcom: sm8250: " Bryan O'Donoghue
@ 2022-09-08 22:30 ` Dmitry Baryshkov
2022-09-26 11:18 ` Bryan O'Donoghue
2022-10-18 3:05 ` (subset) " Bjorn Andersson
13 siblings, 0 replies; 28+ messages in thread
From: Dmitry Baryshkov @ 2022-09-08 22:30 UTC (permalink / raw)
To: Bryan O'Donoghue, robdclark, quic_abhinavk, sean, robh+dt,
krzysztof.kozlowski+dt, quic_mkrishn, agross, andersson,
konrad.dybcio, linux-arm-msm, devicetree
On 07/09/2022 03:00, Bryan O'Donoghue wrote:
> phy-names is both inconsistently used and not actually required by the driver.
>
> Previous discussion with Krzysztof and Rob suggested to remove the property.
>
> https://www.spinics.net/lists/linux-arm-msm/msg116525.html
>
> - Mark as deprecated and not required in yaml
> - Drop from associated dtsi files => grep -r mdss-dsi arch/arm* | grep qcom
>
> Bryan O'Donoghue (11):
> dt-bindings: msm: dsi-controller-main: Drop redundant phy-names
I'm going to take this into msm-next, the rest can go via Bjorn.
> ARM: dts: qcom: apq8064: Drop redundant phy-names from DSI controller
> ARM: dts: qcom: msm8974: Drop redundant phy-names from DSI controller
> arm64: dts: qcom: msm8916: Drop redundant phy-names from DSI
> controller
> arm64: dts: qcom: msm8996: Drop redundant phy-names from DSI
> controller
> arm64: dts: qcom: sc7180: Drop redundant phy-names from DSI controller
> arm64: dts: qcom: sc7280: Drop redundant phy-names from DSI controller
> arm64: dts: qcom: sdm660: Drop redundant phy-names from DSI controller
> arm64: dts: qcom: sdm630: Drop redundant phy-names from DSI controller
> arm64: dts: qcom: sdm845: Drop redundant phy-names from DSI controller
> arm64: dts: qcom: sm8250: Drop redundant phy-names from DSI controller
>
> .../devicetree/bindings/display/msm/dsi-controller-main.yaml | 2 +-
> arch/arm/boot/dts/qcom-apq8064.dtsi | 1 -
> arch/arm/boot/dts/qcom-msm8974.dtsi | 1 -
> arch/arm64/boot/dts/qcom/msm8916.dtsi | 1 -
> arch/arm64/boot/dts/qcom/msm8996.dtsi | 2 --
> arch/arm64/boot/dts/qcom/sc7180.dtsi | 1 -
> arch/arm64/boot/dts/qcom/sc7280.dtsi | 1 -
> arch/arm64/boot/dts/qcom/sdm630.dtsi | 1 -
> arch/arm64/boot/dts/qcom/sdm660.dtsi | 1 -
> arch/arm64/boot/dts/qcom/sdm845.dtsi | 2 --
> arch/arm64/boot/dts/qcom/sm8250.dtsi | 2 --
> 11 files changed, 1 insertion(+), 14 deletions(-)
>
--
With best wishes
Dmitry
^ permalink raw reply [flat|nested] 28+ messages in thread
* Re: [PATCH 01/11] dt-bindings: msm: dsi-controller-main: Drop redundant phy-names
2022-09-07 0:00 ` [PATCH 01/11] dt-bindings: msm: dsi-controller-main: Drop redundant phy-names Bryan O'Donoghue
2022-09-08 14:11 ` Krzysztof Kozlowski
@ 2022-09-08 22:31 ` Dmitry Baryshkov
1 sibling, 0 replies; 28+ messages in thread
From: Dmitry Baryshkov @ 2022-09-08 22:31 UTC (permalink / raw)
To: Bryan O'Donoghue, robdclark, quic_abhinavk, sean, robh+dt,
krzysztof.kozlowski+dt, quic_mkrishn, agross, andersson,
konrad.dybcio, linux-arm-msm, devicetree
On 07/09/2022 03:00, Bryan O'Donoghue wrote:
> Adding in msm8939 which is based msm8916 dtsi I stumbled across a binding
> check complaining about the phy name for msm8916 which we were reusing for
> msm8939.
>
> The currently inconsistent upstream dtsi naming of "dsi" and "dsi-phy" is
> not captured in the yaml for this driver.
>
> The driver however doesn't care about the name of DSI phy, hence the yaml
> check is redundant.
>
> Both Krzysztof and Rob suggested we could drop the phy-names entirely if it
> really isn't a dependency.
>
> So, drop the inconsistent and unnecessary phy-names field from the yaml.
>
> Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> ---
> .../devicetree/bindings/display/msm/dsi-controller-main.yaml | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
> index 880bfe930830c..3742e2ab4fb1d 100644
> --- a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
> +++ b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
> @@ -49,6 +49,7 @@ properties:
> maxItems: 1
>
> phy-names:
> + deprecated: true
> const: dsi
>
> "#address-cells": true
> @@ -133,7 +134,6 @@ required:
> - clocks
> - clock-names
> - phys
> - - phy-names
> - assigned-clocks
> - assigned-clock-parents
> - power-domains
--
With best wishes
Dmitry
^ permalink raw reply [flat|nested] 28+ messages in thread
* Re: [PATCH 00/11] qcom,mdss-dsi-ctrl: Remove redundant phy-names from yaml and dtsi
2022-09-07 0:00 [PATCH 00/11] qcom,mdss-dsi-ctrl: Remove redundant phy-names from yaml and dtsi Bryan O'Donoghue
` (11 preceding siblings ...)
2022-09-08 22:30 ` [PATCH 00/11] qcom,mdss-dsi-ctrl: Remove redundant phy-names from yaml and dtsi Dmitry Baryshkov
@ 2022-09-26 11:18 ` Bryan O'Donoghue
2022-10-18 3:05 ` (subset) " Bjorn Andersson
13 siblings, 0 replies; 28+ messages in thread
From: Bryan O'Donoghue @ 2022-09-26 11:18 UTC (permalink / raw)
To: robdclark, quic_abhinavk, dmitry.baryshkov, sean, robh+dt,
krzysztof.kozlowski+dt, quic_mkrishn, agross, andersson,
konrad.dybcio, linux-arm-msm, devicetree
On 07/09/2022 01:00, Bryan O'Donoghue wrote:
> phy-names is both inconsistently used and not actually required by the driver.
>
> Previous discussion with Krzysztof and Rob suggested to remove the property.
>
> https://www.spinics.net/lists/linux-arm-msm/msg116525.html
>
> - Mark as deprecated and not required in yaml
> - Drop from associated dtsi files => grep -r mdss-dsi arch/arm* | grep qcom
>
> Bryan O'Donoghue (11):
> dt-bindings: msm: dsi-controller-main: Drop redundant phy-names
> ARM: dts: qcom: apq8064: Drop redundant phy-names from DSI controller
> ARM: dts: qcom: msm8974: Drop redundant phy-names from DSI controller
> arm64: dts: qcom: msm8916: Drop redundant phy-names from DSI
> controller
> arm64: dts: qcom: msm8996: Drop redundant phy-names from DSI
> controller
> arm64: dts: qcom: sc7180: Drop redundant phy-names from DSI controller
> arm64: dts: qcom: sc7280: Drop redundant phy-names from DSI controller
> arm64: dts: qcom: sdm660: Drop redundant phy-names from DSI controller
> arm64: dts: qcom: sdm630: Drop redundant phy-names from DSI controller
> arm64: dts: qcom: sdm845: Drop redundant phy-names from DSI controller
> arm64: dts: qcom: sm8250: Drop redundant phy-names from DSI controller
>
> .../devicetree/bindings/display/msm/dsi-controller-main.yaml | 2 +-
> arch/arm/boot/dts/qcom-apq8064.dtsi | 1 -
> arch/arm/boot/dts/qcom-msm8974.dtsi | 1 -
> arch/arm64/boot/dts/qcom/msm8916.dtsi | 1 -
> arch/arm64/boot/dts/qcom/msm8996.dtsi | 2 --
> arch/arm64/boot/dts/qcom/sc7180.dtsi | 1 -
> arch/arm64/boot/dts/qcom/sc7280.dtsi | 1 -
> arch/arm64/boot/dts/qcom/sdm630.dtsi | 1 -
> arch/arm64/boot/dts/qcom/sdm660.dtsi | 1 -
> arch/arm64/boot/dts/qcom/sdm845.dtsi | 2 --
> arch/arm64/boot/dts/qcom/sm8250.dtsi | 2 --
> 11 files changed, 1 insertion(+), 14 deletions(-)
>
Ping.
https://lore.kernel.org/lkml/20220925172443.92900-1-luca@z3ntu.xyz/T/
---
bod
^ permalink raw reply [flat|nested] 28+ messages in thread
* Re: (subset) [PATCH 00/11] qcom,mdss-dsi-ctrl: Remove redundant phy-names from yaml and dtsi
2022-09-07 0:00 [PATCH 00/11] qcom,mdss-dsi-ctrl: Remove redundant phy-names from yaml and dtsi Bryan O'Donoghue
` (12 preceding siblings ...)
2022-09-26 11:18 ` Bryan O'Donoghue
@ 2022-10-18 3:05 ` Bjorn Andersson
13 siblings, 0 replies; 28+ messages in thread
From: Bjorn Andersson @ 2022-10-18 3:05 UTC (permalink / raw)
To: dmitry.baryshkov, sean, devicetree, robdclark,
krzysztof.kozlowski+dt, agross, quic_abhinavk, robh+dt,
bryan.odonoghue, konrad.dybcio, linux-arm-msm, quic_mkrishn
On Wed, 7 Sep 2022 01:00:54 +0100, Bryan O'Donoghue wrote:
> phy-names is both inconsistently used and not actually required by the driver.
>
> Previous discussion with Krzysztof and Rob suggested to remove the property.
>
> https://www.spinics.net/lists/linux-arm-msm/msg116525.html
>
> - Mark as deprecated and not required in yaml
> - Drop from associated dtsi files => grep -r mdss-dsi arch/arm* | grep qcom
>
> [...]
Applied, thanks!
[02/11] ARM: dts: qcom: apq8064: Drop redundant phy-names from DSI controller
commit: 645953bae8ae89e072af3b78bd648c0035d806c0
[03/11] ARM: dts: qcom: msm8974: Drop redundant phy-names from DSI controller
commit: 0b01159afd5a0b782610fc113c4d8c0594b6e43d
Best regards,
--
Bjorn Andersson <andersson@kernel.org>
^ permalink raw reply [flat|nested] 28+ messages in thread
end of thread, other threads:[~2022-10-18 3:06 UTC | newest]
Thread overview: 28+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-09-07 0:00 [PATCH 00/11] qcom,mdss-dsi-ctrl: Remove redundant phy-names from yaml and dtsi Bryan O'Donoghue
2022-09-07 0:00 ` [PATCH 01/11] dt-bindings: msm: dsi-controller-main: Drop redundant phy-names Bryan O'Donoghue
2022-09-08 14:11 ` Krzysztof Kozlowski
2022-09-08 22:31 ` Dmitry Baryshkov
2022-09-07 0:00 ` [PATCH 02/11] ARM: dts: qcom: apq8064: Drop redundant phy-names from DSI controller Bryan O'Donoghue
2022-09-08 14:12 ` Krzysztof Kozlowski
2022-09-07 0:00 ` [PATCH 03/11] ARM: dts: qcom: msm8974: " Bryan O'Donoghue
2022-09-08 14:12 ` Krzysztof Kozlowski
2022-09-07 0:00 ` [PATCH 04/11] arm64: dts: qcom: msm8916: " Bryan O'Donoghue
2022-09-08 14:12 ` Krzysztof Kozlowski
2022-09-07 0:00 ` [PATCH 05/11] arm64: dts: qcom: msm8996: " Bryan O'Donoghue
2022-09-08 14:12 ` Krzysztof Kozlowski
2022-09-07 0:01 ` [PATCH 06/11] arm64: dts: qcom: sc7180: " Bryan O'Donoghue
2022-09-08 14:12 ` Krzysztof Kozlowski
2022-09-07 0:01 ` [PATCH 07/11] arm64: dts: qcom: sc7280: " Bryan O'Donoghue
2022-09-08 14:12 ` Krzysztof Kozlowski
2022-09-07 0:01 ` [PATCH 08/11] arm64: dts: qcom: sdm660: " Bryan O'Donoghue
2022-09-08 14:12 ` Krzysztof Kozlowski
2022-09-07 0:01 ` [PATCH 09/11] arm64: dts: qcom: sdm630: " Bryan O'Donoghue
2022-09-08 14:13 ` Krzysztof Kozlowski
2022-09-07 0:01 ` [PATCH 10/11] arm64: dts: qcom: sdm845: " Bryan O'Donoghue
2022-09-07 3:13 ` Steev Klimaszewski
2022-09-08 14:13 ` Krzysztof Kozlowski
2022-09-07 0:01 ` [PATCH 11/11] arm64: dts: qcom: sm8250: " Bryan O'Donoghue
2022-09-08 14:13 ` Krzysztof Kozlowski
2022-09-08 22:30 ` [PATCH 00/11] qcom,mdss-dsi-ctrl: Remove redundant phy-names from yaml and dtsi Dmitry Baryshkov
2022-09-26 11:18 ` Bryan O'Donoghue
2022-10-18 3:05 ` (subset) " 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).