RE: [PATCH 03/16] ARM: ep93xx: move platform_data definitions

From: H Hartley Sweeten
Date: Tue Sep 11 2012 - 12:27:03 EST


On Tuesday, September 11, 2012 6:00 AM, Arnd Bergmann wrote:
> Platform data for device drivers should be defined in
> include/linux/platform_data/*.h, not in the architecture
> and platform specific directories.
>
> This moves such data out of the ep93xx include directories
>
> Signed-off-by: Arnd Bergmann <arnd@xxxxxxxx>
> Cc: Grant Likely <grant.likely@xxxxxxxxxxxx>
> Cc: Hartley Sweeten <hsweeten@xxxxxxxxxxxxxxxxxxx>
> Cc: Ryan Mallon <rmallon@xxxxxxxxx>
> Cc: Jeff Garzik <jgarzik@xxxxxxxxx> (supporter:SERIAL ATA
> Cc: Vinod Koul <vinod.koul@xxxxxxxxx>
> Cc: Dan Williams <djbw@xxxxxx>
> Cc: Dmitry Torokhov <dmitry.torokhov@xxxxxxxxx> (maintainer:INPUT
> Cc: Florian Tobias Schandinat <FlorianSchandinat@xxxxxx>
> Cc: Liam Girdwood <lrg@xxxxxx>
> Cc: Mark Brown <broonie@xxxxxxxxxxxxxxxxxxxxxxxxxxx>
> Cc: Jaroslav Kysela <perex@xxxxxxxx>
> Cc: Takashi Iwai <tiwai@xxxxxxx>
> Cc: Mika Westerberg <mika.westerberg@xxxxxx>
> Cc: Axel Lin <axel.lin@xxxxxxxxx>
> ---
> Documentation/spi/ep93xx_spi | 2 +-
> arch/arm/mach-ep93xx/core.c | 6 +++---
> arch/arm/mach-ep93xx/dma.c | 2 +-
> arch/arm/mach-ep93xx/edb93xx.c | 4 ++--
> arch/arm/mach-ep93xx/simone.c | 2 +-
> arch/arm/mach-ep93xx/snappercl15.c | 2 +-
> arch/arm/mach-ep93xx/vision_ep9307.c | 4 ++--
> drivers/ata/pata_ep93xx.c | 2 +-
> drivers/dma/ep93xx_dma.c | 2 +-
> drivers/input/keyboard/ep93xx_keypad.c | 2 +-
> drivers/spi/spi-ep93xx.c | 4 ++--
> drivers/video/ep93xx-fb.c | 2 +-
> .../mach/dma.h => include/linux/platform_data/dma-ep93xx.h | 0
> .../linux/platform_data/keypad-ep93xx.h | 0
> .../ep93xx_spi.h => include/linux/platform_data/spi-ep93xx.h | 0
> .../mach/fb.h => include/linux/platform_data/video-ep93xx.h | 0
> sound/soc/ep93xx/ep93xx-ac97.c | 2 +-
> sound/soc/ep93xx/ep93xx-i2s.c | 2 +-
> sound/soc/ep93xx/ep93xx-pcm.c | 2 +-
> 19 files changed, 20 insertions(+), 20 deletions(-)
> rename arch/arm/mach-ep93xx/include/mach/dma.h => include/linux/platform_data/dma-ep93xx.h (100%)
> rename arch/arm/mach-ep93xx/include/mach/ep93xx_keypad.h => include/linux/platform_data/keypad-ep93xx.h (100%)
> rename arch/arm/mach-ep93xx/include/mach/ep93xx_spi.h => include/linux/platform_data/spi-ep93xx.h (100%)
> rename arch/arm/mach-ep93xx/include/mach/fb.h => include/linux/platform_data/video-ep93xx.h (100%)

Acked-by: H Hartley Sweeten <hsweeten@xxxxxxxxxxxxxxxxxxx>

Arnd,

What about gpio-ep93xx.h? It does not have any platform_data in
it but the gpio driver (driver/gpio/gpio-ep93xx.c) does include it
from the mach-* directory.

Regards,
Hartley

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/