Skip to contentSkip to conversation

DA830: Fix Build Warning

PATCH New s-paulraj at ti.com 2 participants 2 messages Sep 13, 2011

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 DA830: Fix Build Warning

From: Sandeep Paulraj <[email protected]>

This commit fixes a build warning in the DA830 EVM build

---
 include/configs/da830evm.h |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)
include/configs/da830evm.h +2 -1
index d52c08c..66fdea2 100644
@@ -50,7 +50,8 @@
 #define CONFIG_SYS_MALLOC_LEN	(0x10000 + 1*1024*1024) /* malloc() len */
 #define PHYS_SDRAM_1			0xc0000000 /* SDRAM Start */
 #define CONFIG_SYS_MEMTEST_START	(PHYS_SDRAM_1 + 0x2000000)
-#define CONFIG_SYS_MEMTEST_END		(CONFIG_SYS_MEMTEST_START + 32 << 20)
+#define CONFIG_SYS_MEMTEST_END		(CONFIG_SYS_MEMTEST_START + \
+						(32 << 20))
 #define CONFIG_NR_DRAM_BANKS	1 /* we have 1 bank of DRAM */
 #define CONFIG_STACKSIZE	(256*1024) /* regular stack */
 
Signature · 1 line
1.7.0.4
#1 lore raw

PATCH 1/1 DA830: Fix Build Warning

Hi Sandeep,

Le 13/09/2011 21:56, s-paulraj at ti.com a ?crit :

Quoted 16 lines

From: Sandeep Paulraj<[email protected]>

This commit fixes a build warning in the DA830 EVM build

---
  include/configs/da830evm.h |    3 ++-
  1 files changed, 2 insertions(+), 1 deletions(-)
include/configs/da830evm.h +0 -0
index d52c08c..66fdea2 100644

include/configs/da830evm.h line 56

Quoted 8 lines
include/configs/da830evm.h +2 -1
@@ -50,7 +50,8 @@
  #define CONFIG_SYS_MALLOC_LEN	(0x10000 + 1*1024*1024) /* malloc() len */
  #define PHYS_SDRAM_1			0xc0000000 /* SDRAM Start */
  #define CONFIG_SYS_MEMTEST_START	(PHYS_SDRAM_1 + 0x2000000)
-#define CONFIG_SYS_MEMTEST_END		(CONFIG_SYS_MEMTEST_START + 32<<  20)
+#define CONFIG_SYS_MEMTEST_END		(CONFIG_SYS_MEMTEST_START + \
+						(32<<  20))
  #define CONFIG_NR_DRAM_BANKS	1 /* we have 1 bank of DRAM */
  #define CONFIG_STACKSIZE	(256*1024) /* regular stack */

Applied to u-boot-arm/master, thanks!

The current u-boot-arm/master branch is down to 13 boards with warnings or errors, identical to v2011.09-rc1:

--------------------- SUMMARY ---------------------------- Boards compiled: 242
Boards with warnings or errors: 13 ( omap3_beagle actux1_4_16 actux1_8_16 actux1_4_32 actux1_8_32 actux2 actux3 actux4 dvlhost ixdp425 ixdpg425 pdnb3 scpu )
----------------------------------------------------------

The marvell 'next' branch gives the same result, so as soon as I know if that branch is intended as a 'master' (which I'm not sure because its commits are not fixes and were submitted outside of the merge window), I'll pull it and send the pull req for ARM.

Amicalement,

Signature · 1 line
Albert.