mbox series

[00/10] Make Qualcomm QMP PHY dtbs_check warning free

Message ID 20210929034253.24570-1-shawn.guo@linaro.org
Headers show
Series Make Qualcomm QMP PHY dtbs_check warning free | expand

Message

Shawn Guo Sept. 29, 2021, 3:42 a.m. UTC
As I recently add QMP USB3 PHY support for QCM2290 and touch the binding
doc.  Rob's bot report a huge number of dtbs_check warnings on Qualcomm
DTS files.  Although none of them is new warning introduced by my
change, I decided to clean them up.

The first two patches are fixes on QMP PHY binding, and the reset are
all fixes on Qualcomm DTS.  In the end, with this series, we are QMP PHY
dtbs_check warning free!


Shawn Guo (10):
  dt-bindings: phy: qcom,qmp: '#clock-cells' is not required for parent
    node
  dt-bindings: phy: qcom,qmp: IPQ6018 and IPQ8074 PCIe PHY require no
    supply
  arm64: dts: qcom: msm8996: Move '#clock-cells' to QMP PHY child node
  arm64: dts: qcom: Correct QMP PHY child node name
  arm64: dts: qcom: Drop max-microamp and vddp-ref-clk properties from
    QMP PHY
  arm64: dts: qcom: Drop reg-names from QMP PHY nodes
  arm64: dts: qcom: msm8998-clamshell: Add missing vdda supplies
  arm64: dts: qcom: Fix IPQ8074 PCIe PHY nodes
  arm64: dts: qcom: ipq8074-hk01: Add dummy supply for QMP USB3 PHY
  ARM: dts: qcom: sdx55: Drop '#clock-cells' from QMP PHY node

 .../devicetree/bindings/phy/qcom,qmp-phy.yaml | 57 +++++++++----------
 arch/arm/boot/dts/qcom-sdx55.dtsi             |  1 -
 arch/arm64/boot/dts/qcom/apq8096-ifc6640.dts  |  3 -
 arch/arm64/boot/dts/qcom/ipq6018.dtsi         |  2 +-
 arch/arm64/boot/dts/qcom/ipq8074-hk01.dts     |  6 ++
 arch/arm64/boot/dts/qcom/ipq8074.dtsi         | 54 +++++++++++++-----
 arch/arm64/boot/dts/qcom/msm8996.dtsi         | 14 ++---
 .../boot/dts/qcom/msm8998-clamshell.dtsi      |  2 +
 arch/arm64/boot/dts/qcom/msm8998-mtp.dtsi     |  4 --
 .../boot/dts/qcom/msm8998-oneplus-common.dtsi |  4 --
 arch/arm64/boot/dts/qcom/msm8998.dtsi         |  8 +--
 arch/arm64/boot/dts/qcom/qrb5165-rb5.dts      |  2 -
 arch/arm64/boot/dts/qcom/sa8155p-adp.dts      |  2 -
 arch/arm64/boot/dts/qcom/sdm845.dtsi          | 11 ++--
 arch/arm64/boot/dts/qcom/sm8150-hdk.dts       |  2 -
 .../dts/qcom/sm8150-microsoft-surface-duo.dts |  2 -
 arch/arm64/boot/dts/qcom/sm8150-mtp.dts       |  2 -
 arch/arm64/boot/dts/qcom/sm8150.dtsi          |  7 +--
 arch/arm64/boot/dts/qcom/sm8250-hdk.dts       |  2 -
 arch/arm64/boot/dts/qcom/sm8250-mtp.dts       |  2 -
 arch/arm64/boot/dts/qcom/sm8250.dtsi          | 10 ++--
 arch/arm64/boot/dts/qcom/sm8350-hdk.dts       |  2 -
 arch/arm64/boot/dts/qcom/sm8350-mtp.dts       |  2 -
 arch/arm64/boot/dts/qcom/sm8350.dtsi          |  6 +-
 24 files changed, 102 insertions(+), 105 deletions(-)

-- 
2.17.1

Comments

Rob Herring Oct. 4, 2021, 6:41 p.m. UTC | #1
On Wed, 29 Sep 2021 11:42:44 +0800, Shawn Guo wrote:
> '#clock-cells' is not required for parent qmp-phy node but the child

> phy/lane node.

> 

> Signed-off-by: Shawn Guo <shawn.guo@linaro.org>

> ---

>  Documentation/devicetree/bindings/phy/qcom,qmp-phy.yaml | 5 -----

>  1 file changed, 5 deletions(-)

> 


Acked-by: Rob Herring <robh@kernel.org>
Vinod Koul Oct. 5, 2021, 6:34 a.m. UTC | #2
On 29-09-21, 11:42, Shawn Guo wrote:
> '#clock-cells' is not required for parent qmp-phy node but the child

> phy/lane node.


Acked-By: Vinod Koul <vkoul@kernel.org>


-- 
~Vinod
Vinod Koul Oct. 5, 2021, 6:34 a.m. UTC | #3
On 29-09-21, 11:42, Shawn Guo wrote:
> The qmp-phy driver suggests that 'vdda-phy-supply' and 'vdda-pll-supply'

> are not required for IPQ6018 and IPQ8074 QMP PCIe PHY.  Update the

> bindings to reflect it.

> 

> While at it, also correct the clock properies for IPQ8074 QMP PCIe PHY.

> And as the result, 'qcom,ipq8074-qmp-pcie-phy' and

> 'qcom,ipq6018-qmp-pcie-phy' share the same clock, reset and supply

> bindings.


Acked-By: Vinod Koul <vkoul@kernel.org>


-- 
~Vinod