* [U-Boot] [PATCH] fsl_esdhc: Add no-snoop config for default init @ 2012-08-10 21:13 Benoît Thébaudeau 2012-08-11 10:43 ` Stefano Babic 0 siblings, 1 reply; 18+ messages in thread From: Benoît Thébaudeau @ 2012-08-10 21:13 UTC (permalink / raw) To: u-boot cfg->no_snoop was available only through custom fsl_esdhc_mmc init functions. Add a config options to make it available also through the default init. Signed-off-by: Beno?t Th?baudeau <benoit.thebaudeau@advansee.com> Cc: Andy Fleming <afleming@gmail.com> Cc: Stefano Babic <sbabic@denx.de> Cc: Kim Phillips <kim.phillips@freescale.com> --- .../drivers/mmc/fsl_esdhc.c | 3 +++ 1 file changed, 3 insertions(+) diff --git u-boot-4d3c95f.orig/drivers/mmc/fsl_esdhc.c u-boot-4d3c95f/drivers/mmc/fsl_esdhc.c index b6c969d..21154e5 100644 --- u-boot-4d3c95f.orig/drivers/mmc/fsl_esdhc.c +++ u-boot-4d3c95f/drivers/mmc/fsl_esdhc.c @@ -597,6 +597,9 @@ int fsl_esdhc_mmc_init(bd_t *bis) cfg = malloc(sizeof(struct fsl_esdhc_cfg)); memset(cfg, 0, sizeof(struct fsl_esdhc_cfg)); cfg->esdhc_base = CONFIG_SYS_FSL_ESDHC_ADDR; +#ifdef CONFIG_SYS_FSL_ESDHC_NO_SNOOP + cfg->no_snoop = 1; +#endif return fsl_esdhc_initialize(bis, cfg); } ^ permalink raw reply related [flat|nested] 18+ messages in thread
* [U-Boot] [PATCH] fsl_esdhc: Add no-snoop config for default init 2012-08-10 21:13 [U-Boot] [PATCH] fsl_esdhc: Add no-snoop config for default init Benoît Thébaudeau @ 2012-08-11 10:43 ` Stefano Babic 2012-08-11 14:39 ` Benoît Thébaudeau 0 siblings, 1 reply; 18+ messages in thread From: Stefano Babic @ 2012-08-11 10:43 UTC (permalink / raw) To: u-boot On 10/08/2012 23:13, Beno?t Th?baudeau wrote: > cfg->no_snoop was available only through custom fsl_esdhc_mmc init functions. > Add a config options to make it available also through the default init. > Hi Beno?t, > Signed-off-by: Beno?t Th?baudeau <benoit.thebaudeau@advansee.com> > Cc: Andy Fleming <afleming@gmail.com> > Cc: Stefano Babic <sbabic@denx.de> > Cc: Kim Phillips <kim.phillips@freescale.com> > --- > .../drivers/mmc/fsl_esdhc.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git u-boot-4d3c95f.orig/drivers/mmc/fsl_esdhc.c u-boot-4d3c95f/drivers/mmc/fsl_esdhc.c > index b6c969d..21154e5 100644 > --- u-boot-4d3c95f.orig/drivers/mmc/fsl_esdhc.c > +++ u-boot-4d3c95f/drivers/mmc/fsl_esdhc.c > @@ -597,6 +597,9 @@ int fsl_esdhc_mmc_init(bd_t *bis) > cfg = malloc(sizeof(struct fsl_esdhc_cfg)); > memset(cfg, 0, sizeof(struct fsl_esdhc_cfg)); > cfg->esdhc_base = CONFIG_SYS_FSL_ESDHC_ADDR; > +#ifdef CONFIG_SYS_FSL_ESDHC_NO_SNOOP > + cfg->no_snoop = 1; > +#endif > return fsl_esdhc_initialize(bis, cfg); > } > Using a structure to setup the controller avoids to add further CONFIG_ switch. I do not find the needs to have both. Why do you need ? Best regards, Stefano Babic -- ===================================================================== DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de ===================================================================== ^ permalink raw reply [flat|nested] 18+ messages in thread
* [U-Boot] [PATCH] fsl_esdhc: Add no-snoop config for default init 2012-08-11 10:43 ` Stefano Babic @ 2012-08-11 14:39 ` Benoît Thébaudeau 2012-08-11 17:25 ` Stefano Babic 0 siblings, 1 reply; 18+ messages in thread From: Benoît Thébaudeau @ 2012-08-11 14:39 UTC (permalink / raw) To: u-boot Hi Stefano, On 08/11/2012 12:43, Stefano Babic wrote: > On 10/08/2012 23:13, Beno?t Th?baudeau wrote: > > cfg->no_snoop was available only through custom fsl_esdhc_mmc init > > functions. > > Add a config options to make it available also through the default > > init. > > > > Hi Beno?t, > > > Signed-off-by: Beno?t Th?baudeau <benoit.thebaudeau@advansee.com> > > Cc: Andy Fleming <afleming@gmail.com> > > Cc: Stefano Babic <sbabic@denx.de> > > Cc: Kim Phillips <kim.phillips@freescale.com> > > --- > > .../drivers/mmc/fsl_esdhc.c | 3 +++ > > 1 file changed, 3 insertions(+) > > > > diff --git u-boot-4d3c95f.orig/drivers/mmc/fsl_esdhc.c > > u-boot-4d3c95f/drivers/mmc/fsl_esdhc.c > > index b6c969d..21154e5 100644 > > --- u-boot-4d3c95f.orig/drivers/mmc/fsl_esdhc.c > > +++ u-boot-4d3c95f/drivers/mmc/fsl_esdhc.c > > @@ -597,6 +597,9 @@ int fsl_esdhc_mmc_init(bd_t *bis) > > cfg = malloc(sizeof(struct fsl_esdhc_cfg)); > > memset(cfg, 0, sizeof(struct fsl_esdhc_cfg)); > > cfg->esdhc_base = CONFIG_SYS_FSL_ESDHC_ADDR; > > +#ifdef CONFIG_SYS_FSL_ESDHC_NO_SNOOP > > + cfg->no_snoop = 1; > > +#endif > > return fsl_esdhc_initialize(bis, cfg); > > } > > > > Using a structure to setup the controller avoids to add further > CONFIG_ > switch. I do not find the needs to have both. Why do you need ? fsl_esdhc_mmc_init() exists, so it should give access to all features. Moreover, it's shorter to #define a config option than to create custom fsl_esdhc_mmc init functions when only a single eSDHC instance is needed. Besides that, the cache snooping feature is not available on i.MX, which means that without CONFIG_SYS_FSL_ESDHC_NO_SNOOP or an equivalent, boards using fsl_esdhc_mmc_init() will make esdhc_init() access a reserved bit of a reserved register, which is not really recommended. On i.MX, the patch could also be improved by not making this option available, but behaving as if it were set. Best regards, Beno?t ^ permalink raw reply [flat|nested] 18+ messages in thread
* [U-Boot] [PATCH] fsl_esdhc: Add no-snoop config for default init 2012-08-11 14:39 ` Benoît Thébaudeau @ 2012-08-11 17:25 ` Stefano Babic 2012-08-11 17:59 ` Benoît Thébaudeau 0 siblings, 1 reply; 18+ messages in thread From: Stefano Babic @ 2012-08-11 17:25 UTC (permalink / raw) To: u-boot On 11/08/2012 16:39, Beno?t Th?baudeau wrote: > Hi Stefano, > Hi, > fsl_esdhc_mmc_init() exists, so it should give access to all features. This was left for compatibility with PowerPC SOCs. > Moreover, > it's shorter to #define a config option than to create custom fsl_esdhc_mmc init > functions when only a single eSDHC instance is needed. However, the number of CONFIG_ continues to increase, often not well documented. > > Besides that, the cache snooping feature is not available on i.MX, which means > that without CONFIG_SYS_FSL_ESDHC_NO_SNOOP or an equivalent, boards using > fsl_esdhc_mmc_init() will make esdhc_init() access a reserved bit of a reserved > register, which is not really recommended. It should be masked by the i.MX code. You add CONFIG_SYS_FSL_ESDHC_NO_SNOOP and this is set inside the board configuration file. But this is not a board configuration option, rather a SOC option. Then it should the initialization code of i.MX that should hide this bit to the board itself. Instead of this, I think it is better to change cpu_mmc_init() in arch/arm/cpu/armv7/imx-common/cpu.c, setting the structure and fixing the soop bit to 1, because i.MX has not this feature. Then all i.MX boards using ESDHC have the no_snoop bit set. > On i.MX, the patch could also be improved by not making this option available, > but behaving as if it were set. Right. It should not be configurable. We know that the i.MXs have no snoop cabability. Best regards, Stefano -- ===================================================================== DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de ===================================================================== ^ permalink raw reply [flat|nested] 18+ messages in thread
* [U-Boot] [PATCH] fsl_esdhc: Add no-snoop config for default init 2012-08-11 17:25 ` Stefano Babic @ 2012-08-11 17:59 ` Benoît Thébaudeau 2012-08-11 21:57 ` stefano babic 2012-08-12 6:25 ` [U-Boot] [PATCH] fsl_esdhc: Add no-snoop config for default init Stefano Babic 0 siblings, 2 replies; 18+ messages in thread From: Benoît Thébaudeau @ 2012-08-11 17:59 UTC (permalink / raw) To: u-boot Hi Stefano, On 08/11/2012 19:25:26, Stefano Babic wrote: > On 11/08/2012 16:39, Beno?t Th?baudeau wrote: > > Hi Stefano, > > > > Hi, > > > fsl_esdhc_mmc_init() exists, so it should give access to all > > features. > > This was left for compatibility with PowerPC SOCs. OK. > > Moreover, > > it's shorter to #define a config option than to create custom > > fsl_esdhc_mmc init > > functions when only a single eSDHC instance is needed. > > However, the number of CONFIG_ continues to increase, often not well > documented. OK. > > Besides that, the cache snooping feature is not available on i.MX, > > which means > > that without CONFIG_SYS_FSL_ESDHC_NO_SNOOP or an equivalent, boards > > using > > fsl_esdhc_mmc_init() will make esdhc_init() access a reserved bit > > of a reserved > > register, which is not really recommended. > > It should be masked by the i.MX code. You add > CONFIG_SYS_FSL_ESDHC_NO_SNOOP and this is set inside the board > configuration file. But this is not a board configuration option, > rather > a SOC option. Then it should the initialization code of i.MX that > should > hide this bit to the board itself. > > Instead of this, I think it is better to change cpu_mmc_init() in > arch/arm/cpu/armv7/imx-common/cpu.c, setting the structure and fixing > the soop bit to 1, because i.MX has not this feature. Then all i.MX > boards using ESDHC have the no_snoop bit set. That could be a solution. However, that would have to be done for i.MX25 and i.MX35 too (I have patches to add/fix eSDHC support for these that I will post shortly), which means more duplicated code that should rather be centralized somewhere. > > On i.MX, the patch could also be improved by not making this option > > available, > > but behaving as if it were set. > > Right. It should not be configurable. We know that the i.MXs have no > snoop cabability. The solution I would like would be to have some automatic ARCH_IMX (or ARCH_MXC) config like on Linux. Then the fsl_esdhc driver could use this config to remove all cache snooping stuff from its code and from struct fsl_esdhc_cfg. The i.MX board files would have to be updated to no longer refer to the no_snoop field. struct fsl_esdhc_cfg would then have only one field for i.MX, but this is not really an issue. ARCH_IMX could be defined in all imx-regs.h files. Or the fsl_esdhc driver could simply use a #if testing all the supported CONFIG_MX*. What do you think? Best regards, Beno?t ^ permalink raw reply [flat|nested] 18+ messages in thread
* [U-Boot] [PATCH] fsl_esdhc: Add no-snoop config for default init 2012-08-11 17:59 ` Benoît Thébaudeau @ 2012-08-11 21:57 ` stefano babic 2012-08-13 14:18 ` [U-Boot] [PATCH 1/2] imx: Define architecture identifier Benoît Thébaudeau 2012-08-12 6:25 ` [U-Boot] [PATCH] fsl_esdhc: Add no-snoop config for default init Stefano Babic 1 sibling, 1 reply; 18+ messages in thread From: stefano babic @ 2012-08-11 21:57 UTC (permalink / raw) To: u-boot Am 11/08/2012 19:59, schrieb Beno?t Th?baudeau: > Hi Stefano, > Hi Beno?t, > That could be a solution. However, that would have to be done for i.MX25 and > i.MX35 too (I have patches to add/fix eSDHC support for these that I will post > shortly), which means more duplicated code that should rather be centralized > somewhere. Ok, I'll wait for your patches ;-) > >>> On i.MX, the patch could also be improved by not making this option >>> available, >>> but behaving as if it were set. >> >> Right. It should not be configurable. We know that the i.MXs have no >> snoop cabability. > > The solution I would like would be to have some automatic ARCH_IMX (or ARCH_MXC) > config like on Linux. Then the fsl_esdhc driver could use this config to remove > all cache snooping stuff from its code and from struct fsl_esdhc_cfg. > > The i.MX board files would have to be updated to no longer refer to the no_snoop > field. struct fsl_esdhc_cfg would then have only one field for i.MX, but this is > not really an issue. Yes, this is no an issue. > > ARCH_IMX could be defined in all imx-regs.h files. Or the fsl_esdhc driver could > simply use a #if testing all the supported CONFIG_MX*. Let the fsl_edhc checking for CONFIG_MX* results clear for us today, but nobody will remember after some times and if the checked CONFIG_MX* will be changed, it will have drawbacks not so evident on MMC. I prefer the first solution, adding an entry to imx-regs.h. Best regards, Stefano Babic -- ===================================================================== DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: office at denx.de ===================================================================== ^ permalink raw reply [flat|nested] 18+ messages in thread
* [U-Boot] [PATCH 1/2] imx: Define architecture identifier 2012-08-11 21:57 ` stefano babic @ 2012-08-13 14:18 ` Benoît Thébaudeau 2012-08-13 14:18 ` [U-Boot] [PATCH 2/2] fsl_esdhc: Remove cache snooping for i.MX Benoît Thébaudeau 2012-08-13 16:32 ` [U-Boot] [PATCH 1/2] imx: Define architecture identifier Otavio Salvador 0 siblings, 2 replies; 18+ messages in thread From: Benoît Thébaudeau @ 2012-08-13 14:18 UTC (permalink / raw) To: u-boot Define ARCH_IMX for i.MX devices. This is useful to identify features or behaviors common to all i.MX SoCs. The i.MX28 is omitted because its architecture is a bit different (like imx/mxc vs. mxs in Linux). Signed-off-by: Beno?t Th?baudeau <benoit.thebaudeau@advansee.com> Cc: Stefano Babic <sbabic@denx.de> Cc: Andy Fleming <afleming@gmail.com> Cc: Kim Phillips <kim.phillips@freescale.com> --- .../arch/arm/include/asm/arch-imx/imx-regs.h | 3 +++ .../arch/arm/include/asm/arch-mx25/imx-regs.h | 2 ++ .../arch/arm/include/asm/arch-mx27/imx-regs.h | 2 ++ .../arch/arm/include/asm/arch-mx31/imx-regs.h | 2 ++ .../arch/arm/include/asm/arch-mx35/imx-regs.h | 2 ++ .../arch/arm/include/asm/arch-mx5/imx-regs.h | 2 ++ .../arch/arm/include/asm/arch-mx6/imx-regs.h | 2 ++ 7 files changed, 15 insertions(+) diff --git u-boot-4d3c95f.orig/arch/arm/include/asm/arch-imx/imx-regs.h u-boot-4d3c95f/arch/arm/include/asm/arch-imx/imx-regs.h index ec94ba9..35ebf82 100644 --- u-boot-4d3c95f.orig/arch/arm/include/asm/arch-imx/imx-regs.h +++ u-boot-4d3c95f/arch/arm/include/asm/arch-imx/imx-regs.h @@ -1,5 +1,8 @@ #ifndef _IMX_REGS_H #define _IMX_REGS_H + +#define ARCH_IMX + /* ------------------------------------------------------------------------ * Motorola IMX system registers * ------------------------------------------------------------------------ diff --git u-boot-4d3c95f.orig/arch/arm/include/asm/arch-mx25/imx-regs.h u-boot-4d3c95f/arch/arm/include/asm/arch-mx25/imx-regs.h index cf925d7..3e48fe9 100644 --- u-boot-4d3c95f.orig/arch/arm/include/asm/arch-mx25/imx-regs.h +++ u-boot-4d3c95f/arch/arm/include/asm/arch-mx25/imx-regs.h @@ -172,6 +172,8 @@ struct aips_regs { #endif +#define ARCH_IMX + /* AIPS 1 */ #define IMX_AIPS1_BASE (0x43F00000) #define IMX_MAX_BASE (0x43F04000) diff --git u-boot-4d3c95f.orig/arch/arm/include/asm/arch-mx27/imx-regs.h u-boot-4d3c95f/arch/arm/include/asm/arch-mx27/imx-regs.h index ced5b2a..d095c17 100644 --- u-boot-4d3c95f.orig/arch/arm/include/asm/arch-mx27/imx-regs.h +++ u-boot-4d3c95f/arch/arm/include/asm/arch-mx27/imx-regs.h @@ -217,6 +217,8 @@ struct fuse_bank0_regs { #endif +#define ARCH_IMX + #define IMX_IO_BASE 0x10000000 #define IMX_AIPI1_BASE (0x00000 + IMX_IO_BASE) diff --git u-boot-4d3c95f.orig/arch/arm/include/asm/arch-mx31/imx-regs.h u-boot-4d3c95f/arch/arm/include/asm/arch-mx31/imx-regs.h index 7ddbbd6..59fb7c4 100644 --- u-boot-4d3c95f.orig/arch/arm/include/asm/arch-mx31/imx-regs.h +++ u-boot-4d3c95f/arch/arm/include/asm/arch-mx31/imx-regs.h @@ -541,6 +541,8 @@ struct esdc_regs { #endif +#define ARCH_IMX + #define __REG(x) (*((volatile u32 *)(x))) #define __REG16(x) (*((volatile u16 *)(x))) #define __REG8(x) (*((volatile u8 *)(x))) diff --git u-boot-4d3c95f.orig/arch/arm/include/asm/arch-mx35/imx-regs.h u-boot-4d3c95f/arch/arm/include/asm/arch-mx35/imx-regs.h index 3146006..d80d52f 100644 --- u-boot-4d3c95f.orig/arch/arm/include/asm/arch-mx35/imx-regs.h +++ u-boot-4d3c95f/arch/arm/include/asm/arch-mx35/imx-regs.h @@ -25,6 +25,8 @@ #ifndef __ASM_ARCH_MX35_H #define __ASM_ARCH_MX35_H +#define ARCH_IMX + /* * IRAM */ diff --git u-boot-4d3c95f.orig/arch/arm/include/asm/arch-mx5/imx-regs.h u-boot-4d3c95f/arch/arm/include/asm/arch-mx5/imx-regs.h index 7f66b61..24daef0 100644 --- u-boot-4d3c95f.orig/arch/arm/include/asm/arch-mx5/imx-regs.h +++ u-boot-4d3c95f/arch/arm/include/asm/arch-mx5/imx-regs.h @@ -23,6 +23,8 @@ #ifndef __ASM_ARCH_MX5_IMX_REGS_H__ #define __ASM_ARCH_MX5_IMX_REGS_H__ +#define ARCH_IMX + #if defined(CONFIG_MX51) #define IRAM_BASE_ADDR 0x1FFE0000 /* internal ram */ #define IPU_SOC_BASE_ADDR 0x40000000 diff --git u-boot-4d3c95f.orig/arch/arm/include/asm/arch-mx6/imx-regs.h u-boot-4d3c95f/arch/arm/include/asm/arch-mx6/imx-regs.h index 5d77603..1edfc25 100644 --- u-boot-4d3c95f.orig/arch/arm/include/asm/arch-mx6/imx-regs.h +++ u-boot-4d3c95f/arch/arm/include/asm/arch-mx6/imx-regs.h @@ -19,6 +19,8 @@ #ifndef __ASM_ARCH_MX6_IMX_REGS_H__ #define __ASM_ARCH_MX6_IMX_REGS_H__ +#define ARCH_IMX + #define CONFIG_SYS_CACHELINE_SIZE 32 #define ROMCP_ARB_BASE_ADDR 0x00000000 ^ permalink raw reply related [flat|nested] 18+ messages in thread
* [U-Boot] [PATCH 2/2] fsl_esdhc: Remove cache snooping for i.MX 2012-08-13 14:18 ` [U-Boot] [PATCH 1/2] imx: Define architecture identifier Benoît Thébaudeau @ 2012-08-13 14:18 ` Benoît Thébaudeau 2012-08-13 14:24 ` Stefano Babic 2012-08-13 16:32 ` [U-Boot] [PATCH 1/2] imx: Define architecture identifier Otavio Salvador 1 sibling, 1 reply; 18+ messages in thread From: Benoît Thébaudeau @ 2012-08-13 14:18 UTC (permalink / raw) To: u-boot The cache snooping feature of Freescale's eSDHC IP is not available on i.MX, so disable it globally for this architecture. This avoids setting no_snoop for all i.MX boards, and it prevents setting a reserved bit of a reserved register if fsl_esdhc_mmc_init() were used for an i.MX board. Signed-off-by: Beno?t Th?baudeau <benoit.thebaudeau@advansee.com> Cc: Andy Fleming <afleming@gmail.com> Cc: Stefano Babic <sbabic@denx.de> Cc: Kim Phillips <kim.phillips@freescale.com> --- .../board/efikamx/efikamx.c | 4 ++-- .../board/esg/ima3-mx53/ima3-mx53.c | 2 +- .../board/freescale/mx51evk/mx51evk.c | 4 ++-- .../board/freescale/mx53ard/mx53ard.c | 4 ++-- .../board/freescale/mx53evk/mx53evk.c | 4 ++-- .../board/freescale/mx53loco/mx53loco.c | 4 ++-- .../board/freescale/mx53smd/mx53smd.c | 2 +- .../board/freescale/mx6qarm2/mx6qarm2.c | 4 ++-- .../board/freescale/mx6qsabrelite/mx6qsabrelite.c | 4 ++-- .../board/ttcontrol/vision2/vision2.c | 2 +- .../drivers/mmc/fsl_esdhc.c | 2 ++ .../include/fsl_esdhc.h | 3 +++ 12 files changed, 22 insertions(+), 17 deletions(-) diff --git u-boot-4d3c95f.orig/board/efikamx/efikamx.c u-boot-4d3c95f/board/efikamx/efikamx.c index e88b2ed..1e643d7 100644 --- u-boot-4d3c95f.orig/board/efikamx/efikamx.c +++ u-boot-4d3c95f/board/efikamx/efikamx.c @@ -302,8 +302,8 @@ static inline void power_init(void) { } */ #ifdef CONFIG_FSL_ESDHC struct fsl_esdhc_cfg esdhc_cfg[2] = { - {MMC_SDHC1_BASE_ADDR, 1}, - {MMC_SDHC2_BASE_ADDR, 1}, + {MMC_SDHC1_BASE_ADDR}, + {MMC_SDHC2_BASE_ADDR}, }; static inline uint32_t efika_mmc_cd(void) diff --git u-boot-4d3c95f.orig/board/esg/ima3-mx53/ima3-mx53.c u-boot-4d3c95f/board/esg/ima3-mx53/ima3-mx53.c index 9ecf31d..e947330 100644 --- u-boot-4d3c95f.orig/board/esg/ima3-mx53/ima3-mx53.c +++ u-boot-4d3c95f/board/esg/ima3-mx53/ima3-mx53.c @@ -172,7 +172,7 @@ static void setup_iomux_fec(void) } #ifdef CONFIG_FSL_ESDHC -struct fsl_esdhc_cfg esdhc_cfg = { MMC_SDHC1_BASE_ADDR, 1 }; +struct fsl_esdhc_cfg esdhc_cfg = { MMC_SDHC1_BASE_ADDR }; int board_mmc_getcd(struct mmc *mmc) { diff --git u-boot-4d3c95f.orig/board/freescale/mx51evk/mx51evk.c u-boot-4d3c95f/board/freescale/mx51evk/mx51evk.c index 514a7ac..7e50688 100644 --- u-boot-4d3c95f.orig/board/freescale/mx51evk/mx51evk.c +++ u-boot-4d3c95f/board/freescale/mx51evk/mx51evk.c @@ -47,8 +47,8 @@ DECLARE_GLOBAL_DATA_PTR; #ifdef CONFIG_FSL_ESDHC struct fsl_esdhc_cfg esdhc_cfg[2] = { - {MMC_SDHC1_BASE_ADDR, 1}, - {MMC_SDHC2_BASE_ADDR, 1}, + {MMC_SDHC1_BASE_ADDR}, + {MMC_SDHC2_BASE_ADDR}, }; #endif diff --git u-boot-4d3c95f.orig/board/freescale/mx53ard/mx53ard.c u-boot-4d3c95f/board/freescale/mx53ard/mx53ard.c index 2d21584..6fea7e5 100644 --- u-boot-4d3c95f.orig/board/freescale/mx53ard/mx53ard.c +++ u-boot-4d3c95f/board/freescale/mx53ard/mx53ard.c @@ -79,8 +79,8 @@ static void setup_iomux_uart(void) #ifdef CONFIG_FSL_ESDHC struct fsl_esdhc_cfg esdhc_cfg[2] = { - {MMC_SDHC1_BASE_ADDR, 1 }, - {MMC_SDHC2_BASE_ADDR, 1 }, + {MMC_SDHC1_BASE_ADDR}, + {MMC_SDHC2_BASE_ADDR}, }; int board_mmc_getcd(struct mmc *mmc) diff --git u-boot-4d3c95f.orig/board/freescale/mx53evk/mx53evk.c u-boot-4d3c95f/board/freescale/mx53evk/mx53evk.c index 8a6e31d..2a90a4c 100644 --- u-boot-4d3c95f.orig/board/freescale/mx53evk/mx53evk.c +++ u-boot-4d3c95f/board/freescale/mx53evk/mx53evk.c @@ -204,8 +204,8 @@ static void setup_iomux_fec(void) #ifdef CONFIG_FSL_ESDHC struct fsl_esdhc_cfg esdhc_cfg[2] = { - {MMC_SDHC1_BASE_ADDR, 1}, - {MMC_SDHC3_BASE_ADDR, 1}, + {MMC_SDHC1_BASE_ADDR}, + {MMC_SDHC3_BASE_ADDR}, }; int board_mmc_getcd(struct mmc *mmc) diff --git u-boot-4d3c95f.orig/board/freescale/mx53loco/mx53loco.c u-boot-4d3c95f/board/freescale/mx53loco/mx53loco.c index cbdcfad..3195b22 100644 --- u-boot-4d3c95f.orig/board/freescale/mx53loco/mx53loco.c +++ u-boot-4d3c95f/board/freescale/mx53loco/mx53loco.c @@ -165,8 +165,8 @@ static void setup_iomux_fec(void) #ifdef CONFIG_FSL_ESDHC struct fsl_esdhc_cfg esdhc_cfg[2] = { - {MMC_SDHC1_BASE_ADDR, 1}, - {MMC_SDHC3_BASE_ADDR, 1}, + {MMC_SDHC1_BASE_ADDR}, + {MMC_SDHC3_BASE_ADDR}, }; int board_mmc_getcd(struct mmc *mmc) diff --git u-boot-4d3c95f.orig/board/freescale/mx53smd/mx53smd.c u-boot-4d3c95f/board/freescale/mx53smd/mx53smd.c index c237980..b816377 100644 --- u-boot-4d3c95f.orig/board/freescale/mx53smd/mx53smd.c +++ u-boot-4d3c95f/board/freescale/mx53smd/mx53smd.c @@ -129,7 +129,7 @@ static void setup_iomux_fec(void) #ifdef CONFIG_FSL_ESDHC struct fsl_esdhc_cfg esdhc_cfg[1] = { - {MMC_SDHC1_BASE_ADDR, 1}, + {MMC_SDHC1_BASE_ADDR}, }; int board_mmc_getcd(struct mmc *mmc) diff --git u-boot-4d3c95f.orig/board/freescale/mx6qarm2/mx6qarm2.c u-boot-4d3c95f/board/freescale/mx6qarm2/mx6qarm2.c index 340c4c4..69f5781 100644 --- u-boot-4d3c95f.orig/board/freescale/mx6qarm2/mx6qarm2.c +++ u-boot-4d3c95f/board/freescale/mx6qarm2/mx6qarm2.c @@ -116,8 +116,8 @@ static void setup_iomux_enet(void) #ifdef CONFIG_FSL_ESDHC struct fsl_esdhc_cfg usdhc_cfg[2] = { - {USDHC3_BASE_ADDR, 1}, - {USDHC4_BASE_ADDR, 1}, + {USDHC3_BASE_ADDR}, + {USDHC4_BASE_ADDR}, }; int board_mmc_getcd(struct mmc *mmc) diff --git u-boot-4d3c95f.orig/board/freescale/mx6qsabrelite/mx6qsabrelite.c u-boot-4d3c95f/board/freescale/mx6qsabrelite/mx6qsabrelite.c index 01e5083..c40db59 100644 --- u-boot-4d3c95f.orig/board/freescale/mx6qsabrelite/mx6qsabrelite.c +++ u-boot-4d3c95f/board/freescale/mx6qsabrelite/mx6qsabrelite.c @@ -237,8 +237,8 @@ int board_ehci_hcd_init(int port) #ifdef CONFIG_FSL_ESDHC struct fsl_esdhc_cfg usdhc_cfg[2] = { - {USDHC3_BASE_ADDR, 1}, - {USDHC4_BASE_ADDR, 1}, + {USDHC3_BASE_ADDR}, + {USDHC4_BASE_ADDR}, }; int board_mmc_getcd(struct mmc *mmc) diff --git u-boot-4d3c95f.orig/board/ttcontrol/vision2/vision2.c u-boot-4d3c95f/board/ttcontrol/vision2/vision2.c index d68bef7..d4ca510 100644 --- u-boot-4d3c95f.orig/board/ttcontrol/vision2/vision2.c +++ u-boot-4d3c95f/board/ttcontrol/vision2/vision2.c @@ -521,7 +521,7 @@ static void setup_fec(void) } struct fsl_esdhc_cfg esdhc_cfg[1] = { - {MMC_SDHC1_BASE_ADDR, 1}, + {MMC_SDHC1_BASE_ADDR}, }; int get_mmc_getcd(u8 *cd, struct mmc *mmc) diff --git u-boot-4d3c95f.orig/drivers/mmc/fsl_esdhc.c u-boot-4d3c95f/drivers/mmc/fsl_esdhc.c index b6c969d..efdc6be 100644 --- u-boot-4d3c95f.orig/drivers/mmc/fsl_esdhc.c +++ u-boot-4d3c95f/drivers/mmc/fsl_esdhc.c @@ -479,9 +479,11 @@ static int esdhc_init(struct mmc *mmc) while ((esdhc_read32(®s->sysctl) & SYSCTL_RSTA) && --timeout) udelay(1000); +#ifndef ARCH_IMX /* Enable cache snooping */ if (cfg && !cfg->no_snoop) esdhc_write32(®s->scr, 0x00000040); +#endif esdhc_write32(®s->sysctl, SYSCTL_HCKEN | SYSCTL_IPGEN); diff --git u-boot-4d3c95f.orig/include/fsl_esdhc.h u-boot-4d3c95f/include/fsl_esdhc.h index 0e26558..44f9f06 100644 --- u-boot-4d3c95f.orig/include/fsl_esdhc.h +++ u-boot-4d3c95f/include/fsl_esdhc.h @@ -26,6 +26,7 @@ #ifndef __FSL_ESDHC_H__ #define __FSL_ESDHC_H__ +#include <config.h> #include <asm/errno.h> #include <asm/byteorder.h> @@ -167,7 +168,9 @@ struct fsl_esdhc_cfg { u32 esdhc_base; +#ifndef ARCH_IMX u32 no_snoop; +#endif }; /* Select the correct accessors depending on endianess */ ^ permalink raw reply related [flat|nested] 18+ messages in thread
* [U-Boot] [PATCH 2/2] fsl_esdhc: Remove cache snooping for i.MX 2012-08-13 14:18 ` [U-Boot] [PATCH 2/2] fsl_esdhc: Remove cache snooping for i.MX Benoît Thébaudeau @ 2012-08-13 14:24 ` Stefano Babic 2012-08-13 14:39 ` Benoît Thébaudeau 0 siblings, 1 reply; 18+ messages in thread From: Stefano Babic @ 2012-08-13 14:24 UTC (permalink / raw) To: u-boot On 13/08/2012 16:18, Beno?t Th?baudeau wrote: > The cache snooping feature of Freescale's eSDHC IP is not available on i.MX, so > disable it globally for this architecture. This avoids setting no_snoop for all > i.MX boards, and it prevents setting a reserved bit of a reserved register if > fsl_esdhc_mmc_init() were used for an i.MX board. > > Signed-off-by: Beno?t Th?baudeau <benoit.thebaudeau@advansee.com> > Cc: Andy Fleming <afleming@gmail.com> > Cc: Stefano Babic <sbabic@denx.de> > Cc: Kim Phillips <kim.phillips@freescale.com> > --- > .../board/efikamx/efikamx.c | 4 ++-- > .../board/esg/ima3-mx53/ima3-mx53.c | 2 +- > .../board/freescale/mx51evk/mx51evk.c | 4 ++-- > .../board/freescale/mx53ard/mx53ard.c | 4 ++-- > .../board/freescale/mx53evk/mx53evk.c | 4 ++-- > .../board/freescale/mx53loco/mx53loco.c | 4 ++-- > .../board/freescale/mx53smd/mx53smd.c | 2 +- > .../board/freescale/mx6qarm2/mx6qarm2.c | 4 ++-- > .../board/freescale/mx6qsabrelite/mx6qsabrelite.c | 4 ++-- > .../board/ttcontrol/vision2/vision2.c | 2 +- > .../drivers/mmc/fsl_esdhc.c | 2 ++ > .../include/fsl_esdhc.h | 3 +++ > 12 files changed, 22 insertions(+), 17 deletions(-) > Hi Beno?t, > diff --git u-boot-4d3c95f.orig/drivers/mmc/fsl_esdhc.c u-boot-4d3c95f/drivers/mmc/fsl_esdhc.c > index b6c969d..efdc6be 100644 > --- u-boot-4d3c95f.orig/drivers/mmc/fsl_esdhc.c > +++ u-boot-4d3c95f/drivers/mmc/fsl_esdhc.c > @@ -479,9 +479,11 @@ static int esdhc_init(struct mmc *mmc) > while ((esdhc_read32(®s->sysctl) & SYSCTL_RSTA) && --timeout) > udelay(1000); > > +#ifndef ARCH_IMX > /* Enable cache snooping */ > if (cfg && !cfg->no_snoop) > esdhc_write32(®s->scr, 0x00000040); > +#endif > > esdhc_write32(®s->sysctl, SYSCTL_HCKEN | SYSCTL_IPGEN); > > diff --git u-boot-4d3c95f.orig/include/fsl_esdhc.h u-boot-4d3c95f/include/fsl_esdhc.h > index 0e26558..44f9f06 100644 > --- u-boot-4d3c95f.orig/include/fsl_esdhc.h > +++ u-boot-4d3c95f/include/fsl_esdhc.h > @@ -26,6 +26,7 @@ > #ifndef __FSL_ESDHC_H__ > #define __FSL_ESDHC_H__ > > +#include <config.h> > #include <asm/errno.h> > #include <asm/byteorder.h> > > @@ -167,7 +168,9 @@ > > struct fsl_esdhc_cfg { > u32 esdhc_base; > +#ifndef ARCH_IMX > u32 no_snoop; > +#endif > }; Why don't you get rid of no_snoop at all ? Using ARCH_IMX is not needed anymore. Best regards, Stefano Babic -- ===================================================================== DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de ===================================================================== ^ permalink raw reply [flat|nested] 18+ messages in thread
* [U-Boot] [PATCH 2/2] fsl_esdhc: Remove cache snooping for i.MX 2012-08-13 14:24 ` Stefano Babic @ 2012-08-13 14:39 ` Benoît Thébaudeau 2012-08-13 15:16 ` Stefano Babic 0 siblings, 1 reply; 18+ messages in thread From: Benoît Thébaudeau @ 2012-08-13 14:39 UTC (permalink / raw) To: u-boot Hi Stefano, On 08/13/2012 16:24, Stefano Babic wrote: > On 13/08/2012 16:18, Beno?t Th?baudeau wrote: > > The cache snooping feature of Freescale's eSDHC IP is not available > > on i.MX, so > > disable it globally for this architecture. This avoids setting > > no_snoop for all > > i.MX boards, and it prevents setting a reserved bit of a reserved > > register if > > fsl_esdhc_mmc_init() were used for an i.MX board. > > > > Signed-off-by: Beno?t Th?baudeau <benoit.thebaudeau@advansee.com> > > Cc: Andy Fleming <afleming@gmail.com> > > Cc: Stefano Babic <sbabic@denx.de> > > Cc: Kim Phillips <kim.phillips@freescale.com> > > --- > > .../board/efikamx/efikamx.c | 4 ++-- > > .../board/esg/ima3-mx53/ima3-mx53.c | 2 +- > > .../board/freescale/mx51evk/mx51evk.c | 4 ++-- > > .../board/freescale/mx53ard/mx53ard.c | 4 ++-- > > .../board/freescale/mx53evk/mx53evk.c | 4 ++-- > > .../board/freescale/mx53loco/mx53loco.c | 4 ++-- > > .../board/freescale/mx53smd/mx53smd.c | 2 +- > > .../board/freescale/mx6qarm2/mx6qarm2.c | 4 ++-- > > .../board/freescale/mx6qsabrelite/mx6qsabrelite.c | 4 ++-- > > .../board/ttcontrol/vision2/vision2.c | 2 +- > > .../drivers/mmc/fsl_esdhc.c | 2 ++ > > .../include/fsl_esdhc.h | 3 +++ > > 12 files changed, 22 insertions(+), 17 deletions(-) > > > > Hi Beno?t, > > > > diff --git u-boot-4d3c95f.orig/drivers/mmc/fsl_esdhc.c > > u-boot-4d3c95f/drivers/mmc/fsl_esdhc.c > > index b6c969d..efdc6be 100644 > > --- u-boot-4d3c95f.orig/drivers/mmc/fsl_esdhc.c > > +++ u-boot-4d3c95f/drivers/mmc/fsl_esdhc.c > > @@ -479,9 +479,11 @@ static int esdhc_init(struct mmc *mmc) > > while ((esdhc_read32(®s->sysctl) & SYSCTL_RSTA) && --timeout) > > udelay(1000); > > > > +#ifndef ARCH_IMX > > /* Enable cache snooping */ > > if (cfg && !cfg->no_snoop) > > esdhc_write32(®s->scr, 0x00000040); > > +#endif > > > > esdhc_write32(®s->sysctl, SYSCTL_HCKEN | SYSCTL_IPGEN); > > > > diff --git u-boot-4d3c95f.orig/include/fsl_esdhc.h > > u-boot-4d3c95f/include/fsl_esdhc.h > > index 0e26558..44f9f06 100644 > > --- u-boot-4d3c95f.orig/include/fsl_esdhc.h > > +++ u-boot-4d3c95f/include/fsl_esdhc.h > > @@ -26,6 +26,7 @@ > > #ifndef __FSL_ESDHC_H__ > > #define __FSL_ESDHC_H__ > > > > +#include <config.h> > > #include <asm/errno.h> > > #include <asm/byteorder.h> > > > > @@ -167,7 +168,9 @@ > > > > struct fsl_esdhc_cfg { > > u32 esdhc_base; > > +#ifndef ARCH_IMX > > u32 no_snoop; > > +#endif > > }; > > Why don't you get rid of no_snoop at all ? Using ARCH_IMX is not > needed > anymore. Because I didn't know if disabling cache snooping could be a useful option or not for architectures other than i.MX, so I preferred to leave it just in case, even if it seems to be unused. But if you are certain that disabling cache snooping can not be useful for architectures other than i.MX (now or in the future, for boards in or out of tree), I can completely remove no_snoop. Just tell me. Perhaps you have a rule such as "remove any stuff that is unused in mainstream". Best regards, Beno?t ^ permalink raw reply [flat|nested] 18+ messages in thread
* [U-Boot] [PATCH 2/2] fsl_esdhc: Remove cache snooping for i.MX 2012-08-13 14:39 ` Benoît Thébaudeau @ 2012-08-13 15:16 ` Stefano Babic 2012-08-13 15:53 ` [U-Boot] [PATCH v2 " Benoît Thébaudeau 0 siblings, 1 reply; 18+ messages in thread From: Stefano Babic @ 2012-08-13 15:16 UTC (permalink / raw) To: u-boot On 13/08/2012 16:39, Beno?t Th?baudeau wrote: >> >> Why don't you get rid of no_snoop at all ? Using ARCH_IMX is not >> needed >> anymore. > > Because I didn't know if disabling cache snooping could be a useful option or > not for architectures other than i.MX, so I preferred to leave it just in case, > even if it seems to be unused. I introduce no_snoop when I adapted the driver for iMX. I do not see PowerPC boards setting this bit. And checking this, it seems to me that code in arch/arm/cpu/armv7/imx-common/cpu.c is wrong, because cpu_mmc_init() does not set the bit and the scr register is set (or could be set) even for i.MX6 SOCs. Maybe writing at this adress does not cause problems, but it seems wrong. You patch then fixes also this issue. > But if you are certain that disabling cache snooping can not be useful for > architectures other than i.MX (now or in the future, for boards in or out of > tree), I can completely remove no_snoop. Just tell me. IMHO you can drop it. > > Perhaps you have a rule such as "remove any stuff that is unused in mainstream". It would be nice to have it ;-) Best regards, Stefano -- ===================================================================== DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de ===================================================================== ^ permalink raw reply [flat|nested] 18+ messages in thread
* [U-Boot] [PATCH v2 2/2] fsl_esdhc: Remove cache snooping for i.MX 2012-08-13 15:16 ` Stefano Babic @ 2012-08-13 15:53 ` Benoît Thébaudeau 2012-08-13 17:28 ` [U-Boot] [PATCH v3 " Benoît Thébaudeau 0 siblings, 1 reply; 18+ messages in thread From: Benoît Thébaudeau @ 2012-08-13 15:53 UTC (permalink / raw) To: u-boot The cache snooping feature of Freescale's eSDHC IP is not available on i.MX, so disable it globally for this architecture. This avoids setting no_snoop for all i.MX boards, and it prevents setting a reserved bit of a reserved register if fsl_esdhc_mmc_init() is used on i.MX, like in arch/arm/cpu/armv7/imx-common/cpu.c/cpu_mmc_init(). Since no_snoop was only used on i.MX, get rid of it BTW. Signed-off-by: Beno?t Th?baudeau <benoit.thebaudeau@advansee.com> Cc: Andy Fleming <afleming@gmail.com> Cc: Stefano Babic <sbabic@denx.de> Cc: Kim Phillips <kim.phillips@freescale.com> --- Changes for v2: - Get rid of no_snoop. .../board/efikamx/efikamx.c | 4 ++-- .../board/esg/ima3-mx53/ima3-mx53.c | 2 +- .../board/freescale/mx51evk/mx51evk.c | 4 ++-- .../board/freescale/mx53ard/mx53ard.c | 4 ++-- .../board/freescale/mx53evk/mx53evk.c | 4 ++-- .../board/freescale/mx53loco/mx53loco.c | 4 ++-- .../board/freescale/mx53smd/mx53smd.c | 2 +- .../board/freescale/mx6qarm2/mx6qarm2.c | 4 ++-- .../board/freescale/mx6qsabrelite/mx6qsabrelite.c | 4 ++-- .../board/ttcontrol/vision2/vision2.c | 2 +- .../drivers/mmc/fsl_esdhc.c | 5 +++-- .../include/fsl_esdhc.h | 1 - 12 files changed, 20 insertions(+), 20 deletions(-) diff --git u-boot-4d3c95f.orig/board/efikamx/efikamx.c u-boot-4d3c95f/board/efikamx/efikamx.c index e88b2ed..1e643d7 100644 --- u-boot-4d3c95f.orig/board/efikamx/efikamx.c +++ u-boot-4d3c95f/board/efikamx/efikamx.c @@ -302,8 +302,8 @@ static inline void power_init(void) { } */ #ifdef CONFIG_FSL_ESDHC struct fsl_esdhc_cfg esdhc_cfg[2] = { - {MMC_SDHC1_BASE_ADDR, 1}, - {MMC_SDHC2_BASE_ADDR, 1}, + {MMC_SDHC1_BASE_ADDR}, + {MMC_SDHC2_BASE_ADDR}, }; static inline uint32_t efika_mmc_cd(void) diff --git u-boot-4d3c95f.orig/board/esg/ima3-mx53/ima3-mx53.c u-boot-4d3c95f/board/esg/ima3-mx53/ima3-mx53.c index 9ecf31d..e947330 100644 --- u-boot-4d3c95f.orig/board/esg/ima3-mx53/ima3-mx53.c +++ u-boot-4d3c95f/board/esg/ima3-mx53/ima3-mx53.c @@ -172,7 +172,7 @@ static void setup_iomux_fec(void) } #ifdef CONFIG_FSL_ESDHC -struct fsl_esdhc_cfg esdhc_cfg = { MMC_SDHC1_BASE_ADDR, 1 }; +struct fsl_esdhc_cfg esdhc_cfg = { MMC_SDHC1_BASE_ADDR }; int board_mmc_getcd(struct mmc *mmc) { diff --git u-boot-4d3c95f.orig/board/freescale/mx51evk/mx51evk.c u-boot-4d3c95f/board/freescale/mx51evk/mx51evk.c index 514a7ac..7e50688 100644 --- u-boot-4d3c95f.orig/board/freescale/mx51evk/mx51evk.c +++ u-boot-4d3c95f/board/freescale/mx51evk/mx51evk.c @@ -47,8 +47,8 @@ DECLARE_GLOBAL_DATA_PTR; #ifdef CONFIG_FSL_ESDHC struct fsl_esdhc_cfg esdhc_cfg[2] = { - {MMC_SDHC1_BASE_ADDR, 1}, - {MMC_SDHC2_BASE_ADDR, 1}, + {MMC_SDHC1_BASE_ADDR}, + {MMC_SDHC2_BASE_ADDR}, }; #endif diff --git u-boot-4d3c95f.orig/board/freescale/mx53ard/mx53ard.c u-boot-4d3c95f/board/freescale/mx53ard/mx53ard.c index 2d21584..6fea7e5 100644 --- u-boot-4d3c95f.orig/board/freescale/mx53ard/mx53ard.c +++ u-boot-4d3c95f/board/freescale/mx53ard/mx53ard.c @@ -79,8 +79,8 @@ static void setup_iomux_uart(void) #ifdef CONFIG_FSL_ESDHC struct fsl_esdhc_cfg esdhc_cfg[2] = { - {MMC_SDHC1_BASE_ADDR, 1 }, - {MMC_SDHC2_BASE_ADDR, 1 }, + {MMC_SDHC1_BASE_ADDR}, + {MMC_SDHC2_BASE_ADDR}, }; int board_mmc_getcd(struct mmc *mmc) diff --git u-boot-4d3c95f.orig/board/freescale/mx53evk/mx53evk.c u-boot-4d3c95f/board/freescale/mx53evk/mx53evk.c index 8a6e31d..2a90a4c 100644 --- u-boot-4d3c95f.orig/board/freescale/mx53evk/mx53evk.c +++ u-boot-4d3c95f/board/freescale/mx53evk/mx53evk.c @@ -204,8 +204,8 @@ static void setup_iomux_fec(void) #ifdef CONFIG_FSL_ESDHC struct fsl_esdhc_cfg esdhc_cfg[2] = { - {MMC_SDHC1_BASE_ADDR, 1}, - {MMC_SDHC3_BASE_ADDR, 1}, + {MMC_SDHC1_BASE_ADDR}, + {MMC_SDHC3_BASE_ADDR}, }; int board_mmc_getcd(struct mmc *mmc) diff --git u-boot-4d3c95f.orig/board/freescale/mx53loco/mx53loco.c u-boot-4d3c95f/board/freescale/mx53loco/mx53loco.c index cbdcfad..3195b22 100644 --- u-boot-4d3c95f.orig/board/freescale/mx53loco/mx53loco.c +++ u-boot-4d3c95f/board/freescale/mx53loco/mx53loco.c @@ -165,8 +165,8 @@ static void setup_iomux_fec(void) #ifdef CONFIG_FSL_ESDHC struct fsl_esdhc_cfg esdhc_cfg[2] = { - {MMC_SDHC1_BASE_ADDR, 1}, - {MMC_SDHC3_BASE_ADDR, 1}, + {MMC_SDHC1_BASE_ADDR}, + {MMC_SDHC3_BASE_ADDR}, }; int board_mmc_getcd(struct mmc *mmc) diff --git u-boot-4d3c95f.orig/board/freescale/mx53smd/mx53smd.c u-boot-4d3c95f/board/freescale/mx53smd/mx53smd.c index c237980..b816377 100644 --- u-boot-4d3c95f.orig/board/freescale/mx53smd/mx53smd.c +++ u-boot-4d3c95f/board/freescale/mx53smd/mx53smd.c @@ -129,7 +129,7 @@ static void setup_iomux_fec(void) #ifdef CONFIG_FSL_ESDHC struct fsl_esdhc_cfg esdhc_cfg[1] = { - {MMC_SDHC1_BASE_ADDR, 1}, + {MMC_SDHC1_BASE_ADDR}, }; int board_mmc_getcd(struct mmc *mmc) diff --git u-boot-4d3c95f.orig/board/freescale/mx6qarm2/mx6qarm2.c u-boot-4d3c95f/board/freescale/mx6qarm2/mx6qarm2.c index 340c4c4..69f5781 100644 --- u-boot-4d3c95f.orig/board/freescale/mx6qarm2/mx6qarm2.c +++ u-boot-4d3c95f/board/freescale/mx6qarm2/mx6qarm2.c @@ -116,8 +116,8 @@ static void setup_iomux_enet(void) #ifdef CONFIG_FSL_ESDHC struct fsl_esdhc_cfg usdhc_cfg[2] = { - {USDHC3_BASE_ADDR, 1}, - {USDHC4_BASE_ADDR, 1}, + {USDHC3_BASE_ADDR}, + {USDHC4_BASE_ADDR}, }; int board_mmc_getcd(struct mmc *mmc) diff --git u-boot-4d3c95f.orig/board/freescale/mx6qsabrelite/mx6qsabrelite.c u-boot-4d3c95f/board/freescale/mx6qsabrelite/mx6qsabrelite.c index 01e5083..c40db59 100644 --- u-boot-4d3c95f.orig/board/freescale/mx6qsabrelite/mx6qsabrelite.c +++ u-boot-4d3c95f/board/freescale/mx6qsabrelite/mx6qsabrelite.c @@ -237,8 +237,8 @@ int board_ehci_hcd_init(int port) #ifdef CONFIG_FSL_ESDHC struct fsl_esdhc_cfg usdhc_cfg[2] = { - {USDHC3_BASE_ADDR, 1}, - {USDHC4_BASE_ADDR, 1}, + {USDHC3_BASE_ADDR}, + {USDHC4_BASE_ADDR}, }; int board_mmc_getcd(struct mmc *mmc) diff --git u-boot-4d3c95f.orig/board/ttcontrol/vision2/vision2.c u-boot-4d3c95f/board/ttcontrol/vision2/vision2.c index d68bef7..d4ca510 100644 --- u-boot-4d3c95f.orig/board/ttcontrol/vision2/vision2.c +++ u-boot-4d3c95f/board/ttcontrol/vision2/vision2.c @@ -521,7 +521,7 @@ static void setup_fec(void) } struct fsl_esdhc_cfg esdhc_cfg[1] = { - {MMC_SDHC1_BASE_ADDR, 1}, + {MMC_SDHC1_BASE_ADDR}, }; int get_mmc_getcd(u8 *cd, struct mmc *mmc) diff --git u-boot-4d3c95f.orig/drivers/mmc/fsl_esdhc.c u-boot-4d3c95f/drivers/mmc/fsl_esdhc.c index b6c969d..1ed89a3 100644 --- u-boot-4d3c95f.orig/drivers/mmc/fsl_esdhc.c +++ u-boot-4d3c95f/drivers/mmc/fsl_esdhc.c @@ -479,9 +479,10 @@ static int esdhc_init(struct mmc *mmc) while ((esdhc_read32(®s->sysctl) & SYSCTL_RSTA) && --timeout) udelay(1000); +#ifndef ARCH_IMX /* Enable cache snooping */ - if (cfg && !cfg->no_snoop) - esdhc_write32(®s->scr, 0x00000040); + esdhc_write32(®s->scr, 0x00000040); +#endif esdhc_write32(®s->sysctl, SYSCTL_HCKEN | SYSCTL_IPGEN); diff --git u-boot-4d3c95f.orig/include/fsl_esdhc.h u-boot-4d3c95f/include/fsl_esdhc.h index 0e26558..4e321e7 100644 --- u-boot-4d3c95f.orig/include/fsl_esdhc.h +++ u-boot-4d3c95f/include/fsl_esdhc.h @@ -167,7 +167,6 @@ struct fsl_esdhc_cfg { u32 esdhc_base; - u32 no_snoop; }; /* Select the correct accessors depending on endianess */ ^ permalink raw reply related [flat|nested] 18+ messages in thread
* [U-Boot] [PATCH v3 2/2] fsl_esdhc: Remove cache snooping for i.MX 2012-08-13 15:53 ` [U-Boot] [PATCH v2 " Benoît Thébaudeau @ 2012-08-13 17:28 ` Benoît Thébaudeau 0 siblings, 0 replies; 18+ messages in thread From: Benoît Thébaudeau @ 2012-08-13 17:28 UTC (permalink / raw) To: u-boot The cache snooping feature of Freescale's eSDHC IP is not available on i.MX, so disable it globally for this architecture. This avoids setting no_snoop for all i.MX boards, and it prevents setting a reserved bit of a reserved register if fsl_esdhc_mmc_init() is used on i.MX, like in arch/arm/cpu/armv7/imx-common/cpu.c/cpu_mmc_init(). Since no_snoop was only used on i.MX, get rid of it BTW. Signed-off-by: Beno?t Th?baudeau <benoit.thebaudeau@advansee.com> Cc: Andy Fleming <afleming@gmail.com> Cc: Stefano Babic <sbabic@denx.de> Cc: Kim Phillips <kim.phillips@freescale.com> --- Changes for v2: - Get rid of no_snoop. Changes for v3: - Rename ARCH_IMX to ARCH_MXC to emphasize the difference between mxc and mxs. .../board/efikamx/efikamx.c | 4 ++-- .../board/esg/ima3-mx53/ima3-mx53.c | 2 +- .../board/freescale/mx51evk/mx51evk.c | 4 ++-- .../board/freescale/mx53ard/mx53ard.c | 4 ++-- .../board/freescale/mx53evk/mx53evk.c | 4 ++-- .../board/freescale/mx53loco/mx53loco.c | 4 ++-- .../board/freescale/mx53smd/mx53smd.c | 2 +- .../board/freescale/mx6qarm2/mx6qarm2.c | 4 ++-- .../board/freescale/mx6qsabrelite/mx6qsabrelite.c | 4 ++-- .../board/ttcontrol/vision2/vision2.c | 2 +- .../drivers/mmc/fsl_esdhc.c | 5 +++-- .../include/fsl_esdhc.h | 1 - 12 files changed, 20 insertions(+), 20 deletions(-) diff --git u-boot-4d3c95f.orig/board/efikamx/efikamx.c u-boot-4d3c95f/board/efikamx/efikamx.c index e88b2ed..1e643d7 100644 --- u-boot-4d3c95f.orig/board/efikamx/efikamx.c +++ u-boot-4d3c95f/board/efikamx/efikamx.c @@ -302,8 +302,8 @@ static inline void power_init(void) { } */ #ifdef CONFIG_FSL_ESDHC struct fsl_esdhc_cfg esdhc_cfg[2] = { - {MMC_SDHC1_BASE_ADDR, 1}, - {MMC_SDHC2_BASE_ADDR, 1}, + {MMC_SDHC1_BASE_ADDR}, + {MMC_SDHC2_BASE_ADDR}, }; static inline uint32_t efika_mmc_cd(void) diff --git u-boot-4d3c95f.orig/board/esg/ima3-mx53/ima3-mx53.c u-boot-4d3c95f/board/esg/ima3-mx53/ima3-mx53.c index 9ecf31d..e947330 100644 --- u-boot-4d3c95f.orig/board/esg/ima3-mx53/ima3-mx53.c +++ u-boot-4d3c95f/board/esg/ima3-mx53/ima3-mx53.c @@ -172,7 +172,7 @@ static void setup_iomux_fec(void) } #ifdef CONFIG_FSL_ESDHC -struct fsl_esdhc_cfg esdhc_cfg = { MMC_SDHC1_BASE_ADDR, 1 }; +struct fsl_esdhc_cfg esdhc_cfg = { MMC_SDHC1_BASE_ADDR }; int board_mmc_getcd(struct mmc *mmc) { diff --git u-boot-4d3c95f.orig/board/freescale/mx51evk/mx51evk.c u-boot-4d3c95f/board/freescale/mx51evk/mx51evk.c index 514a7ac..7e50688 100644 --- u-boot-4d3c95f.orig/board/freescale/mx51evk/mx51evk.c +++ u-boot-4d3c95f/board/freescale/mx51evk/mx51evk.c @@ -47,8 +47,8 @@ DECLARE_GLOBAL_DATA_PTR; #ifdef CONFIG_FSL_ESDHC struct fsl_esdhc_cfg esdhc_cfg[2] = { - {MMC_SDHC1_BASE_ADDR, 1}, - {MMC_SDHC2_BASE_ADDR, 1}, + {MMC_SDHC1_BASE_ADDR}, + {MMC_SDHC2_BASE_ADDR}, }; #endif diff --git u-boot-4d3c95f.orig/board/freescale/mx53ard/mx53ard.c u-boot-4d3c95f/board/freescale/mx53ard/mx53ard.c index 2d21584..6fea7e5 100644 --- u-boot-4d3c95f.orig/board/freescale/mx53ard/mx53ard.c +++ u-boot-4d3c95f/board/freescale/mx53ard/mx53ard.c @@ -79,8 +79,8 @@ static void setup_iomux_uart(void) #ifdef CONFIG_FSL_ESDHC struct fsl_esdhc_cfg esdhc_cfg[2] = { - {MMC_SDHC1_BASE_ADDR, 1 }, - {MMC_SDHC2_BASE_ADDR, 1 }, + {MMC_SDHC1_BASE_ADDR}, + {MMC_SDHC2_BASE_ADDR}, }; int board_mmc_getcd(struct mmc *mmc) diff --git u-boot-4d3c95f.orig/board/freescale/mx53evk/mx53evk.c u-boot-4d3c95f/board/freescale/mx53evk/mx53evk.c index 8a6e31d..2a90a4c 100644 --- u-boot-4d3c95f.orig/board/freescale/mx53evk/mx53evk.c +++ u-boot-4d3c95f/board/freescale/mx53evk/mx53evk.c @@ -204,8 +204,8 @@ static void setup_iomux_fec(void) #ifdef CONFIG_FSL_ESDHC struct fsl_esdhc_cfg esdhc_cfg[2] = { - {MMC_SDHC1_BASE_ADDR, 1}, - {MMC_SDHC3_BASE_ADDR, 1}, + {MMC_SDHC1_BASE_ADDR}, + {MMC_SDHC3_BASE_ADDR}, }; int board_mmc_getcd(struct mmc *mmc) diff --git u-boot-4d3c95f.orig/board/freescale/mx53loco/mx53loco.c u-boot-4d3c95f/board/freescale/mx53loco/mx53loco.c index cbdcfad..3195b22 100644 --- u-boot-4d3c95f.orig/board/freescale/mx53loco/mx53loco.c +++ u-boot-4d3c95f/board/freescale/mx53loco/mx53loco.c @@ -165,8 +165,8 @@ static void setup_iomux_fec(void) #ifdef CONFIG_FSL_ESDHC struct fsl_esdhc_cfg esdhc_cfg[2] = { - {MMC_SDHC1_BASE_ADDR, 1}, - {MMC_SDHC3_BASE_ADDR, 1}, + {MMC_SDHC1_BASE_ADDR}, + {MMC_SDHC3_BASE_ADDR}, }; int board_mmc_getcd(struct mmc *mmc) diff --git u-boot-4d3c95f.orig/board/freescale/mx53smd/mx53smd.c u-boot-4d3c95f/board/freescale/mx53smd/mx53smd.c index c237980..b816377 100644 --- u-boot-4d3c95f.orig/board/freescale/mx53smd/mx53smd.c +++ u-boot-4d3c95f/board/freescale/mx53smd/mx53smd.c @@ -129,7 +129,7 @@ static void setup_iomux_fec(void) #ifdef CONFIG_FSL_ESDHC struct fsl_esdhc_cfg esdhc_cfg[1] = { - {MMC_SDHC1_BASE_ADDR, 1}, + {MMC_SDHC1_BASE_ADDR}, }; int board_mmc_getcd(struct mmc *mmc) diff --git u-boot-4d3c95f.orig/board/freescale/mx6qarm2/mx6qarm2.c u-boot-4d3c95f/board/freescale/mx6qarm2/mx6qarm2.c index 340c4c4..69f5781 100644 --- u-boot-4d3c95f.orig/board/freescale/mx6qarm2/mx6qarm2.c +++ u-boot-4d3c95f/board/freescale/mx6qarm2/mx6qarm2.c @@ -116,8 +116,8 @@ static void setup_iomux_enet(void) #ifdef CONFIG_FSL_ESDHC struct fsl_esdhc_cfg usdhc_cfg[2] = { - {USDHC3_BASE_ADDR, 1}, - {USDHC4_BASE_ADDR, 1}, + {USDHC3_BASE_ADDR}, + {USDHC4_BASE_ADDR}, }; int board_mmc_getcd(struct mmc *mmc) diff --git u-boot-4d3c95f.orig/board/freescale/mx6qsabrelite/mx6qsabrelite.c u-boot-4d3c95f/board/freescale/mx6qsabrelite/mx6qsabrelite.c index 01e5083..c40db59 100644 --- u-boot-4d3c95f.orig/board/freescale/mx6qsabrelite/mx6qsabrelite.c +++ u-boot-4d3c95f/board/freescale/mx6qsabrelite/mx6qsabrelite.c @@ -237,8 +237,8 @@ int board_ehci_hcd_init(int port) #ifdef CONFIG_FSL_ESDHC struct fsl_esdhc_cfg usdhc_cfg[2] = { - {USDHC3_BASE_ADDR, 1}, - {USDHC4_BASE_ADDR, 1}, + {USDHC3_BASE_ADDR}, + {USDHC4_BASE_ADDR}, }; int board_mmc_getcd(struct mmc *mmc) diff --git u-boot-4d3c95f.orig/board/ttcontrol/vision2/vision2.c u-boot-4d3c95f/board/ttcontrol/vision2/vision2.c index d68bef7..d4ca510 100644 --- u-boot-4d3c95f.orig/board/ttcontrol/vision2/vision2.c +++ u-boot-4d3c95f/board/ttcontrol/vision2/vision2.c @@ -521,7 +521,7 @@ static void setup_fec(void) } struct fsl_esdhc_cfg esdhc_cfg[1] = { - {MMC_SDHC1_BASE_ADDR, 1}, + {MMC_SDHC1_BASE_ADDR}, }; int get_mmc_getcd(u8 *cd, struct mmc *mmc) diff --git u-boot-4d3c95f.orig/drivers/mmc/fsl_esdhc.c u-boot-4d3c95f/drivers/mmc/fsl_esdhc.c index b6c969d..3f8d30d 100644 --- u-boot-4d3c95f.orig/drivers/mmc/fsl_esdhc.c +++ u-boot-4d3c95f/drivers/mmc/fsl_esdhc.c @@ -479,9 +479,10 @@ static int esdhc_init(struct mmc *mmc) while ((esdhc_read32(®s->sysctl) & SYSCTL_RSTA) && --timeout) udelay(1000); +#ifndef ARCH_MXC /* Enable cache snooping */ - if (cfg && !cfg->no_snoop) - esdhc_write32(®s->scr, 0x00000040); + esdhc_write32(®s->scr, 0x00000040); +#endif esdhc_write32(®s->sysctl, SYSCTL_HCKEN | SYSCTL_IPGEN); diff --git u-boot-4d3c95f.orig/include/fsl_esdhc.h u-boot-4d3c95f/include/fsl_esdhc.h index 0e26558..4e321e7 100644 --- u-boot-4d3c95f.orig/include/fsl_esdhc.h +++ u-boot-4d3c95f/include/fsl_esdhc.h @@ -167,7 +167,6 @@ struct fsl_esdhc_cfg { u32 esdhc_base; - u32 no_snoop; }; /* Select the correct accessors depending on endianess */ ^ permalink raw reply related [flat|nested] 18+ messages in thread
* [U-Boot] [PATCH 1/2] imx: Define architecture identifier 2012-08-13 14:18 ` [U-Boot] [PATCH 1/2] imx: Define architecture identifier Benoît Thébaudeau 2012-08-13 14:18 ` [U-Boot] [PATCH 2/2] fsl_esdhc: Remove cache snooping for i.MX Benoît Thébaudeau @ 2012-08-13 16:32 ` Otavio Salvador 2012-08-13 17:27 ` [U-Boot] [PATCH v3 1/2] mxc: " Benoît Thébaudeau 1 sibling, 1 reply; 18+ messages in thread From: Otavio Salvador @ 2012-08-13 16:32 UTC (permalink / raw) To: u-boot On Mon, Aug 13, 2012 at 11:18 AM, Beno?t Th?baudeau <benoit.thebaudeau@advansee.com> wrote: > Define ARCH_IMX for i.MX devices. This is useful to identify features or > behaviors common to all i.MX SoCs. > > The i.MX28 is omitted because its architecture is a bit different (like imx/mxc > vs. mxs in Linux). > > Signed-off-by: Beno?t Th?baudeau <benoit.thebaudeau@advansee.com> > Cc: Stefano Babic <sbabic@denx.de> > Cc: Andy Fleming <afleming@gmail.com> > Cc: Kim Phillips <kim.phillips@freescale.com> Wouldn't be better to use ARCH_MXC in this case? -- Otavio Salvador O.S. Systems E-mail: otavio at ossystems.com.br http://www.ossystems.com.br Mobile: +55 53 9981-7854 http://projetos.ossystems.com.br ^ permalink raw reply [flat|nested] 18+ messages in thread
* [U-Boot] [PATCH v3 1/2] mxc: Define architecture identifier 2012-08-13 16:32 ` [U-Boot] [PATCH 1/2] imx: Define architecture identifier Otavio Salvador @ 2012-08-13 17:27 ` Benoît Thébaudeau 2012-09-01 7:35 ` Stefano Babic 0 siblings, 1 reply; 18+ messages in thread From: Benoît Thébaudeau @ 2012-08-13 17:27 UTC (permalink / raw) To: u-boot Define ARCH_MXC for i.MX devices. This is useful to identify features or behaviors common to all i.MX SoCs. The i.MX28 is omitted because its architecture is a bit different (like imx/mxc vs. mxs in Linux). Signed-off-by: Beno?t Th?baudeau <benoit.thebaudeau@advansee.com> Cc: Stefano Babic <sbabic@denx.de> Cc: Andy Fleming <afleming@gmail.com> Cc: Kim Phillips <kim.phillips@freescale.com> --- Changes for v2: None. Changes for v3: - Rename ARCH_IMX to ARCH_MXC to emphasize the difference between mxc and mxs. .../arch/arm/include/asm/arch-imx/imx-regs.h | 3 +++ .../arch/arm/include/asm/arch-mx25/imx-regs.h | 2 ++ .../arch/arm/include/asm/arch-mx27/imx-regs.h | 2 ++ .../arch/arm/include/asm/arch-mx31/imx-regs.h | 2 ++ .../arch/arm/include/asm/arch-mx35/imx-regs.h | 2 ++ .../arch/arm/include/asm/arch-mx5/imx-regs.h | 2 ++ .../arch/arm/include/asm/arch-mx6/imx-regs.h | 2 ++ 7 files changed, 15 insertions(+) diff --git u-boot-4d3c95f.orig/arch/arm/include/asm/arch-imx/imx-regs.h u-boot-4d3c95f/arch/arm/include/asm/arch-imx/imx-regs.h index ec94ba9..4de0779 100644 --- u-boot-4d3c95f.orig/arch/arm/include/asm/arch-imx/imx-regs.h +++ u-boot-4d3c95f/arch/arm/include/asm/arch-imx/imx-regs.h @@ -1,5 +1,8 @@ #ifndef _IMX_REGS_H #define _IMX_REGS_H + +#define ARCH_MXC + /* ------------------------------------------------------------------------ * Motorola IMX system registers * ------------------------------------------------------------------------ diff --git u-boot-4d3c95f.orig/arch/arm/include/asm/arch-mx25/imx-regs.h u-boot-4d3c95f/arch/arm/include/asm/arch-mx25/imx-regs.h index cf925d7..672f9d7 100644 --- u-boot-4d3c95f.orig/arch/arm/include/asm/arch-mx25/imx-regs.h +++ u-boot-4d3c95f/arch/arm/include/asm/arch-mx25/imx-regs.h @@ -172,6 +172,8 @@ struct aips_regs { #endif +#define ARCH_MXC + /* AIPS 1 */ #define IMX_AIPS1_BASE (0x43F00000) #define IMX_MAX_BASE (0x43F04000) diff --git u-boot-4d3c95f.orig/arch/arm/include/asm/arch-mx27/imx-regs.h u-boot-4d3c95f/arch/arm/include/asm/arch-mx27/imx-regs.h index ced5b2a..546b8a1 100644 --- u-boot-4d3c95f.orig/arch/arm/include/asm/arch-mx27/imx-regs.h +++ u-boot-4d3c95f/arch/arm/include/asm/arch-mx27/imx-regs.h @@ -217,6 +217,8 @@ struct fuse_bank0_regs { #endif +#define ARCH_MXC + #define IMX_IO_BASE 0x10000000 #define IMX_AIPI1_BASE (0x00000 + IMX_IO_BASE) diff --git u-boot-4d3c95f.orig/arch/arm/include/asm/arch-mx31/imx-regs.h u-boot-4d3c95f/arch/arm/include/asm/arch-mx31/imx-regs.h index 7ddbbd6..766049b 100644 --- u-boot-4d3c95f.orig/arch/arm/include/asm/arch-mx31/imx-regs.h +++ u-boot-4d3c95f/arch/arm/include/asm/arch-mx31/imx-regs.h @@ -541,6 +541,8 @@ struct esdc_regs { #endif +#define ARCH_MXC + #define __REG(x) (*((volatile u32 *)(x))) #define __REG16(x) (*((volatile u16 *)(x))) #define __REG8(x) (*((volatile u8 *)(x))) diff --git u-boot-4d3c95f.orig/arch/arm/include/asm/arch-mx35/imx-regs.h u-boot-4d3c95f/arch/arm/include/asm/arch-mx35/imx-regs.h index 3146006..d6d7225 100644 --- u-boot-4d3c95f.orig/arch/arm/include/asm/arch-mx35/imx-regs.h +++ u-boot-4d3c95f/arch/arm/include/asm/arch-mx35/imx-regs.h @@ -25,6 +25,8 @@ #ifndef __ASM_ARCH_MX35_H #define __ASM_ARCH_MX35_H +#define ARCH_MXC + /* * IRAM */ diff --git u-boot-4d3c95f.orig/arch/arm/include/asm/arch-mx5/imx-regs.h u-boot-4d3c95f/arch/arm/include/asm/arch-mx5/imx-regs.h index 7f66b61..fb8668f 100644 --- u-boot-4d3c95f.orig/arch/arm/include/asm/arch-mx5/imx-regs.h +++ u-boot-4d3c95f/arch/arm/include/asm/arch-mx5/imx-regs.h @@ -23,6 +23,8 @@ #ifndef __ASM_ARCH_MX5_IMX_REGS_H__ #define __ASM_ARCH_MX5_IMX_REGS_H__ +#define ARCH_MXC + #if defined(CONFIG_MX51) #define IRAM_BASE_ADDR 0x1FFE0000 /* internal ram */ #define IPU_SOC_BASE_ADDR 0x40000000 diff --git u-boot-4d3c95f.orig/arch/arm/include/asm/arch-mx6/imx-regs.h u-boot-4d3c95f/arch/arm/include/asm/arch-mx6/imx-regs.h index 5d77603..76fab09 100644 --- u-boot-4d3c95f.orig/arch/arm/include/asm/arch-mx6/imx-regs.h +++ u-boot-4d3c95f/arch/arm/include/asm/arch-mx6/imx-regs.h @@ -19,6 +19,8 @@ #ifndef __ASM_ARCH_MX6_IMX_REGS_H__ #define __ASM_ARCH_MX6_IMX_REGS_H__ +#define ARCH_MXC + #define CONFIG_SYS_CACHELINE_SIZE 32 #define ROMCP_ARB_BASE_ADDR 0x00000000 ^ permalink raw reply related [flat|nested] 18+ messages in thread
* [U-Boot] [PATCH v3 1/2] mxc: Define architecture identifier 2012-08-13 17:27 ` [U-Boot] [PATCH v3 1/2] mxc: " Benoît Thébaudeau @ 2012-09-01 7:35 ` Stefano Babic 0 siblings, 0 replies; 18+ messages in thread From: Stefano Babic @ 2012-09-01 7:35 UTC (permalink / raw) To: u-boot On 13/08/2012 19:27, Beno?t Th?baudeau wrote: > Define ARCH_MXC for i.MX devices. This is useful to identify features or > behaviors common to all i.MX SoCs. > > The i.MX28 is omitted because its architecture is a bit different (like imx/mxc > vs. mxs in Linux). > > Signed-off-by: Beno?t Th?baudeau <benoit.thebaudeau@advansee.com> > Cc: Stefano Babic <sbabic@denx.de> > Cc: Andy Fleming <afleming@gmail.com> > Cc: Kim Phillips <kim.phillips@freescale.com> > --- Hi Beno?t, I checked again this patch and I already set this patch in patchwork as "merged" (=awaiting upstream), but I could not find it in u-boot-imx. Something went wrong.. I have applied it now to u-boot-imx. Best regards, Stefano Babic -- ===================================================================== DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de ===================================================================== ^ permalink raw reply [flat|nested] 18+ messages in thread
* [U-Boot] [PATCH] fsl_esdhc: Add no-snoop config for default init 2012-08-11 17:59 ` Benoît Thébaudeau 2012-08-11 21:57 ` stefano babic @ 2012-08-12 6:25 ` Stefano Babic 2012-08-12 13:52 ` Benoît Thébaudeau 1 sibling, 1 reply; 18+ messages in thread From: Stefano Babic @ 2012-08-12 6:25 UTC (permalink / raw) To: u-boot On 11/08/2012 19:59, Beno?t Th?baudeau wrote: Hi Beno?t, > That could be a solution. However, that would have to be done for i.MX25 and > i.MX35 too (I have patches to add/fix eSDHC support for these that I will post > shortly), which means more duplicated code that should rather be centralized > somewhere. can I ask you which kind of problems you have seen on a i.MX35 ? I was curious after your e-amil and I enabled the FSL_ESDHC driver on a mx35pdk - it seems working out of the box, I have not changed the driver. Best regards, Stefano Babic -- ===================================================================== DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de ===================================================================== ^ permalink raw reply [flat|nested] 18+ messages in thread
* [U-Boot] [PATCH] fsl_esdhc: Add no-snoop config for default init 2012-08-12 6:25 ` [U-Boot] [PATCH] fsl_esdhc: Add no-snoop config for default init Stefano Babic @ 2012-08-12 13:52 ` Benoît Thébaudeau 0 siblings, 0 replies; 18+ messages in thread From: Benoît Thébaudeau @ 2012-08-12 13:52 UTC (permalink / raw) To: u-boot Hi Stefano, On 08/12/2012 8:25, Stefano Babic wrote: > On 11/08/2012 19:59, Beno?t Th?baudeau wrote: > > Hi Beno?t, > > > That could be a solution. However, that would have to be done for > > i.MX25 and > > i.MX35 too (I have patches to add/fix eSDHC support for these that > > I will post > > shortly), which means more duplicated code that should rather be > > centralized > > somewhere. > > can I ask you which kind of problems you have seen on a i.MX35 ? I > was > curious after your e-amil and I enabled the FSL_ESDHC driver on a > mx35pdk - it seems working out of the box, I have not changed the > driver. Yes, it's not a big deal. The driver itself is fine. The issues are clock and init issues in the mx25- and mx35-specific files (clock.c, generic.c and the associated headers I think). I don't remember precisely what applied to mx25 and/or mx35, but what I fixed is: - init code referring to mxc mmc instead of fsl esdhc, - wrong handling of the cpu_mmc_init return value in some cases, - wrong source clock reported to the fsl_esdhc driver (not the eSDHC clock, and not making any difference between the clocks of the various eSDHC instances), so that if it works it's only by chance, and the actual SD clock will likely be different from what mmcinfo reports, depending on the selected board clock dividers. Best regards, Beno?t ^ permalink raw reply [flat|nested] 18+ messages in thread
end of thread, other threads:[~2012-09-01 7:35 UTC | newest] Thread overview: 18+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2012-08-10 21:13 [U-Boot] [PATCH] fsl_esdhc: Add no-snoop config for default init Benoît Thébaudeau 2012-08-11 10:43 ` Stefano Babic 2012-08-11 14:39 ` Benoît Thébaudeau 2012-08-11 17:25 ` Stefano Babic 2012-08-11 17:59 ` Benoît Thébaudeau 2012-08-11 21:57 ` stefano babic 2012-08-13 14:18 ` [U-Boot] [PATCH 1/2] imx: Define architecture identifier Benoît Thébaudeau 2012-08-13 14:18 ` [U-Boot] [PATCH 2/2] fsl_esdhc: Remove cache snooping for i.MX Benoît Thébaudeau 2012-08-13 14:24 ` Stefano Babic 2012-08-13 14:39 ` Benoît Thébaudeau 2012-08-13 15:16 ` Stefano Babic 2012-08-13 15:53 ` [U-Boot] [PATCH v2 " Benoît Thébaudeau 2012-08-13 17:28 ` [U-Boot] [PATCH v3 " Benoît Thébaudeau 2012-08-13 16:32 ` [U-Boot] [PATCH 1/2] imx: Define architecture identifier Otavio Salvador 2012-08-13 17:27 ` [U-Boot] [PATCH v3 1/2] mxc: " Benoît Thébaudeau 2012-09-01 7:35 ` Stefano Babic 2012-08-12 6:25 ` [U-Boot] [PATCH] fsl_esdhc: Add no-snoop config for default init Stefano Babic 2012-08-12 13:52 ` Benoît Thébaudeau
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox