[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