Hi Samuel, Hi On Mon, Oct 01, 2012 at 04:52:02, Samuel Ortiz wrote: > Hi Jonathan, > > On Sat, Sep 29, 2012 at 10:50:19AM +0100, Jonathan Cameron wrote: > > On 09/26/2012 11:09 AM, Patil, Rachna wrote: > > > Hi Shubhrajyoti, > > > > > > On Wed, Sep 26, 2012 at 12:10:51, Datta, Shubhrajyoti wrote: > > >> On Wednesday 26 September 2012 10:50 AM, Patil, Rachna wrote: > > >>> This patch adds support for suspend/resume of TSC/ADC MFDevice. > > >> this should be merged with the patch adding support else we may end > > >> up in a case where patch a does the runtime calls and the call back > > >> handlers added later. > > > > > > I am adding both the runtime calls and the handlers in this patch. > > > I think maintaining this as a separate patch in better in terms of > > > readability as well. > > I would prefer this merged into patch 4 as it's a simple addition to a > > driver that should arguably have been there in the first place. > I agree. And I would even go as far as saying that having all the MFD parts > from this patchset merged into patch #2 would make sense. > Rachna, could you please re-work this patchset in such way, and adress my > comments on patch #2 ? I could then take the MFD parts and then Jonathan and > Dmitry take their parts. There are build time dependencies, but neither the > input nor the ADC driver could be actually built as long as the MFD one is > not merged (provided that the subdevices driver comes with the right Kconfig > dependecy).
I will re-work the patchset and address all the review comments accordingly. Thanks & Regards, Rachna > > Cheers, > Samuel. > > -- > Intel Open Source Technology Centre > http://oss.intel.com/ > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
