Hi Tom,
On Sun, Mar 10, 2013 at 6:27 AM, Tom Rini <tr...@ti.com> wrote: > -----BEGIN PGP SIGNED MESSAGE----- > Hash: SHA1 > > On 03/09/2013 01:30 PM, Simon Glass wrote: > > Hi Otavio, > > > > On Sat, Mar 9, 2013 at 4:31 AM, Otavio Salvador > > <ota...@ossystems.com.br <mailto:ota...@ossystems.com.br>> wrote: > > > > On Fri, Mar 8, 2013 at 8:45 PM, Simon Glass <s...@chromium.org > > <mailto:s...@chromium.org>> wrote: > >> This enables generic board support so that ARM boards can define > >> CONFIG_SYS_GENERIC_BOARD. > >> > >> Signed-off-by: Simon Glass <s...@chromium.org > > <mailto:s...@chromium.org>> > >> --- Changes in v8: - Define __HAVE_ARCH_GENERIC_BOARD in ARM's > >> config.mk > > <http://config.mk> > >> > >> Changes in v7: None Changes in v6: None Changes in v5: None > >> Changes in v4: None Changes in v3: None Changes in v2: None > >> > >> arch/arm/config.mk <http://config.mk> | 3 +++ > >> arch/arm/include/asm/u-boot.h | 9 +++++++++ arch/arm/lib/Makefile > >> | 3 +++ 3 files changed, 15 insertions(+) > >> > >> diff --git a/arch/arm/config.mk <http://config.mk> > > b/arch/arm/config.mk <http://config.mk> > >> index 24b9d7c..a0c89b7 100644 --- a/arch/arm/config.mk > >> <http://config.mk> +++ b/arch/arm/config.mk <http://config.mk> @@ > >> -31,6 +31,9 @@ CONFIG_STANDALONE_LOAD_ADDR = 0xc100000 endif > >> endif > >> > >> +# Support generic board on ARM +__HAVE_ARCH_GENERIC_BOARD := y > >> + PLATFORM_CPPFLAGS += -DCONFIG_ARM -D__ARM__ > >> > >> # Choose between ARM/Thumb instruction sets diff --git > >> a/arch/arm/include/asm/u-boot.h > > b/arch/arm/include/asm/u-boot.h > >> index 2ba98bc..8e7e27b 100644 --- > >> a/arch/arm/include/asm/u-boot.h +++ > >> b/arch/arm/include/asm/u-boot.h @@ -36,6 +36,12 @@ #ifndef > >> _U_BOOT_H_ #define _U_BOOT_H_ 1 > >> > >> +#ifdef CONFIG_SYS_GENERIC_BOARD +/* Use the generic board which > >> requires a unified bd_info */ +#include <asm-generic/u-boot.h> > >> +#else + +#ifndef __ASSEMBLY__ typedef struct bd_info { unsigned > >> int bi_baudrate; /* serial console baudrate */ ulong > >> bi_arch_number; /* unique id for this board */ @@ -49,6 +55,9 @@ > >> typedef struct bd_info { ulong size; } > >> bi_dram[CONFIG_NR_DRAM_BANKS]; } bd_t; +#endif + +#endif /* > >> nCONFIG_SYS_GENERIC_BOARD */ > > > > Typo? > > > > > > The 'n' is intended to mean 'not'. Perhaps I should use ! instead? > > !CONFIG is the style of the rest of the codebase. Thanks. > OK, I will resend the affected patch(es). Regards, Simon
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
