Re: [PATCH v3] gpio: aspeed: Add banks Y, Z, AA, AB and AC

From: Joel Stanley
Date: Fri Jan 27 2017 - 01:05:05 EST


On Fri, Jan 27, 2017 at 3:24 PM, Andrew Jeffery <andrew@xxxxxxxx> wrote:
> This is less straight-forward than one would hope, as some banks only
> have 4 pins rather than 8, others are output only, yet more (W and
> X, already supported) are input-only, and in the case of the g4 SoC bank
> AC doesn't exist.
>
> Add some structs to describe the varying properties of different banks
> and integrate mechanisms to deny requests for unsupported
> configurations.
>
> Signed-off-by: Andrew Jeffery <andrew@xxxxxxxx>

Acked-by: Joel Stanley <joel@xxxxxxxxx>

Cheers,

Joel

> ---
>
> Since v2:
>
> Drop linux/gpio.h include and change away from using GPIOF_* macros
>
> Since v1:
>
> Fix types for for_each_clear_bit() iteration
>
> drivers/gpio/gpio-aspeed.c | 173 +++++++++++++++++++++++++++++++++++++++++----
> 1 file changed, 159 insertions(+), 14 deletions(-)