Hi Higor, Hi Stefano,
On 17 août 2011, at 09:31, Igor Grinberg wrote:
>
> On 08/12/11 09:49, Stefano Babic wrote:
>> On 08/12/2011 01:41 AM, Eric Jarrige wrote:
>>
>> Hi Eric,
>>
>>>>> +int board_init(void) +{ + gd->bd->bi_arch_number =
>>>>> CONFIG_MACH_TYPE;
>>>> Is there no MACH_TYPE for this board ? It is uncommon for an ARM
>>>> board. Should this board run Linux ?
>>> The MACH_TYPE for this board is 906. This board runs linux and the
>>> integration in linux is on going.
>> Ok, I have understood. The only thing you can directly set
>> MACH_TYPE_APF9328 here. IMHO it is more readable.
>
> You should not do this here unless you support multiple mach types in a
> single file.
> Please, see the CONFIG_MACH_TYPE in the U-Boot documentation (README file).
Here is the README sentence regarding CONFIG_MACH_TYPE:
> CONFIG_MACH_TYPE [relevant for ARM only][mandatory]
>
> This setting is mandatory for all boards that have only one
> machine type and must be used to specify the machine type
> number as it appears in the ARM machine registry
> (see http://www.arm.linux.org.uk/developer/machines/).
> Only boards that have multiple machine types supported
> in a single configuration file and the machine type is
> runtime discoverable, do not have to use this setting.
My understanding is:
1) CONFIG_MACH_TYPE has to be defined in apf9328.h
2) as gd->bd->bi_arch_number is already initialize in board.c i should
completely remove this second initialization from the apf9382.c
arch/arm/lib/board.c:
262 void board_init_f(ulong bootflag)
..
277
278 #ifdef CONFIG_MACH_TYPE
279 gd->bd->bi_arch_number = CONFIG_MACH_TYPE; /* board id for Linux */
280 #endif
Please let me know if missed something.
Thx and best regards,
Eric
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
