Re: [PATCH 0/2] pinctl: armada-37xx: fix for pins 32+

From: Linus Walleij
Date: Tue Jun 25 2019 - 08:31:15 EST


On Tue, Jun 18, 2019 at 6:01 PM <alpawi@xxxxxxxxxx> wrote:

> From: Patrick Williams <alpawi@xxxxxxxxxx>
>
> The 37xx GPIO config registers are only 32 bits long and
> span 2 registers for the NB GPIO controller. The function
> to calculate the offset was missing the increase to the
> config register.
>
> I have tested both raw gpio access and interrupts using
> libgpiod utilities on an Espressonbin.
>
> The first patch is a simple rename of a function because
> the original name implied it was doing IO itself ("update
> reg"). This patch could be dropped if undesired.
>
> The second patch contains the fix for GPIOs 32+.

This looks good overall. I am waiting for a maintainer review.
If nothing happens in a week, poke me and I'll just apply
the patches.

Yours,
Linus Walleij