[U-Boot] [PATCH 08/29] MPC512x: add support for ARIA board

Wolfgang Denk wd at denx.de
Mon May 11 09:38:59 CEST 2009


Dear Stefan,

In message <200905110917.21667.sr at denx.de> you wrote:
>
> > +phys_size_t initdram (int board_type)
> > +{
> > +	u32 msize = 0;
> > +
> > +	msize = fixed_sdram ();
> > +
> > +	return msize;
> > +}
> 
> Or just:
> 
> phys_size_t initdram (int board_type)
> {
> 	return fixed_sdram ();
> }

Right, will change.

> And I noticed that are mixing the coding style's in this file (space before 
> "(" of functions): fixed_sdram () vs. out_be(...). Better it would be 
> consistent.

Indeed. Will change, too. Thanks for pointing out.

[I'll continue to collect review comments for a  few  more  days  and
then  post  patch  series  V2;  in the meantime, the current, updated
code, rebased against "next", is available in the  "aria"  branch  of
the u-boot-testing repository.]

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
There comes to all races an ultimate crisis which  you  have  yet  to
face  ....  One  day  our  minds became so powerful we dared think of
ourselves as gods.
	-- Sargon, "Return to Tomorrow", stardate 4768.3


More information about the U-Boot mailing list