[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