Re: [PATCH v3 05/10] dt-bindings: pinctrl: Add bindings for new Ingenic SoCs.

[Paul Cercueil <paul@crapouillou.net>]

Le mer. 17 mars 2021 à 17:57, 周琰杰 (Zhou Yanjie) 
<zhouyanjie@wanyeetech.com> a écrit :
> Add the pinctrl bindings for the JZ4730 SoC, the JZ4750 SoC,
> the JZ4755 SoC, the JZ4775 SoC and the X2000 SoC from Ingenic.
> 
> Signed-off-by: 周琰杰 (Zhou Yanjie) <zhouyanjie@wanyeetech.com>
> ---
> 
> Notes:
>     v2:
>     New patch.
> 
>     v2->v3:
>     No change.
> 
>  .../bindings/pinctrl/ingenic,pinctrl.yaml          | 23 
> ++++++++++++++++++----
>  1 file changed, 19 insertions(+), 4 deletions(-)
> 
> diff --git 
> a/Documentation/devicetree/bindings/pinctrl/ingenic,pinctrl.yaml 
> b/Documentation/devicetree/bindings/pinctrl/ingenic,pinctrl.yaml
> index 44c04d1..60604fc 100644
> --- a/Documentation/devicetree/bindings/pinctrl/ingenic,pinctrl.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/ingenic,pinctrl.yaml
> @@ -17,10 +17,12 @@ description: >
>    naming scheme "PxN" where x is a character identifying the GPIO 
> port with
>    which the pin is associated and N is an integer from 0 to 31 
> identifying the
>    pin within that GPIO port. For example PA0 is the first pin in 
> GPIO port A,
> -  and PB31 is the last pin in GPIO port B. The JZ4740, the X1000 and 
> the X1830
> -  contains 4 GPIO ports, PA to PD, for a total of 128 pins. The 
> JZ4760, the
> -  JZ4770 and the JZ4780 contains 6 GPIO ports, PA to PF, for a total 
> of 192
> -  pins.
> +  and PB31 is the last pin in GPIO port B. The JZ4730, the JZ4740, 
> the X1000
> +  and the X1830 contains 4 GPIO ports, PA to PD, for a total of 128 
> pins. The
> +  X2000 contains 5 GPIO ports, PA to PE, for a total of 160 pins. 
> The JZ4750,
> +  the JZ4755 the JZ4760, the JZ4770 and the JZ4780 contains 6 GPIO 
> ports, PA
> +  to PF, for a total of 192 pins. The JZ4775 contains 7 GPIO ports, 
> PA to PG,
> +  for a total of 224 pins.

While we're at it, the JZ4725B has also 4 GPIO ports.

> 
>  maintainers:
>    - Paul Cercueil <paul@crapouillou.net>
> @@ -32,20 +34,28 @@ properties:
>    compatible:
>      oneOf:
>        - enum:
> +          - ingenic,jz4730-pinctrl
>            - ingenic,jz4740-pinctrl
>            - ingenic,jz4725b-pinctrl
> +          - ingenic,jz4750-pinctrl
> +          - ingenic,jz4755-pinctrl
>            - ingenic,jz4760-pinctrl
>            - ingenic,jz4770-pinctrl
> +          - ingenic,jz4775-pinctrl
>            - ingenic,jz4780-pinctrl
>            - ingenic,x1000-pinctrl
>            - ingenic,x1500-pinctrl
>            - ingenic,x1830-pinctrl
> +          - ingenic,x2000-pinctrl
>        - items:
>            - const: ingenic,jz4760b-pinctrl
>            - const: ingenic,jz4760-pinctrl
>        - items:
>            - const: ingenic,x1000e-pinctrl
>            - const: ingenic,x1000-pinctrl
> +      - items:
> +          - const: ingenic,x2000e-pinctrl
> +          - const: ingenic,x2000-pinctrl
> 
>    reg:
>      maxItems: 1
> @@ -62,14 +72,19 @@ patternProperties:
>      properties:
>        compatible:
>          enum:
> +          - ingenic,jz4730-gpio
>            - ingenic,jz4740-gpio
>            - ingenic,jz4725b-gpio
> +          - ingenic,jz4750-gpio
> +          - ingenic,jz4755-gpio
>            - ingenic,jz4760-gpio
>            - ingenic,jz4770-gpio
> +          - ingenic,jz4775-gpio
>            - ingenic,jz4780-gpio
>            - ingenic,x1000-gpio
>            - ingenic,x1500-gpio
>            - ingenic,x1830-gpio
> +          - ingenic,x2000-gpio
> 
>        reg:
>          items:
> --
> 2.7.4
>