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

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

[Michael Walle]

Call ide_init() in the board initialization if IDE support is compiled in.

Signed-off-by: Michael Walle <michael@walle.cc>
---
 arch/arm/lib/board.c |    5 +++++
 1 files changed, 5 insertions(+), 0 deletions(-)

diff --git a/arch/arm/lib/board.c b/arch/arm/lib/board.c
index 705d4d2..9addab8 100644
--- a/arch/arm/lib/board.c
+++ b/arch/arm/lib/board.c
@@ -596,6 +596,11 @@ void board_init_r(gd_t *id, ulong dest_addr)
 #endif
 #endif
 
+#if defined(CONFIG_CMD_IDE)
+	puts ("IDE:   ");
+	ide_init ();
+#endif
+
 #ifdef CONFIG_POST
 	post_run(NULL, POST_RAM | post_bootmode_get(0));
 #endif
-- 
1.7.2.5

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

[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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
Url : http://lists.denx.de/pipermail/u-boot/attachments/20111007/cccbb489/attachment.pgp 

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

[Michael Walle]

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?

-- 
Michael

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

[Albert ARIBAUD]

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.

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

[Mike Frysinger]

On Tuesday 18 October 2011 17:42:26 Albert ARIBAUD wrote:
> 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
> > 
> > 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.

ACK to this.  would be nice to get existing ports (ppc/m68k/sh/sparc/x86) to 
migrate their users to "ide reset" too ...
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
Url : http://lists.denx.de/pipermail/u-boot/attachments/20111018/719d932c/attachment.pgp