[PATCH 12/16] ARM: S5PC100: Move sdhci helpers from plat-s5pc1xx to mach-s5pc100

Marek Szyprowski m.szyprowski at samsung.com
Wed May 12 03:24:08 EDT 2010


Hello,

On Wednesday, May 12, 2010 9:05 AM Kukjin Kim wrote:

> Marek Szyprowski wrote:
> >
> > Move sdhci helpers from plat-s5pc1xx to mach-s5pc100.
> >
> > Signed-off-by: Marek Szyprowski <m.szyprowski at samsung.com>
> > Signed-off-by: Kyungmin Park <kyungmin.park at samsung.com>
> > ---
> >  arch/arm/mach-s5pc100/Kconfig                      |   11 +++++++++++
> >  arch/arm/mach-s5pc100/Makefile                     |    2 ++
> >  arch/arm/mach-s5pc100/cpu.c                        |    6 ++++++
> >  arch/arm/mach-s5pc100/include/mach/map.h           |    8 ++++++++
> >  .../setup-sdhci-gpio.c                             |    2 +-
> >  5 files changed, 28 insertions(+), 1 deletions(-)
> >  rename arch/arm/{plat-s5pc1xx => mach-s5pc100}/setup-sdhci-gpio.c (97%)
> >
> > diff --git a/arch/arm/mach-s5pc100/Kconfig b/arch/arm/mach-
> s5pc100/Kconfig
> > index f2d5c23..af64da3 100644
> > --- a/arch/arm/mach-s5pc100/Kconfig
> > +++ b/arch/arm/mach-s5pc100/Kconfig
> > @@ -24,6 +24,17 @@ config S5PC100_SETUP_I2C1
> >  	help
> >  	  Common setup code for i2c bus 1.
> >
> > +config S5PC100_SETUP_SDHCI
> > +        bool
> > +        select S5PC100_SETUP_SDHCI_GPIO
> > +        help
> > +          Internal helper functions for S5PC100 based SDHCI systems
> > +
> 
> Not aligned.
> Please fix it.

ok

> 
> > +config S5PC100_SETUP_SDHCI_GPIO
> > +	bool
> > +	help
> > +	  Common setup code for SDHCI gpio.
> > +
> >  config MACH_SMDKC100
> >  	bool "SMDKC100"
> >  	select CPU_S5PC100
> > diff --git a/arch/arm/mach-s5pc100/Makefile
> b/arch/arm/mach-s5pc100/Makefile
> > index 522d8b7..9242f14 100644
> > --- a/arch/arm/mach-s5pc100/Makefile
> > +++ b/arch/arm/mach-s5pc100/Makefile
> > @@ -18,6 +18,8 @@ obj-$(CONFIG_CPU_S5PC100)	+= setup-i2c0.o
> >
> >  obj-$(CONFIG_S5PC100_SETUP_FB_24BPP) += setup-fb-24bpp.o
> >  obj-$(CONFIG_S5PC100_SETUP_I2C1) += setup-i2c1.o
> > +obj-$(CONFIG_S5PC100_SETUP_SDHCI)       += setup-sdhci.o
> > +obj-$(CONFIG_S5PC100_SETUP_SDHCI_GPIO)	+= setup-sdhci-gpio.o
> 
> How about union the setup-sdhci-gpio.c to setup-sdhci file?
> I think don't need to separate setup-sdhci and setup-sdhci-gpio file.

I just followed the style of other Samsung machines (s3c64xx mainly).
If required I can merge these two files together.

> ...

Best regards
--
Marek Szyprowski
Samsung Poland R&D Center





More information about the linux-arm-kernel mailing list