From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751849Ab2LJFot (ORCPT ); Mon, 10 Dec 2012 00:44:49 -0500 Received: from va3ehsobe003.messaging.microsoft.com ([216.32.180.13]:9578 "EHLO va3outboundpool.messaging.microsoft.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750815Ab2LJFos (ORCPT ); Mon, 10 Dec 2012 00:44:48 -0500 X-Forefront-Antispam-Report: CIP:59.163.77.45;KIP:(null);UIP:(null);IPV:NLI;H:KCHJEXHC01.kpit.com;RD:59.163.77.45.static.vsnl.net.in;EFVD:NLI X-SpamScore: -3 X-BigFish: VPS-3(zz98dI9371I936eI1432Izz1de0h1202h1e76h1d1ah1d2ahzz8275bhz2dh2a8h668h839h93fhd24hd2bhf0ah1288h12a5h12a9h12bdh137ah139eh13b6h1441h1504h1537h162dh1631h1758hbe9i1155h) Subject: Re: [Patch v3 3/7] rtc: DA9055 RTC driver From: Ashish Jangam To: Andrew Morton CC: , , , , , Alessandro Zummo , shubhro In-Reply-To: <1354094680.10949.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> Content-Type: text/plain; charset="UTF-8" Date: Mon, 10 Dec 2012 11:12:29 +0530 Message-ID: <1355118149.4272.1.camel@dhruva> MIME-Version: 1.0 X-Mailer: Evolution 2.28.3 Content-Transfer-Encoding: 7bit X-Originating-IP: [10.10.38.47] X-OriginatorOrg: kpitcummins.com Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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.