On Tue, Jul 02, 2013 at 05:37:43, Kevin Hilman wrote: > Hebbar Gururaja <gururaja.hebbar@xxxxxx> writes: > > > Since now rtc-omap driver itself calls deice_init_wakeup(dev, true), > > duplicate call from the rtc device registration can be removed. > > > > This is basically a partial revert of the prev commit > > > > commit 75c99bb0006ee065b4e2995078d779418b0fab54 > > Author: Sekhar Nori <nsekhar@xxxxxx> > > > > davinci: da8xx/omap-l1: mark RTC as a wakeup source > > > > Signed-off-by: Hebbar Gururaja <gururaja.hebbar@xxxxxx> > > Cc: Sekhar Nori <nsekhar@xxxxxx> > > Cc: Kevin Hilman <khilman@xxxxxxxxxx> > > Cc: Russell King <linux@xxxxxxxxxxxxxxxx> > > > > --- > > :100644 100644 bf57252... 85a900c... M arch/arm/mach-davinci/devices-da8xx.c > > arch/arm/mach-davinci/devices-da8xx.c | 9 +-------- > > 1 file changed, 1 insertion(+), 8 deletions(-) > > > > diff --git a/arch/arm/mach-davinci/devices-da8xx.c b/arch/arm/mach-davinci/devices-da8xx.c > > index bf57252..85a900c 100644 > > --- a/arch/arm/mach-davinci/devices-da8xx.c > > +++ b/arch/arm/mach-davinci/devices-da8xx.c > > @@ -827,14 +827,7 @@ static struct platform_device da8xx_rtc_device = { > > > > int da8xx_register_rtc(void) > > { > > - int ret; > > - > > - ret = platform_device_register(&da8xx_rtc_device); > > - if (!ret) > > - /* Atleast on DA850, RTC is a wakeup source */ > > - device_init_wakeup(&da8xx_rtc_device.dev, true); > > - > > - return ret; > > + return platform_device_register(&da8xx_rtc_device); > > nit: extra space between 'return' and 'platform_' Thanks for the review. V2 will soon follow. > > > } > > > > static void __iomem *da8xx_ddr2_ctlr_base; > > Otherwise, > > Acked-by: Kevin Hilman <khilman@xxxxxxxxxx> > > Regards, Gururaja -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html