Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Tue, 23 Mar 2021 16:25:25 -0600
From:   Rob Herring <robh@kernel.org>
To:     =?iso-8859-1?Q?=C1lvaro_Fern=E1ndez?= Rojas <noltari@gmail.com>
Cc:     Linus Walleij <linus.walleij@linaro.org>,
        Bartosz Golaszewski <bgolaszewski@baylibre.com>,
        Lee Jones <lee.jones@linaro.org>,
        Michael Walle <michael@walle.cc>,
        Florian Fainelli <f.fainelli@gmail.com>,
        bcm-kernel-feedback-list@broadcom.com,
        Jonas Gorski <jonas.gorski@gmail.com>,
        Necip Fazil Yildiran <fazilyildiran@gmail.com>,
        Andy Shevchenko <andy.shevchenko@gmail.com>,
        linux-gpio@vger.kernel.org, devicetree@vger.kernel.org,
        linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH v8 17/22] dt-bindings: add BCM63268 pincontroller binding
 documentation
Message-ID: <20210323222525.GA1449625@robh.at.kernel.org>
References: <20210317143803.26127-1-noltari@gmail.com>
 <20210317143803.26127-18-noltari@gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=iso-8859-1
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <20210317143803.26127-18-noltari@gmail.com>
Precedence: bulk

On Wed, Mar 17, 2021 at 03:37:58PM +0100, ?lvaro Fern?ndez Rojas wrote:
> Add binding documentation for the pincontrol core found in the BCM63268
> family SoCs.
> 
> Co-developed-by: Jonas Gorski <jonas.gorski@gmail.com></jonas.gorski>

checkpatch.pl highlights an error here.

> Signed-off-by: Jonas Gorski <jonas.gorski@gmail.com>
> Signed-off-by: ?lvaro Fern?ndez Rojas <noltari@gmail.com>
> ---
>  v8: add changes suggested by Rob Herring
>  v7: add changes suggested by Rob Herring
>  v6: add changes suggested by Rob Herring
>  v5: change Documentation to dt-bindings in commit title
>  v4: no changes
>  v3: add new gpio node
>  v2: remove interrupts
> 
>  .../pinctrl/brcm,bcm63268-pinctrl.yaml        | 166 ++++++++++++++++++
>  1 file changed, 166 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/pinctrl/brcm,bcm63268-pinctrl.yaml
> 
> diff --git a/Documentation/devicetree/bindings/pinctrl/brcm,bcm63268-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/brcm,bcm63268-pinctrl.yaml
> new file mode 100644
> index 000000000000..1acd6ef38c5f
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/pinctrl/brcm,bcm63268-pinctrl.yaml
> @@ -0,0 +1,166 @@
> +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/pinctrl/brcm,bcm63268-pinctrl.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Broadcom BCM63268 pin controller
> +
> +maintainers:
> +  - ?lvaro Fern?ndez Rojas <noltari@gmail.com>
> +  - Jonas Gorski <jonas.gorski@gmail.com>
> +
> +description:
> +  Bindings for Broadcom's BCM63268 memory-mapped pin controller.
> +
> +properties:
> +  compatible:
> +    const: brcm,bcm63268-pinctrl
> +
> +  reg:
> +    maxItems: 3
> +
> +patternProperties:
> +  '-pins$':
> +    type: object
> +    $ref: pinmux-node.yaml#
> +
> +    properties:
> +      function:
> +        $ref: "pinmux-node.yaml#/properties/function"

Drop this. The previous $ref already does the same thing.

> +        enum: [ serial_led_clk, serial_led_data, hsspi_cs4, hsspi_cs5,
> +                hsspi_cs6, hsspi_cs7, adsl_spi_miso, adsl_spi_mosi,
> +                vreq_clk, pcie_clkreq_b, robosw_led_clk, robosw_led_data,
> +                nand, gpio35_alt, dectpd, vdsl_phy_override_0,
> +                vdsl_phy_override_1, vdsl_phy_override_2,
> +                vdsl_phy_override_3, dsl_gpio8, dsl_gpio9 ]
> +
> +      pins:
> +        $ref: "pinmux-node.yaml#/properties/pins"

And this.

Same applies to the other pinctrl bindings.

> +        enum: [ gpio0, gpio1, gpio16, gpio17, gpio8, gpio9, gpio18, gpio19,
> +                gpio22, gpio23, gpio30, gpio31, nand_grp, gpio35
> +                dectpd_grp, vdsl_phy_override_0_grp,
> +                vdsl_phy_override_1_grp, vdsl_phy_override_2_grp,
> +                vdsl_phy_override_3_grp, dsl_gpio8, dsl_gpio9 ]
> +
> +required:
> +  - compatible
> +  - reg
> +
> +additionalProperties: false
> +
> +examples:
> +  - |
> +    pinctrl@10 {
> +      compatible = "brcm,bcm63268-pinctrl";
> +      reg = <0x10 0x4>, <0x18 0x8>, <0x38 0x4>;
> +
> +      pinctrl_serial_led: serial_led-pins {
> +        pinctrl_serial_led_clk: serial_led_clk-pins {
> +          function = "serial_led_clk";
> +          pins = "gpio0";
> +        };
> +
> +        pinctrl_serial_led_data: serial_led_data-pins {
> +          function = "serial_led_data";
> +          pins = "gpio1";
> +        };
> +      };
> +
> +      pinctrl_hsspi_cs4: hsspi_cs4-pins {
> +        function = "hsspi_cs4";
> +        pins = "gpio16";
> +      };
> +
> +      pinctrl_hsspi_cs5: hsspi_cs5-pins {
> +        function = "hsspi_cs5";
> +        pins = "gpio17";
> +      };
> +
> +      pinctrl_hsspi_cs6: hsspi_cs6-pins {
> +        function = "hsspi_cs6";
> +        pins = "gpio8";
> +      };
> +
> +      pinctrl_hsspi_cs7: hsspi_cs7-pins {
> +        function = "hsspi_cs7";
> +        pins = "gpio9";
> +      };
> +
> +      pinctrl_adsl_spi: adsl_spi-pins {
> +        pinctrl_adsl_spi_miso: adsl_spi_miso-pins {
> +          function = "adsl_spi_miso";
> +          pins = "gpio18";
> +        };
> +
> +        pinctrl_adsl_spi_mosi: adsl_spi_mosi-pins {
> +          function = "adsl_spi_mosi";
> +          pins = "gpio19";
> +        };
> +      };
> +
> +      pinctrl_vreq_clk: vreq_clk-pins {
> +        function = "vreq_clk";
> +        pins = "gpio22";
> +      };
> +
> +      pinctrl_pcie_clkreq_b: pcie_clkreq_b-pins {
> +        function = "pcie_clkreq_b";
> +        pins = "gpio23";
> +      };
> +
> +      pinctrl_robosw_led_clk: robosw_led_clk-pins {
> +        function = "robosw_led_clk";
> +        pins = "gpio30";
> +      };
> +
> +      pinctrl_robosw_led_data: robosw_led_data-pins {
> +        function = "robosw_led_data";
> +        pins = "gpio31";
> +      };
> +
> +      pinctrl_nand: nand-pins {
> +        function = "nand";
> +        group = "nand_grp";
> +      };
> +
> +      pinctrl_gpio35_alt: gpio35_alt-pins {
> +        function = "gpio35_alt";
> +        pin = "gpio35";
> +      };
> +
> +      pinctrl_dectpd: dectpd-pins {
> +        function = "dectpd";
> +        group = "dectpd_grp";
> +      };
> +
> +      pinctrl_vdsl_phy_override_0: vdsl_phy_override_0-pins {
> +        function = "vdsl_phy_override_0";
> +        group = "vdsl_phy_override_0_grp";
> +      };
> +
> +      pinctrl_vdsl_phy_override_1: vdsl_phy_override_1-pins {
> +        function = "vdsl_phy_override_1";
> +        group = "vdsl_phy_override_1_grp";
> +      };
> +
> +      pinctrl_vdsl_phy_override_2: vdsl_phy_override_2-pins {
> +        function = "vdsl_phy_override_2";
> +        group = "vdsl_phy_override_2_grp";
> +      };
> +
> +      pinctrl_vdsl_phy_override_3: vdsl_phy_override_3-pins {
> +        function = "vdsl_phy_override_3";
> +        group = "vdsl_phy_override_3_grp";
> +      };
> +
> +      pinctrl_dsl_gpio8: dsl_gpio8-pins {
> +        function = "dsl_gpio8";
> +        group = "dsl_gpio8";
> +      };
> +
> +      pinctrl_dsl_gpio9: dsl_gpio9-pins {
> +        function = "dsl_gpio9";
> +        group = "dsl_gpio9";
> +      };
> +    };
> -- 
> 2.20.1
>