Acked-by: Jaehoon Chung <jh80.chung@xxxxxxxxxxx> Best Regards, Jaehoon Chung On 12/18/2014 06:41 PM, Ulf Hansson wrote: > Since mmc_of_parse() may fail, let's deal with it and thus do proper > error handling. > > Cc: Jaehoon Chung <jh80.chung@xxxxxxxxxxx> > Signed-off-by: Ulf Hansson <ulf.hansson@xxxxxxxxxx> > --- > drivers/mmc/host/sdhci-s3c.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/drivers/mmc/host/sdhci-s3c.c b/drivers/mmc/host/sdhci-s3c.c > index c45b893..a282822 100644 > --- a/drivers/mmc/host/sdhci-s3c.c > +++ b/drivers/mmc/host/sdhci-s3c.c > @@ -607,7 +607,9 @@ static int sdhci_s3c_probe(struct platform_device *pdev) > pm_runtime_use_autosuspend(&pdev->dev); > pm_suspend_ignore_children(&pdev->dev, 1); > > - mmc_of_parse(host->mmc); > + ret = mmc_of_parse(host->mmc); > + if (ret) > + goto err_req_regs; > > ret = sdhci_add_host(host); > if (ret) { > -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html