On Thu, Apr 13, 2017 at 12:27 PM, Nandor Han <[email protected]> wrote:
> This is a simple driver that provides a /sys/class/gpio > interface for controlling and configuring the GPIO lines. > It does not provide support for chip select or interrupts. > > Signed-off-by: Nandor Han <[email protected]> > Signed-off-by: Semi Malinen <[email protected]> I almost want to make the driver depend on !GPIO_SYSFS because of this commit message. DO NOT USE OR ENCOURAGE THE USE OF THE GPIO SYSFS INTERFACE. Use the character device. Use the example in tools/gpio/* as a guideline and testbed. Use libgpiod as a rich userspace. And the commit message should state that this is a driver for such and such Exar hardware instead. Thanks. > +#include <linux/bitops.h> > +#include <linux/gpio/driver.h> > +#include <linux/kernel.h> > +#include <linux/module.h> > +#include <linux/mutex.h> > +#include <linux/of_device.h> > +#include <linux/of_gpio.h> > +#include <linux/spi/spi.h> > +#include <linux/regmap.h> You are missing #include <linux/seq_file.h> and that is why the build robot is complaining. Yours, Linus Walleij
