On Tue, Mar 14, 2017 at 8:38 PM, Tom Rini trini@konsulko.com wrote:
On Wed, Mar 08, 2017 at 11:55:36AM -0500, Jon Mason wrote:
Add support for the Broadcom Northstar2 SoC and SVK (bcm958712k). The BCM5871X is a series of quad-core 64-bit 2GHz ARMv8 Cortex-A57 processors targeting a broad range of networking applications.
Signed-off-by: Jon Mason jon.mason@broadcom.com
+(bcm958712k) In file included from include/config.h:5:0, +(bcm958712k) from ../include/common.h:21, +(bcm958712k) from ../board/broadcom/bcmns2/northstar2.c:6: +(bcm958712k) #define V2M_BASE 0x80000000 +(bcm958712k) ^ +(bcm958712k) ../include/configs/bcm_northstar2.h:16:25: note: in expansion of macro ‘V2M_BASE’ +(bcm958712k) #define PHYS_SDRAM_1 V2M_BASE +(bcm958712k) ^ +(bcm958712k) ../include/configs/bcm_northstar2.h:21:33: note: in expansion of macro ‘PHYS_SDRAM_1’ +(bcm958712k) #define CONFIG_SYS_SDRAM_BASE PHYS_SDRAM_1 +(bcm958712k) ^ +(bcm958712k) ../board/broadcom/bcmns2/northstar2.c:41:30: note: in expansion of macro ‘ CONFIG_SYS_SDRAM_BASE’ +(bcm958712k) gd->ram_size = get_ram_size(CONFIG_SYS_SDRAM_BASE, +(bcm958712k) ^ +(bcm958712k) In file included from ../board/broadcom/bcmns2/northstar2.c:6:0: +(bcm958712k) ../include/common.h:467:6: note: expected ‘long int *’ but argument is of type ‘unsigned int’ +(bcm958712k) long get_ram_size (long *, long); +(bcm958712k) ^ w+(bcm958712k) ../board/broadcom/bcmns2/northstar2.c: In function ‘dram_init’: w+(bcm958712k) ../include/configs/bcm_northstar2.h:15:21: warning: passing argument 1 of
Sorry, I needed to cast the CONFIG_SYS_SDRAM_BASE in get_ram_size() as a long *, similar to other users. I'll send out a v3 which addresses this issue.
Thanks, Jon
-- Tom
U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot