| Message ID | 20221006140637.246665-1-krzysztof.kozlowski@linaro.org |
|---|---|
| Headers | show |
| Series | pinctrl/arm64: qcom: continued - fix Qualcomm TLMM pinctrl schema warnings (5th set) | expand |
Hi Krzysztof, On Thu, Oct 06, 2022 at 04:06:03PM +0200, Krzysztof Kozlowski wrote: > [...] > Krzysztof Kozlowski (34): > dt-bindings: pinctrl: qcom,mdm9607: drop ref to pinctrl.yaml > dt-bindings: pinctrl: qcom,sc7280: drop checks used in common TLMM > dt-bindings: pinctrl: qcom,sc8180x: drop ref to pinctrl.yaml > dt-bindings: pinctrl: qcom,sc8180x: drop checks used in common TLMM > dt-bindings: pinctrl: qcom,sc8280xp: drop checks used in common TLMM > dt-bindings: pinctrl: qcom,sm6115: use common TLMM schema > dt-bindings: pinctrl: qcom,sm6125: drop checks used in common TLMM > dt-bindings: pinctrl: qcom,sm6125: drop ref to pinctrl.yaml > dt-bindings: pinctrl: qcom,sm6350: drop ref to pinctrl.yaml > dt-bindings: pinctrl: qcom,sm6350: drop checks used in common TLMM > dt-bindings: pinctrl: qcom,sm6375-tlmm: drop ref to pinctrl.yaml > dt-bindings: pinctrl: qcom,sm6375-tlmm: drop checks used in common > TLMM > dt-bindings: pinctrl: qcom,sm8250: use common TLMM schema > dt-bindings: pinctrl: qcom,sm8350: drop ref to pinctrl.yaml > dt-bindings: pinctrl: qcom,sm8350: drop checks used in common TLMM > dt-bindings: pinctrl: qcom,sm8450: drop checks used in common TLMM > dt-bindings: pinctrl: qcom,mdm9607: minor style cleanups > dt-bindings: pinctrl: qcom,msm8909-tlmm: minor style cleanups > dt-bindings: pinctrl: qcom,qcm2290: minor style cleanups > dt-bindings: pinctrl: qcom,sdx65: minor style cleanups > dt-bindings: pinctrl: qcom,sc8180x: minor style cleanups > dt-bindings: pinctrl: qcom,sc8280xp: minor style cleanups > dt-bindings: pinctrl: qcom,sm6115: minor style cleanups > dt-bindings: pinctrl: qcom,sm6125: minor style cleanups > dt-bindings: pinctrl: qcom,sm6350: minor style cleanups > dt-bindings: pinctrl: qcom,sm6375: minor style cleanups > dt-bindings: pinctrl: qcom,sm8250: minor style cleanups > dt-bindings: pinctrl: qcom,sm8350: minor style cleanups > dt-bindings: pinctrl: qcom,sm8450: minor style cleanups > dt-bindings: pinctrl: qcom,sc7280-lpass-lpi: minor style cleanups > dt-bindings: pinctrl: qcom,sm8250-lpass-lpi: minor style cleanups > dt-bindings: pinctrl: qcom,sc8280xp-lpass-lpi: minor style cleanups > dt-bindings: pinctrl: qcom,sm8450-lpass-lpi: minor style cleanups > dt-bindings: pinctrl: qcom: adjust description > > .../pinctrl/qcom,ipq6018-pinctrl.yaml | 5 +- > .../pinctrl/qcom,mdm9607-pinctrl.yaml | 16 ++--- > .../pinctrl/qcom,msm8226-pinctrl.yaml | 5 +- > .../bindings/pinctrl/qcom,msm8909-tlmm.yaml | 11 ++- > .../pinctrl/qcom,msm8953-pinctrl.yaml | 5 +- > .../pinctrl/qcom,qcm2290-pinctrl.yaml | 10 ++- > .../qcom,sc7280-lpass-lpi-pinctrl.yaml | 13 ++-- > .../bindings/pinctrl/qcom,sc7280-pinctrl.yaml | 16 +---- > .../pinctrl/qcom,sc8180x-pinctrl.yaml | 27 +++---- > .../qcom,sc8280xp-lpass-lpi-pinctrl.yaml | 13 ++-- > .../pinctrl/qcom,sc8280xp-pinctrl.yaml | 22 ++---- > .../bindings/pinctrl/qcom,sdx55-pinctrl.yaml | 5 +- > .../bindings/pinctrl/qcom,sdx65-pinctrl.yaml | 10 ++- > .../bindings/pinctrl/qcom,sm6115-pinctrl.yaml | 53 +++----------- > .../bindings/pinctrl/qcom,sm6125-pinctrl.yaml | 30 +++----- > .../bindings/pinctrl/qcom,sm6350-pinctrl.yaml | 23 ++---- > .../bindings/pinctrl/qcom,sm6375-tlmm.yaml | 23 ++---- > .../qcom,sm8250-lpass-lpi-pinctrl.yaml | 16 ++--- > .../bindings/pinctrl/qcom,sm8250-pinctrl.yaml | 70 ++++++------------- > .../bindings/pinctrl/qcom,sm8350-pinctrl.yaml | 23 ++---- > .../qcom,sm8450-lpass-lpi-pinctrl.yaml | 13 ++-- > .../bindings/pinctrl/qcom,sm8450-pinctrl.yaml | 22 ++---- Just a random thought since you are already doing minor style cleanups here: Some of these files are named incorrectly, e.g. qcom,sm8450-pinctrl.yaml actually documents "qcom,sm8450-tlmm". I noticed this while adding qcom,msm8909-tlmm but I have to admit that it did not bother me enough to actually prepare a patch for this (and now it would just conflict with all your patches). :) No need to change anything here, just thought I'd mention it. Thanks! Stephan
On Thu, 06 Oct 2022 16:06:04 +0200, Krzysztof Kozlowski wrote: > The binding references common Qualcomm TLMM pin controller schema, which > references pinctrl.yaml. > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > --- > .../devicetree/bindings/pinctrl/qcom,mdm9607-pinctrl.yaml | 1 - > 1 file changed, 1 deletion(-) > Acked-by: Rob Herring <robh@kernel.org>
On Thu, 06 Oct 2022 16:06:06 +0200, Krzysztof Kozlowski wrote: > The binding references common Qualcomm TLMM pin controller schema, which > references pinctrl.yaml. > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > --- > .../devicetree/bindings/pinctrl/qcom,sc8180x-pinctrl.yaml | 1 - > 1 file changed, 1 deletion(-) > Acked-by: Rob Herring <robh@kernel.org>
On Thu, 06 Oct 2022 16:06:08 +0200, Krzysztof Kozlowski wrote: > The common Qualcomm TLMM pin controller schema already brings > requirement of function for GPIO pins. > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > --- > .../bindings/pinctrl/qcom,sc8280xp-pinctrl.yaml | 11 +---------- > 1 file changed, 1 insertion(+), 10 deletions(-) > Acked-by: Rob Herring <robh@kernel.org>
On Thu, 06 Oct 2022 16:06:12 +0200, Krzysztof Kozlowski wrote: > The binding references common Qualcomm TLMM pin controller schema, which > references pinctrl.yaml. > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > --- > .../devicetree/bindings/pinctrl/qcom,sm6350-pinctrl.yaml | 1 - > 1 file changed, 1 deletion(-) > Acked-by: Rob Herring <robh@kernel.org>
On Thu, 06 Oct 2022 16:06:15 +0200, Krzysztof Kozlowski wrote: > The common Qualcomm TLMM pin controller schema already brings > requirement of function for GPIO pins. > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > --- > .../devicetree/bindings/pinctrl/qcom,sm6375-tlmm.yaml | 11 +---------- > 1 file changed, 1 insertion(+), 10 deletions(-) > Acked-by: Rob Herring <robh@kernel.org>
On Thu, 06 Oct 2022 16:06:21 +0200, Krzysztof Kozlowski wrote: > Drop "binding" from description (and align it with other Qualcomm > pinctrl bindings), use double quotes consistently and drop redundant > quotes. > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > --- > .../bindings/pinctrl/qcom,msm8909-tlmm.yaml | 11 +++++------ > 1 file changed, 5 insertions(+), 6 deletions(-) > Acked-by: Rob Herring <robh@kernel.org>
On Thu, 06 Oct 2022 16:06:25 +0200, Krzysztof Kozlowski wrote: > Drop "binding" from description (and align it with other Qualcomm > pinctrl bindings), use double quotes consistently and drop redundant > quotes. > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > --- > .../bindings/pinctrl/qcom,sc8280xp-pinctrl.yaml | 11 +++++------ > 1 file changed, 5 insertions(+), 6 deletions(-) > Acked-by: Rob Herring <robh@kernel.org>
On Thu, 06 Oct 2022 16:06:27 +0200, Krzysztof Kozlowski wrote: > Drop "binding" from description (and align it with other Qualcomm > pinctrl bindings), drop redundant minItems, use double quotes > consistently and drop redundant quotes. > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > --- > .../bindings/pinctrl/qcom,sm6125-pinctrl.yaml | 18 ++++++++---------- > 1 file changed, 8 insertions(+), 10 deletions(-) > Acked-by: Rob Herring <robh@kernel.org>
On Thu, 06 Oct 2022 16:06:29 +0200, Krzysztof Kozlowski wrote: > Drop "binding" from description (and align it with other Qualcomm > pinctrl bindings), use double quotes consistently and drop redundant > quotes. > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > --- > .../devicetree/bindings/pinctrl/qcom,sm6375-tlmm.yaml | 11 +++++------ > 1 file changed, 5 insertions(+), 6 deletions(-) > Acked-by: Rob Herring <robh@kernel.org>
On Thu, 06 Oct 2022 16:06:33 +0200, Krzysztof Kozlowski wrote: > Drop "binding" from description (and align it with other Qualcomm > pinctrl bindings), use double quotes consistently and drop redundant > quotes. > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > --- > .../pinctrl/qcom,sc7280-lpass-lpi-pinctrl.yaml | 13 ++++++------- > 1 file changed, 6 insertions(+), 7 deletions(-) > Acked-by: Rob Herring <robh@kernel.org>
On Thu, Oct 6, 2022 at 4:06 PM Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> wrote: > Patches are organized not by file, but rather type of change > 1. First patches is for common TLMM schema and dropping unneeded refs. > 2. Last patches are pure cleanups without functional impact. > > Overview > ======== > This is the *fifth* patchset around Qualcomm pinctrl in recent days: > 1. First round of TLMM fixes: merged > 2. LPASS fixes: > https://lore.kernel.org/linux-devicetree/20220927153429.55365-1-krzysztof.kozlowski@linaro.org/ > 3. ARMv7 TLMM fixes: > https://lore.kernel.org/linux-devicetree/20221006124659.217540-1-krzysztof.kozlowski@linaro.org/ > 4. ARMv8 remaining TLMM fixes: > https://lore.kernel.org/linux-devicetree/20220930192954.242546-1-krzysztof.kozlowski@linaro.org/ > 5. Fifth clean - styles and using common TLMM schema: *THIS PATCHSET* Blanket: Acked-by: Linus Walleij <linus.walleij@linaro.org> Please send me pull requests for any parts that need to go into the pinctrl tree. Yours, Linus Walleij
Hi, Patches are organized not by file, but rather type of change 1. First patches is for common TLMM schema and dropping unneeded refs. 2. Last patches are pure cleanups without functional impact. Overview ======== This is the *fifth* patchset around Qualcomm pinctrl in recent days: 1. First round of TLMM fixes: merged 2. LPASS fixes: https://lore.kernel.org/linux-devicetree/20220927153429.55365-1-krzysztof.kozlowski@linaro.org/ 3. ARMv7 TLMM fixes: https://lore.kernel.org/linux-devicetree/20221006124659.217540-1-krzysztof.kozlowski@linaro.org/ 4. ARMv8 remaining TLMM fixes: https://lore.kernel.org/linux-devicetree/20220930192954.242546-1-krzysztof.kozlowski@linaro.org/ 5. Fifth clean - styles and using common TLMM schema: *THIS PATCHSET* Dependencies ============ 1. Almost no dependencies - logically the bindings patch "dt-bindings: pinctrl: qcom,sm8250: drop checks used in common TLMM" depends on patchset #3 above. This is not a hard-dependency, everything will compile nicely, no warnings. 2. dt-bindings are independent of DTS patches. Best regards, Krzysztof Krzysztof Kozlowski (34): dt-bindings: pinctrl: qcom,mdm9607: drop ref to pinctrl.yaml dt-bindings: pinctrl: qcom,sc7280: drop checks used in common TLMM dt-bindings: pinctrl: qcom,sc8180x: drop ref to pinctrl.yaml dt-bindings: pinctrl: qcom,sc8180x: drop checks used in common TLMM dt-bindings: pinctrl: qcom,sc8280xp: drop checks used in common TLMM dt-bindings: pinctrl: qcom,sm6115: use common TLMM schema dt-bindings: pinctrl: qcom,sm6125: drop checks used in common TLMM dt-bindings: pinctrl: qcom,sm6125: drop ref to pinctrl.yaml dt-bindings: pinctrl: qcom,sm6350: drop ref to pinctrl.yaml dt-bindings: pinctrl: qcom,sm6350: drop checks used in common TLMM dt-bindings: pinctrl: qcom,sm6375-tlmm: drop ref to pinctrl.yaml dt-bindings: pinctrl: qcom,sm6375-tlmm: drop checks used in common TLMM dt-bindings: pinctrl: qcom,sm8250: use common TLMM schema dt-bindings: pinctrl: qcom,sm8350: drop ref to pinctrl.yaml dt-bindings: pinctrl: qcom,sm8350: drop checks used in common TLMM dt-bindings: pinctrl: qcom,sm8450: drop checks used in common TLMM dt-bindings: pinctrl: qcom,mdm9607: minor style cleanups dt-bindings: pinctrl: qcom,msm8909-tlmm: minor style cleanups dt-bindings: pinctrl: qcom,qcm2290: minor style cleanups dt-bindings: pinctrl: qcom,sdx65: minor style cleanups dt-bindings: pinctrl: qcom,sc8180x: minor style cleanups dt-bindings: pinctrl: qcom,sc8280xp: minor style cleanups dt-bindings: pinctrl: qcom,sm6115: minor style cleanups dt-bindings: pinctrl: qcom,sm6125: minor style cleanups dt-bindings: pinctrl: qcom,sm6350: minor style cleanups dt-bindings: pinctrl: qcom,sm6375: minor style cleanups dt-bindings: pinctrl: qcom,sm8250: minor style cleanups dt-bindings: pinctrl: qcom,sm8350: minor style cleanups dt-bindings: pinctrl: qcom,sm8450: minor style cleanups dt-bindings: pinctrl: qcom,sc7280-lpass-lpi: minor style cleanups dt-bindings: pinctrl: qcom,sm8250-lpass-lpi: minor style cleanups dt-bindings: pinctrl: qcom,sc8280xp-lpass-lpi: minor style cleanups dt-bindings: pinctrl: qcom,sm8450-lpass-lpi: minor style cleanups dt-bindings: pinctrl: qcom: adjust description .../pinctrl/qcom,ipq6018-pinctrl.yaml | 5 +- .../pinctrl/qcom,mdm9607-pinctrl.yaml | 16 ++--- .../pinctrl/qcom,msm8226-pinctrl.yaml | 5 +- .../bindings/pinctrl/qcom,msm8909-tlmm.yaml | 11 ++- .../pinctrl/qcom,msm8953-pinctrl.yaml | 5 +- .../pinctrl/qcom,qcm2290-pinctrl.yaml | 10 ++- .../qcom,sc7280-lpass-lpi-pinctrl.yaml | 13 ++-- .../bindings/pinctrl/qcom,sc7280-pinctrl.yaml | 16 +---- .../pinctrl/qcom,sc8180x-pinctrl.yaml | 27 +++---- .../qcom,sc8280xp-lpass-lpi-pinctrl.yaml | 13 ++-- .../pinctrl/qcom,sc8280xp-pinctrl.yaml | 22 ++---- .../bindings/pinctrl/qcom,sdx55-pinctrl.yaml | 5 +- .../bindings/pinctrl/qcom,sdx65-pinctrl.yaml | 10 ++- .../bindings/pinctrl/qcom,sm6115-pinctrl.yaml | 53 +++----------- .../bindings/pinctrl/qcom,sm6125-pinctrl.yaml | 30 +++----- .../bindings/pinctrl/qcom,sm6350-pinctrl.yaml | 23 ++---- .../bindings/pinctrl/qcom,sm6375-tlmm.yaml | 23 ++---- .../qcom,sm8250-lpass-lpi-pinctrl.yaml | 16 ++--- .../bindings/pinctrl/qcom,sm8250-pinctrl.yaml | 70 ++++++------------- .../bindings/pinctrl/qcom,sm8350-pinctrl.yaml | 23 ++---- .../qcom,sm8450-lpass-lpi-pinctrl.yaml | 13 ++-- .../bindings/pinctrl/qcom,sm8450-pinctrl.yaml | 22 ++---- 22 files changed, 136 insertions(+), 295 deletions(-)