On Mon, Aug 23, 2010 at 9:18 AM, Kukjin Kim <kgene.kim@xxxxxxxxxxx> wrote: > Kyungmin Park wrote: >> >> NAK. >> > I don't know why I need your ack for this...if any opinions, just comments > is enough. Okay I said in other word, I can't agree this patch. > >> This approach don't make a common GPIO framework. I already send the >> common GPIO framework which send the base address to GPIO framework >> and handle it regradless GPIO is one or three. >> > I don't think that your that RFC GPIO patch is common GPIO framework. > >> > http://lists.infradead.org/pipermail/linux-arm-kernel/2010-August/022513.htm > l >> >> With this patch, we can use the common GPIO framework and remove the >> VA_GPIO dependency. >> > Why do you think your patch is common? In another patch, you just write the driver strength with readl/writel. even though gpiolib already has interface, s5p_gpio_set_drvstr and how to you implement the gpio_to_irq at both GPIO and EINT? The "common" means GPIO related function should use the gpiolib functions only. No direct access. If your patch match these criteria then I'll ack your patch. All > >> Thank you, >> Kyungmin Park >> >> On Fri, Aug 20, 2010 at 9:33 PM, Kukjin Kim <kgene.kim@xxxxxxxxxxx> wrote: >> > From: Jongpill Lee <boyko.lee@xxxxxxxxxxx> >> > >> > This patch adds initial map for GPIO2 and GPIO3. >> > S5PV310/S5PC210 has separated GPIO1, GPIO2 and GPIO3. >> > >> > Signed-off-by: Jongpill Lee <boyko.lee@xxxxxxxxxxx> >> > Signed-off-by: Kukjin Kim <kgene.kim@xxxxxxxxxxx> >> > --- >> > arch/arm/mach-s5pv310/cpu.c | 10 ++++++++++ >> > arch/arm/plat-s5p/include/plat/map-s5p.h | 2 ++ >> > 2 files changed, 12 insertions(+), 0 deletions(-) >> > >> > diff --git a/arch/arm/mach-s5pv310/cpu.c b/arch/arm/mach-s5pv310/cpu.c >> > index 196c9f1..db4f55a 100644 >> > --- a/arch/arm/mach-s5pv310/cpu.c >> > +++ b/arch/arm/mach-s5pv310/cpu.c >> > @@ -45,6 +45,16 @@ static struct map_desc s5pv310_iodesc[] __initdata = > { >> > .pfn = __phys_to_pfn(S5PV310_PA_L2CC), >> > .length = SZ_4K, >> > .type = MT_DEVICE, >> > + }, { >> > + .virtual = (unsigned long)S5P_VA_GPIO2, >> > + .pfn = __phys_to_pfn(S5PV310_PA_GPIO2), >> > + .length = SZ_4K, >> > + .type = MT_DEVICE, >> > + }, { >> > + .virtual = (unsigned long)S5P_VA_GPIO3, >> > + .pfn = __phys_to_pfn(S5PV310_PA_GPIO3), >> > + .length = SZ_256K, >> > + .type = MT_DEVICE, >> > }, >> > }; >> > >> > diff --git a/arch/arm/plat-s5p/include/plat/map-s5p.h b/arch/arm/plat- >> s5p/include/plat/map-s5p.h >> > index 54e9fb9..bc52595 100644 >> > --- a/arch/arm/plat-s5p/include/plat/map-s5p.h >> > +++ b/arch/arm/plat-s5p/include/plat/map-s5p.h >> > @@ -15,6 +15,8 @@ >> > >> > #define S5P_VA_CHIPID S3C_ADDR(0x00700000) >> > #define S5P_VA_GPIO S3C_ADDR(0x00500000) >> > +#define S5P_VA_GPIO2 S3C_ADDR(0x00510000) >> > +#define S5P_VA_GPIO3 S3C_ADDR(0x00520000) >> > #define S5P_VA_SYSTIMER S3C_ADDR(0x01200000) >> > #define S5P_VA_SROMC S3C_ADDR(0x01100000) >> > >> > -- > > > Thanks. > > Best regards, > Kgene. > -- > Kukjin Kim <kgene.kim@xxxxxxxxxxx>, Senior Engineer, > SW Solution Development Team, Samsung Electronics Co., Ltd. > > -- > 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 > -- 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
 |