All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] ARM: Samsung: s5p-fimc: decrease IO registers memory region size
@ 2010-09-09  2:22 ` Marek Szyprowski
  0 siblings, 0 replies; 4+ messages in thread
From: Marek Szyprowski @ 2010-09-09  2:22 UTC (permalink / raw)
  To: linux-samsung-soc, linux-arm-kernel
  Cc: s.nawrocki, m.szyprowski, kyungmin.park, kgene.kim, ben-linux

From: Sylwester Nawrocki <s.nawrocki@samsung.com>

IO registers region size of all FIMC versions is less than 1kB so there
is no need to reserve 1M.

Signed-off-by: Sylwester Nawrocki <s.nawrocki@samsung.com>
Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com>
Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
---
 arch/arm/plat-s5p/dev-fimc0.c |    2 +-
 arch/arm/plat-s5p/dev-fimc1.c |    2 +-
 arch/arm/plat-s5p/dev-fimc2.c |    2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/arch/arm/plat-s5p/dev-fimc0.c b/arch/arm/plat-s5p/dev-fimc0.c
index f9203fa..608770f 100644
--- a/arch/arm/plat-s5p/dev-fimc0.c
+++ b/arch/arm/plat-s5p/dev-fimc0.c
@@ -19,7 +19,7 @@
 static struct resource s5p_fimc0_resource[] = {
 	[0] = {
 		.start	= S5P_PA_FIMC0,
-		.end	= S5P_PA_FIMC0 + SZ_1M - 1,
+		.end	= S5P_PA_FIMC0 + SZ_4K - 1,
 		.flags	= IORESOURCE_MEM,
 	},
 	[1] = {
diff --git a/arch/arm/plat-s5p/dev-fimc1.c b/arch/arm/plat-s5p/dev-fimc1.c
index 7ed91a3..76e3a97 100644
--- a/arch/arm/plat-s5p/dev-fimc1.c
+++ b/arch/arm/plat-s5p/dev-fimc1.c
@@ -19,7 +19,7 @@
 static struct resource s5p_fimc1_resource[] = {
 	[0] = {
 		.start	= S5P_PA_FIMC1,
-		.end	= S5P_PA_FIMC1 + SZ_1M - 1,
+		.end	= S5P_PA_FIMC1 + SZ_4K - 1,
 		.flags	= IORESOURCE_MEM,
 	},
 	[1] = {
diff --git a/arch/arm/plat-s5p/dev-fimc2.c b/arch/arm/plat-s5p/dev-fimc2.c
index 5543d6373..24d2981 100644
--- a/arch/arm/plat-s5p/dev-fimc2.c
+++ b/arch/arm/plat-s5p/dev-fimc2.c
@@ -19,7 +19,7 @@
 static struct resource s5p_fimc2_resource[] = {
 	[0] = {
 		.start	= S5P_PA_FIMC2,
-		.end	= S5P_PA_FIMC2 + SZ_1M - 1,
+		.end	= S5P_PA_FIMC2 + SZ_4K - 1,
 		.flags	= IORESOURCE_MEM,
 	},
 	[1] = {
-- 
1.7.2.2

^ permalink raw reply related	[flat|nested] 4+ messages in thread
* [PATCH] ARM: Samsung: s5p-fimc: decrease IO registers memory region size
@ 2010-09-09  2:22 ` Marek Szyprowski
  0 siblings, 0 replies; 4+ messages in thread
From: Marek Szyprowski @ 2010-09-09  2:22 UTC (permalink / raw)
  To: linux-arm-kernel

From: Sylwester Nawrocki <s.nawrocki@samsung.com>

IO registers region size of all FIMC versions is less than 1kB so there
is no need to reserve 1M.

Signed-off-by: Sylwester Nawrocki <s.nawrocki@samsung.com>
Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com>
Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
---
 arch/arm/plat-s5p/dev-fimc0.c |    2 +-
 arch/arm/plat-s5p/dev-fimc1.c |    2 +-
 arch/arm/plat-s5p/dev-fimc2.c |    2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/arch/arm/plat-s5p/dev-fimc0.c b/arch/arm/plat-s5p/dev-fimc0.c
index f9203fa..608770f 100644
--- a/arch/arm/plat-s5p/dev-fimc0.c
+++ b/arch/arm/plat-s5p/dev-fimc0.c
@@ -19,7 +19,7 @@
 static struct resource s5p_fimc0_resource[] = {
 	[0] = {
 		.start	= S5P_PA_FIMC0,
-		.end	= S5P_PA_FIMC0 + SZ_1M - 1,
+		.end	= S5P_PA_FIMC0 + SZ_4K - 1,
 		.flags	= IORESOURCE_MEM,
 	},
 	[1] = {
diff --git a/arch/arm/plat-s5p/dev-fimc1.c b/arch/arm/plat-s5p/dev-fimc1.c
index 7ed91a3..76e3a97 100644
--- a/arch/arm/plat-s5p/dev-fimc1.c
+++ b/arch/arm/plat-s5p/dev-fimc1.c
@@ -19,7 +19,7 @@
 static struct resource s5p_fimc1_resource[] = {
 	[0] = {
 		.start	= S5P_PA_FIMC1,
-		.end	= S5P_PA_FIMC1 + SZ_1M - 1,
+		.end	= S5P_PA_FIMC1 + SZ_4K - 1,
 		.flags	= IORESOURCE_MEM,
 	},
 	[1] = {
diff --git a/arch/arm/plat-s5p/dev-fimc2.c b/arch/arm/plat-s5p/dev-fimc2.c
index 5543d6373..24d2981 100644
--- a/arch/arm/plat-s5p/dev-fimc2.c
+++ b/arch/arm/plat-s5p/dev-fimc2.c
@@ -19,7 +19,7 @@
 static struct resource s5p_fimc2_resource[] = {
 	[0] = {
 		.start	= S5P_PA_FIMC2,
-		.end	= S5P_PA_FIMC2 + SZ_1M - 1,
+		.end	= S5P_PA_FIMC2 + SZ_4K - 1,
 		.flags	= IORESOURCE_MEM,
 	},
 	[1] = {
-- 
1.7.2.2

^ permalink raw reply related	[flat|nested] 4+ messages in thread
* RE: [PATCH] ARM: Samsung: s5p-fimc: decrease IO registers memory region size
  2010-09-09  2:22 ` Marek Szyprowski
@ 2010-09-09 11:26   ` Kukjin Kim
  -1 siblings, 0 replies; 4+ messages in thread
From: Kukjin Kim @ 2010-09-09 11:26 UTC (permalink / raw)
  To: 'Marek Szyprowski', linux-samsung-soc, linux-arm-kernel
  Cc: kyungmin.park, ben-linux, s.nawrocki

Marek Szyprowski wrote:
> 
> From: Sylwester Nawrocki <s.nawrocki@samsung.com>
> 
> IO registers region size of all FIMC versions is less than 1kB so there
> is no need to reserve 1M.
> 
> Signed-off-by: Sylwester Nawrocki <s.nawrocki@samsung.com>
> Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com>
> Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
> ---
>  arch/arm/plat-s5p/dev-fimc0.c |    2 +-
>  arch/arm/plat-s5p/dev-fimc1.c |    2 +-
>  arch/arm/plat-s5p/dev-fimc2.c |    2 +-
>  3 files changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/arch/arm/plat-s5p/dev-fimc0.c b/arch/arm/plat-s5p/dev-fimc0.c
> index f9203fa..608770f 100644
> --- a/arch/arm/plat-s5p/dev-fimc0.c
> +++ b/arch/arm/plat-s5p/dev-fimc0.c
> @@ -19,7 +19,7 @@
>  static struct resource s5p_fimc0_resource[] = {
>  	[0] = {
>  		.start	= S5P_PA_FIMC0,
> -		.end	= S5P_PA_FIMC0 + SZ_1M - 1,
> +		.end	= S5P_PA_FIMC0 + SZ_4K - 1,
>  		.flags	= IORESOURCE_MEM,
>  	},
>  	[1] = {
> diff --git a/arch/arm/plat-s5p/dev-fimc1.c b/arch/arm/plat-s5p/dev-fimc1.c
> index 7ed91a3..76e3a97 100644
> --- a/arch/arm/plat-s5p/dev-fimc1.c
> +++ b/arch/arm/plat-s5p/dev-fimc1.c
> @@ -19,7 +19,7 @@
>  static struct resource s5p_fimc1_resource[] = {
>  	[0] = {
>  		.start	= S5P_PA_FIMC1,
> -		.end	= S5P_PA_FIMC1 + SZ_1M - 1,
> +		.end	= S5P_PA_FIMC1 + SZ_4K - 1,
>  		.flags	= IORESOURCE_MEM,
>  	},
>  	[1] = {
> diff --git a/arch/arm/plat-s5p/dev-fimc2.c b/arch/arm/plat-s5p/dev-fimc2.c
> index 5543d6373..24d2981 100644
> --- a/arch/arm/plat-s5p/dev-fimc2.c
> +++ b/arch/arm/plat-s5p/dev-fimc2.c
> @@ -19,7 +19,7 @@
>  static struct resource s5p_fimc2_resource[] = {
>  	[0] = {
>  		.start	= S5P_PA_FIMC2,
> -		.end	= S5P_PA_FIMC2 + SZ_1M - 1,
> +		.end	= S5P_PA_FIMC2 + SZ_4K - 1,
>  		.flags	= IORESOURCE_MEM,
>  	},
>  	[1] = {
> --
> 1.7.2.2

Ok...will apply.

Thanks.

Best regards,
Kgene.
--
Kukjin Kim <kgene.kim@samsung.com>, Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.

^ permalink raw reply	[flat|nested] 4+ messages in thread
* [PATCH] ARM: Samsung: s5p-fimc: decrease IO registers memory region size
@ 2010-09-09 11:26   ` Kukjin Kim
  0 siblings, 0 replies; 4+ messages in thread
From: Kukjin Kim @ 2010-09-09 11:26 UTC (permalink / raw)
  To: linux-arm-kernel

Marek Szyprowski wrote:
> 
> From: Sylwester Nawrocki <s.nawrocki@samsung.com>
> 
> IO registers region size of all FIMC versions is less than 1kB so there
> is no need to reserve 1M.
> 
> Signed-off-by: Sylwester Nawrocki <s.nawrocki@samsung.com>
> Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com>
> Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
> ---
>  arch/arm/plat-s5p/dev-fimc0.c |    2 +-
>  arch/arm/plat-s5p/dev-fimc1.c |    2 +-
>  arch/arm/plat-s5p/dev-fimc2.c |    2 +-
>  3 files changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/arch/arm/plat-s5p/dev-fimc0.c b/arch/arm/plat-s5p/dev-fimc0.c
> index f9203fa..608770f 100644
> --- a/arch/arm/plat-s5p/dev-fimc0.c
> +++ b/arch/arm/plat-s5p/dev-fimc0.c
> @@ -19,7 +19,7 @@
>  static struct resource s5p_fimc0_resource[] = {
>  	[0] = {
>  		.start	= S5P_PA_FIMC0,
> -		.end	= S5P_PA_FIMC0 + SZ_1M - 1,
> +		.end	= S5P_PA_FIMC0 + SZ_4K - 1,
>  		.flags	= IORESOURCE_MEM,
>  	},
>  	[1] = {
> diff --git a/arch/arm/plat-s5p/dev-fimc1.c b/arch/arm/plat-s5p/dev-fimc1.c
> index 7ed91a3..76e3a97 100644
> --- a/arch/arm/plat-s5p/dev-fimc1.c
> +++ b/arch/arm/plat-s5p/dev-fimc1.c
> @@ -19,7 +19,7 @@
>  static struct resource s5p_fimc1_resource[] = {
>  	[0] = {
>  		.start	= S5P_PA_FIMC1,
> -		.end	= S5P_PA_FIMC1 + SZ_1M - 1,
> +		.end	= S5P_PA_FIMC1 + SZ_4K - 1,
>  		.flags	= IORESOURCE_MEM,
>  	},
>  	[1] = {
> diff --git a/arch/arm/plat-s5p/dev-fimc2.c b/arch/arm/plat-s5p/dev-fimc2.c
> index 5543d6373..24d2981 100644
> --- a/arch/arm/plat-s5p/dev-fimc2.c
> +++ b/arch/arm/plat-s5p/dev-fimc2.c
> @@ -19,7 +19,7 @@
>  static struct resource s5p_fimc2_resource[] = {
>  	[0] = {
>  		.start	= S5P_PA_FIMC2,
> -		.end	= S5P_PA_FIMC2 + SZ_1M - 1,
> +		.end	= S5P_PA_FIMC2 + SZ_4K - 1,
>  		.flags	= IORESOURCE_MEM,
>  	},
>  	[1] = {
> --
> 1.7.2.2

Ok...will apply.

Thanks.

Best regards,
Kgene.
--
Kukjin Kim <kgene.kim@samsung.com>, Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.

^ permalink raw reply	[flat|nested] 4+ messages in thread
end of thread, other threads:[~2010-09-09 11:26 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-09-09  2:22 [PATCH] ARM: Samsung: s5p-fimc: decrease IO registers memory region size Marek Szyprowski
2010-09-09  2:22 ` Marek Szyprowski
2010-09-09 11:26 ` Kukjin Kim
2010-09-09 11:26   ` Kukjin Kim

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.