Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755661Ab2JCGti (ORCPT ); Wed, 3 Oct 2012 02:49:38 -0400 Received: from comal.ext.ti.com ([198.47.26.152]:58077 "EHLO comal.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755611Ab2JCGtg convert rfc822-to-8bit (ORCPT ); Wed, 3 Oct 2012 02:49:36 -0400 From: "Patil, Rachna" To: Samuel Ortiz , Jonathan Cameron CC: "Datta, Shubhrajyoti" , "linux-kernel@vger.kernel.org" , "linux-input@vger.kernel.org" , "linux-iio@vger.kernel.org" , Dmitry Torokhov , Dmitry Torokhov , Jonathan Cameron Subject: RE: [PATCH v4 5/5] MFD: ti_tscadc: add suspend/resume functionality Thread-Topic: [PATCH v4 5/5] MFD: ti_tscadc: add suspend/resume functionality Thread-Index: AQHNm6lzpqtiuVnbHEGO6q7HwPN1EJeb0DaAgACUJzCABFfIgIACdR8AgAP8asA= Date: Wed, 3 Oct 2012 06:49:19 +0000 Message-ID: <4CE347531D4CA947960AF71FF095B9323E959D06@DBDE01.ent.ti.com> References: <1348636814-14129-1-git-send-email-rachna@ti.com> <1348636814-14129-6-git-send-email-rachna@ti.com> <5062A373.4000707@ti.com> <4CE347531D4CA947960AF71FF095B9323E95738D@DBDE01.ent.ti.com> <5066C45B.3000500@kernel.org> <20120930232202.GF12231@sortiz-mobl> In-Reply-To: <20120930232202.GF12231@sortiz-mobl> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [172.24.170.142] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 8BIT MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1925 Lines: 43 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 majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/