[U-Boot] [PATCH 4/4] ARM: atmel: add sama5d3xek support

[Tom Rini]

On Thu, Feb 28, 2013 at 03:00:47PM +0800, Bo Shen wrote:

> Add sama5d3xek support with following feature
>   - boot from NAND flash, PMECC support, 4bit ECC @ 512 bytes sector
>   - boot from SPI flash support
>   - boot from SD card support
>   - LCD support
>   - EMAC support
>   - USB support
> 
> Signed-off-by: Bo Shen <voice.shen at atmel.com>

Some minor comments:

[snip]
> +	if (cpu_is_sama5d3())
> +		switch (extension_id) {
> +		case ARCH_EXID_SAMA5D31:
> +			return CONFIG_SYS_AT91_D31_CPU_NAME;
> +		case ARCH_EXID_SAMA5D33:
> +			return CONFIG_SYS_AT91_D33_CPU_NAME;
> +		case ARCH_EXID_SAMA5D34:
> +			return CONFIG_SYS_AT91_D34_CPU_NAME;
> +		case ARCH_EXID_SAMA5D35:
> +			return CONFIG_SYS_AT91_D35_CPU_NAME;
> +		default:
> +			return CONFIG_SYS_AT91_UNKNOWN_CPU;

These aren't configurable, and are used once.  Just put the strings
here.

> @@ -0,0 +1,268 @@
> +/*
> + * Configuation settings for the SAMA5D3xEK board.
[snip]
> +#undef CONFIG_USE_IRQ			/* we don't need IRQ/FIQ stuff	*/
> +
> +#undef CONFIG_CMDLINE_TAG		/* enable passing of ATAGs	*/
> +#undef CONFIG_SETUP_MEMORY_TAGS
> +#undef CONFIG_INITRD_TAG

Just leave these, and the other #undef's out.

> +/*
> + * Command line configuration.
> + */
> +#include <config_cmd_default.h>
> +#undef CONFIG_CMD_FPGA
> +#undef CONFIG_CMD_IMI
> +#undef CONFIG_CMD_IMLS
> +#undef CONFIG_CMD_AUTOSCRIPT
> +#undef CONFIG_CMD_LOADS

These are fine to leave in 'tho.

> +#ifdef CONFIG_USE_IRQ
> +#error CONFIG_USE_IRQ not supported
> +#endif

Just drop that part.  And please check things with checkpatch.pl, I
thought I saw a '#define<tab>FOO' in there.  Thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20130304/731f37da/attachment.pgp>