On Mon, Aug 23, 2010 at 11:15 AM, Kukjin Kim <kgene....@samsung.com> wrote:
> Kyungmin Park wrote:
>>
>> On Mon, Aug 23, 2010 at 9:18 AM, Kukjin Kim <kgene....@samsung.com> 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
>>
> I missed gpio driver strength driver. Will fix it.
> But this issue is not regarding gpio driver strength.
>
>> and how to you implement the gpio_to_irq at both GPIO and EINT?
>>
> Will sort it out.
>
>> The "common" means GPIO related function should use the gpiolib
>> functions only. No direct access.
>>
> It means don't need to use your previous patch which is just RFC regarding
> removing VA_GPIO.
and make it for multiple GPIOs such as s5pc210. also make it simple
for GPIOs and EINT.
It's also cover the gpio_to_irq to use the similar scheme either GPIOs or EINT.
>
>> If your patch match these criteria then I'll ack your patch.
>>
> Hmm...
>
>> All
>> >
>> >> Thank you,
>> >> Kyungmin Park
>> >>
>> >> On Fri, Aug 20, 2010 at 9:33 PM, Kukjin Kim <kgene....@samsung.com>
> wrote:
>> >> > From: Jongpill Lee <boyko....@samsung.com>
>> >> >
>> >> > This patch adds initial map for GPIO2 and GPIO3.
>> >> > S5PV310/S5PC210 has separated GPIO1, GPIO2 and GPIO3.
>> >> >
>> >> > Signed-off-by: Jongpill Lee <boyko....@samsung.com>
>> >> > Signed-off-by: Kukjin Kim <kgene....@samsung.com>
>> >> > ---
>> >> > 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....@samsung.com>, 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 majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
