sdhci_get_of_property may failed while probing, so we check the return value here. Cc: Barry Song <baohua@xxxxxxxxxx> Cc: Weijun Yang <york.yang@xxxxxxx> Signed-off-by: Shawn Lin <shawn.lin@xxxxxxxxxxxxxx> --- drivers/mmc/host/sdhci-sirf.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/mmc/host/sdhci-sirf.c b/drivers/mmc/host/sdhci-sirf.c index 34866f6..3f8499f 100644 --- a/drivers/mmc/host/sdhci-sirf.c +++ b/drivers/mmc/host/sdhci-sirf.c @@ -195,7 +195,9 @@ static int sdhci_sirf_probe(struct platform_device *pdev) priv = sdhci_pltfm_priv(pltfm_host); priv->gpio_cd = gpio_cd; - sdhci_get_of_property(pdev); + ret = sdhci_get_of_property(pdev); + if (ret) + return ret; ret = clk_prepare_enable(pltfm_host->clk); if (ret) -- 2.3.7 -- 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