[U-Boot] [PATCH 1/7] mmc: atmel_sdhci: fix uninitialized pointer deref on probe
Eugen.Hristev at microchip.com
Eugen.Hristev at microchip.com
Wed Aug 7 10:02:04 UTC 2019
Tested-by: Eugen Hristev <eugen.hristev at microchip.com>
This has to go as soon as possible into the tree.
The at91 boards do not boot without this. The commit checking for
sdhci-caps-mask introduces a crash at this point.
I can take it through atmel tree if needed.
Thanks,
Eugen
On 06.08.2019 05:47, Peng Fan wrote:
>
> Commit 3d296365e4e8 ("mmc: sdhci: Add support for
> sdhci-caps-mask") sdhci_setup_cfg() expects a valid sdhci_host mmc
> field.
>
> Fixes: 3d296365e4e8 ("mmc: sdhci: Add support for sdhci-caps-mask")
> Cc: Faiz Abbas <faiz_abbas at ti.com>
> Cc: Wenyou Yang <wenyou.yang at atmel.com>
> Signed-off-by: Peng Fan <peng.fan at nxp.com>
> ---
> drivers/mmc/atmel_sdhci.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/mmc/atmel_sdhci.c b/drivers/mmc/atmel_sdhci.c
> index 4be47ba75e..d930ed8da0 100644
> --- a/drivers/mmc/atmel_sdhci.c
> +++ b/drivers/mmc/atmel_sdhci.c
> @@ -88,13 +88,13 @@ static int atmel_sdhci_probe(struct udevice *dev)
> return -EINVAL;
>
> host->max_clk = max_clk;
> + host->mmc = &plat->mmc;
> + host->mmc->dev = dev;
>
> ret = sdhci_setup_cfg(&plat->cfg, host, 0, ATMEL_SDHC_MIN_FREQ);
> if (ret)
> return ret;
>
> - host->mmc = &plat->mmc;
> - host->mmc->dev = dev;
> host->mmc->priv = host;
> upriv->mmc = host->mmc;
>
>
More information about the U-Boot
mailing list