[U-Boot] [PATCH] MMC: Do not write to mmc_uclass_priv if it was not allocated

Simon Glass sjg at chromium.org
Wed Jun 29 05:27:49 CEST 2016


Hi Matt,

On 25 June 2016 at 14:57, Matt Corallo <linux at bluematt.me> wrote:
> Signed-off-by: Matt Corallo <git at bluematt.me>
> ---
>  drivers/mmc/mmc.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/mmc/mmc.c b/drivers/mmc/mmc.c
> index aabfc71..eba20f0 100644
> --- a/drivers/mmc/mmc.c
> +++ b/drivers/mmc/mmc.c
> @@ -1734,7 +1734,8 @@ int mmc_init(struct mmc *mmc)
>  #ifdef CONFIG_DM_MMC
>         struct mmc_uclass_priv *upriv = dev_get_uclass_priv(mmc->dev);
>
> -       upriv->mmc = mmc;
> +       if (upriv)
> +               upriv->mmc = mmc;
>  #endif
>         if (mmc->has_init)
>                 return 0;
> --
> 2.1.4

Can you please add a commit message explaining why this is needed and
what it fixes?  How can mmc_init() be called before the MMC device is
there? Is this related to this patch?

http://patchwork.ozlabs.org/patch/640735/

Regards,
Simon


More information about the U-Boot mailing list