From: Tom Rini <trini@ti.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v8 13/31] arm: Enable generic board support
Date: Sun, 10 Mar 2013 09:27:29 -0400 [thread overview]
Message-ID: <513C8A41.6090106@ti.com> (raw)
In-Reply-To: <CAPnjgZ0+fARyS6ko01ZOy-k+D4CPG-BcLisw3aR5u_KNYg1EhQ@mail.gmail.com>
-----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
> <otavio at ossystems.com.br <mailto:otavio@ossystems.com.br>> wrote:
>
> On Fri, Mar 8, 2013 at 8:45 PM, Simon Glass <sjg@chromium.org
> <mailto:sjg@chromium.org>> wrote:
>> This enables generic board support so that ARM boards can define
>> CONFIG_SYS_GENERIC_BOARD.
>>
>> Signed-off-by: Simon Glass <sjg@chromium.org
> <mailto:sjg@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.
- --
Tom
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.11 (GNU/Linux)
Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/
iQIcBAEBAgAGBQJRPIpAAAoJENk4IS6UOR1WvHsP/RxMk8w1XFl6lMmsJysrdwRd
4pGVk1x1LjRpnleo3nJgQIX5e458g6oM7tra0JQ1RvbxNEZhWiJon/P0kun4gTAt
CPzCNZt0JyAXTu/+MyNTALuLMwY30JSzBR6Tg1uTgrfBDcQVOlRkwCwU91oDBeQH
Dbf8KYfBmOoHy3/uLu/JwEAbAOQFxb4VdTaXnwM9Vp4sDX2TiP1gyJVDT668/i07
SSwYiIE022zJKe/MSmiWzXwClUDKoKwFOsv15y+O2Wv5Dr+uSSd6uCj+tUlEBFmP
uhOZPveAbhL2Myhti1j0GkZD/VSeUhHbAXH5OATQRCJHXZGLSsynxgl4fTFK1Mnn
QLcYGyTWGOzba/ytlvSpCfHrQL+dh48bV8Cz3Mvj1iQyZyMRbfC6JMaZu2AFsqvE
vQni+ZhuE4FIBLt7mD3e5aqDM2Z6kijpDk95sU0G6LwcUJW1KeFTnqKxTE8BjNu4
dmFsqpNXSDCkgCGdnOasEfXzGmUMy/DlwWtPxXxa/CnFcxDkrh7to9kWW5xYYrL8
zItHpvLeuZAbzHCtjJsewjCx9QXcZ+jvpSbpjUrCEAM29ifxbKukRBb6chU3XDaN
KJwRIWlhtkTGXEoUmSsurQGPDD2SvDq10wgml1cQoCB9n756IQBVKruFV+dHjdgw
0TybImGlHorgFeZJh8pX
=zX5G
-----END PGP SIGNATURE-----
next prev parent reply other threads:[~2013-03-10 13:27 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-03-08 23:45 [U-Boot] [PATCH v8 07/31] __HAVE_ARCH_GENERIC_BOARD controls availabilty of generic board Simon Glass
2013-03-08 23:45 ` [U-Boot] [PATCH v8 13/31] arm: Enable generic board support Simon Glass
2013-03-09 12:31 ` Otavio Salvador
2013-03-09 18:30 ` Simon Glass
2013-03-09 19:06 ` Otavio Salvador
2013-03-10 13:27 ` Tom Rini [this message]
2013-03-11 16:17 ` Simon Glass
2013-03-08 23:45 ` [U-Boot] [PATCH v8 17/31] ppc: " Simon Glass
2013-03-08 23:45 ` [U-Boot] [PATCH v8 21/31] x86: " Simon Glass
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=513C8A41.6090106@ti.com \
--to=trini@ti.com \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox