From: Danny Kukawka <danny.kukawka@web.de>
To: hal@lists.freedesktop.org
Cc: Matthew Garrett <mjg59@srcf.ucam.org>,
acpi4asus-user@lists.sourceforge.net,
Alan Jenkins <sourcejedi.lkml@googlemail.com>,
Corentin Chary <corentin.chary@gmail.com>,
linux acpi <linux-acpi@vger.kernel.org>
Subject: Re: Illuminated keyboard on ASUS U50VG laptop, backlight or led ?
Date: Sat, 1 Aug 2009 19:21:22 +0200 [thread overview]
Message-ID: <200908011921.23990.danny.kukawka@web.de> (raw)
In-Reply-To: <9b2b86520908010930n4b47c6c2uc5608ae7d81b0b6b@mail.gmail.com>
On Samstag, 1. August 2009, Alan Jenkins wrote:
> On 8/1/09, Corentin Chary <corentin.chary@gmail.com> wrote:
[...]
> >
> > It seems there is also a KeyboardBacklight interface which is what we
> > want, but I didn't find how to use it
> > without an addon (like addon-macbookpro-backlight).
>
> But if you look at the methods, they are exactly the same, right?
Yes, they are.
> I
> think the idea is to use an FDI file to match our LED device (or
> possibly create it as a hal device... I'm still really fuzzy on hal),
> and then mark it as having the "keyboard_backlight" capability.
No, it wouldn't work this way. At least not with the current key.
I changed the led addon in my repo to check if a led has
leds.function=kbd_backlight and provide then the
org.freedesktop.Hal.Device.KeyboardBacklight interface instead of
org.freedesktop.Hal.Device.Leds. Please check if this helps.
see:
http://cgit.freedesktop.org/~dkukawka/hal/commit/?id=c900d0e5012b1dd0e0fa6e20fd0d92a59b3de093
[...]
> > I believe that we need a keyboard backlight addon that respond to
> > KEY_KBDILLUM*
> > and update the corresponding /sys/class/vendor:color:kbd_backlight.
>
> Oops, thats not the addons job. It's for something like the gnome
> power manager. It looks like g-p-m already picks up the device with a
> "keyboard_backlight" capability, and react to keyboard brightness key
> events.
Correct. It's not the job of the addon to react on keyevents. This is up to a
desktop application which listen to the events and call then the interface.
The same way it work for e.g. the brightness of your laptop panel.
Danny
next prev parent reply other threads:[~2009-08-01 17:21 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-07-24 7:34 Illuminated keyboard on ASUS U50VG laptop, backlight or led ? Corentin Chary
2009-07-28 19:51 ` Corentin Chary
2009-07-28 20:04 ` Matthew Garrett
2009-08-01 10:40 ` Corentin Chary
2009-08-01 11:19 ` Alan Jenkins
2009-08-01 12:40 ` Corentin Chary
2009-08-01 13:21 ` Danny Kukawka
2009-08-01 15:50 ` Corentin Chary
2009-08-01 16:30 ` Alan Jenkins
2009-08-01 17:21 ` Danny Kukawka [this message]
2009-08-01 19:43 ` Alan Jenkins
2009-08-01 19:47 ` Alan Jenkins
2009-08-02 7:20 ` Corentin Chary
2009-08-06 10:07 ` Corentin Chary
2009-08-06 10:48 ` Danny Kukawka
2009-08-06 11:59 ` Corentin Chary
2009-08-06 16:20 ` Danny Kukawka
2009-08-06 17:00 ` Corentin Chary
2009-12-07 7:51 ` Corentin Chary
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=200908011921.23990.danny.kukawka@web.de \
--to=danny.kukawka@web.de \
--cc=acpi4asus-user@lists.sourceforge.net \
--cc=corentin.chary@gmail.com \
--cc=hal@lists.freedesktop.org \
--cc=linux-acpi@vger.kernel.org \
--cc=mjg59@srcf.ucam.org \
--cc=sourcejedi.lkml@googlemail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox