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.thebaud...@advansee.com> > Cc: Andy Fleming <aflem...@gmail.com> > Cc: Stefano Babic <sba...@denx.de> > Cc: Kim Phillips <kim.phill...@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: sba...@denx.de ===================================================================== _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot