[U-Boot] arm: suen3, suen3_v1, mgcoge2_arm_p1a support

Heiko Schocher hs at denx.de
Wed Jan 27 08:23:24 CET 2010


Hello Wolfgang,

Wolfgang Denk wrote:
> In message <4B459395.2050301 at denx.de> you wrote:
>> This patch adds support for the Keymile SUEN3 board variants which
>> are based on the Marvell Kirkwood (88F6281) SoC. All variants
>> uses common code stored in board/keymile/km_arm/km_arm.c
>>
>> mgcoge2_arm_p1a board:
>> This adds support for the ARM part of the mgcoge2. The suen3
>> target was moved to the correct suen3 p1b version. There is a
>> difference between the GPIO configuration between suen3 and mgcoge2.
> ...
> 
> 
>> --- /dev/null
>> +++ b/include/configs/km-arm.h
> ...
>> +#if 1 // test-only cramfs ...
>> +#define CONFIG_SYS_MAX_FLASH_SECT	256
> 
> Please do not add dead code (the "#if 1" here),
> and do not use C++ comments (please fix this globally).

Ok, fixed.

>> +#ifndef __ASSEMBLY__
>> +typedef struct {
>> +	unsigned long size;			/* total bank size in bytes		*/
>> +	unsigned short sector_count;		/* number of erase units		*/
>> +	unsigned long flash_id;			/* combined device & manufacturer code	*/
>> +	unsigned long start[CONFIG_SYS_MAX_FLASH_SECT];   /* virtual sector start address */
>> +	unsigned char protect[CONFIG_SYS_MAX_FLASH_SECT]; /* sector protection status	*/
>> +} flash_info_t;
> 
> Lines way too long. Please fix globally.
> 
> Umm.. why cannot you use the struct flash_info_t as declared in
> "include/flash.h" ??

Hmm.. this is, because keymile uses a cramfs without a NOR-flash.
And in fs/cramfs/cramfs.c this struct is used ...

I made in the meantime a patch, with solves this problem. I post
it soon for discussion. I also send an updated version of this
patch without this struct in the board config.

Thanks for the review!

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


More information about the U-Boot mailing list