From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ben Dooks Subject: Re: [PATCH] ARM: S3C24XX: Fix the S3C24XX_VA_GPIO mapping Date: Thu, 21 Oct 2010 23:25:46 +0100 Message-ID: <4CC0BDEA.8090407@fluff.org> References: <1287581622-8335-1-git-send-email-kgene.kim@samsung.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Return-path: Received: from dsl78-143-211-26.in-addr.fast.co.uk ([78.143.211.26]:57336 "EHLO ben-laptop" rhost-flags-OK-FAIL-OK-FAIL) by vger.kernel.org with ESMTP id S1752053Ab0JUWZw (ORCPT ); Thu, 21 Oct 2010 18:25:52 -0400 In-Reply-To: <1287581622-8335-1-git-send-email-kgene.kim@samsung.com> Sender: linux-samsung-soc-owner@vger.kernel.org List-Id: linux-samsung-soc@vger.kernel.org To: Kukjin Kim Cc: linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org, ben-linux@fluff.org 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 > Signed-off-by: Kukjin Kim > Cc: Ben Dooks > --- > 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-s3c24xx/include/plat/map.h > 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. > -#define S3C2410_PA_GPIO (0x56000000) > -#define S3C24XX_VA_GPIO ((S3C24XX_PA_GPIO - S3C24XX_PA_UART) + S3C24XX_VA_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 From mboxrd@z Thu Jan 1 00:00:00 1970 From: ben-linux@fluff.org (Ben Dooks) Date: Thu, 21 Oct 2010 23:25:46 +0100 Subject: [PATCH] ARM: S3C24XX: Fix the S3C24XX_VA_GPIO mapping In-Reply-To: <1287581622-8335-1-git-send-email-kgene.kim@samsung.com> References: <1287581622-8335-1-git-send-email-kgene.kim@samsung.com> Message-ID: <4CC0BDEA.8090407@fluff.org> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org 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 > Signed-off-by: Kukjin Kim > Cc: Ben Dooks > --- > 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-s3c24xx/include/plat/map.h > 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. > -#define S3C2410_PA_GPIO (0x56000000) > -#define S3C24XX_VA_GPIO ((S3C24XX_PA_GPIO - S3C24XX_PA_UART) + S3C24XX_VA_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