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: off...@denx.de ===================================================================== _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot