[U-Boot] [PATCH][v3] mpc8308_p1m: support for MPC8308 P1M board

Wolfgang Denk wd at denx.de
Sat Sep 18 22:35:54 CEST 2010


Dear Kim Phillips,

In message <20100913201208.461501b3.kim.phillips at freescale.com> you wrote:
>
> >  MAINTAINERS                     |    1 +
> >  board/mpc8308_p1m/Makefile      |   52 ++++
> >  board/mpc8308_p1m/config.mk     |    1 +
> >  board/mpc8308_p1m/mpc8308_p1m.c |  122 +++++++++
> >  board/mpc8308_p1m/sdram.c       |   93 +++++++
> >  boards.cfg                      |    1 +
> >  include/configs/mpc8308_p1m.h   |  548 +++++++++++++++++++++++++++++++++++++++
> 
> missing MAKEALL entry.

Not needed any more since we pick this off from boards.cfg for
"simple" boards.

> > +++ b/board/mpc8308_p1m/config.mk
> > @@ -0,0 +1 @@
> > +TEXT_BASE = 0xFC000000
> 
> ifndef TEXT_BASE
> TEXT_BASE = 0xFC000000
> endif

Why?

> > +int board_eth_init(bd_t *bis)
> > +{
> > +	int rv, num_if = 0;
> > +
> > +	/* Initialize TSECs first */
> > +	if ((rv = cpu_eth_init(bis)) >= 0)
> 
> no assignments in if statements.

Why?

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
Never ascribe to malice that which can  adequately  be  explained  by
stupidity.


More information about the U-Boot mailing list