lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Fri, 14 Sep 2012 05:16:37 +0000 From: "Patil, Rachna" <rachna@...com> To: Venu Byravarasu <vbyravarasu@...dia.com>, Lars-Peter Clausen <lars@...afoo.de> CC: "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>, "linux-input@...r.kernel.org" <linux-input@...r.kernel.org>, "linux-iio@...r.kernel.org" <linux-iio@...r.kernel.org>, Samuel Ortiz <sameo@...ux.intel.com>, Dmitry Torokhov <dmitry.torokhov@...il.com>, Dmitry Torokhov <dtor@...l.ru>, Jonathan Cameron <jic23@....ac.uk> Subject: RE: [PATCH v3 5/5] MFD: ti_tscadc: add suspend/resume functionality On Fri, Sep 14, 2012 at 10:39:20, Venu Byravarasu wrote: > > -----Original Message----- > > From: linux-kernel-owner@...r.kernel.org [mailto:linux-kernel- > > owner@...r.kernel.org] On Behalf Of Patil, Rachna > > Sent: Friday, September 14, 2012 10:29 AM > > To: Lars-Peter Clausen > > Cc: linux-kernel@...r.kernel.org; linux-input@...r.kernel.org; linux- > > iio@...r.kernel.org; Samuel Ortiz; Dmitry Torokhov; Dmitry Torokhov; > > Jonathan Cameron > > Subject: RE: [PATCH v3 5/5] MFD: ti_tscadc: add suspend/resume > > functionality > > > > On Thu, Sep 13, 2012 at 18:31:35, Lars-Peter Clausen wrote: > > > On 09/13/2012 12:40 PM, Patil, Rachna wrote: > > > > This patch adds support for suspend/resume of TSC/ADC MFDevice. > > > > > > > > Signed-off-by: Patil, Rachna <rachna@...com> > > > > --- > > > > Changes in v2: > > > > Added this patch newly in this patch series. > > > > > > > > Changes in v3: > > > > No changes. > > > > > > > > drivers/iio/adc/ti_adc.c | 32 > > ++++++++++++++++++++++++++++++++ > > > > drivers/input/touchscreen/ti_tsc.c | 33 > > +++++++++++++++++++++++++++++++++ > > > > drivers/mfd/ti_tscadc.c | 33 > > ++++++++++++++++++++++++++++++++- > > > > include/linux/mfd/ti_tscadc.h | 3 +++ > > > > 4 files changed, 100 insertions(+), 1 deletions(-) > > > > > > > > diff --git a/drivers/iio/adc/ti_adc.c b/drivers/iio/adc/ti_adc.c > > > > index 56f8af2..69f19f0 100644 > > > > --- a/drivers/iio/adc/ti_adc.c > > > > +++ b/drivers/iio/adc/ti_adc.c > > > > @@ -207,6 +207,36 @@ static int __devexit tiadc_remove(struct > > platform_device *pdev) > > > > return 0; > > > > } > > > > > > > [...] > > > > static struct platform_driver tiadc_driver = { > > > > .driver = { > > > > .name = "tiadc", > > > > @@ -214,6 +244,8 @@ static struct platform_driver tiadc_driver = { > > > > }, > > > > .probe = tiadc_probe, > > > > .remove = __devexit_p(tiadc_remove), > > > > + .suspend = adc_suspend, > > > > + .resume = adc_resume, > > > > }; > > > > > > > > > > > > > Using the suspend/resume callbacks is deprecated, please use > > dev_pm_ops. > > > Same comment applies to the other two drivers. > > > > Ok. I will make changes in all the 3 driver to use dev_pm_ops instead > > of suspend/resume callbacks. > > Probably you might need to protect suspend/resumes with CONFIG_PM. Yes, I will add that as well. Regards, Rachna -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists