[U-Boot] [PATCH 1/3] ARM: Add support for EP93xx SoCs

Wolfgang Denk wd at denx.de
Tue Jan 19 22:52:03 CET 2010


Dear Matthias Kaehlcke,

In message <20100119191617.GK16182 at darwin> you wrote:
> Add support for the Cirrus EP93xx platform
> 
> Signed-off-by: Matthias Kaehlcke <matthias at kaehlcke.net>
...
> --- /dev/null
> +++ b/include/asm-arm/arch-ep93xx/ep93xx.h
> @@ -0,0 +1,596 @@
> +/* -----------------------------------------------------------------------------
> + * Cirrus Logic EP93xx register definitions.
...
> + */
...
> +/* -----------------------------------------------------------------------------
> + * 0x80000000 - 0x8000FFFF: DMA
> + */

etc.: incorrect multiline comment style. Please fix globally.

And please get rid of the trailing empty line(s).

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 are bugs and then there are bugs.  And then there are bugs.
                                                    - Karl Lehenbauer


More information about the U-Boot mailing list