Re: [PATCH] pinctrl: intel: Add Intel Sunrisepoint pin controller and GPIO support

[Mika Westerberg <mika.westerberg@linux.intel.com>]

On Tue, Apr 07, 2015 at 03:18:20PM +0200, Linus Walleij wrote:
> On Mon, Mar 30, 2015 at 4:31 PM, Mika Westerberg
> <mika.westerberg@linux.intel.com> wrote:
> 
> > This driver supports pinctrl/GPIO hardware found on Intel Sunrisepoint (a
> > Skylake PCH) providing users a pinctrl and GPIO interfaces (including GPIO
> > interrupts).
> >
> > The driver is split into core and platform parts so that the same core
> > driver can be reused in other drivers for other Intel GPIO hardware that is
> > based on the same host controller design.
> >
> > Signed-off-by: Mathias Nyman <mathias.nyman@linux.intel.com>
> > Signed-off-by: Mika Westerberg <mika.westerberg@linux.intel.com>
> 
> How could I possibly resist this gorgeous driver that just
> does everything right? Patch applied for 4.1.

Thanks!

> > +config PINCTRL_INTEL
> > +       tristate
> > +       select PINMUX
> > +       select PINCONF
> > +       select GENERIC_PINCONF
> > +       select GPIOLIB
> > +       select GPIOLIB_IRQCHIP
> 
> Is this opening a refactoring possibility sharing more code/infrastructure
> amongst the Intel drivers or just something that will be used for
> new SoCs going forward?

We are going to use the same core for newer hardware as much as
possible. Hopefully all new SoCs will have GPIO controller which is
based on the same hardware design.

For the existing, unfortunately the hardware differentiates too much
from Sunrisepoint that it does not make much sense to factor them to use
this new core.