[U-Boot] [PATCH 8/8] arm, davinci: add support for am1808 based enbw_cmc board

[Heiko Schocher]

Hello Igor,

Igor Grinberg wrote:
> Hi Heiko,
> 
> On 09/15/11 08:59, Heiko Schocher wrote:
>> - booting from NOR Flash with direct boot method
>> - POST support
>> - LOGBUF support
>>
>> Signed-off-by: Heiko Schocher <hs at denx.de>
>> Cc: Paulraj Sandeep <s-paulraj at ti.com>
>> Cc: Albert ARIBAUD <albert.u.boot at aribaud.net>
>> ---
> 
> [...]
> 
>> diff --git a/board/enbw/enbw_cmc/enbw_cmc.c b/board/enbw/enbw_cmc/enbw_cmc.c
>> new file mode 100644
>> index 0000000..1b9ee67
>> --- /dev/null
>> +++ b/board/enbw/enbw_cmc/enbw_cmc.c
> 
> [...]
> 
>> +
>> +int board_init(void)
>> +{
>> +#ifndef CONFIG_USE_IRQ
>> +	irq_init();
>> +#endif
>> +
>> +	/* arch number of the board */
>> +	gd->bd->bi_arch_number = 3585;
> 
> There is a new standard option for assigning the machid.
> Now, this should be done in the board config file and the above two lines removed.
> Please, see the CONFIG_MACH_TYPE in the U-Boot documentation (README file).

Ah, thanks!

> [...]
> 
>> diff --git a/include/configs/enbw_cmc.h b/include/configs/enbw_cmc.h
>> new file mode 100644
>> index 0000000..5de752f
>> --- /dev/null
>> +++ b/include/configs/enbw_cmc.h
> 
> [...]
> 
>> +/*
>> + * Board
>> + */
>> +#define CONFIG_DRIVER_TI_EMAC
>> +#define CONFIG_USE_NAND
> 
> Consider adding here:
> #define MACH_TYPE_ENBW_CMC	3585
> #define CONFIG_MACH_TYPE	MACH_TYPE_ENBW_CMC

Yep! Change this for v2, thanks!

bye,
Heiko
-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany