[U-Boot-Users] [PATCH]ppc_4xx:netstal HCU5 board: added various fixes and POST
Wolfgang Denk
wd at denx.de
Mon Jan 14 21:20:35 CET 2008
In message <200801141530.23152.niklausgiger at gmx.ch> you wrote:
> Moved some code to nestal/common/nm_bsp.c.
> Added support for the vxWorks EDR.
> Enable trace for Lauterbach, if present.
> Added support for POST.
...
> + if (mfspr(dbcr0) & 0x80000000) { /* External debugger alive */
> + /* enable trace facilty for Lauterback
/*
* enable trace facilty for Lauterbach
...
> - /*-------------------------------------------------------------------+
> - | Set priority for all PLB3 devices to 0.
> - | Set PLB3 arbiter to fair mode.
> - +-------------------------------------------------------------------*/
> + /*-------------------------------------------------------------------
> + * Set priority for all PLB3 devices to 0.
> + * Set PLB3 arbiter to fair mode.
> + *-------------------------------------------------------------------*/
...
> - /*-------------------------------------------------------------------+
> - | Set priority for all PLB4 devices to 0.
> - +-------------------------------------------------------------------*/
> + /*-------------------------------------------------------------------
> + * Set priority for all PLB4 devices to 0.
> + *-------------------------------------------------------------------*/
If you change multiline comments, then please do it right. The
preferred style for long (multi-line) comments is:
/*
* This is the preferred style for multi-line
* comments in the Linux kernel source code.
* Please use it consistently.
*
* Description: A column of asterisks on the left side,
* with beginning and ending almost-blank lines.
*/
> + /* TLB#12: */
> + tlbentry( CFG_CS_2 + 0x1000000, SZ_16M, CFG_CS_2 + 0x1000000, 1, AC_R|AC_W|AC_X|SA_G|SA_I )
Here and in some other places: please mind the maximum line length.
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
"If the code and the comments disagree, then both are probably
wrong." - Norm Schryer
More information about the U-Boot
mailing list