On Mon, Jan 29, 2024 at 06:36:29PM +0100, Uwe Kleine-König wrote: > Hello Sascha, > > On Thu, Jan 04, 2024 at 03:17:45PM +0100, Sascha Hauer wrote: > > diff --git a/drivers/mci/imx-esdhc-pbl.c b/drivers/mci/imx-esdhc-pbl.c > > index c13830d726..2c74d101e4 100644 > > --- a/drivers/mci/imx-esdhc-pbl.c > > +++ b/drivers/mci/imx-esdhc-pbl.c > > @@ -9,6 +9,8 @@ > > #include <asm/sections.h> > > #include <asm/cache.h> > > #include <mach/imx/xload.h> > > +#include <firmware.h> > > +#include <asm/atf_common.h> > > #ifdef CONFIG_ARCH_IMX > > #include <mach/imx/atf.h> > > #include <mach/imx/imx6-regs.h> > > @@ -319,7 +321,8 @@ static int layerscape_esdhc_load_image(struct fsl_esdhc_host *host, void *adr, u > > val |= div_val; > > sdhci_write32(&host->sdhci, SDHCI_CLOCK_CONTROL__TIMEOUT_CONTROL__SOFTWARE_RESET, val); > > > > - sdhci_write32(&host->sdhci, ESDHC_DMA_SYSCTL, ESDHC_SYSCTL_DMA_SNOOP); > > + sdhci_write32(&host->sdhci, ESDHC_DMA_SYSCTL, > > + ESDHC_SYSCTL_DMA_SNOOP | ESDHC_SYSCTL_PERIPHERAL_CLK_SEL); > > This affects ls1046, too. Is this intended? It's not intended to do this in this patch, nevertheless I beliebe it's the right thing to do also for LS1046a Sascha -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
