[U-Boot] [PATCH 3/3] mx28evk: add SPI support

[Marek Vasut <marek.vasut@gmail.com>]

> On 01/18/2012 05:07 PM, Marek Vasut wrote:
> >> This patch adds SPI support for the MX28EVK. Support for
> >> an optionally installed SPI flash is also added. An example
> >> configuration for redundant envrionment from SPI flash is also
> >> added but disabled by default.
> >> 
> >> This patch has been tested on a MX28EVK Rev. D with an installed
> >> SST25VF032B 32Mbit SPI flash.
> >> 
> >> Signed-off-by: Matthias Fuchs <matthias.fuchs@esd.eu>
> >> ---
> >> 
> >>  board/freescale/mx28evk/iomux.c |    8 +++++++
> >>  include/configs/mx28evk.h       |   44
> >> 
> >> ++++++++++++++++++++++++++++++++++++-- 2 files changed, 49
> >> insertions(+), 3 deletions(-)
> >> 
> >> diff --git a/board/freescale/mx28evk/iomux.c
> >> b/board/freescale/mx28evk/iomux.c index 904e3f3..396761b 100644
> >> --- a/board/freescale/mx28evk/iomux.c
> >> +++ b/board/freescale/mx28evk/iomux.c
> >> @@ -28,6 +28,7 @@
> >> 
> >>  #define	MUX_CONFIG_SSP0	(MXS_PAD_3V3 | MXS_PAD_8MA | MXS_PAD_PULLUP)
> >>  #define	MUX_CONFIG_ENET	(MXS_PAD_3V3 | MXS_PAD_8MA | MXS_PAD_PULLUP)
> >>  #define	MUX_CONFIG_EMI	(MXS_PAD_3V3 | MXS_PAD_12MA | MXS_PAD_NOPULL)
> >> 
> >> +#define	MUX_CONFIG_SSP2	(MXS_PAD_3V3 | MXS_PAD_4MA | MXS_PAD_PULLUP)
> >> 
> >>  const iomux_cfg_t iomux_setup[] = {
> >>  
> >>  	/* DUART */
> >> 
> >> @@ -130,6 +131,13 @@ const iomux_cfg_t iomux_setup[] = {
> >> 
> >>  	MX28_PAD_EMI_CE0N__EMI_CE0N | MUX_CONFIG_EMI,
> >>  	MX28_PAD_EMI_CE1N__EMI_CE1N | MUX_CONFIG_EMI,
> >>  	MX28_PAD_EMI_CKE__EMI_CKE | MUX_CONFIG_EMI,
> >> 
> >> +
> >> +	/* SPI2 (for SPI flash) */
> >> +	MX28_PAD_SSP2_SCK__SSP2_SCK | MUX_CONFIG_SSP2,
> >> +	MX28_PAD_SSP2_MOSI__SSP2_CMD | MUX_CONFIG_SSP2,
> >> +	MX28_PAD_SSP2_MISO__SSP2_D0 | MUX_CONFIG_SSP2,
> >> +	MX28_PAD_SSP2_SS0__SSP2_D3 |
> >> +		(MXS_PAD_3V3 | MXS_PAD_8MA | MXS_PAD_PULLUP),
> >> 
> >>  };
> >>  
> >>  void board_init_ll(void)
> >> 
> >> diff --git a/include/configs/mx28evk.h b/include/configs/mx28evk.h
> >> index 8f791aa..04967d7 100644
> >> --- a/include/configs/mx28evk.h
> >> +++ b/include/configs/mx28evk.h
> >> @@ -64,6 +64,8 @@
> >> 
> >>  #define CONFIG_CMD_NET
> >>  #define CONFIG_CMD_NFS
> >>  #define CONFIG_CMD_PING
> >> 
> >> +#define CONFIG_CMD_SF
> >> +#define CONFIG_CMD_SPI
> >> 
> >>  #define CONFIG_CMD_USB
> >>  
> >>  /*
> >> 
> >> @@ -127,9 +129,11 @@
> >> 
> >>   * MMC Driver
> >>   */
> >>  
> >>  #define CONFIG_ENV_IS_IN_MMC
> >> 
> >> -#define CONFIG_ENV_OFFSET	(256 * 1024)
> >> -#define CONFIG_ENV_SIZE	(16 * 1024)
> >> -#define CONFIG_SYS_MMC_ENV_DEV 0
> >> +#ifdef CONFIG_ENV_IS_IN_MMC
> >> + #define CONFIG_ENV_OFFSET	(256 * 1024)
> >> + #define CONFIG_ENV_SIZE	(16 * 1024)
> >> + #define CONFIG_SYS_MMC_ENV_DEV 0
> >> +#endif
> >> 
> >>  #define CONFIG_CMD_SAVEENV
> >>  #ifdef	CONFIG_CMD_MMC
> >>  #define CONFIG_MMC
> >> 
> >> @@ -170,6 +174,40 @@
> >> 
> >>  #endif
> >>  
> >>  /*
> >> 
> >> + * SPI
> >> + */
> >> +#ifdef CONFIG_CMD_SPI
> >> +#define CONFIG_HARD_SPI
> >> +#define CONFIG_MXS_SPI
> >> +#define CONFIG_SPI_HALF_DUPLEX
> >> +#define CONFIG_DEFAULT_SPI_BUS		2
> >> +#define CONFIG_DEFAULT_SPI_MODE		SPI_MODE_0
> >> +
> >> +/* SPI Flash */
> >> +#ifdef CONFIG_CMD_SF
> >> +#define CONFIG_SPI_FLASH
> >> +/* this may vary and depends on the installed chip */
> >> +#define CONFIG_SPI_FLASH_SST
> >> +#define CONFIG_SF_DEFAULT_MODE		SPI_MODE_0
> >> +#define CONFIG_SF_DEFAULT_SPEED		24000000
> >> +
> >> +/* (redundant) environemnt in SPI flash */
> >> +#undef CONFIG_ENV_IS_IN_SPI_FLASH
> >> +#ifdef CONFIG_ENV_IS_IN_SPI_FLASH
> >> +#define CONFIG_SYS_REDUNDAND_ENVIRONMENT
> >> +#define CONFIG_ENV_SIZE			0x1000		/* 4KB */
> >> +#define CONFIG_ENV_OFFSET		0x40000		/* 256K */
> >> +#define CONFIG_ENV_OFFSET_REDUND	(CONFIG_ENV_OFFSET + CONFIG_ENV_SIZE)
> >> +#define CONFIG_ENV_SECT_SIZE		0x1000
> >> +#define CONFIG_ENV_SPI_CS		0
> >> +#define CONFIG_ENV_SPI_BUS		2
> >> +#define CONFIG_ENV_SPI_MAX_HZ		24000000
> >> +#define CONFIG_ENV_SPI_MODE		SPI_MODE_0
> >> +#endif
> >> +#endif
> >> +#endif
> >> +
> >> +/*
> >> 
> >>   * Boot Linux
> >>   */
> >>  
> >>  #define CONFIG_CMDLINE_TAG
> > 
> > Hm, good ... so the SPI flash works now with no modifications ?
> 
> Of course not! The SPI driver needs to be fixed. Either by my previously
> posted patch or by god's hands :-)
> 
> Fabio asked for this patch in order to test my SPI driver fix.
> But I think this is independant from each other.
> 
> Matthias

Ok, I see ... sigh, can you please do such negotiations on the mailing list so 
people are informed about such?

M