[U-Boot] [PATCH] imx: Add titanium board support (i.MX6 based)
Fabio Estevam
festevam at gmail.com
Thu Apr 11 05:36:36 CEST 2013
Hi Stefan,
On Wed, Apr 10, 2013 at 4:17 AM, Stefan Roese <sr at denx.de> wrote:
> Titanium is a i.MX6 based board from ProjectionDesign / Barco. This
> patch adds support for this board with the newly introduced NAND
> support for i.MX6.
>
> Signed-off-by: Stefan Roese <sr at denx.de>
> ---
> MAINTAINERS | 2 +
> board/freescale/titanium/Makefile | 36 ++++
> board/freescale/titanium/imximage.cfg | 181 +++++++++++++++++
> board/freescale/titanium/titanium.c | 372 ++++++++++++++++++++++++++++++++++
> boards.cfg | 1 +
> include/configs/titanium.h | 274 +++++++++++++++++++++++++
> 6 files changed, 866 insertions(+)
> create mode 100644 board/freescale/titanium/Makefile
> create mode 100644 board/freescale/titanium/imximage.cfg
> create mode 100644 board/freescale/titanium/titanium.c
> create mode 100644 include/configs/titanium.h
It would be helpful to add a small README explaining how to flash and
boot from NAND.
> +/*
> + * Siste adresse i RAM. Starter på 0x10000000 og med 512MB blir dette
> + * siste 0x30000000 som er 768MB - sabrelite: 0x00000027
> + */
Maybe translate it to English?
> +#define CONFIG_MACH_TYPE 3769
You could do:
#define MACH_TYPE_TITANIUM 3769
#define CONFIG_MACH_TYPE MACH_TYPE_TITANIUM
> +
> +#include <asm/arch/imx-regs.h>
> +#include <asm/imx-common/gpio.h>
You could move the defines to the start of this file.
> +/* Size of malloc() pool */
> +#define CONFIG_SYS_MALLOC_LEN (2 * 1024 * 1024)
What about SZ_2M ?
> +#define CONFIG_ARP_TIMEOUT 200UL
Is this needed?
> +/* Physical Memory Map */
> +#define CONFIG_NR_DRAM_BANKS 1
> +#define PHYS_SDRAM MMDC0_ARB_BASE_ADDR
> +#define PHYS_SDRAM_SIZE (1u * 512 * 1024 * 1024)
What about using SZ_512M instead? Same for other (x << y) sizes passed
in other places.
Regards,
Fabio Estevam
More information about the U-Boot
mailing list