[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