[U-Boot] [PATCH V2] mvsata_ide: adjust port init sequence
Prafulla Wadaskar
prafulla at marvell.com
Tue Sep 7 15:32:15 CEST 2010
> -----Original Message-----
> From: Albert ARIBAUD [mailto:albert.aribaud at free.fr]
> Sent: Tuesday, September 07, 2010 4:53 PM
> To: Prafulla Wadaskar
> Cc: u-boot at lists.denx.de; Prabhanjan Sarnaik; Ashish Karkare
> Subject: Re: [PATCH V2] mvsata_ide: adjust port init sequence
>
> Le 07/09/2010 08:42, Prafulla Wadaskar a écrit :
> >> diff --git a/drivers/block/mvsata_ide.c
> b/drivers/block/mvsata_ide.c
>
> >> @@ -125,15 +129,17 @@ int ide_preinit(void)
>
> >> + return 1;
> >
> > How about returning negative values for errors ?
>
> Function ide_preinit() is called from cmd_ide.c:ide_init(),
> which does
> not distinguish positive vs negative return values, ony zero vs
> non-zero. So what would be the point of returning negative
> rather than
> positive values?
Negative always represents errors,
whereas positive may represent some valid return state.
In this case it may not be that important.
Regards..
Prafulla . .
>
> Amicalement,
> --
> Albert.
>
More information about the U-Boot
mailing list