Hello Wolfgang,
Wolfgang Denk wrote:
> In message <4b459395.2050...@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
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
