On 07/04/2016 01:03 PM, Alim Akhtar wrote: > At the end of s3c_rtc_probe(), s3c_rtc_disable_clk() being called with rtc > clock already disabled, which looks extra and unnecessary call. > Lets clean it up.
Does not look right. Till that place, the clocks are enabled. Then s3c_rtc_setaie() is called which expects that clocks are disabled... otherwise counters get mixed. So overall this looks like wrong approach unless I am missing something? Best regards, Krzysztof > > Signed-off-by: Alim Akhtar <alim.akh...@samsung.com> > --- > drivers/rtc/rtc-s3c.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/drivers/rtc/rtc-s3c.c b/drivers/rtc/rtc-s3c.c > index d01ad7e..b083840 100644 > --- a/drivers/rtc/rtc-s3c.c > +++ b/drivers/rtc/rtc-s3c.c > @@ -577,8 +577,6 @@ static int s3c_rtc_probe(struct platform_device *pdev) > > s3c_rtc_setfreq(info, 1); > > - s3c_rtc_disable_clk(info); > - > return 0; > > err_nortc:
