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;
Message-ID: <e6f7a913-f434-9ea4-30f5-56b6990cc8bf@gmail.com>
Date:   Sun, 11 Sep 2022 16:02:47 +0300
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101
 Thunderbird/91.12.0
Subject: Re: [PATCH 03/40] dt-bindings: pinctrl: qcom,sm6115-pinctrl: fix
 indentation in example
Content-Language: en-US
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
References: <20220911111200.199182-1-krzysztof.kozlowski@linaro.org>
 <20220911111200.199182-4-krzysztof.kozlowski@linaro.org>
From:   Iskren Chernev <iskren.chernev@gmail.com>
In-Reply-To: <20220911111200.199182-4-krzysztof.kozlowski@linaro.org>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: bulk


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>;
> +            };
>          };
> +    };