diff mbox

[U-Boot,v5,2/2] common/board_f: Fix size variable

Message ID 1399076885-449-2-git-send-email-yorksun@freescale.com
State Accepted
Delegated to: Tom Rini
Headers show

Commit Message

York Sun May 3, 2014, 12:28 a.m. UTC 
DRAM size should use 64-bit variable when the size could be more than 4GB.
Caught and verified on P4080DS with 4GB DDR.

Signed-off-by: York Sun <yorksun@freescale.com>
---
Change log
 v5: no change since v1

 common/board_f.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Simon Glass May 5, 2014, 4:56 p.m. UTC | #1 
On 2 May 2014 18:28, York Sun <yorksun@freescale.com> wrote:
> DRAM size should use 64-bit variable when the size could be more than 4GB.
> Caught and verified on P4080DS with 4GB DDR.
>
> Signed-off-by: York Sun <yorksun@freescale.com>

Acked-by: Simon Glass <sjg@chromium.org>
Tom Rini May 13, 2014, 1:54 a.m. UTC | #2 
On Fri, May 02, 2014 at 05:28:05PM -0700, York Sun wrote:

> DRAM size should use 64-bit variable when the size could be more than 4GB.
> Caught and verified on P4080DS with 4GB DDR.
> 
> Signed-off-by: York Sun <yorksun@freescale.com>
> Acked-by: Simon Glass <sjg@chromium.org>

Applied to u-boot/master, thanks!
diff mbox

Patch

diff --git a/common/board_f.c b/common/board_f.c
index 7601a98..c268cf9 100644
--- a/common/board_f.c
+++ b/common/board_f.c
@@ -194,7 +194,7 @@  static int init_func_ram(void)
 
 static int show_dram_config(void)
 {
-	ulong size;
+	unsigned long long size;
 
 #ifdef CONFIG_NR_DRAM_BANKS
 	int i;