lkml.org 
[lkml]   [2022]   [Apr]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH 10/14] dt-bindings: pinctrl: rt2880: fix binding name, pin groups and functions
From
On 13/04/2022 08:07, Arınç ÜNAL wrote:
> Change binding name from ralink,rt2880-pinmux to ralink,rt2880-pinctrl.
> This is the binding for the Ralink RT2880 pinctrl subdriver.

What I don't see here is why you are doing this. pinmux/pinctrl have the
same meaning, I guess?

>
> Current pin group and function bindings are for MT7621. Put bindings for
> RT2880 instead.
>
> Signed-off-by: Arınç ÜNAL <arinc.unal@arinc9.com>
> ---
> ...pinmux.yaml => ralink,rt2880-pinctrl.yaml} | 24 +++++++++----------
> 1 file changed, 12 insertions(+), 12 deletions(-)
> rename Documentation/devicetree/bindings/pinctrl/{ralink,rt2880-pinmux.yaml => ralink,rt2880-pinctrl.yaml} (56%)
>
> diff --git a/Documentation/devicetree/bindings/pinctrl/ralink,rt2880-pinmux.yaml b/Documentation/devicetree/bindings/pinctrl/ralink,rt2880-pinctrl.yaml
> similarity index 56%
> rename from Documentation/devicetree/bindings/pinctrl/ralink,rt2880-pinmux.yaml
> rename to Documentation/devicetree/bindings/pinctrl/ralink,rt2880-pinctrl.yaml
> index 9de8b0c075e2..c657bbf9fdda 100644
> --- a/Documentation/devicetree/bindings/pinctrl/ralink,rt2880-pinmux.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/ralink,rt2880-pinctrl.yaml
> @@ -1,21 +1,23 @@
> # SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
> %YAML 1.2
> ---
> -$id: http://devicetree.org/schemas/pinctrl/ralink,rt2880-pinmux.yaml#
> +$id: http://devicetree.org/schemas/pinctrl/ralink,rt2880-pinctrl.yaml#
> $schema: http://devicetree.org/meta-schemas/core.yaml#
>
> -title: Ralink rt2880 pinmux controller
> +title: Ralink RT2880 Pin Controller
>
> maintainers:
> + - Arınç ÜNAL <arinc.unal@arinc9.com>

Mention this in commit msg.

> - Sergio Paracuellos <sergio.paracuellos@gmail.com>
>
> description:
> - The rt2880 pinmux can only set the muxing of pin groups. Muxing indiviual pins
> + Ralink RT2880 pin controller for RT2880 SoC.
> + The pin controller can only set the muxing of pin groups. Muxing indiviual pins
> is not supported. There is no pinconf support.
>
> properties:
> compatible:
> - const: ralink,rt2880-pinmux
> + const: ralink,rt2880-pinctrl

you need to deprecate old property and add a new one.


>
> patternProperties:
> '-pins$':
> @@ -28,14 +30,12 @@ patternProperties:
>
> properties:
> groups:
> - description: Name of the pin group to use for the functions.
> - enum: [i2c, jtag, mdio, pcie, rgmii1, rgmii2, sdhci, spi,
> - uart1, uart2, uart3, wdt]
> + description: The pin group to select.
> + enum: [i2c, spi, uartlite, jtag, mdio, sdram, pci]
> +
> function:
> - description: The mux function to select
> - enum: [gpio, i2c, i2s, jtag, mdio, nand1, nand2, pcie refclk,
> - pcie rst, pcm, rgmii1, rgmii2, sdhci, spdif2, spdif3,
> - spi, uart1, uart2, uart3, wdt refclk, wdt rst]
> + description: The mux function to select.
> + enum: [gpio, i2c, spi, uartlite, jtag, mdio, sdram, pci]
>

These were all incorrect for rt2880, I understand?


Best regards,
Krzysztof

\
 
 \ /
  Last update: 2022-04-13 17:27    [W:0.282 / U:0.416 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site