On Tue, Mar 14, 2017 at 8:38 PM, Tom Rini <tr...@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.ma...@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 > _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
