[PATCH 5/6] dt-bindings: pinctrl: qcom: msm8960: rename msmgpio node to tlmm

Antony Kurniawan Soemardi posted 6 patches 4 months, 4 weeks ago
There is a newer version of this series
[PATCH 5/6] dt-bindings: pinctrl: qcom: msm8960: rename msmgpio node to tlmm
Posted by Antony Kurniawan Soemardi 4 months, 4 weeks ago
Rename the GPIO controller node from "msmgpio" to "tlmm" to match the
convention used by other Qualcomm SoCs.

Suggested-by: Shinjo Park <peremen@gmail.com>
Signed-off-by: Antony Kurniawan Soemardi <linux@smankusors.com>
---
 Documentation/devicetree/bindings/pinctrl/qcom,msm8960-pinctrl.yaml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,msm8960-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,msm8960-pinctrl.yaml
index 46618740bd312b975321427e5ffa34811c68e652..03a3692770142048391577401869cea9a7735053 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,msm8960-pinctrl.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,msm8960-pinctrl.yaml
@@ -107,12 +107,12 @@ examples:
   - |
     #include <dt-bindings/interrupt-controller/arm-gic.h>
 
-    msmgpio: pinctrl@800000 {
+    tlmm: pinctrl@800000 {
         compatible = "qcom,msm8960-pinctrl";
         reg = <0x800000 0x4000>;
         #gpio-cells = <2>;
         gpio-controller;
-        gpio-ranges = <&msmgpio 0 0 152>;
+        gpio-ranges = <&tlmm 0 0 152>;
         interrupts = <GIC_SPI 16 IRQ_TYPE_LEVEL_HIGH>;
         interrupt-controller;
         #interrupt-cells = <2>;

-- 
2.34.1
Re: [PATCH 5/6] dt-bindings: pinctrl: qcom: msm8960: rename msmgpio node to tlmm
Posted by Bjorn Andersson 4 months, 3 weeks ago
On Sun, Sep 14, 2025 at 06:34:59PM +0000, Antony Kurniawan Soemardi wrote:
> Rename the GPIO controller node from "msmgpio" to "tlmm" to match the
> convention used by other Qualcomm SoCs.
> 
> Suggested-by: Shinjo Park <peremen@gmail.com>
> Signed-off-by: Antony Kurniawan Soemardi <linux@smankusors.com>

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


Note that this patch is in a different subsystem and is picked up by a
different maintainer and there are no dependencies with the other
patches. It would therefor have been preferable if you sent this patch
alone.

No need to do anything this time though, I expect Linus to fish out this
patch from the series.

Thank you,
Bjorn

> ---
>  Documentation/devicetree/bindings/pinctrl/qcom,msm8960-pinctrl.yaml | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,msm8960-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,msm8960-pinctrl.yaml
> index 46618740bd312b975321427e5ffa34811c68e652..03a3692770142048391577401869cea9a7735053 100644
> --- a/Documentation/devicetree/bindings/pinctrl/qcom,msm8960-pinctrl.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/qcom,msm8960-pinctrl.yaml
> @@ -107,12 +107,12 @@ examples:
>    - |
>      #include <dt-bindings/interrupt-controller/arm-gic.h>
>  
> -    msmgpio: pinctrl@800000 {
> +    tlmm: pinctrl@800000 {
>          compatible = "qcom,msm8960-pinctrl";
>          reg = <0x800000 0x4000>;
>          #gpio-cells = <2>;
>          gpio-controller;
> -        gpio-ranges = <&msmgpio 0 0 152>;
> +        gpio-ranges = <&tlmm 0 0 152>;
>          interrupts = <GIC_SPI 16 IRQ_TYPE_LEVEL_HIGH>;
>          interrupt-controller;
>          #interrupt-cells = <2>;
> 
> -- 
> 2.34.1
>
Re: [PATCH 5/6] dt-bindings: pinctrl: qcom: msm8960: rename msmgpio node to tlmm
Posted by Linus Walleij 3 months, 4 weeks ago
On Tue, Sep 16, 2025 at 5:11 PM Bjorn Andersson <andersson@kernel.org> wrote:
> On Sun, Sep 14, 2025 at 06:34:59PM +0000, Antony Kurniawan Soemardi wrote:
> > Rename the GPIO controller node from "msmgpio" to "tlmm" to match the
> > convention used by other Qualcomm SoCs.
> >
> > Suggested-by: Shinjo Park <peremen@gmail.com>
> > Signed-off-by: Antony Kurniawan Soemardi <linux@smankusors.com>
>
> Reviewed-by: Bjorn Andersson <andersson@kernel.org>
>
> Note that this patch is in a different subsystem and is picked up by a
> different maintainer and there are no dependencies with the other
> patches. It would therefor have been preferable if you sent this patch
> alone.
>
> No need to do anything this time though, I expect Linus to fish out this
> patch from the series.

I fished out this patch and applied it to the pin control tree :D

Yours,
Linus Walleij