[PATCH 2/2] gpio: aspeed: Support banks Y, Z, AA and BB
Andrew Jeffery
andrew at aj.id.au
Tue Jan 10 11:29:54 AEDT 2017
Hi Xo,
On Mon, 2017-01-09 at 16:24 -0800, Xo Wang wrote:
> Hi Joel,
>
> > On Sun, Dec 11, 2016 at 8:06 PM, Joel Stanley <joel at jms.id.au> wrote:
> > These banks have extra limitations compared to those currently
> > implemented. We must detect when these unique banks are being used and
> > special case their IRQ and input state.
> >
> > TODO: Do what it says on the label
> >
> > > Signed-off-by: Joel Stanley <joel at jms.id.au>
> > ---
> > drivers/gpio/gpio-aspeed.c | 15 ++++++++++++++-
> > 1 file changed, 14 insertions(+), 1 deletion(-)
> >
> > diff --git a/drivers/gpio/gpio-aspeed.c b/drivers/gpio/gpio-aspeed.c
> > index d7198536de13..c4d87a2a489f 100644
> > --- a/drivers/gpio/gpio-aspeed.c
> > +++ b/drivers/gpio/gpio-aspeed.c
> > @@ -65,10 +65,14 @@ static const struct aspeed_gpio_bank aspeed_gpio_banks[] = {
> > .names = { "U", "V", "W", "X" },
> > },
> > /*
> > - * A bank exists for { 'Y', 'Z', "AA", "AB" }, but is not implemented.
> > * Only half of GPIOs Y support interrupt configuration, and none of Z,
> > * AA or AB do as they are output only.
>
> Is this comment accurate? I don't see anything in the AST2500
> datasheet about GPIOs Z, AA, and AB being output only.
These banks are output-only on the AST2400. The comment is not accurate
for the 2500 as far as I can see. Again this was a patch that needed
some improvement, we just haven't got around to cleaning it up.
Cheers,
Andrew
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: This is a digitally signed message part
URL: <http://lists.ozlabs.org/pipermail/openbmc/attachments/20170110/be893af3/attachment.sig>
More information about the openbmc
mailing list