m68knommu: refactor Coldfire GPIO not to require GPIOLIB, eliminate mcf_gpio_chips.
authorSteven King <sfking@fdwdc.com>
Mon, 21 May 2012 20:10:19 +0000 (13:10 -0700)
committerGreg Ungerer <gerg@uclinux.org>
Sun, 15 Jul 2012 23:59:21 +0000 (09:59 +1000)
commiteac57949947fa24f47a2e993a1dbbfdb573b4301
tree7ef7fadfbbb7153c788dbc036445ed1f74012f28
parent14be4252eae2ce40abec5872bf6011891d753433
m68knommu: refactor Coldfire GPIO not to require GPIOLIB, eliminate mcf_gpio_chips.

If we're not connecting external GPIO extenders via i2c or spi or whatever, we
probably don't need GPIOLIB.  If we provide an alternate implementation of
the GPIOLIB functions to use when only on-chip GPIO is needed, we can change
ARCH_REQUIRE_GPIOLIB to ARCH_WANTS_OPTIONAL_GPIOLIB so that GPIOLIB becomes
optional.

The downside is that in the GPIOLIB=n case, we lose all error checking done by
gpiolib, ie multiply allocating the gpio, free'ing gpio etc., so that the
only checking that can be done is if we reference a gpio on an external part.
Targets that need the extra error checking can still select GPIOLIB=y.

For the case where GPIOLIB=y, we can simplify the table of gpio chips to use a
single chip, eliminating the tables of chips in the 5xxx.c files.  The
original motivation for the definition of multiple chips was to match the way
many of the Coldfire variants defined their gpio as a spare array in memory.
However, all this really gains us is some error checking when we request a
gpio, gpiolib can check that it doesn't fall in one of the holes.  If thats
important, I think we can still come up with a better way of accomplishing
that.

Also in this patch is some general cleanup and reorganizing of the gpio header
files (I'm sure I must have had a reason why I sometimes used a prefix of
mcf_gpio and other times mcfgpio but for the life of me I can't think of it
now).

Signed-off-by: Steven King <sfking@fdwdc.com>
Signed-off-by: Greg Ungerer <gerg@uclinux.org>
18 files changed:
arch/m68k/Kconfig.cpu
arch/m68k/include/asm/gpio.h
arch/m68k/include/asm/mcfgpio.h
arch/m68k/include/asm/pinmux.h [deleted file]
arch/m68k/platform/coldfire/Makefile
arch/m68k/platform/coldfire/gpio.c
arch/m68k/platform/coldfire/m5206.c
arch/m68k/platform/coldfire/m520x.c
arch/m68k/platform/coldfire/m523x.c
arch/m68k/platform/coldfire/m5249.c
arch/m68k/platform/coldfire/m5272.c
arch/m68k/platform/coldfire/m527x.c
arch/m68k/platform/coldfire/m528x.c
arch/m68k/platform/coldfire/m5307.c
arch/m68k/platform/coldfire/m532x.c
arch/m68k/platform/coldfire/m5407.c
arch/m68k/platform/coldfire/m54xx.c
arch/m68k/platform/coldfire/pinmux.c [deleted file]