Re: [PATCH v2 0/4] pinctrl: ingenic: add support for x1600 SoC and MII and I2S for jz4730

From: Paul Cercueil
Date: Fri Feb 28 2025 - 13:29:40 EST


Hi Nikolaus,

Le vendredi 28 février 2025 à 14:33 +0100, H. Nikolaus Schaller a
écrit :
> PATCH V2 2025-02-28 14:33:57:
> Fix pwm5/pwm6/pwm7 pin groups (each one can be muxed to one of two
> pads while pwm0-4 have only one pad) for X1600.
>
> PATCH V1 2025-02-26 18:14:53:
> This series expands pinctrl support for some Ingenic/Lumissil SoC.
> For the jz4730 we add MII and I2S pinctrl and general x1600 support.
>
> The x1600 parts were jointly developed.
>
> Code was tested on LX16 board (x1600) and Alpha400 (jz4730) and
> on CI20 (jz4780).
>
> Co-authored-by: Andreas Kemnade <andreas@xxxxxxxxxxxx>
> Co-authored-by: H. Nikolaus Schaller <hns@xxxxxxxxxxxxx>

Reviewed-by: Paul Cercueil <paul@xxxxxxxxxxxxxxx>

For the whole patchset, with the nit on patch 4/4.

Either fix it when sending a v3 with my reviewed-by tag above, unless
Linus does it when merging.

Cheers,
-Paul

>
>
> H. Nikolaus Schaller (3):
>   bindings: ingenic,pinctrl: add x1600
>   pinctrl: ingenic: jz4730: add pinmux for MII
>   pinctrl: ingenic: jz4730: add pinmux for I2S interface
>
> Paul Boddie (1):
>   pinctrl: ingenic: add x1600 support
>
>  .../bindings/pinctrl/ingenic,pinctrl.yaml     |   2 +
>  drivers/pinctrl/pinctrl-ingenic.c             | 261
> +++++++++++++++++-
>  2 files changed, 261 insertions(+), 2 deletions(-)
>