[U-Boot] [PATCH v3] add ASTRO MCF5373L board
Wolfgang Denk
wd at denx.de
Mon Jan 18 00:40:45 CET 2010
Dear Wolfgang Wegner,
In message <1263231799-28685-1-git-send-email-w.wegner at astro-kom.de> you wrote:
> This patch adds support for ASTRO board(s) based on MCF5373L.
>
> Signed-off-by: Wolfgang Wegner <w.wegner at astro-kom.de>
...
> + /* GPIO configuration for bus should be set correctly from reset,
> + * so we do not care! First, set up address space: at this point,
> + * we should be running from internal SRAM;
> + * so use CONFIG_SYS_SDRAM_BASE as the base address for SDRAM,
> + * and do not care where it is
> + */
Incorrect multiline comment style.
> + /* I am not sure from the data sheet, but it seems burst length
> + * has to be 8 for the 16 bit data bus we use;
> + * so these values are for BL = 8
> + */
This happens in a number or places, please fix globally.
> +/*-----------------------------------------------------------------------
> + * UART update subcommand codes
> + */
These, too, please.
> + /*****************************************
> + ** Flash program routine **
> + ** only called in flash mode **
> + *****************************************
> + */
And these as well.
> +/*
> + *-----------------------------------------------------------------------------
> + * End of user parameters to be customized
> + *-----------------------------------------------------------------------------
> + */
> +
> +/* ---
> + * Defines memory range for test
> + * ---
> + */
...and these.
Amazing haw many different comment styles you are using in a single
patch. Guess you can apply for a record ;-)
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
1000 pains = 1 Megahertz
More information about the U-Boot
mailing list