| Message ID | 20220228123019.382037-1-bhupesh.sharma@linaro.org |
|---|---|
| Headers | show |
| Series | Fix dtbs_check warning(s) for Qualcomm QMP PHY | expand |
On Mon, Feb 28, 2022 at 06:00:18PM +0530, Bhupesh Sharma wrote: > Currently the qcom,qmp-phy dt-binding doesn't describe > the 'reg' and '#phy-cells' properties for the phy@ subnode. > > Fix the same. > > Cc: Bjorn Andersson <bjorn.andersson@linaro.org> > Cc: Rob Herring <robh@kernel.org> > Signed-off-by: Bhupesh Sharma <bhupesh.sharma@linaro.org> > --- > Documentation/devicetree/bindings/phy/qcom,qmp-phy.yaml | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/Documentation/devicetree/bindings/phy/qcom,qmp-phy.yaml b/Documentation/devicetree/bindings/phy/qcom,qmp-phy.yaml > index 70e7cd75e173..f8a43acd229a 100644 > --- a/Documentation/devicetree/bindings/phy/qcom,qmp-phy.yaml > +++ b/Documentation/devicetree/bindings/phy/qcom,qmp-phy.yaml > @@ -115,6 +115,13 @@ patternProperties: > Each device node of QMP phy is required to have as many child nodes as > the number of lanes the PHY has. > properties: > + reg: > + minItems: 1 > + maxItems: 6 You need to define what each entry is. > + > + "#phy-cells": > + const: 0 > + > "#clock-cells": > enum: [ 0, 1, 2 ] > > -- > 2.35.1 > >
Changes since v1: ----------------- - v1 can be found here: https://www.spinics.net/lists/linux-arm-msm/msg104871.html - Addressed review comments from Dmitry. - Addressed the remaining dtbs_check warnings rasied by Rob's bot on v1. Shawn Guo recently fixed a large number of 'dtbs_check' warnings on Qualcomm DTS files for qmp phys. However a few warnings are still left (possibly due to new dts files being added over time). So, I decided to clean them up. After this series, '$ make dtbs_check' leads to no warning for QCOM QMP PHY. Cc: Bjorn Andersson <bjorn.andersson@linaro.org> Cc: Rob Herring <robh@kernel.org> Bhupesh Sharma (8): dt-bindings: phy: qcom,qmp: Mark '#clock-cells' as a 'optional' property dt-bindings: phy: qcom,qmp: Describe 'vdda-max-microamp' & 'vdda-pll-max-microamp' properties arm64: dts: qcom: msm8996-xiaomi: Drop max-microamp and vddp-ref-clk properties from QMP PHY arm64: dts: qcom: sc7280: Fix qmp phy node (use phy@ instead of lanes@) arm64: dts: qcom: sm8450: Fix qmp ufs phy node (use phy@ instead of lanes@) arm64: dts: qcom: ipq6018: Fix qmp usb3 phy node dt-bindings: phy: qcom,qmp: Describe phy@ subnode properly dt-bindings: phy: qcom,qmp: Mark 'vdda-pll-supply' & 'vdda-phy-supply' as optional .../devicetree/bindings/phy/qcom,qmp-phy.yaml | 26 +++++++++++++------ arch/arm64/boot/dts/qcom/ipq6018.dtsi | 4 +-- .../boot/dts/qcom/msm8996-xiaomi-common.dtsi | 5 ---- arch/arm64/boot/dts/qcom/sc7280.dtsi | 2 +- arch/arm64/boot/dts/qcom/sm8450.dtsi | 2 +- 5 files changed, 22 insertions(+), 17 deletions(-)