[U-Boot] [PATCH] arm: add ide initialization

Albert ARIBAUD albert.u.boot at aribaud.net
Tue Oct 18 23:42:26 CEST 2011


Hi Michael,

Le 07/10/2011 23:50, Michael Walle a écrit :
> Am Freitag 07 Oktober 2011, 19:27:11 schrieb Mike Frysinger:
>> On Thursday 06 October 2011 18:23:06 Michael Walle wrote:
>>> --- a/arch/arm/lib/board.c
>>> +++ b/arch/arm/lib/board.c
>>>
>>> +#if defined(CONFIG_CMD_IDE)
>>> +	puts ("IDE:   ");
>>> +	ide_init ();
>>> +#endif
>>
>> NAK: no space before that open paren
>> -mike
>
> as long as the style is the only objection, i'm glad to resend a fixed patch
> :)
>
> Albert, are there any other objections?

NAK: the fact that some board has an IDE interface does not mean it must 
initialize it in all boot cases; and the rule with devices in U-Boot is 
'turn it on *only* when you're about to use it'.

Boards that always want to initialize IDE on boot can and will do so by 
putting 'ide reset' first in their bootcmd environment variable.

Amicalement,
-- 
Albert.


More information about the U-Boot mailing list