g-p-m should display battery status on KEY_BATTERY

[Martin Willemoes Hansen]

Breezy on a HP NC4200 laptop.

The key for "battery info" does nothing.
The key is located on the same key as f8.

[Matt Zimmerman (mdz)]

What should it do?

[Martin Willemoes Hansen (mwh)]

Created an attachment (id=4100)
This is what the battery button does in windows.

[Daniel Silverstone (dsilvers)]

Once we have a keysym/code we can make this button activate g-p-m's battery info

[Martin Willemoes Hansen (mwh)]

I think this is the keysym/code: 0x9f

[Martin Willemoes Hansen (mwh)]

Ah, sorry the above is wrong, its for the presentation button, the battery button has this keysym/code: 0xd9

[Martin Willemoes Hansen (mwh)]

Same issue for Flight 5

[Andreas Schildbach (schildbach)]

On the Dell Latitude X1 running Breezy, neither xev nor showkey capture Fn-F3 (Battery).

However, I see this output in kern.log:

Mar 15 19:27:52 localhost kernel: [4297940.314000] atkbd.c: Unknown key pressed (translated set 2, code 0x87 on isa0060/serio0).
Mar 15 19:27:52 localhost kernel: [4297940.314000] atkbd.c: Use 'setkeycodes e007 <keycode>' to make it known.

[Paul Sladen (sladen)]

KEY_BATTERY is currently aliased to KEY_F24 until such a point as there's proper kernel assignment <256; see:

  grep -i battery /usr/share/hotkey-setup/*.hk

I think Kinnison has already done the code to gnome-power-manager to pop-down the notification box, so I guess they just need matching up.

[Richard Hughes (richard-hughes)]

Does the hal input addon monitor F24? You can test by doing lshal -m and then pressing the button. It's trivial to add to g-p-m if it does.

[Paul Sladen (sladen)]

Yes, that's the other thing I was going to suggest:

  $ sudo acpi_fakekey 194
  interface=org.freedesktop.Hal.Device; member=Condition
 string "ButtonPressed" string "f24"

If you'd prefer that, 'hald-addon-keyboard' can be patched to send "battery" for that key and then you wouldn't have to worry when/if we get a better assignment as it'll be abstracted away.

[Richard Hughes (richard-hughes)]

"battery-info" is probably better, and I can patch g-p-m to listen for "battery-info" instead of "www" which is what it listens now for.

[Matthew Garrett (mjg59)]

KEY_BATTERY ought to be keycode 236. I've submitted a patch to upstream.
Can we go with that rather than F24?

--
Matthew Garrett | <email address hidden>

[Richard Hughes (richard-hughes)]

Matthew, can you send a patch to hal-devel for the addon for key 236, and I'll change g-p-m to listen for "battery-info" rather than "www". Many thanks.

[Paul Sladen (sladen)]

IIRC, all the other 'hal-keyboard-addon' codes are the KEY_NAME from 'linux/input.h' processed with:

  s/^KEY_//;y/A-Z/a-z/

so if Matthew sent 'KEY_BATTERY' upstream that would be just "battery".

[Richard Hughes (richard-hughes)]

Sure, that makes sense. Matthew, how's that upstream patch going?

[Matthew Garrett (mjg59)]

It's been accepted upstream, but doesn't seem to have been committed anywhere yet.

[Paul Sladen (sladen)]

g-p-m can be fixed to listen for "battery" independently.

Then when it's sorted upstream only acpi-support, hotkey-setup and hal-addon-keyboard will need patching and for the moment it should allow KEY_WWW to operate again normally.

[Richard Hughes (richard-hughes)]

2006-03-22 Richard Hughes <email address hidden>
 * src/gpm-manager.c: Use the button "battery" rather than "www" to show the battery status. This depends on mjg59's kernel addition and the use of a HAL with the new button patched in.

[Martin Willemoes Hansen (mwh)]

Last tested on Dapper Flight 5

magic number of the hotkey: 0xd9

Laptop: Compaq nc4200

[Paul Sladen (sladen)]

Martin, please could you file a new bug against 'hotkey-setup' for your laptop with the dmidecode and keycode information on:

  https://wiki.ubuntu.com/LaptopTestingTeam/HotkeyResearch

Many Thanks,

At the moment, the status of this bug is that:

  hotkey-setup/acpi-support maps $KEY to KEY_BATTERY (depending on the laptop)
  KEY_BATTERY is currently aliased to something else while we wait for a code from upstream
  hald-addon-keyboard needs to convert "KEY_BATTERY" to send "battery" key signal over DBus
  gnome-power-manager listens for "battery" signal and displays dialogue.

Only the first and last are currently done.

[Martin Willemoes Hansen (mwh)]

Okay, done, it is available here:
https://launchpad.net/distros/ubuntu/+source/hotkey-setup/+bug/36605

[Paul Sladen (sladen)]

The rest of the stack should be fixed, I think it's just waiting on a fixed 'g-p-m' pulling in.

Richard: this fix should go in 2.14.1 :)

[Richard Hughes (richard-hughes)]

Paul, the "www" -> "battery" fix is already in the 2.14 branch, so it'll be in 2.14.1

[Paul Sladen (sladen)]

 hal (0.5.7-1ubuntu9) dapper; urgency=low
 .
   * debian/patches/10_volume_and_aliased_keys.patch (addon-keyboard):
     - Add macro, volume* and mute keys (special codes <128)
     - Add in 'aliased-keys' mirrored onto F19..F23
     - Add battery key at value 236, (waiting for upstream), [Malone: #22451]

[apelete (apelete)]

Ok, "www" shortcut works now (on Asus V6 laptop here), but there still is something annoying:
Why does the battery's state tooltip always appears when you use the www button ? I mean it launches the browser now, but it also makes that unwanted tooltip to appear (and it doesn't dissapear by itself, you have to close it yourself).
Will this be the final behaviour of the www shortcut ?

[Matthew Garrett (mjg59)]

No, this will not be the final behaviour.

[Andreas Schildbach (schildbach)]

What's the status of this bug? In Dapper Beta 1, the Battery Hotkey still does not have any visible effect.

[Daniel Silverstone (dsilvers)]

This was fixed in the 2.14.3-0ubuntu1 upload.

[Andreas Schildbach (schildbach)]

I can confirm that this works now on Flight 7. Thanks a lot!

[Paul Sladen (sladen)]

gnome-power-manager (2.14.3-0ubuntu1) dapper; urgency=low

  * New upstream release on stable branch
    Highlights since 2.14.0 include:
     - Use 'battery' rather than 'www' closes: launchpad #22451:

[Martin Willemoes Hansen (mwh)]

Works on nc4200, Dapper final release