Dear Mike Rapoport,
In message <1257955131-16729-1-git-send-email-m...@compulab.co.il> you wrote:
> Add CM-T35 board support
>
> --
> v2 changes:
> - rename board config file from omap3_cm-t35.h to cm-t35.h
> - remove SZ_xx references
> - add MAKEALL/MAINTEINERS entries
> --
>
> Signed-off-by: Mike Rapoport <m...@compulab.co.il>
The Signed-off-by: line belongs _above_ the "--" line.
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -677,6 +677,10 @@ Stelian Pop <stelian....@leadtechdesign.com>
> at91sam9263ek ARM926EJS (AT91SAM9263 SoC)
> at91sam9rlek ARM926EJS (AT91SAM9RL SoC)
>
> +Mike Rapoport <m...@compulab.co.il>
> +
> + omap3_cm-t35 ARM CORTEX-A8 (OMAP3xx SoC)
Please fix the board name.
> diff --git a/board/cm-t35/cm-t35.c b/board/cm-t35/cm-t35.c
> new file mode 100644
> index 0000000..b3eb087
> --- /dev/null
> +++ b/board/cm-t35/cm-t35.c
> + udelay(1);
> + twl4030_i2c_write_u8(TWL4030_CHIP_GPIO, 0x02,
> + TWL4030_BASEADD_GPIO+0x0C);
> +
> +}
Delete this empty line, please (please check globally).
> +++ b/board/cm-t35/cm-t35.h
...
> +#define MUX_CM_T35() \
> + /*SDRC*/\
> + MUX_VAL(CP(SDRC_D0), (IEN | PTD | DIS | M0)) /*SDRC_D0*/\
...
Indentation by TAB, please.
> +++ b/board/cm-t35/config.mk
> @@ -0,0 +1,30 @@
...
> +# For use with external or internal boots.
> +TEXT_BASE = 0x80e80000
> \ No newline at end of file
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Please fix.
> diff --git a/include/configs/cm-t35.h b/include/configs/cm-t35.h
> new file mode 100755
> index 0000000..b881112
> --- /dev/null
> +++ b/include/configs/cm-t35.h
...
> +#ifndef __ASSEMBLY__
> +extern struct gpmc *gpmc_cfg;
> +extern unsigned int boot_flash_base;
> +extern volatile unsigned int boot_flash_env_addr;
> +extern unsigned int boot_flash_off;
> +extern unsigned int boot_flash_sec;
> +extern unsigned int boot_flash_type;
> +#endif
These should not be needed in your board config file. Please move to a
more appropriate header.
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: w...@denx.de
Conquest is easy. Control is not.
-- Kirk, "Mirror, Mirror", stardate unknown
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
