linux-kernel - Re: [PATCH v5 2/3] pinctrl: renesas: Add support for RZ/T2H

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAMuHMdXTs114PLGm-fSrCw2MwK1zv8CMooWVwUvkrSAzY1ZpVw@mail.gmail.com>
Date: Mon, 11 Aug 2025 14:52:57 +0200
From: Geert Uytterhoeven <geert@...ux-m68k.org>
To: Prabhakar <prabhakar.csengg@...il.com>
Cc: Linus Walleij <linus.walleij@...aro.org>, Rob Herring <robh@...nel.org>, 
	Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley <conor+dt@...nel.org>, 
	Magnus Damm <magnus.damm@...il.com>, Bartosz Golaszewski <brgl@...ev.pl>, linux-renesas-soc@...r.kernel.org, 
	linux-gpio@...r.kernel.org, devicetree@...r.kernel.org, 
	linux-kernel@...r.kernel.org, Biju Das <biju.das.jz@...renesas.com>, 
	Fabrizio Castro <fabrizio.castro.jz@...esas.com>, 
	Thierry Bultel <thierry.bultel.yh@...renesas.com>, 
	Lad Prabhakar <prabhakar.mahadev-lad.rj@...renesas.com>
Subject: Re: [PATCH v5 2/3] pinctrl: renesas: Add support for RZ/T2H

On Fri, 8 Aug 2025 at 15:30, Prabhakar <prabhakar.csengg@...il.com> wrote:
> From: Thierry Bultel <thierry.bultel.yh@...renesas.com>
>
> Add the pinctrl and gpio driver for RZ/T2H
>
> Signed-off-by: Thierry Bultel <thierry.bultel.yh@...renesas.com>
> Co-developed-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@...renesas.com>
> Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@...renesas.com>
> ---
> v4->v5:
> - Updated Kconfig description to include RZ/N2H SoC
> - Grouped PM_* macros
> - Made use of BIT() macro for PM_INPUT/OUTPUT
> - Added necessary new lines for readability
> - Dropped port_pins from struct rzt2h_pinctrl_data and
>   made use of rzt2h_gpio_names directly
> - Added comment for mutex and lock in struct rzt2h_pinctrl
>   to clarify its use
> - Replaced u16 to unsigned in rzt2h_pinctrl_write*/rzt2h_pinctrl_read*
> - Updated rzt2h_validate_pin() to make used of BIT() macro instead of
>   open coding (1 << pin)
> - Added consistent spacing around colons
> - Inverted the logic to reduce indentation
> - Updated if checks to use `reg & PM_OUTPUT/INPUT` instead of
>   `reg == PM_OUTPUT/INPUT`
> - Dropped assigning drv_data for individual pins
> - Opencoded devm_platform_ioremap_resource_byname() for "srs" resource
>   to avoid error message print as the resource is optional
> - Replaced set_rv with set
> - Added RZT2H_GET_BASE() macro and simplified
>   rzt2h_pinctrl_write*/rzt2h_pinctrl_read* macros
> - Moved rzt2h_gpio_direction_output() below rzt2h_gpio_direction_input()
> - Used consistent name ie reg64/reg16 in rzt2h_pinctrl_set_pfc_mode()

Reviewed-by: Geert Uytterhoeven <geert+renesas@...der.be>
i.e. will queue in renesas-pinctrl for v6.18.

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@...ux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds