Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Mon, 26 Sep 2022 22:41:55 -0500
From:   Bjorn Andersson <andersson@kernel.org>
To:     Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Cc:     Andy Gross <agross@kernel.org>,
        Konrad Dybcio <konrad.dybcio@somainline.org>,
        Linus Walleij <linus.walleij@linaro.org>,
        Rob Herring <robh+dt@kernel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Stephan Gerhold <stephan@gerhold.net>,
        Shawn Guo <shawn.guo@linaro.org>,
        Vinod Koul <vkoul@kernel.org>,
        krishna Lanka <quic_vamslank@quicinc.com>,
        Sivaprakash Murugesan <sivaprak@codeaurora.org>,
        linux-arm-msm@vger.kernel.org, linux-gpio@vger.kernel.org,
        devicetree@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v2 00/33] pinctrl/arm64: qcom: continued - fix Qualcomm
 TLMM pinctrl schema warnings
Message-ID: <20220927034155.2bwxfq4g4m6ovjod@builder.lan>
References: <20220926074415.53100-1-krzysztof.kozlowski@linaro.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20220926074415.53100-1-krzysztof.kozlowski@linaro.org>
Precedence: bulk

On Mon, Sep 26, 2022 at 09:43:42AM +0200, Krzysztof Kozlowski wrote:
> Hi,
> 
> Changes since v1
> ================
> v1: https://lore.kernel.org/linux-devicetree/20220924080459.13084-1-krzysztof.kozlowski@linaro.org/T/#t
> 1. As Stephan suggested, move check for function on non-GPIO pins to common
>    TLMM schema.  This affects few commits in the set named:
>    "use common TLMM schema"
>    "do not require function on non-GPIOs"
> 
> Overview
> ========
> This is the third, independent patchset around Qualcomm pinctrl in recent days:
> 1. First round of TLMM fixes: merged
> 2. LPASS fixes:
>    https://lore.kernel.org/linux-devicetree/20220922195651.345369-1-krzysztof.kozlowski@linaro.org/T/#t
> 3. ARMv7 TLMM fixes: *THIS PATCHSET*
> 4. ARMv8 remaining TLMM fixes: v1 send:
>    https://lore.kernel.org/linux-devicetree/20220925110608.145728-1-krzysztof.kozlowski@linaro.org/T/#t
> 
> Dependencies
> ============
> 1. No dependencies.
> 2. dt-bindings are independent of DTS patches.
> 
> Best regards,
> Krzysztof

Reviewed-by: Bjorn Andersson <andersson@kernel.org>

Regards,
Bjorn

> 
> Krzysztof Kozlowski (33):
>   arm64: dts: qcom: ipq6018-cp01-c1: correct blspi1 pins
>   arm64: dts: qcom: ipq6018: align TLMM pin configuration with DT schema
>   ARM: dts: qcom: sdx55: add gpio-ranges to TLMM pinctrl
>   ARM: dts: qcom: sdx55: align TLMM pin configuration with DT schema
>   ARM: dts: qcom: msm8226: align TLMM pin configuration with DT schema
>   ARM: dts: qcom: msm8974: align TLMM pin configuration with DT schema
>   dt-bindings: pinctrl: qcom,tlmm-common: add common check for function
>   dt-bindings: pinctrl: qcom,ipq6018: add qpic_pad function
>   dt-bindings: pinctrl: qcom,ipq6018: increase number of pins in pinmux
>   dt-bindings: pinctrl: qcom,ipq6018: fix matching pin config
>   dt-bindings: pinctrl: qcom,ipq6018: use common TLMM schema
>   dt-bindings: pinctrl: qcom,ipq6018: fix indentation in example
>   dt-bindings: pinctrl: qcom,msm8226: fix matching pin config
>   dt-bindings: pinctrl: qcom,msm8226: use common TLMM schema
>   dt-bindings: pinctrl: qcom,msm8226: add functions and input-enable
>   dt-bindings: pinctrl: qcom,msm8226: fix indentation in example
>   dt-bindings: pinctrl: qcom,msm8909-tlmm: fix matching pin config
>   dt-bindings: pinctrl: qcom,msm8909-tlmm: do not require function on
>     non-GPIOs
>   dt-bindings: pinctrl: qcom,msm8909-tlmm: fix indentation in example
>   dt-bindings: pinctrl: qcom,msm8953: fix matching pin config
>   dt-bindings: pinctrl: qcom,msm8953: use common TLMM schema
>   dt-bindings: pinctrl: qcom,msm8953: fix indentation in example
>   dt-bindings: pinctrl: qcom,mdm9607: do not require function on
>     non-GPIOs
>   dt-bindings: pinctrl: qcom,mdm9607: fix indentation in example
>   dt-bindings: pinctrl: qcom,qcm2290: fix matching pin config
>   dt-bindings: pinctrl: qcom,qcm2290: use common TLMM schema
>   dt-bindings: pinctrl: qcom,sdx55: fix matching pin config
>   dt-bindings: pinctrl: qcom,sdx55: use common TLMM schema
>   dt-bindings: pinctrl: qcom,sdx55: fix indentation in example
>   dt-bindings: pinctrl: qcom,sdx65: fix matching pin config
>   dt-bindings: pinctrl: qcom,sdx65: use common TLMM schema
>   dt-bindings: pinctrl: qcom,sc7280: fix matching pin config
>   dt-bindings: pinctrl: qcom,sc8280xp: fix indentation in example
>     (remaining piece)
> 
>  .../pinctrl/qcom,ipq6018-pinctrl.yaml         | 60 +++++++++--------
>  .../pinctrl/qcom,mdm9607-pinctrl.yaml         | 23 ++++---
>  .../pinctrl/qcom,msm8226-pinctrl.yaml         | 63 +++++++++---------
>  .../bindings/pinctrl/qcom,msm8909-tlmm.yaml   | 64 +++++++++----------
>  .../pinctrl/qcom,msm8953-pinctrl.yaml         | 51 ++++++++-------
>  .../pinctrl/qcom,qcm2290-pinctrl.yaml         | 11 ++--
>  .../bindings/pinctrl/qcom,sc7280-pinctrl.yaml | 14 +++-
>  .../pinctrl/qcom,sc8280xp-pinctrl.yaml        |  4 +-
>  .../bindings/pinctrl/qcom,sdx55-pinctrl.yaml  | 51 ++++++++-------
>  .../bindings/pinctrl/qcom,sdx65-pinctrl.yaml  | 12 ++--
>  .../bindings/pinctrl/qcom,tlmm-common.yaml    | 20 ++++--
>  arch/arm/boot/dts/qcom-apq8026-lg-lenok.dts   |  6 +-
>  arch/arm/boot/dts/qcom-msm8226.dtsi           | 24 +++----
>  .../qcom-msm8974-lge-nexus5-hammerhead.dts    | 30 ++++-----
>  .../boot/dts/qcom-sdx55-telit-fn980-tlb.dts   | 45 +++++--------
>  arch/arm/boot/dts/qcom-sdx55.dtsi             |  1 +
>  arch/arm64/boot/dts/qcom/ipq6018-cp01-c1.dts  |  8 ++-
>  arch/arm64/boot/dts/qcom/ipq6018.dtsi         |  4 +-
>  18 files changed, 264 insertions(+), 227 deletions(-)
> 
> -- 
> 2.34.1
>