[PATCH] ARM: S3C24XX: Fix the S3C24XX_VA_GPIO mapping
kgene.kim at samsung.com
Thu Oct 21 21:41:25 EDT 2010
Ben Dooks wrote:
> On 20/10/10 14:33, Kukjin Kim wrote:
> > This patch changes the S3C24XX_VA_GPIO for to fix following BUG.
> > BUG: not creating mapping for 0x56000000 at 0x01000000 in user region
> > It is due to commit 8fecfe9d(ARM: SAMSUNG: Move the start address of
> > Samsung SoCs' VA space) which changes S3C_ADDR_BASE from 0xF4000000
> > to 0xFA000000.
> > Reported-by: Abhilash Kesavan <a.kesavan at samsung.com>
> > Signed-off-by: Kukjin Kim <kgene.kim at samsung.com>
> > Cc: Ben Dooks <ben-linux at fluff.org>
> > ---
> > arch/arm/plat-s3c24xx/include/plat/map.h | 14 +++-----------
> > 1 files changed, 3 insertions(+), 11 deletions(-)
> > diff --git a/arch/arm/plat-s3c24xx/include/plat/map.h b/arch/arm/plat-
> > index bd534d3..771491e 100644
> > --- a/arch/arm/plat-s3c24xx/include/plat/map.h
> > +++ b/arch/arm/plat-s3c24xx/include/plat/map.h
> > @@ -61,17 +61,9 @@
> > /* GPIO ports */
> > -/* the calculation for the VA of this must ensure that
> > - * it is the same distance apart from the UART in the
> > - * phsyical address space, as the initial mapping for the IO
> > - * is done as a 1:1 mapping. This puts it (currently) at
> > - * 0xFA800000, which is not in the way of any current mapping
> > - * by the base system.
> > -*/
> This comment is really important, and it is possible this will
> break some of the s3c24xx platforms duing the boot process as
> there can be problems during the changes of mmu when booting.
I understand, thanks for your pointing out.
Firstly, will drop this..and update other regarding patch.
I think, we need to sort VA mapping out later ;-)
> > -#define S3C2410_PA_GPIO (0x56000000)
> > -#define S3C24XX_VA_GPIO ((S3C24XX_PA_GPIO - S3C24XX_PA_UART) +
> > -#define S3C24XX_SZ_GPIO SZ_1M
> > +#define S3C2410_PA_GPIO (0x56000000)
> > +#define S3C24XX_VA_GPIO S3C2410_ADDR(0x00F00000)
> > +#define S3C24XX_SZ_GPIO SZ_1M
> > /* ISA style IO, for each machine to sort out mappings for, if it
Kukjin Kim <kgene.kim at samsung.com>, Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.
More information about the linux-arm-kernel