[U-Boot] [PATCH V2 6/6] arm: mx5: Add support for DENX M53EVK

[Marek Vasut]

Dear Benoît Thébaudeau,

> Dear Marek Vasut,
> 
> On Sunday, April 21, 2013 5:52:27 PM, Marek Vasut wrote:
> > Add basic support for the DENX M53EVK board. Currently supported is:
> > MMC (incl. booting)
> 
>              ^
> Can you clarify this, please?

Use u-boot.imx for SD booting as usual.

> spl_boot_device() points only to NAND, so
> you're clearly talking about hardware MMC boot, and not about hardware
> NAND boot followed by SPL payload fetched from MMC. But MMC boot does not
> need SPL here, in which case you will have to generate a simple
> u-boot.imx, or you will rather want to use u-boot-with-spl.imx for SD
> (NAND header dropped to leave room for MBR). And in the latter case, why
> have spl_boot_device() point to NAND for MMC boot?

No, regular u-boot.imx will be used for SD boot.

> > NAND (incl. booting)
> > Ethernet, I2C, USB, SATA, RTC.
> > 
> > Signed-off-by: Marek Vasut <marex at denx.de>
> > Cc: Albert ARIBAUD <albert.u.boot at aribaud.net>
> > Cc: Benoît Thébaudeau <benoit.thebaudeau at advansee.com>
> > Cc: Fabio Estevam <fabio.estevam at freescale.com>
> > Cc: Scott Wood <scottwood at freescale.com>
> > Cc: Stefano Babic <sbabic at denx.de>
> > Cc: Tom Rini <trini at ti.com>
> > Cc: Wolfgang Denk <wd at denx.de>
> > ---

[...]

> > +static void m53_set_nand(void)
> > +{
> > +	u32 i;
> > +
> > +	#define M4IF_GENP_WEIM_MM		0x00000001
> > +	#define WEIM_GCR2_MUX16_BYP_GRANT	0x00001000
> > +
> > +	/* NAND flash is muxed on ATA pins */
> > +	setbits_le32(M4IF_BASE_ADDR + 0xc, M4IF_GENP_WEIM_MM);
> 
> This should be clrbits_le32().

Why?

> > +
> > +	/* Wait for Grant/Ack sequence (see EIM_CSnGCR2:MUX16_BYP_GRANT) */
> > +	for (i = 0x4; i < 0x94; i += 0x18)
> > +		clrbits_le32(WEIM_BASE_ADDR + i, WEIM_GCR2_MUX16_BYP_GRANT);
> > +
> > +	mxc_set_clock(0, 33, MXC_NFC_CLK);
> > +	enable_nfc_clk(1);
> > +}

[...]