Messages in this thread | | | Subject | Re: [RFC PATCH 1/2] rtc: s3c: Remove unnecessary call to disable already disabled clock | From | Krzysztof Kozlowski <> | Date | Tue, 05 Jul 2016 09:18:00 +0200 |
| |
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.akhtar@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:
| |