[U-Boot] [PATCH 08/11] Blackfin: bf60x: add rsi/sdh support
Wolfgang Denk
wd at denx.de
Sun Feb 17 21:15:11 CET 2013
Dear Sonic Zhang,
In message <CAJxxZ0OTwow6X2KX8yAujtrxc6qicVPMO2rP4MHwJQVQUbhecw at mail.gmail.com> you wrote:
>
> >> @@ -113,16 +131,19 @@ static int sdh_setup_data(struct mmc *mmc, struct mmc_data *data)
> > ...
> >> - int ret = 0;
> > ...
> >> - return ret;
> >> + return 0;
> >
> > If this function can always only return 0, then please make it void.
> >
> > Please fix globally.
>
> I am sorry, this function can't be changed to void, because it may
> return error. See bellow code.
You are contradicting yourself. You _always_ return a 0 here. Your
code CANNOT return an error code.
Best regards,
Wolfgang Denk
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Build a system that even a fool can use and only a fool will want to
use it.
More information about the U-Boot
mailing list