On 9 September 2016 at 15:30, Tummala, Sahitya <stummala@xxxxxxxxxxxxxx> wrote: > Hi Pramod, <snip> >> + ret = clk_prepare_enable(msm_host->clk); >> + if (ret) { >> + dev_err(dev, "clk_enable failed: %d\n", ret); > > A minor comment - Both error prints related to clock enable are same. Better > to print the clock name as well to know which clock enable got failed. > Thanks Sahitya for comments. Will take care of this in next version. >> + return 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