* [U-Boot] [PATCH] nitrogen6x: Fix RAM size variable
@ 2013-03-13 22:50 Fabio Estevam
2013-03-13 22:50 ` [U-Boot] [PATCH 2/2] nitrogen: Use unsigned long to specify the total RAM size Fabio Estevam
2013-03-13 22:52 ` [U-Boot] [PATCH] nitrogen6x: Fix RAM size variable Fabio Estevam
0 siblings, 2 replies; 3+ messages in thread
From: Fabio Estevam @ 2013-03-13 22:50 UTC (permalink / raw)
To: u-boot
From: Fabio Estevam <fabio.estevam@freescale.com>
Fix the following build error:
nitrogen6x.c:89:17: error: 'CONFIG_DDR_MB' undeclared (first use in
this function)
nitrogen6x.c:89:17: note: each undeclared identifier is reported only
once for each function it appears in
Reported-by: Albert Aribaud <albert.u.boot@aribaud.net>
Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
---
boards.cfg | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/boards.cfg b/boards.cfg
index 02115bf..9fd4bd6 100644
--- a/boards.cfg
+++ b/boards.cfg
@@ -260,7 +260,7 @@ nitrogen6dl2g arm armv7 nitrogen6x boundar
nitrogen6q arm armv7 nitrogen6x boundary mx6 nitrogen6x:IMX_CONFIG=board/boundary/nitrogen6x/nitrogen6q.cfg,MX6Q,DDR_MB=1024
nitrogen6q2g arm armv7 nitrogen6x boundary mx6 nitrogen6x:IMX_CONFIG=board/boundary/nitrogen6x/nitrogen6q2g.cfg,MX6Q,DDR_MB=2048
nitrogen6s arm armv7 nitrogen6x boundary mx6 nitrogen6x:IMX_CONFIG=board/boundary/nitrogen6x/nitrogen6s.cfg,MX6S,DDR_MB=512
-nitrogen6s1g arm armv7 nitrogen6x boundary mx6 nitrogen6x:IMX_CONFIG=board/boundary/nitrogen6x/nitrogen6s1g.cfg,MX6S,SDRAM_MB=1024
+nitrogen6s1g arm armv7 nitrogen6x boundary mx6 nitrogen6x:IMX_CONFIG=board/boundary/nitrogen6x/nitrogen6s1g.cfg,MX6S,DDR_MB=1024
wandboard_dl arm armv7 wandboard - mx6 wandboard:IMX_CONFIG=board/boundary/nitrogen6x/nitrogen6dl.cfg,MX6DL,DDR_MB=1024
wandboard_solo arm armv7 wandboard - mx6 wandboard:IMX_CONFIG=board/boundary/nitrogen6x/nitrogen6s.cfg,MX6S,DDR_MB=512
cm_t35 arm armv7 cm_t35 - omap3
--
1.7.9.5
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [U-Boot] [PATCH 2/2] nitrogen: Use unsigned long to specify the total RAM size
2013-03-13 22:50 [U-Boot] [PATCH] nitrogen6x: Fix RAM size variable Fabio Estevam
@ 2013-03-13 22:50 ` Fabio Estevam
2013-03-13 22:52 ` [U-Boot] [PATCH] nitrogen6x: Fix RAM size variable Fabio Estevam
1 sibling, 0 replies; 3+ messages in thread
From: Fabio Estevam @ 2013-03-13 22:50 UTC (permalink / raw)
To: u-boot
From: Fabio Estevam <fabio.estevam@freescale.com>>
When building for the nitrogen boards with 2GB the following warning happens:
nitrogen6x.c:89:38: warning: integer overflow in expression [-Woverflow]
2GB can not fit in 32-bits, so use ulong instead.
Reported-by: Albert Aribaud <albert.u.boot@aribaud.net>
Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
---
board/boundary/nitrogen6x/nitrogen6x.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/board/boundary/nitrogen6x/nitrogen6x.c b/board/boundary/nitrogen6x/nitrogen6x.c
index 0dbb6d2..e5634d8 100644
--- a/board/boundary/nitrogen6x/nitrogen6x.c
+++ b/board/boundary/nitrogen6x/nitrogen6x.c
@@ -86,7 +86,7 @@ DECLARE_GLOBAL_DATA_PTR;
int dram_init(void)
{
- gd->ram_size = CONFIG_DDR_MB * 1024 * 1024;
+ gd->ram_size = (ulong)(CONFIG_DDR_MB * 1024 * 1024);
return 0;
}
--
1.7.9.5
^ permalink raw reply related [flat|nested] 3+ messages in thread* [U-Boot] [PATCH] nitrogen6x: Fix RAM size variable
2013-03-13 22:50 [U-Boot] [PATCH] nitrogen6x: Fix RAM size variable Fabio Estevam
2013-03-13 22:50 ` [U-Boot] [PATCH 2/2] nitrogen: Use unsigned long to specify the total RAM size Fabio Estevam
@ 2013-03-13 22:52 ` Fabio Estevam
1 sibling, 0 replies; 3+ messages in thread
From: Fabio Estevam @ 2013-03-13 22:52 UTC (permalink / raw)
To: u-boot
On Wed, Mar 13, 2013 at 7:50 PM, Fabio Estevam <festevam@gmail.com> wrote:
> From: Fabio Estevam <fabio.estevam@freescale.com>
>
> Fix the following build error:
>
> nitrogen6x.c:89:17: error: 'CONFIG_DDR_MB' undeclared (first use in
> this function)
> nitrogen6x.c:89:17: note: each undeclared identifier is reported only
> once for each function it appears in
>
> Reported-by: Albert Aribaud <albert.u.boot@aribaud.net>
> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
Sorry, please discard this one.
It contains the wandboard patches.
Will use the clean u-boot-imx and generate v2.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2013-03-13 22:52 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-03-13 22:50 [U-Boot] [PATCH] nitrogen6x: Fix RAM size variable Fabio Estevam
2013-03-13 22:50 ` [U-Boot] [PATCH 2/2] nitrogen: Use unsigned long to specify the total RAM size Fabio Estevam
2013-03-13 22:52 ` [U-Boot] [PATCH] nitrogen6x: Fix RAM size variable Fabio Estevam
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.