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

Heiko Schocher hs at denx.de
Mon Jan 18 09:34:08 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).

Ups, sorry.

>> +#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.. I take a look at this point... Stefan? Maybe, you can help here,
because this comes from your original port, I think.

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