From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dmitry Torokhov Subject: Re: [PATCH 3/3] wm97xx-core: Support use as a wakeup source Date: Mon, 14 Apr 2008 16:34:13 -0400 Message-ID: <20080414163033.ZZRA012@mailhub.coreip.homeip.net> References: <20080414173841.GA2213@rakim.wolfsonmicro.main> <1208194775-4293-1-git-send-email-broonie@opensource.wolfsonmicro.com> <1208194775-4293-2-git-send-email-broonie@opensource.wolfsonmicro.com> <1208194775-4293-3-git-send-email-broonie@opensource.wolfsonmicro.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from yw-out-2324.google.com ([74.125.46.30]:29396 "EHLO yw-out-2324.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752003AbYDNUeW (ORCPT ); Mon, 14 Apr 2008 16:34:22 -0400 Received: by yw-out-2324.google.com with SMTP id 5so786144ywb.1 for ; Mon, 14 Apr 2008 13:34:17 -0700 (PDT) Content-Disposition: inline In-Reply-To: <1208194775-4293-3-git-send-email-broonie@opensource.wolfsonmicro.com> Sender: linux-input-owner@vger.kernel.org List-Id: linux-input@vger.kernel.org To: Mark Brown Cc: Andrew Morton , Jiri Kosina , linux-input@vger.kernel.org, linux-kernel@vger.kernel.org, David Brownell Hi Mark, On Mon, Apr 14, 2008 at 06:39:35PM +0100, Mark Brown wrote: > The PENDOWN signal from the WM97xx touch screen controllers can be used > to generate a wakeup event when the system is suspended. Provide a new > machine configuration option 'suspend_mode' allowing machine drivers to > enable this. If no suspend_mode is provided then the touch panel will be > powered down when the system is suspended. > Hmm, another unique module option... How about using device_can_wakeup() infrastructure that we already have. I'm adding David Brownell to CC since I believe he's doing a lot of similar work. > Signed-off-by: Mark Brown > --- > drivers/input/touchscreen/wm97xx-core.c | 22 ++++++++++++++++++++++ > include/linux/wm97xx.h | 3 +++ > 2 files changed, 25 insertions(+), 0 deletions(-) > > diff --git a/drivers/input/touchscreen/wm97xx-core.c b/drivers/input/touchscreen/wm97xx-core.c > index fec07c2..e1a1ca0 100644 > --- a/drivers/input/touchscreen/wm97xx-core.c > +++ b/drivers/input/touchscreen/wm97xx-core.c > @@ -689,10 +689,32 @@ static int wm97xx_remove(struct device *dev) > static int wm97xx_suspend(struct device *dev, pm_message_t state) > { > struct wm97xx *wm = dev_get_drvdata(dev); > + u16 reg; > + int suspend_mode; > + > + if (wm->mach_ops) > + suspend_mode = wm->mach_ops->suspend_mode; > + else > + suspend_mode = 0; > > if (wm->input_dev->users) > cancel_delayed_work_sync(&wm->ts_reader); > > + /* Power down the digitiser (bypassing the cache for resume) */ > + reg = wm97xx_reg_read(wm, AC97_WM97XX_DIGITISER2); > + reg &= ~WM97XX_PRP_DET_DIG; > + if (wm->input_dev->users && suspend_mode) > + reg |= suspend_mode; > + wm->ac97->bus->ops->write(wm->ac97, AC97_WM97XX_DIGITISER2, reg); > + > + /* WM9713 has an additional power bit - turn it off if there > + * are no users or if suspend mode is zero. */ > + if (wm->id == WM9713_ID2 && > + (!wm->input_dev->users || !suspend_mode)) { > + reg = wm97xx_reg_read(wm, AC97_EXTENDED_MID) | 0x8000; > + wm97xx_reg_write(wm, AC97_EXTENDED_MID, reg); > + } > + > return 0; > } > > diff --git a/include/linux/wm97xx.h b/include/linux/wm97xx.h > index ed01c7d..2082833 100644 > --- a/include/linux/wm97xx.h > +++ b/include/linux/wm97xx.h > @@ -258,6 +258,9 @@ struct wm97xx_mach_ops { > /* pre and post sample - can be used to minimise any analog noise */ > void (*pre_sample) (int); /* function to run before sampling */ > void (*post_sample) (int); /* function to run after sampling */ > + > + /* WM97XX_PRP value to configure while system is suspended */ > + u16 suspend_mode; > }; > > struct wm97xx { > -- > 1.5.5 > -- Dmitry