[U-Boot] [PATCH] add support for EMK TOP7000 CPU module (new files)

Wolfgang Denk wd at denx.de
Fri Jun 4 14:36:05 CEST 2010


Dear "Reinhard Meyer (-VC)",

In message <4C08EC47.5070603 at emk-elektronik.de> you wrote:
> This Patch adds support for the EMK TOP7000 CPU Module
> Part 1: new files
> 
> Signed-off-by: Reinhard Meyer reinhard.meyer at emk-elektronik.de

But please clean up the code berfore resubmitting...


> --- /dev/null
> +++ b/board/emk/top7000/top7000.c
...
> +static const struct sdram_config sdram_config = {
> +    .data_bits    = SDRAM_DATA_16BIT,
> +    .row_bits    = 13,
> +    .col_bits    = 9,
> +    .bank_bits    = 2,
> +    .cas        = 3,

Indentation by TAB only - please fix globally.

> +#ifdef CONFIG_CMD_I2C
> +    // first select SCL and SDA gpio

No C++ comments - please fix globally.

> +    portmux_select_gpio(PORTMUX_PORT_A,(1<<SDA_PIN), 

No trailing white space - please fix globally

> +    portmux_select_gpio(PORTMUX_PORT_A,(1<<SDA_PIN), 
> PORTMUX_DIR_OUTPUT|PORTMUX_INIT_LOW|PORTMUX_OPEN_DRAIN);

Long lines wrapped by mailer. Please fix your mailer setup (or rather
use "git send-email" for sending patches.

Note: line too long - please fix globally.


> +    if (expected_size != actual_size)
> +        printf("Warning: Only %lu of %lu MiB SDRAM is working\n",
> +                actual_size >> 20, expected_size >> 20);

Curly braces needed for multiline statements - please fix globally.

> +int misc_init_r (void)
------------------^
> +{
> +    /* read 'factory' part of EEPROM */
> +    extern void read_factory_r (void);
> +    read_factory_r ();
---------------------^

No spaces after function names - please fix globally.

...
> --- /dev/null
> +++ b/board/emk/top7000/u-boot.lds
> @@ -0,0 +1,72 @@
> +/* -*- Fundamental -*-

Incorrect multiline comment style - please fix globally.


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
PoB = "Prisoner of Bill" -- those held captive, unwillingly or other-
wise, by the contemptible Microsoft monopoly.
         -- Tom Christiansen in <6abo45$3lc$2 at csnews.cs.colorado.edu>


More information about the U-Boot mailing list