Re: [PATCH v2 2/3] rtc: mediatek: Add MT6397 RTC driver

2015-03-31 Thread Andrew Morton
On Mon, 30 Mar 2015 16:15:50 +0800 Eddie Huang wrote: > > > +out_rtc: > > > + rtc_device_unregister(rtc->rtc_dev); > > > > This is wrong. Whenever you jump to this label the RTC device has not > > been registered yet. > > Oops, will fix in next round. Please ensure that Uwe's review

Re: [PATCH v2 2/3] rtc: mediatek: Add MT6397 RTC driver

2015-03-31 Thread Tomasz Figa
Hi Eddie, Please see my response inline. On Tue, Mar 31, 2015 at 6:44 PM, Eddie Huang wrote: [snip] >> > + ret = mtk_rtc_read(rtc, RTC_BBPU, ); >> > + if (ret < 0) >> > + goto exit; >> > + >> > + while (data & RTC_BBPU_CBUSY) { >> > + cpu_relax();

Re: [PATCH v2 2/3] rtc: mediatek: Add MT6397 RTC driver

2015-03-31 Thread Eddie Huang
Hi Tomasz, On Mon, 2015-03-30 at 16:41 +0900, Tomasz Figa wrote: > Hi Eddie, > > Please see my comments inline. > > On Wed, Mar 18, 2015 at 2:45 PM, Eddie Huang wrote: > > From: Tianping Fang > > > > Add Mediatek MT6397 RTC driver > > [snip] > > > +#define RTC_BBPU 0x > >

Re: [PATCH v2 2/3] rtc: mediatek: Add MT6397 RTC driver

2015-03-31 Thread Tomasz Figa
Hi Eddie, Please see my response inline. On Tue, Mar 31, 2015 at 6:44 PM, Eddie Huang eddie.hu...@mediatek.com wrote: [snip] + ret = mtk_rtc_read(rtc, RTC_BBPU, data); + if (ret 0) + goto exit; + + while (data RTC_BBPU_CBUSY) { +

Re: [PATCH v2 2/3] rtc: mediatek: Add MT6397 RTC driver

2015-03-31 Thread Eddie Huang
Hi Tomasz, On Mon, 2015-03-30 at 16:41 +0900, Tomasz Figa wrote: Hi Eddie, Please see my comments inline. On Wed, Mar 18, 2015 at 2:45 PM, Eddie Huang eddie.hu...@mediatek.com wrote: From: Tianping Fang tianping.f...@mediatek.com Add Mediatek MT6397 RTC driver [snip] +#define

Re: [PATCH v2 2/3] rtc: mediatek: Add MT6397 RTC driver

2015-03-31 Thread Andrew Morton
On Mon, 30 Mar 2015 16:15:50 +0800 Eddie Huang eddie.hu...@mediatek.com wrote: +out_rtc: + rtc_device_unregister(rtc-rtc_dev); This is wrong. Whenever you jump to this label the RTC device has not been registered yet. Oops, will fix in next round. Please ensure that Uwe's

Re: [PATCH v2 2/3] rtc: mediatek: Add MT6397 RTC driver

2015-03-30 Thread Eddie Huang
Hi Dmitry, On Fri, 2015-03-20 at 22:25 -0700, Dmitry Torokhov wrote: > Hi Eddie, > > > On Tue, Mar 17, 2015 at 10:45 PM, Eddie Huang > wrote: > > +static int mtk_rtc_probe(struct platform_device *pdev) > > +{ > > + struct resource *res; > > + struct mt6397_chip *mt6397_chip =

Re: [PATCH v2 2/3] rtc: mediatek: Add MT6397 RTC driver

2015-03-30 Thread Tomasz Figa
Hi Eddie, Please see my comments inline. On Wed, Mar 18, 2015 at 2:45 PM, Eddie Huang wrote: > From: Tianping Fang > > Add Mediatek MT6397 RTC driver [snip] > +#define RTC_BBPU 0x > +#define RTC_WRTGR 0x003c > +#define RTC_IRQ_EN 0x0004 > +#define

Re: [PATCH v2 2/3] rtc: mediatek: Add MT6397 RTC driver

2015-03-30 Thread Tomasz Figa
Hi Eddie, Please see my comments inline. On Wed, Mar 18, 2015 at 2:45 PM, Eddie Huang eddie.hu...@mediatek.com wrote: From: Tianping Fang tianping.f...@mediatek.com Add Mediatek MT6397 RTC driver [snip] +#define RTC_BBPU 0x +#define RTC_WRTGR 0x003c

Re: [PATCH v2 2/3] rtc: mediatek: Add MT6397 RTC driver

2015-03-30 Thread Eddie Huang
Hi Dmitry, On Fri, 2015-03-20 at 22:25 -0700, Dmitry Torokhov wrote: Hi Eddie, On Tue, Mar 17, 2015 at 10:45 PM, Eddie Huang eddie.hu...@mediatek.com wrote: +static int mtk_rtc_probe(struct platform_device *pdev) +{ + struct resource *res; + struct mt6397_chip

Re: [PATCH v2 2/3] rtc: mediatek: Add MT6397 RTC driver

2015-03-20 Thread Dmitry Torokhov
Hi Eddie, On Tue, Mar 17, 2015 at 10:45 PM, Eddie Huang wrote: > +static int mtk_rtc_probe(struct platform_device *pdev) > +{ > + struct resource *res; > + struct mt6397_chip *mt6397_chip = dev_get_drvdata(pdev->dev.parent); > + struct mt6397_rtc *rtc; > + int ret = 0; >

Re: [PATCH v2 2/3] rtc: mediatek: Add MT6397 RTC driver

2015-03-20 Thread Dmitry Torokhov
Hi Eddie, On Tue, Mar 17, 2015 at 10:45 PM, Eddie Huang eddie.hu...@mediatek.com wrote: +static int mtk_rtc_probe(struct platform_device *pdev) +{ + struct resource *res; + struct mt6397_chip *mt6397_chip = dev_get_drvdata(pdev-dev.parent); + struct mt6397_rtc *rtc; +

[PATCH v2 2/3] rtc: mediatek: Add MT6397 RTC driver

2015-03-17 Thread Eddie Huang
From: Tianping Fang Add Mediatek MT6397 RTC driver Signed-off-by: Tianping Fang Signed-off-by: Eddie Huang --- drivers/rtc/Kconfig | 10 ++ drivers/rtc/Makefile | 1 + drivers/rtc/rtc-mt6397.c | 454 +++ 3 files changed, 465

[PATCH v2 2/3] rtc: mediatek: Add MT6397 RTC driver

2015-03-17 Thread Eddie Huang
From: Tianping Fang tianping.f...@mediatek.com Add Mediatek MT6397 RTC driver Signed-off-by: Tianping Fang tianping.f...@mediatek.com Signed-off-by: Eddie Huang eddie.hu...@mediatek.com --- drivers/rtc/Kconfig | 10 ++ drivers/rtc/Makefile | 1 + drivers/rtc/rtc-mt6397.c | 454