> On 19.11.2019, at 12:04, Heiko Stuebner <he...@sntech.de> wrote: > > From: Heiko Stuebner <heiko.stueb...@theobroma-systems.com> > > Rockchips dwmmc controllers can't do dma to non-ddr addresses, > like for example the soc-internal sram but during boot parts of > TrustedFirmware need to be placed there from the read FIT image. > > So add handling for a u-boot,spl-fifo-mode to not put the mmc > controllers into fifo mode for all time. > > The regular fifo-mode property still takes precedent and only > if not set do we check for the spl-specific property. > > Suggested-by: Philipp Tomsich <philipp.toms...@theobroma-systems.com> > Signed-off-by: Heiko Stuebner <heiko.stueb...@theobroma-systems.com>
Signed-off-by: Philipp Tomsich <philipp.toms...@theobroma-systems.com> > --- > drivers/mmc/rockchip_dw_mmc.c | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/drivers/mmc/rockchip_dw_mmc.c b/drivers/mmc/rockchip_dw_mmc.c > index b2a1201631..a0e1be8794 100644 > --- a/drivers/mmc/rockchip_dw_mmc.c > +++ b/drivers/mmc/rockchip_dw_mmc.c > @@ -72,6 +72,11 @@ static int rockchip_dwmmc_ofdata_to_platdata(struct > udevice *dev) > return -EINVAL; > priv->fifo_mode = dev_read_bool(dev, "fifo-mode"); > > +#ifdef CONFIG_SPL_BUILD > + if (!priv->fifo_mode) > + priv->fifo_mode = dev_read_bool(dev, "u-boot,spl-fifo-mode"); > +#endif Nitpick: I would have used an “|=" instead of the "if(…)”. > + > /* > * 'clock-freq-min-max' is deprecated > * (see > https://github.com/torvalds/linux/commit/b023030f10573de738bbe8df63d43acab64c9f7b) > -- > 2.24.0 > _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
