[PATCH 4/4] gpio: aspeed: Add interfaces for co-processor to grab GPIOs

Benjamin Herrenschmidt benh at kernel.crashing.org
Fri Jun 15 09:40:53 AEST 2018

On Thu, 2018-06-14 at 10:59 +0200, Linus Walleij wrote:
> Overall looks fine!
> You definately need something like this for handling this special case.
> > +#include <linux/gpio/consumer.h>
> Why do you need this?
> I don't see that you use any functions from it.
> > +#include "gpiolib.h"
> I'm not so happy about this either, what is this needed for?
> It seems to me you can remove both includes, but admittedly I miss
> fine details all the time.

I wish I could :-) This is the main wart in there to be honest.

consumer.h is needed to build gpiolib.h

gpiolib.h is needed for gpio_chip_hwgpio() which needs the definition
of gpio_desc, etc...

I didn't find a simple way out if it... the "API" I expose to the copro
driver takes a gpio_desc which is the clean thing to do but I can't get
to the underlying GPIO number without gpiolib.


More information about the Linux-aspeed mailing list