Hello,
On Monday, May 17, 2010 1:25 PM Kukjin Kim wrote:
> Marek Szyprowski wrote:
> >
> > Move gpio-lib support from plat-s5pc1xx to mach-s5pc100. Only basic gpio
> > functionality is now supported. Gpio interrupts are disabled temporarly
> > and will be added later.
> >
> > Signed-off-by: Marek Szyprowski <m.szyprowski@xxxxxxxxxxx>
> > Signed-off-by: Kyungmin Park <kyungmin.park@xxxxxxxxxxx>
> > ---
> > arch/arm/mach-s5pc100/Makefile | 2 +-
> > .../gpiolib.c => mach-s5pc100/gpio.c} | 11 +++++++----
> > .../plat => mach-s5pc100/include/mach}/regs-gpio.h | 0
> > arch/arm/mach-s5pc100/mach-smdkc100.c | 1 -
> > arch/arm/plat-s5pc1xx/Makefile | 3 +--
> > 5 files changed, 9 insertions(+), 8 deletions(-)
> > rename arch/arm/{plat-s5pc1xx/gpiolib.c => mach-s5pc100/gpio.c} (99%)
> > rename arch/arm/{plat-s5pc1xx/include/plat =>
> mach-s5pc100/include/mach}/regs-
> > gpio.h (100%)
> >
> > diff --git a/arch/arm/mach-s5pc100/Makefile
> b/arch/arm/mach-s5pc100/Makefile
> > index 809ff10..e1afdbf 100644
> > --- a/arch/arm/mach-s5pc100/Makefile
> > +++ b/arch/arm/mach-s5pc100/Makefile
> > @@ -11,7 +11,7 @@ obj- :=
> >
> > # Core support for S5PC100 system
> >
> > -obj-$(CONFIG_CPU_S5PC100) += cpu.o
> > +obj-$(CONFIG_CPU_S5PC100) += cpu.o gpio.o
>
> How about changing gpiolib like gpiolib.c of mach-s3c64xx/ and mach-
> s5pv210/.
> I will change the file name of mach-s5p6440/ also.
I you don't mind I wanted first move the existing and working code and
then, later update it to match common s5p style.
> >
> > # Helper and device support
> >
> > diff --git a/arch/arm/plat-s5pc1xx/gpiolib.c b/arch/arm/mach-
> s5pc100/gpio.c
> > similarity index 99%
> > rename from arch/arm/plat-s5pc1xx/gpiolib.c
> > rename to arch/arm/mach-s5pc100/gpio.c
> > index 5a97a8f..c8e8336 100644
> > --- a/arch/arm/plat-s5pc1xx/gpiolib.c
> > +++ b/arch/arm/mach-s5pc100/gpio.c
> > @@ -17,11 +17,11 @@
> > #include <linux/gpio.h>
> >
> > #include <mach/map.h>
> > +#include <mach/regs-gpio.h>
> >
> > #include <plat/gpio-core.h>
> > #include <plat/gpio-cfg.h>
> > #include <plat/gpio-cfg-helpers.h>
> > -#include <plat/regs-gpio.h>
> >
> > /* S5PC100 GPIO bank summary:
> > *
> > @@ -61,6 +61,7 @@
> > * L3 8 4Bit None
> > */
> >
> > +#if 0
> > static int s5pc1xx_gpiolib_to_irq(struct gpio_chip *chip, unsigned int
> offset)
> > {
> > return S3C_IRQ_GPIO(chip->base + offset);
> > @@ -84,7 +85,7 @@ static int s5pc1xx_gpiolib_to_eint(struct gpio_chip
> *chip,
> > unsigned int offset)
> > return IRQ_EINT(24 + offset);
> > return -EINVAL;
> > }
> > -
> > +#endif
> > static struct s3c_gpio_cfg gpio_cfg = {
> > .set_config = s3c_gpio_setcfg_s3c64xx_4bit,
> > .set_pull = s3c_gpio_setpull_updown,
> > @@ -386,7 +387,7 @@ extern void s5pc1xx_irq_gpioint_handler(unsigned int
> irq,
> > struct irq_desc *desc)
> >
> > static __init void s5pc100_gpiolib_link(struct s3c_gpio_chip *chip)
> > {
> > -
> > +#if 0
> > /* Interrupt */
> > if (chip->config == &gpio_cfg) {
> > int i, irq;
> > @@ -402,6 +403,7 @@ static __init void s5pc100_gpiolib_link(struct
> s3c_gpio_chip
> > *chip)
> > }
> > } else if (chip->config == &gpio_cfg_eint)
> > chip->chip.to_irq = s5pc1xx_gpiolib_to_eint;
> > +#endif
> > }
> >
> > static __init int s5pc1xx_gpiolib_init(void)
> > @@ -417,9 +419,10 @@ static __init int s5pc1xx_gpiolib_init(void)
> >
> > samsung_gpiolib_add_4bit_chips(s5pc100_gpio_chips,
> > ARRAY_SIZE(s5pc100_gpio_chips));
> > +#if 0
> > /* Interrupt */
> > set_irq_chained_handler(IRQ_GPIOINT, s5pc1xx_irq_gpioint_handler);
> > -
> > +#endif
> > return 0;
> > }
> > core_initcall(s5pc1xx_gpiolib_init);
>
> Is there any way to avoid compile compatibility without #if 0?
The other way would be to completely remove that code and add it in the
next patches. I don't think that this is really required.
> ...
Best regards
--
Marek Szyprowski
Samsung Poland R&D Center
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
