linux-gpio.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Iskren Chernev <iskren.chernev@gmail.com>
To: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
	Andy Gross <agross@kernel.org>,
	Bjorn Andersson <andersson@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>,
	Martin Botka <martin.botka@somainline.org>,
	Vinod Koul <vkoul@kernel.org>,
	linux-arm-msm@vger.kernel.org, linux-gpio@vger.kernel.org,
	devicetree@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 03/40] dt-bindings: pinctrl: qcom,sm6115-pinctrl: fix indentation in example
Date: Sun, 11 Sep 2022 16:02:47 +0300	[thread overview]
Message-ID: <e6f7a913-f434-9ea4-30f5-56b6990cc8bf@gmail.com> (raw)
In-Reply-To: <20220911111200.199182-4-krzysztof.kozlowski@linaro.org>



On 9/11/22 14:11, Krzysztof Kozlowski wrote:
> Bindings example should be indented with 4-spaces.
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Reviewed-by: Iskren Chernev <iskren.chernev@gmail.com>

> ---
>  .../bindings/pinctrl/qcom,sm6115-pinctrl.yaml | 80 +++++++++----------
>  1 file changed, 40 insertions(+), 40 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sm6115-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sm6115-pinctrl.yaml
> index 28b29bf714b4..e39fbb36d8c1 100644
> --- a/Documentation/devicetree/bindings/pinctrl/qcom,sm6115-pinctrl.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/qcom,sm6115-pinctrl.yaml
> @@ -150,44 +150,44 @@ additionalProperties: false
>
>  examples:
>    - |
> -        #include <dt-bindings/interrupt-controller/arm-gic.h>
> -        tlmm: pinctrl@500000 {
> -                compatible = "qcom,sm6115-tlmm";
> -                reg = <0x500000 0x400000>,
> -                        <0x900000 0x400000>,
> -                        <0xd00000 0x400000>;
> -                reg-names = "west", "south", "east";
> -                interrupts = <GIC_SPI 227 IRQ_TYPE_LEVEL_HIGH>;
> -                gpio-controller;
> -                #gpio-cells = <2>;
> -                interrupt-controller;
> -                #interrupt-cells = <2>;
> -                gpio-ranges = <&tlmm 0 0 114>;
> -
> -                sdc2_on_state: sdc2-on-state {
> -                        clk-pins {
> -                                pins = "sdc2_clk";
> -                                bias-disable;
> -                                drive-strength = <16>;
> -                        };
> -
> -                        cmd-pins {
> -                                pins = "sdc2_cmd";
> -                                bias-pull-up;
> -                                drive-strength = <10>;
> -                        };
> -
> -                        data-pins {
> -                                pins = "sdc2_data";
> -                                bias-pull-up;
> -                                drive-strength = <10>;
> -                        };
> -
> -                        sd-cd-pins {
> -                                pins = "gpio88";
> -                                function = "gpio";
> -                                bias-pull-up;
> -                                drive-strength = <2>;
> -                        };
> -                };
> +    #include <dt-bindings/interrupt-controller/arm-gic.h>
> +    tlmm: pinctrl@500000 {
> +        compatible = "qcom,sm6115-tlmm";
> +        reg = <0x500000 0x400000>,
> +              <0x900000 0x400000>,
> +              <0xd00000 0x400000>;
> +        reg-names = "west", "south", "east";
> +        interrupts = <GIC_SPI 227 IRQ_TYPE_LEVEL_HIGH>;
> +        gpio-controller;
> +        #gpio-cells = <2>;
> +        interrupt-controller;
> +        #interrupt-cells = <2>;
> +        gpio-ranges = <&tlmm 0 0 114>;
> +
> +        sdc2_on_state: sdc2-on-state {
> +            clk-pins {
> +                pins = "sdc2_clk";
> +                bias-disable;
> +                drive-strength = <16>;
> +            };
> +
> +            cmd-pins {
> +                pins = "sdc2_cmd";
> +                bias-pull-up;
> +                drive-strength = <10>;
> +            };
> +
> +            data-pins {
> +                pins = "sdc2_data";
> +                bias-pull-up;
> +                drive-strength = <10>;
> +            };
> +
> +            sd-cd-pins {
> +                pins = "gpio88";
> +                function = "gpio";
> +                bias-pull-up;
> +                drive-strength = <2>;
> +            };
>          };
> +    };

  reply	other threads:[~2022-09-11 13:02 UTC|newest]

Thread overview: 46+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-09-11 11:11 [PATCH 00/40] pinctrl/arm64: qcom: fix some of Qualcomm pinctrl schema warnings Krzysztof Kozlowski
2022-09-11 11:11 ` [PATCH 01/40] dt-bindings: pinctrl: qcom,sm6115-pinctrl: fix matching pin config Krzysztof Kozlowski
2022-09-11 12:59   ` Iskren Chernev
2022-09-11 13:26     ` Krzysztof Kozlowski
2022-09-11 11:11 ` [PATCH 02/40] dt-bindings: pinctrl: qcom,sm6115-pinctrl: fix matching SD card pins Krzysztof Kozlowski
2022-09-11 13:00   ` Iskren Chernev
2022-09-13 15:17   ` Rob Herring
2022-09-11 11:11 ` [PATCH 03/40] dt-bindings: pinctrl: qcom,sm6115-pinctrl: fix indentation in example Krzysztof Kozlowski
2022-09-11 13:02   ` Iskren Chernev [this message]
2022-09-11 11:11 ` [PATCH 04/40] dt-bindings: pinctrl: qcom,sm6125-pinctrl: fix matching pin config Krzysztof Kozlowski
2022-09-11 11:11 ` [PATCH 05/40] dt-bindings: pinctrl: qcom,sm6125-pinctrl: do not require function on non-GPIOs Krzysztof Kozlowski
2022-09-11 11:11 ` [PATCH 06/40] dt-bindings: pinctrl: qcom,sm6125-pinctrl: extend example Krzysztof Kozlowski
2022-09-11 11:11 ` [PATCH 07/40] dt-bindings: pinctrl: qcom,sm6350-pinctrl: fix matching pin config Krzysztof Kozlowski
2022-09-11 11:11 ` [PATCH 08/40] dt-bindings: pinctrl: qcom,sm6350-pinctrl: do not require function on non-GPIOs Krzysztof Kozlowski
2022-09-11 11:11 ` [PATCH 09/40] dt-bindings: pinctrl: qcom,sm6350-pinctrl: fix indentation in example Krzysztof Kozlowski
2022-09-11 11:11 ` [PATCH 10/40] dt-bindings: pinctrl: qcom,sm6375-pinctrl: fix matching pin config Krzysztof Kozlowski
2022-09-11 11:11 ` [PATCH 11/40] dt-bindings: pinctrl: qcom,sm6375-pinctrl: do not require function on non-GPIOs Krzysztof Kozlowski
2022-09-11 11:11 ` [PATCH 12/40] dt-bindings: pinctrl: qcom,sm6375-pinctrl: fix indentation in example Krzysztof Kozlowski
2022-09-11 11:11 ` [PATCH 13/40] dt-bindings: pinctrl: qcom,sm8250-pinctrl: do not require function on non-GPIOs Krzysztof Kozlowski
2022-09-11 11:11 ` [PATCH 14/40] dt-bindings: pinctrl: qcom,sm8250-pinctrl: reference tlmm common pins Krzysztof Kozlowski
2022-09-11 11:11 ` [PATCH 15/40] dt-bindings: pinctrl: qcom,sm8250-pinctrl: fix indentation in example Krzysztof Kozlowski
2022-09-11 11:11 ` [PATCH 16/40] dt-bindings: pinctrl: qcom,sm8350-pinctrl: fix matching pin config Krzysztof Kozlowski
2022-09-11 11:11 ` [PATCH 17/40] dt-bindings: pinctrl: qcom,sm8350-pinctrl: fix indentation in example Krzysztof Kozlowski
2022-09-11 11:11 ` [PATCH 18/40] dt-bindings: pinctrl: qcom,sm8350-pinctrl: do not require function on non-GPIOs Krzysztof Kozlowski
2022-09-11 11:11 ` [PATCH 19/40] dt-bindings: pinctrl: qcom,sm8450-pinctrl: fix matching pin config Krzysztof Kozlowski
2022-09-11 11:11 ` [PATCH 20/40] dt-bindings: pinctrl: qcom,sm8450-pinctrl: fix indentation in example Krzysztof Kozlowski
2022-09-11 11:11 ` [PATCH 21/40] dt-bindings: pinctrl: qcom,sm8450-pinctrl: do not require function on non-GPIOs Krzysztof Kozlowski
2022-09-11 11:11 ` [PATCH 22/40] dt-bindings: pinctrl: qcom,sm8450-pinctrl: add gpio-line-names Krzysztof Kozlowski
2022-09-11 11:11 ` [PATCH 23/40] dt-bindings: pinctrl: qcom,sc7280-pinctrl: correct number of GPIOs Krzysztof Kozlowski
2022-09-11 11:11 ` [PATCH 24/40] dt-bindings: pinctrl: qcom,sc7280-pinctrl: do not require function on non-GPIOs Krzysztof Kozlowski
2022-09-11 11:11 ` [PATCH 25/40] dt-bindings: pinctrl: qcom,sc7280-pinctrl: add gpio-line-names Krzysztof Kozlowski
2022-09-11 11:11 ` [PATCH 26/40] dt-bindings: pinctrl: qcom,sc7280-pinctrl: reference tlmm schema Krzysztof Kozlowski
2022-09-11 11:11 ` [PATCH 27/40] dt-bindings: pinctrl: qcom,sc7280-pinctrl: fix indentation in example Krzysztof Kozlowski
2022-09-11 11:11 ` [PATCH 28/40] dt-bindings: pinctrl: qcom,sc8180x-pinctrl: fix matching pin config Krzysztof Kozlowski
2022-09-11 11:11 ` [PATCH 29/40] dt-bindings: pinctrl: qcom,sc8180x-pinctrl: do not require function on non-GPIOs Krzysztof Kozlowski
2022-09-11 11:11 ` [PATCH 30/40] dt-bindings: pinctrl: qcom,sc8180x-pinctrl: fix indentation in example Krzysztof Kozlowski
2022-09-11 11:11 ` [PATCH 31/40] dt-bindings: pinctrl: qcom,sc8280xp-pinctrl: fix matching pin config Krzysztof Kozlowski
2022-09-11 11:11 ` [PATCH 32/40] dt-bindings: pinctrl: qcom,sc8280xp-pinctrl: do not require function on non-GPIOs Krzysztof Kozlowski
2022-09-11 11:11 ` [PATCH 33/40] dt-bindings: pinctrl: qcom,sc8280xp-pinctrl: fix indentation in example Krzysztof Kozlowski
2022-09-11 11:11 ` [PATCH 34/40] arm64: dts: qcom: sm6125: align TLMM pin configuration with DT schema Krzysztof Kozlowski
2022-09-11 11:11 ` [PATCH 35/40] arm64: dts: qcom: sm6350: " Krzysztof Kozlowski
2022-09-11 11:11 ` [PATCH 36/40] arm64: dts: qcom: sm8350-sagami: correct TS pin property Krzysztof Kozlowski
2022-09-11 11:11 ` [PATCH 37/40] arm64: dts: qcom: sm8350: align TLMM pin configuration with DT schema Krzysztof Kozlowski
2022-09-11 11:11 ` [PATCH 38/40] arm64: dts: qcom: sm8450: " Krzysztof Kozlowski
2022-09-11 11:11 ` [PATCH 39/40] arm64: dts: qcom: sc7280: " Krzysztof Kozlowski
2022-09-11 11:12 ` [PATCH 40/40] arm64: dts: qcom: sc7280-herobrine: correct TLMM gpio-line-names Krzysztof Kozlowski

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=e6f7a913-f434-9ea4-30f5-56b6990cc8bf@gmail.com \
    --to=iskren.chernev@gmail.com \
    --cc=agross@kernel.org \
    --cc=andersson@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=konrad.dybcio@somainline.org \
    --cc=krzysztof.kozlowski+dt@linaro.org \
    --cc=krzysztof.kozlowski@linaro.org \
    --cc=linus.walleij@linaro.org \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=linux-gpio@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=martin.botka@somainline.org \
    --cc=robh+dt@kernel.org \
    --cc=vkoul@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).