Re: [ibm-acpi-devel] [External] Re: [PATCH v4] platform/x86: thinkpad_acpi: lap or desk mode interface

2020-07-27 Thread Bastien Nocera
- Original Message - > Hello Bastien > > >-Original Message----- > >From: Bastien Nocera > > >- Original Message - > >> Newer Lenovo Thinkpad platforms have support to identify whether the > >> system is on-lap or n

Re: [ibm-acpi-devel] [PATCH v4] platform/x86: thinkpad_acpi: lap or desk mode interface

2020-07-26 Thread Bastien Nocera
ed by userspace for thermal mode and WWAN > functionality > > Co-developed-by: Nitin Joshi > Signed-off-by: Nitin Joshi > Reviewed-by: Sugumaran > Signed-off-by: Mark Pearson You can add my: Reviewed-by: Bastien Nocera Cheers __

Re: [ibm-acpi-devel] [PATCH 3/3] platform/x86: thinkpad_acpi: Map Clipping tool hotkey to KEY_SELECTIVE_SCREENSHOT

2020-07-17 Thread Bastien Nocera
hotkey to the new > KEY_SELECTIVE_SCREENSHOT key-code. > > Signed-off-by: Hans de Goede Added 5 years ago for the Carbon X1 2014, and finally getting a keycode ;) Reviewed-by: Bastien Nocera > --- > drivers/platform/x86/thinkpad_acpi.c | 2 +- > 1 file changed, 1 insertion

Re: [ibm-acpi-devel] [PATCH] thinkpad_acpi: support HKEY interface version 0x200

2016-04-19 Thread Bastien Nocera
On Tue, 2016-04-19 at 14:42 +0200, Bjørn Mork wrote: > Bastien Nocera writes: > > > > > From: Shuduo Sang > > > > The Thinkpad X1 Carbon 2nd generation (2014) ships with BIOS that > > will > > return HKEY interface version 0x200. It needs thinkpad-ac

[ibm-acpi-devel] [PATCH] thinkpad_acpi: support HKEY interface version 0x200

2016-04-19 Thread Bastien Nocera
From: Shuduo Sang The Thinkpad X1 Carbon 2nd generation (2014) ships with BIOS that will return HKEY interface version 0x200. It needs thinkpad-acpi support otherwise it will be routed to wrong branch and the hotkey mask will be wrong. https://bugzilla.kernel.org/show_bug.cgi?id=114731 Signed-o

Re: [ibm-acpi-devel] adaptive keyboard off after suspend on linux kernel 4.4 and 4.5, (thinkpad x1 gen2)

2016-02-04 Thread Bastien Nocera
EY interface: 0x200 > > Those are known harmless conditions, and HKEY 0x200 interface *is* the > adaptative keyboard version of the HKEY event interface, but it is only > "partially" known... > > And as you can see, it apparently still has some rough edges ;-) > >

Re: [ibm-acpi-devel] [patch 2/2] thinkpad_acpi: off by one in adaptive_keyboard_hotkey_notify_hotkey()

2015-03-11 Thread Bastien Nocera
084 ('thinkpad_acpi: Add support for more adaptive > kbd buttons') > Signed-off-by: Dan Carpenter Acked-By: Bastien Nocera > diff --git a/drivers/platform/x86/thinkpad_acpi.c > b/drivers/platform/x86/thinkpad_acpi.c > index 024861d..7769575 100644 > --- a/drivers/platf

Re: [ibm-acpi-devel] [patch 1/2] thinkpad_acpi: signedness bugs getting current_mode

2015-03-11 Thread Bastien Nocera
On Wed, 2015-03-11 at 12:34 +0300, Dan Carpenter wrote: > This needs to be signed for the error handling to work. Valid modes > are small positive integers. > > Fixes: b790ceeb0fd9 ('thinkpad_acpi: Add adaptive_kbd_mode sysfs > attr') Signed-off-by: Dan Carpenter

[ibm-acpi-devel] [PATCH] thinkpad_acpi: use DEVICE_ATTR_* macros

2015-03-02 Thread Bastien Nocera
Use the DEVICE_ATTR_* macros to reduce boiler plate. Signed-off-by: Bastien Nocera --- drivers/platform/x86/thinkpad_acpi.c | 72 ++-- 1 file changed, 20 insertions(+), 52 deletions(-) diff --git a/drivers/platform/x86/thinkpad_acpi.c b/drivers/platform/x86

[ibm-acpi-devel] [PATCH 1/4] thinkpad_acpi: Remember adaptive kbd presence

2015-03-02 Thread Bastien Nocera
Rather than checking on each suspend and resume whether the laptop has an adaptive keyboard, check when the driver is initialised. Signed-off-by: Bastien Nocera --- drivers/platform/x86/thinkpad_acpi.c | 37 ++-- 1 file changed, 18 insertions(+), 19 deletions

[ibm-acpi-devel] [PATCH 3/4] thinkpad_acpi: Add adaptive_kbd_mode sysfs attr

2015-03-02 Thread Bastien Nocera
Add a sysfs attribute to allow privileged users to change the keyboard mode. This could be used by desktop environments to change the keyboard mode depending on the application focused, as the Windows application does. Signed-off-by: Bastien Nocera --- Documentation/laptops/thinkpad-acpi.txt

[ibm-acpi-devel] [PATCH 4/4] thinkpad_acpi: Add support for more adaptive kbd buttons

2015-03-02 Thread Bastien Nocera
This commit adds new elements to the ThinkPad keymaps, and will send key events for keys for which an input.h declaration exists. Signed-off-by: Bastien Nocera --- drivers/platform/x86/thinkpad_acpi.c | 80 +++- 1 file changed, 79 insertions(+), 1 deletion

[ibm-acpi-devel] [PATCH 2/4] thinkpad_acpi: Factor out get/set adaptive kbd mode

2015-03-02 Thread Bastien Nocera
Move the getting/setting of the adaptive keyboard mode to separate functions, so that we can reuse them later through sysfs attributes. Signed-off-by: Bastien Nocera --- drivers/platform/x86/thinkpad_acpi.c | 61 ++-- 1 file changed, 38 insertions(+), 23

Re: [ibm-acpi-devel] [PATCH 1/7] thinkpad_acpi: Remember adaptive kbd presence

2015-02-26 Thread Bastien Nocera
On Wed, 2015-02-25 at 22:18 -0800, Darren Hart wrote: > On Fri, Feb 20, 2015 at 03:44:10PM +0100, Bastien Nocera wrote: > > Rather than checking on each suspend and resume whether the laptop > > has an adaptive keyboard, check when the driver is initialised. > > Bastien,

Re: [ibm-acpi-devel] [PATCH 5/7] input: Add keycodes used by Lenovo Carbon X1 2014

2015-02-25 Thread Bastien Nocera
On Fri, 2015-02-20 at 19:54 +0100, Bastien Nocera wrote: > Hey Dmitry, > > On Fri, 2015-02-20 at 10:40 -0800, Dmitry Torokhov wrote: > > On Fri, Feb 20, 2015 at 10:04:04AM -0500, Benjamin Tissoires wrote: > > > On Fri, Feb 20, 2015 at 9:44 AM, Bastien Nocera wrote: > &

Re: [ibm-acpi-devel] [PATCH 3/6] thinkpad_acpi: Add adaptive_kbd_mode sysfs attr

2015-02-23 Thread Bastien Nocera
On Mon, 2015-02-23 at 14:00 -0300, Henrique de Moraes Holschuh wrote: > On Fri, Feb 20, 2015, at 02:28, Darren Hart wrote: > > On Wed, Feb 18, 2015 at 09:53:44PM +0100, Bastien Nocera wrote: > > > > Commit message please. > > > > > Signed-off-by: Bastien Nocer

Re: [ibm-acpi-devel] [PATCH 5/6] input: Add keycodes used by Lenovo Carbon X1 2014

2015-02-23 Thread Bastien Nocera
On Mon, 2015-02-23 at 07:49 -0300, Henrique de Moraes Holschuh wrote: > On Mon, Feb 23, 2015, at 07:15, Bastien Nocera wrote: > > On Wed, 2015-02-18 at 21:34 -0200, Henrique de Moraes Holschuh wrote: > > > On Wed, 18 Feb 2015, Bastien Nocera wrote: > > > >

Re: [ibm-acpi-devel] [PATCH 7/7] thinkpad_acpi: use DEVICE_ATTR_* macros

2015-02-23 Thread Bastien Nocera
On Mon, 2015-02-23 at 07:25 -0300, Henrique de Moraes Holschuh wrote: > On Fri, Feb 20, 2015, at 11:44, Bastien Nocera wrote: > > Use the DEVICE_ATTR_RW macro to reduce boiler plate > > So far, so good, but you also used the DEVICE_ATTR_RO macro... > > > and move the attr

Re: [ibm-acpi-devel] [PATCH 5/6] input: Add keycodes used by Lenovo Carbon X1 2014

2015-02-23 Thread Bastien Nocera
On Wed, 2015-02-18 at 21:34 -0200, Henrique de Moraes Holschuh wrote: > On Wed, 18 Feb 2015, Bastien Nocera wrote: > > Signed-off-by: Bastien Nocera > > This one requires an ACK from the input maintainers, or no go... Any comments about th

Re: [ibm-acpi-devel] [PATCH 6/7] thinkpad_acpi: More Carbon X1 2014 keys

2015-02-23 Thread Bastien Nocera
On Fri, 2015-02-20 at 11:27 -0800, Andy Lutomirski wrote: > On 02/20/2015 06:44 AM, Bastien Nocera wrote: > > + KEY_MICCANCEL_MODE, /* Microphone cancellation */ > > What's this? We already have KEY_MICMUTE. Is this different? If so, > can you improve the

Re: [ibm-acpi-devel] [PATCH 5/7] input: Add keycodes used by Lenovo Carbon X1 2014

2015-02-20 Thread Bastien Nocera
Hey Dmitry, On Fri, 2015-02-20 at 10:40 -0800, Dmitry Torokhov wrote: > On Fri, Feb 20, 2015 at 10:04:04AM -0500, Benjamin Tissoires wrote: > > On Fri, Feb 20, 2015 at 9:44 AM, Bastien Nocera wrote: > > > Signed-off-by: Bastien Nocera > > > --- > > &

Re: [ibm-acpi-devel] [PATCH 4/6] thinkpad_acpi: Add support for more adaptive kbd buttons

2015-02-20 Thread Bastien Nocera
On Thu, 2015-02-19 at 21:35 -0800, Darren Hart wrote: > On Wed, Feb 18, 2015 at 09:53:48PM +0100, Bastien Nocera wrote: > > This commit adds new elements to the ThinkPad keymaps, and > > will send key events for keys for which an input.h declaration > > exists. > >

[ibm-acpi-devel] [PATCH 2/7] thinkpad_acpi: Factor out get/set adaptive kbd mode

2015-02-20 Thread Bastien Nocera
Move the getting/setting of the adaptive keyboard mode to separate functions, so that we can reuse them later through sysfs attributes. Signed-off-by: Bastien Nocera --- drivers/platform/x86/thinkpad_acpi.c | 61 ++-- 1 file changed, 38 insertions(+), 23

[ibm-acpi-devel] [PATCH 1/7] thinkpad_acpi: Remember adaptive kbd presence

2015-02-20 Thread Bastien Nocera
Rather than checking on each suspend and resume whether the laptop has an adaptive keyboard, check when the driver is initialised. Signed-off-by: Bastien Nocera --- drivers/platform/x86/thinkpad_acpi.c | 37 ++-- 1 file changed, 18 insertions(+), 19 deletions

Re: [ibm-acpi-devel] [PATCH 3/7] thinkpad_acpi: Add adaptive_kbd_mode sysfs attr

2015-02-20 Thread Bastien Nocera
On Fri, 2015-02-20 at 17:27 +0200, Mika Westerberg wrote: > On Fri, Feb 20, 2015 at 04:22:07PM +0100, Bastien Nocera wrote: > > On Fri, 2015-02-20 at 17:20 +0200, Mika Westerberg wrote: > > > On Fri, Feb 20, 2015 at 03:44:22PM +0100, Bastien Nocera wrote: > > > >

Re: [ibm-acpi-devel] [PATCH 3/7] thinkpad_acpi: Add adaptive_kbd_mode sysfs attr

2015-02-20 Thread Bastien Nocera
On Fri, 2015-02-20 at 17:20 +0200, Mika Westerberg wrote: > On Fri, Feb 20, 2015 at 03:44:22PM +0100, Bastien Nocera wrote: > > Add a sysfs attribute to allow privileged users to change the keyboard > > mode. This could be used by desktop environments to change the keyboard > &

[ibm-acpi-devel] [PATCH 5/7] input: Add keycodes used by Lenovo Carbon X1 2014

2015-02-20 Thread Bastien Nocera
Signed-off-by: Bastien Nocera --- include/uapi/linux/input.h | 10 ++ 1 file changed, 10 insertions(+) diff --git a/include/uapi/linux/input.h b/include/uapi/linux/input.h index b0a8130..affe40e 100644 --- a/include/uapi/linux/input.h +++ b/include/uapi/linux/input.h @@ -740,6 +740,16

[ibm-acpi-devel] [PATCH 7/7] thinkpad_acpi: use DEVICE_ATTR_* macros

2015-02-20 Thread Bastien Nocera
Use the DEVICE_ATTR_RW macro to reduce boiler plate and move the attribute declaration to get rid of function signatures. Signed-off-by: Bastien Nocera --- drivers/platform/x86/thinkpad_acpi.c | 72 ++-- 1 file changed, 20 insertions(+), 52 deletions(-) diff

[ibm-acpi-devel] [PATCH 4/7] thinkpad_acpi: Add support for more adaptive kbd buttons

2015-02-20 Thread Bastien Nocera
This commit adds new elements to the ThinkPad keymaps, and will send key events for keys for which an input.h declaration exists. Signed-off-by: Bastien Nocera --- drivers/platform/x86/thinkpad_acpi.c | 80 +++- 1 file changed, 79 insertions(+), 1 deletion

[ibm-acpi-devel] [PATCH 6/7] thinkpad_acpi: More Carbon X1 2014 keys

2015-02-20 Thread Bastien Nocera
Add support for newly added evdev keycodes. Signed-off-by: Bastien Nocera --- drivers/platform/x86/thinkpad_acpi.c | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/drivers/platform/x86/thinkpad_acpi.c b/drivers/platform/x86/thinkpad_acpi.c index 57130a1

[ibm-acpi-devel] [PATCH 3/7] thinkpad_acpi: Add adaptive_kbd_mode sysfs attr

2015-02-20 Thread Bastien Nocera
Add a sysfs attribute to allow privileged users to change the keyboard mode. This could be used by desktop environments to change the keyboard mode depending on the application focused, as the Windows application does. Signed-off-by: Bastien Nocera --- drivers/platform/x86/thinkpad_acpi.c | 69

Re: [ibm-acpi-devel] [PATCH 3/6] thinkpad_acpi: Add adaptive_kbd_mode sysfs attr

2015-02-20 Thread Bastien Nocera
On Thu, 2015-02-19 at 21:28 -0800, Darren Hart wrote: > On Wed, Feb 18, 2015 at 09:53:44PM +0100, Bastien Nocera wrote: > > Commit message please. Added. > > Signed-off-by: Bastien Nocera > > --- > > drivers/platform/x86/thinkpad_acpi.c | 71 > > ++

Re: [ibm-acpi-devel] [PATCH 2/6] thinkpad_acpi: Factor out get/set adaptive kbd mode

2015-02-20 Thread Bastien Nocera
On Thu, 2015-02-19 at 21:22 -0800, Darren Hart wrote: > On Wed, Feb 18, 2015 at 09:53:35PM +0100, Bastien Nocera wrote: > > Please provide a commit message. There is always something to say beyond what > is > in the subject. In this case, I suggest the motivation and justifica

Re: [ibm-acpi-devel] [PATCH 1/6] thinkpad_acpi: Remember adaptive kbd presence

2015-02-20 Thread Bastien Nocera
On Thu, 2015-02-19 at 21:08 -0800, Darren Hart wrote: > On Wed, Feb 18, 2015 at 09:53:28PM +0100, Bastien Nocera wrote: > > Rather than checking on each suspend and resume whether the laptop > > has an adaptive keyboard, check when the driver is initialised. > > Reasonable. &

[ibm-acpi-devel] [PATCH 2/6] thinkpad_acpi: Factor out get/set adaptive kbd mode

2015-02-18 Thread Bastien Nocera
Signed-off-by: Bastien Nocera --- drivers/platform/x86/thinkpad_acpi.c | 61 ++-- 1 file changed, 38 insertions(+), 23 deletions(-) diff --git a/drivers/platform/x86/thinkpad_acpi.c b/drivers/platform/x86/thinkpad_acpi.c index 80db3ce..a6dd017 100644 --- a

[ibm-acpi-devel] [PATCH 5/6] input: Add keycodes used by Lenovo Carbon X1 2014

2015-02-18 Thread Bastien Nocera
Signed-off-by: Bastien Nocera --- include/uapi/linux/input.h | 10 ++ 1 file changed, 10 insertions(+) diff --git a/include/uapi/linux/input.h b/include/uapi/linux/input.h index b0a8130..affe40e 100644 --- a/include/uapi/linux/input.h +++ b/include/uapi/linux/input.h @@ -740,6 +740,16

[ibm-acpi-devel] [PATCH 1/6] thinkpad_acpi: Remember adaptive kbd presence

2015-02-18 Thread Bastien Nocera
Rather than checking on each suspend and resume whether the laptop has an adaptive keyboard, check when the driver is initialised. Signed-off-by: Bastien Nocera --- drivers/platform/x86/thinkpad_acpi.c | 38 ++-- 1 file changed, 19 insertions(+), 19 deletions

[ibm-acpi-devel] [PATCH 3/6] thinkpad_acpi: Add adaptive_kbd_mode sysfs attr

2015-02-18 Thread Bastien Nocera
Signed-off-by: Bastien Nocera --- drivers/platform/x86/thinkpad_acpi.c | 71 +++- 1 file changed, 62 insertions(+), 9 deletions(-) diff --git a/drivers/platform/x86/thinkpad_acpi.c b/drivers/platform/x86/thinkpad_acpi.c index a6dd017..562d958 100644 --- a

[ibm-acpi-devel] [PATCH 4/6] thinkpad_acpi: Add support for more adaptive kbd buttons

2015-02-18 Thread Bastien Nocera
This commit adds new elements to the ThinkPad keymaps, and will send key events for keys for which an input.h declaration exists. Signed-off-by: Bastien Nocera --- drivers/platform/x86/thinkpad_acpi.c | 80 +++- 1 file changed, 79 insertions(+), 1 deletion

[ibm-acpi-devel] [PATCH 6/6] thinkpad_acpi: More Carbon X1 2014 keys

2015-02-18 Thread Bastien Nocera
Add support for newly added evdev keycodes. Signed-off-by: Bastien Nocera --- drivers/platform/x86/thinkpad_acpi.c | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/drivers/platform/x86/thinkpad_acpi.c b/drivers/platform/x86/thinkpad_acpi.c index a47a56a

[ibm-acpi-devel] More X1 Carbon (2nd gen) adaptive keyboard support

2015-01-13 Thread Bastien Nocera
Hey, I'm currently working on adding support for the extra "adaptive" keys for the X1 Carbon 2014 (2nd gen). Below is the (non-working) patch to add support for some of those keys. I'm working with Benjamin Tissoires to add the necessary, missing, Linux keycodes. A couple of questions though: -

[ibm-acpi-devel] Keyboard backlight support for Lenovo X1 Carbon Touch

2013-10-05 Thread Bastien Nocera
Heya, With Matthew Garrett's help, I've been able to detect and change the keyboard backlight on my laptop using software only. The laptop has 3 levels of backlight (with one of them being 'off') and can, under Linux, only be controlled using Fn+Space key combination. I wanted to control the keyb

Re: [ibm-acpi-devel] nvram polling for hardware mixer

2007-05-29 Thread Bastien Nocera
On Tue, 2007-05-29 at 18:45 -0300, Henrique de Moraes Holschuh wrote: > On Tue, 29 May 2007, Richard Hughes wrote: > > On Tue, 2007-05-29 at 22:11 +0100, Bastien Nocera wrote: > > > > Please review, and make sure I'm on the right lines. I also can't > > >

Re: [ibm-acpi-devel] nvram polling for hardware mixer

2007-05-29 Thread Bastien Nocera
rack in the builtin ALSA device instead? And then we'd get feedback from that one... -- Bastien Nocera <[EMAIL PROTECTED]> - This SF.net email is sponsored by DB2 Express Download DB2 Express C - the FREE ve

Re: [ibm-acpi-devel] [patch] add INPUT key reporting to thinkpad_acpi

2007-05-21 Thread Bastien Nocera
ocess the relevant event* device to get the events... > > Yes, I found that KEY_POWER would be routed, but KEY_FN_F1 would not be. > I suspect something is filtering these keys. X probably doesn't know about those keys, thus doesn't pass them on. We need to use the proper ke

Re: [ibm-acpi-devel] nvram keys to userspace

2007-05-16 Thread Bastien Nocera
ould be > to get new keys if needed. It seems crazy we are using infrastructure, > if the infrastructure can't cope with the new demands. I'll see what i > can do. It's new keys added to the input layer's input.h, nothing to do with user-space. Which keys would nee