mbox series

[00/11] qcom,mdss-dsi-ctrl: Remove redundant phy-names from yaml and dtsi

Message ID 20220907000105.786265-1-bryan.odonoghue@linaro.org (mailing list archive)
Headers show
Series qcom,mdss-dsi-ctrl: Remove redundant phy-names from yaml and dtsi | expand

Message

Bryan O'Donoghue Sept. 7, 2022, midnight UTC
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(-)

Comments

Dmitry Baryshkov Sept. 8, 2022, 10:30 p.m. UTC | #1
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(-)
>
Bryan O'Donoghue Sept. 26, 2022, 11:18 a.m. UTC | #2
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
Bjorn Andersson Oct. 18, 2022, 3:05 a.m. UTC | #3
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,