Re: [PATCH v2 1/3] gpio: brcmstb: correct hwirq to bank map
From: Linus Walleij
Date: Tue Jan 27 2026 - 17:16:01 EST
On Tue, Jan 27, 2026 at 10:47 PM Florian Fainelli
<florian.fainelli@xxxxxxxxxxxx> wrote:
> From: Doug Berger <opendmb@xxxxxxxxx>
>
> The brcmstb_gpio_hwirq_to_bank() function was designed to
> accommodate the downward numbering of dynamic GPIOs by
> traversing the bank list in the reverse order. However, the
> dynamic numbering has changed to increment upward which can
> produce an incorrect mapping.
>
> The function is modified to no longer assume an ordering of
> the list to accommodate either option.
>
> Fixes: 7b61212f2a07 ("gpiolib: Get rid of ARCH_NR_GPIOS")
> Signed-off-by: Doug Berger <opendmb@xxxxxxxxx>
> Signed-off-by: Florian Fainelli <florian.fainelli@xxxxxxxxxxxx>
This looks right.
Reviewed-by: Linus Walleij <linusw@xxxxxxxxxx>
Yours,
Linus Walleij