Re: [PATCH 00/21] gpio: ab8500: Bring AB8500 back to life

From: Lee Jones
Date: Fri Jan 04 2013 - 09:46:59 EST

On Fri, 04 Jan 2013, Linus Walleij wrote:

> On Wed, Dec 19, 2012 at 3:18 PM, Linus Walleij <linus.walleij@xxxxxxxxxx> wrote:
> > On Fri, Dec 14, 2012 at 5:19 PM, Lee Jones <lee.jones@xxxxxxxxxx> wrote:
> >
> >> The ab8500 GPIO driver is currently 'BROKEN', due to a lack of
> >> updates and recent frame-work adoption. This patch-set aims to
> >> bring the driver back to life so we may make use of it again.
> >>
> >> This first stage is to bring it into line with ST-Ericsson's
> >> internal track, which has been keeping up-to-date. After these
> >> patches have been accepted I will then overhaul the driver to
> >> use the most recent APIs and frame-works, such as pinctrl and
> >> Device Tree.
> >>
> >> Please give these patches an easy ride. I know they're not
> >> perfect, but they do provide a great base on with I can form
> >> the overhaul. Thanks for your understanding.
> >
> > OK I've applied these 21 patches on a branch in the pinctrl
> > tree to be used as a merge base.
> No I'll have to scrap this topic branch, as it does not compile.
> Analysing the problem I see that part of the reason why it does
> not compile is that it is adding stuff referencing the global
> numberspace, like biasing.
> Which is pinctrl business. So the proper feedback is that it
> should be rewritten to use pinctrl.
> Which is in the works.
> So the plan is to mail out an all-in-one pinctrl replacement
> patch instead, deleting the old AB8500 GPIO driver and
> replacing it fully with a pinctrl driver.

Makes sense, although that's going to be a hell of a patch-set.

Lee Jones
Linaro ST-Ericsson Landing Team Lead â Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at
Please read the FAQ at