[PATCH v3 1/5] gpio: introduce gpio-mvebu driver for Marvell SoCs
Linus Walleij
linus.walleij at linaro.org
Fri Sep 14 23:15:23 EST 2012
On Thu, Sep 13, 2012 at 5:54 PM, Thomas Petazzoni
<thomas.petazzoni at free-electrons.com> wrote:
> This driver aims at replacing the arch/arm/plat-orion/gpio.c driver,
> and is designed to be compatible with all Marvell EBU SoCs: Orion,
> Kirkwood, Dove, Armada 370/XP and Discovery.
>
> It has been successfully tested on Dove and Armada XP at the moment.
>
> Compared to the plat-orion driver, this new driver has the following
> added benefits:
>
> *) Support for Armada 370 and Armada XP
> *) It is integrated with the mvebu pinctrl driver so that GPIO pins
> are properly muxed, and the GPIO driver knows which GPIO pins are
> output-only or input-only.
> *) Properly placed in drivers/gpio
> *) More extensible mechanism to support platform differences. The
> plat-orion driver uses a simple mask-offset DT property, which
> works fine for Discovery MV78200 but not for Armada XP. The new
> driver uses different compatible strings to identify the different
> variants of the GPIO controllers.
>
> Signed-off-by: Thomas Petazzoni <thomas.petazzoni at free-electrons.com>
> Cc: Grant Likely <grant.likely at secretlab.ca>
> Cc: Linus Walleij <linus.walleij at stericsson.com>
> Cc: Andrew Lunn <andrew at lunn.ch>
> Cc: Jason Cooper <jason at lakedaemon.net>
> Cc: Gregory Clement <gregory.clement at free-electrons.com>
> Tested-by: Sebastian Hesselbarth <sebastian.hesselbarth at gmail.com>
This looks very good to me, I'm probably unable to see any
further mistakes at this point:
Reviewed-by: Linus Walleij <linus.walleij at linaro.org>
Yours,
Linus Walleij
More information about the devicetree-discuss
mailing list