Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1308179

Re: [PATCH] thinkpad_acpi: Add support for keyboard backlight

From Pali Rohár <pali.rohar@gmail.com>
Newsgroups linux.kernel
Subject Re: [PATCH] thinkpad_acpi: Add support for keyboard backlight
Date 2016-01-13 10:00 +0100
Message-ID <qQprA-5DL-5@gated-at.bofh.it> (permalink)
References <qJj7z-1O0-1@gated-at.bofh.it> <qPQ0O-6AV-9@gated-at.bofh.it> <qPQk9-6HH-1@gated-at.bofh.it> <qPQWT-7fk-19@gated-at.bofh.it> <qQf8T-6R1-21@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


On Tuesday 12 January 2016 22:58:04 Pavel Machek wrote:
> Hi!
> 
> On Mon 2016-01-11 21:03:01, Pali Rohár wrote:
> > On Monday 11 January 2016 20:28:00 Henrique de Moraes Holschuh wrote
> > > The two features are not the same (and are handled differently by the
> > > firmware, for whatever reason), although they do serve the same
> > > purpose. I don't think a thinkpad will ever have both features at
> > > the same time, so I have no idea why they changed the firmware
> > > interface.
> > 
> > Maybe we should decide if ::kbd_backlight LED suffix could be used also 
> > for other LED devices and not only for those which are physically under 
> > the keyboard.
> 
> Another problem is that N900 has _6_ backlight LEDs. Named
> lp5523::kb1..6. ... Which does means desktop software will probably
> not pick them up :-(. 
> 
> I guess we could have "/sys/class/kbd_light/brightness" that would
> control all of them with one write.

Probably... But there is problem that lp5523 is not ordinary on/off
light, it can be programmed to execute own "light" application.

> Next question is.. apparently there are some keyboards that have
> per-key RGB backlight... but maybe we can just call that "weird
> enough" and ignore...

First we need to defines stable kernel ABI for keyboard backlight. And I
suggest to use existing convention used by upower/console-kit and other
userspace apps...

-- 
Pali Rohár
pali.rohar@gmail.com

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

Re: [PATCH] thinkpad_acpi: Add support for keyboard backlight Pavel Machek <pavel@ucw.cz> - 2016-01-04 21:20 +0100
  Re: [PATCH] thinkpad_acpi: Add support for keyboard backlight Pali Rohár <pali.rohar@gmail.com> - 2016-01-04 21:30 +0100
    Re: [PATCH] thinkpad_acpi: Add support for keyboard backlight Pali Rohár <pali.rohar@gmail.com> - 2016-01-04 21:50 +0100
  Re: [PATCH] thinkpad_acpi: Add support for keyboard backlight Henrique de Moraes Holschuh <hmh@hmh.eng.br> - 2016-01-09 18:40 +0100
    Re: [PATCH] thinkpad_acpi: Add support for keyboard backlight Henrique de Moraes Holschuh <hmh@hmh.eng.br> - 2016-01-09 18:50 +0100
      Re: [PATCH] thinkpad_acpi: Add support for keyboard backlight Darren Hart <dvhart@infradead.org> - 2016-01-11 20:10 +0100
        Re: [PATCH] thinkpad_acpi: Add support for keyboard backlight Henrique de Moraes Holschuh <hmh@hmh.eng.br> - 2016-01-11 20:30 +0100
          Re: [PATCH] thinkpad_acpi: Add support for keyboard backlight Pali Rohár <pali.rohar@gmail.com> - 2016-01-11 21:10 +0100
            Re: [PATCH] thinkpad_acpi: Add support for keyboard backlight Johannes Stezenbach <js@sig21.net> - 2016-01-11 22:20 +0100
              Re: [PATCH] thinkpad_acpi: Add support for keyboard backlight Henrique de Moraes Holschuh <hmh@hmh.eng.br> - 2016-01-12 17:10 +0100
                Re: [ibm-acpi-devel] [PATCH] thinkpad_acpi: Add support for  keyboard backlight Yves-Alexis Perez <corsac@debian.org> - 2016-01-12 17:30 +0100
                Re: [PATCH] thinkpad_acpi: Add support for keyboard backlight Johannes Stezenbach <js@sig21.net> - 2016-01-12 17:40 +0100
                Re: [PATCH] thinkpad_acpi: Add support for keyboard backlight Henrique de Moraes Holschuh <hmh@hmh.eng.br> - 2016-01-12 19:00 +0100
                Re: [PATCH] thinkpad_acpi: Add support for keyboard backlight Johannes Stezenbach <js@sig21.net> - 2016-01-12 23:10 +0100
                Re: [ibm-acpi-devel] [PATCH] thinkpad_acpi: Add support for keyboard  backlight Kevin Locke <kevin@kevinlocke.name> - 2016-01-12 19:20 +0100
                Re: [ibm-acpi-devel] [PATCH] thinkpad_acpi: Add support for keyboard  backlight Henrique de Moraes Holschuh <hmh@hmh.eng.br> - 2016-01-12 19:30 +0100
                Re: [ibm-acpi-devel] [PATCH] thinkpad_acpi: Add support for keyboard  backlight Kevin Locke <kevin@kevinlocke.name> - 2016-01-12 19:40 +0100
            Re: [PATCH] thinkpad_acpi: Add support for keyboard backlight Henrique de Moraes Holschuh <hmh@hmh.eng.br> - 2016-01-12 17:10 +0100
            Re: [PATCH] thinkpad_acpi: Add support for keyboard backlight Pavel Machek <pavel@ucw.cz> - 2016-01-12 23:00 +0100
              Re: [PATCH] thinkpad_acpi: Add support for keyboard backlight Pali Rohár <pali.rohar@gmail.com> - 2016-01-13 10:00 +0100
                Re: [PATCH] thinkpad_acpi: Add support for keyboard backlight Pavel Machek <pavel@ucw.cz> - 2016-01-13 20:10 +0100
                Re: [PATCH] thinkpad_acpi: Add support for keyboard backlight Pavel Machek <pavel@ucw.cz> - 2016-01-13 20:20 +0100
          Re: [ibm-acpi-devel] [PATCH] thinkpad_acpi: Add support for  keyboard backlight Yves-Alexis Perez <corsac@debian.org> - 2016-01-11 23:50 +0100
          Re: [PATCH] thinkpad_acpi: Add support for keyboard backlight Darren Hart <dvhart@infradead.org> - 2016-01-12 18:00 +0100
            Re: [PATCH] thinkpad_acpi: Add support for keyboard backlight Henrique de Moraes Holschuh <hmh@hmh.eng.br> - 2016-01-12 19:00 +0100

csiph-web