From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755294Ab2LLVLM (ORCPT ); Wed, 12 Dec 2012 16:11:12 -0500 Received: from mail.linuxfoundation.org ([140.211.169.12]:55681 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754076Ab2LLVLK (ORCPT ); Wed, 12 Dec 2012 16:11:10 -0500 Date: Wed, 12 Dec 2012 13:11:08 -0800 From: Andrew Morton To: Ashish Jangam Cc: , , , , , Alessandro Zummo , shubhro Subject: Re: [Patch v3 3/7] rtc: DA9055 RTC driver Message-Id: <20121212131108.1e158ae2.akpm@linux-foundation.org> In-Reply-To: <1355118149.4272.1.camel@dhruva> References: <1349952025.9228.3.camel@dhruva> <1350986593.23353.3.camel@dhruva> <1353665463.1760.3000.camel@dhruva> <20121127142358.9a4bc20d.akpm@linux-foundation.org> <1354094680.10949.1.camel@dhruva> <1355118149.4272.1.camel@dhruva> X-Mailer: Sylpheed 3.0.2 (GTK+ 2.20.1; x86_64-pc-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, 10 Dec 2012 11:12:29 +0530 Ashish Jangam wrote: > On Wed, 2012-11-28 at 14:54 +0530, Ashish Jangam wrote: > > On Tue, 2012-11-27 at 14:23 -0800, Andrew Morton wrote: > > > On Fri, 23 Nov 2012 15:41:03 +0530 > > > Ashish Jangam wrote: > > > > > > > On Tue, 2012-10-23 at 15:33 +0530, Ashish Jangam wrote: > > > > > Does this patch looks good? > > > > > On Thu, 2012-10-11 at 16:10 +0530, Ashish Jangam wrote: > > > > > > This is the RTC patch for the DA9055 PMIC. This patch has got dependency on > > > > > > the DA9055 MFD core. > > > > > > > > > > > > This patch is functionally tested on Samsung SMDKV6410. > > > > > > "shubhro " made a comment, but it was ignored: > > > > > > : On Thu, 11 Oct 2012 16:53:34 +0530 > > > : shubhro wrote: > > > : > > > : > On Thursday 11 October 2012 04:10 PM, Ashish Jangam wrote: > > > : > > +#define da9055_rtc_freeze NULL > > > : > > +#endif > > > : > > + > > > : > > +static const struct dev_pm_ops da9055_rtc_pm_ops = { > > > : > > + .suspend = da9055_rtc_suspend, > > > : > > + .resume = da9055_rtc_resume, > > > : > > + > > > : > > + .freeze = da9055_rtc_freeze, > > > : > > + .thaw = da9055_rtc_resume, > > > : > > + .restore = da9055_rtc_resume, > > > : > > + > > > : > > + .poweroff = da9055_rtc_suspend, > > > : > > +}; > > > : > You may want to use simple dev pm ops > > To support generic PM ops these additional ops were supported. > Are there any further comments on this patch. Is the patch usable against current mainline? "This patch has got dependency on the DA9055 MFD core". What is the status of that? If the patch is ready to go then please refresh it, retest and resend?