[PATCH V2 2/2] ARM: S5PV310: Add support GPIOlib
Jongsun Han
jongsun.han at samsung.com
Thu Oct 14 19:55:03 EDT 2010
Sangbeom Kim wrote:
> >
> > From: Jongpill Lee <boyko.lee at samsung.com>
> >
> > This patch adds GPIOlib support for S5PV310 and S5PC210.
> >
> > Signed-off-by: Jongpill Lee <boyko.lee at samsung.com>
> > Signed-off-by: Sangbeom Kim <sbkim73 at samsung.com>
> > ---
> > NOTE:
> > This patch used mapped VA IO as GPIO virtual address.
> > because firstly other Samsung SoC's GPIO used same method.
> > and I think that it's better to keep the same method.
> > And ioremap also can be used for it.
> > But in the GPIO, there is no need to allocate VA area dynamically.
> > because GPIO can be used in any drivers anytime so it can't iounmap GPIO
> > VA area.
> >
> Ok...
>
> (snip)
>
> > #define MAX_COMBINER_NR 40
> >
> > +#define S5P_IRQ_EINT_BASE COMBINER_IRQ(MAX_COMBINER_NR, 0)
> > +
> > +#define S5P_EINT_BASE1 (S5P_IRQ_EINT_BASE + 0)
> > +#define S5P_EINT_BASE2 (S5P_IRQ_EINT_BASE + 16)
> > +
>
> Hmm...
> If you want to add external interrupt, should be changed NR_IRQS too.
>
> Will apply with fixing NR_IRQS.
>
It seems that S5P_IRQ_EINT_BASE has wrong definition.
S5P_IRQ_EINT_BASE has to be COMBINER_IRQ(MAX_COMBINER_NR - 1, 0).
Thanks.
> Thanks.
>
> Best regards,
> Kgene.
> --
> Kukjin Kim <kgene.kim at samsung.com>, Senior Engineer,
> SW Solution Development Team, Samsung Electronics Co., Ltd.
>
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
More information about the linux-arm-kernel
mailing list