On Thu, Oct 10, 2013 at 03:24:46PM -0700, Andrew Morton wrote: > On Wed, 9 Oct 2013 17:59:21 +0530 Laxman Dewangan <ldewangan@xxxxxxxxxx> wrote: > > +config RTC_DRV_AS3722 > > + tristate "ams AS3722 RTC driver" > > + depends on MFD_AS3722 > ... the rtc patch depends on an mfd patch, so everyone ends up sitting > around looking at everyone else, wondering who will merge it all. With the MFD ones providing you trust the interfaces with the core are going to be OK there's no problem to just merge them via individual trees - the dependency on the MFD will prevent bisection problems. > > +#ifdef CONFIG_PM > > Most (but not all!) rtc drivers use CONFIG_PM_SLEEP. People have madly > mucked with CONFIG_PM* and I don't know the difference and I don't know > what's going on and I don't know of a convenient place to go to find > out. If you work it out, please be sure to tell me! But as soon as I > figure it out I'm sure they'll go and madly muck with it again. The two effective options for this stuff are PM_SLEEP (for system suspend) and PM_RUNTIME (for runtime suspend which RTCs are unlikely to use I guess). PM mostly just enables selection of the other two.
Attachment:
signature.asc
Description: Digital signature
