Re[2]: [RFC] Add Generic GPIO driver
Alexander Shiyan
shc_work at mail.ru
Tue Feb 19 02:09:23 EST 2013
> On Mon, Feb 18, 2013 at 02:13:22PM +0400, Alexander Shiyan wrote:
> > > On Fri, Feb 15, 2013 at 10:49:27PM +0400, Alexander Shiyan wrote:
> > > > This patch adds generic memory-mapped GPIO controller support.
> > > > Code taken from Linux Kernel and adopted for barebox.
> > >
> > > I'm fine with this if you add a user for it. I wonder though if it's
> > > worth it to have this driver for barebox. If I have a driver in the
> > > kernel that already uses it, then this makes it simple to copy it, but
> > > should I have to write a new gpio driver, I'm unsure if it's simpler
> > > to write a new gpio driver or to register with the generic gpio
> > > driver.
> >
> > Here's an example that I'm checking:
>
> Ok, this indeed looks nice and short. You convinced me ;)
>
> As said, I'm fine with the generic gpio driver if you add the below as
> first user.
I still beg to consider patch for generic GPIO separately.
Then I will send you two completely different versions of the implementation
GPIO for CLPS711X for consideration.
...
> > /*
> > * Copyright (C) 2013 Alexander Shiyan <shc_work at mail.ru>
> > *
> > * This program is free software; you can redistribute it and/or
> > * modify it under the terms of the GNU General Public License as
> > * published by the Free Software Foundation; either version 2 of
> > * the License, or (at your option) any later version.
> > */
> >
> > #include <init.h>
> > #include <common.h>
> > #include <malloc.h>
> >
> > #include <linux/basic_mmio_gpio.h>
...
---
More information about the barebox
mailing list