On Thu, Aug 13, 2015 at 12:37:48PM +0530, Keerthy wrote: > Configure the clock source to either internal clock > or external clock based on the availability of the clocks. > External clock is preferred as it can be ticking during suspend. > > Signed-off-by: Keerthy <j-keerthy@xxxxxx> > @@ -627,6 +642,17 @@ static int omap_rtc_probe(struct platform_device *pdev) > if (reg != new_ctrl) > rtc_write(rtc, OMAP_RTC_CTRL_REG, new_ctrl); > > + /* > + * If we have the external clock then > + * Switch to external clock so we can keep ticking > + * acorss suspend > + */ You should fix up the comment (e.g. stray capitalisation and typo) as well. Perhaps reword as /* * If we have an external clock then switch to it so we can keep * ticking across suspend. */ Johan -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html