diff mbox

panasonic-laptop: use correct key names for sleep states in driver

Message ID 1236164306.3936.20.camel@hughsie-work.lan (mailing list archive)
State Not Applicable, archived
Headers show

Commit Message

Richard Hughes March 4, 2009, 10:58 a.m. UTC
On Wed, 2009-03-04 at 10:54 +0000, Richard Hughes wrote:
> On Mon, 2009-03-02 at 08:24 -0300, Henrique de Moraes Holschuh wrote:
> > FWIW, I think it is a good idea, and I'd take patches for
> > thinkpad-acpi.
> 
> Patch attached for thinkpad-acpi. KEY_HIBERNATE is already in
> linux-next, but has not yet been pushed to master. Please review,
> thanks.

Patch also attached for panasonic-laptop. Harald, can you please review
this. Thanks.

Richard

Comments

Harald Welte March 4, 2009, 11:58 p.m. UTC | #1
On Wed, Mar 04, 2009 at 10:58:26AM +0000, Richard Hughes wrote:
> On Wed, 2009-03-04 at 10:54 +0000, Richard Hughes wrote:
> > On Mon, 2009-03-02 at 08:24 -0300, Henrique de Moraes Holschuh wrote:
> > > FWIW, I think it is a good idea, and I'd take patches for
> > > thinkpad-acpi.
> > 
> > Patch attached for thinkpad-acpi. KEY_HIBERNATE is already in
> > linux-next, but has not yet been pushed to master. Please review,
> > thanks.
> 
> Patch also attached for panasonic-laptop. Harald, can you please review
> this. Thanks.

Acked-by: Harald Welte <laforge@gnumonks.org>

(I've just started three weeks vacation and won't be able to respond to mails
until March 26th)
Richard Hughes March 5, 2009, 9:09 a.m. UTC | #2
On Thu, 2009-03-05 at 00:58 +0100, Harald Welte wrote:
> On Wed, Mar 04, 2009 at 10:58:26AM +0000, Richard Hughes wrote:
> > On Wed, 2009-03-04 at 10:54 +0000, Richard Hughes wrote:
> > > On Mon, 2009-03-02 at 08:24 -0300, Henrique de Moraes Holschuh wrote:
> > > > FWIW, I think it is a good idea, and I'd take patches for
> > > > thinkpad-acpi.
> > > 
> > > Patch attached for thinkpad-acpi. KEY_HIBERNATE is already in
> > > linux-next, but has not yet been pushed to master. Please review,
> > > thanks.
> > 
> > Patch also attached for panasonic-laptop. Harald, can you please review
> > this. Thanks.
> 
> Acked-by: Harald Welte <laforge@gnumonks.org>

Dmitry, could you pull this and the thinkpad acpi patch into linux-next
please. Thanks.

Richard.


--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

From 11df58e5c55e6286f03b29b6b9820ed4b13095db Mon Sep 17 00:00:00 2001
From: Richard Hughes <richard@hughsie.com>
Date: Wed, 4 Mar 2009 10:57:36 +0000
Subject: panasonic-laptop: use correct key names for sleep keys in driver

Use the new key KEY_HIBERNATE and correct the use of KEY_SUSPEND.

KEY_HIBERNATE is used when the decal on the key indicates sleep to disk,
KEY_SUSPEND is used when the decal indicates sleep to memory, and
KEY_SLEEP is used when the sleep type is unknown or unspecified

We've already converted Xorg, HAL, and most of userspace to use the same names.

With this patch, the sleep keys can be labeled consistently all the way
through the stack. More patches to other device drivers are to follow.

Signed-off-by: Richard Hughes <hughsient@gmail.com>
---
 drivers/platform/x86/panasonic-laptop.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/platform/x86/panasonic-laptop.c b/drivers/platform/x86/panasonic-laptop.c
index c47a44d..63a1828 100644
--- a/drivers/platform/x86/panasonic-laptop.c
+++ b/drivers/platform/x86/panasonic-laptop.c
@@ -205,10 +205,10 @@  static const int initial_keymap[KEYMAP_SIZE] = {
 	/*  4 */ KEY_MUTE,
 	/*  5 */ KEY_VOLUMEDOWN,
 	/*  6 */ KEY_VOLUMEUP,
-	/*  7 */ KEY_SLEEP,
+	/*  7 */ KEY_SUSPEND,
 	/*  8 */ KEY_PROG1, /* Change CPU boost */
 	/*  9 */ KEY_BATTERY,
-	/* 10 */ KEY_SUSPEND,
+	/* 10 */ KEY_HIBERNATE,
 };
 
 struct pcc_acpi {
-- 
1.6.1.3