Bug 25517 - Initial Maemo keyboard mappings
Summary: Initial Maemo keyboard mappings
Status: VERIFIED FIXED
Alias: None
Product: xkeyboard-config
Classification: Unclassified
Component: General (show other bugs)
Version: unspecified
Hardware: Other All
: medium normal
Assignee: xkb
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 25507
  Show dependency treegraph
 
Reported: 2009-12-08 06:44 UTC by Rami Ylimaki
Modified: 2009-12-15 07:36 UTC (History)
1 user (show)

See Also:
i915 platform:
i915 features:


Attachments
Maemo keyboard layout files (62.86 KB, patch)
2009-12-08 06:51 UTC, Rami Ylimaki
Details | Splinter Review
Enable Maemo keyboards layouts files. (7.65 KB, patch)
2009-12-08 07:01 UTC, Rami Ylimaki
Details | Splinter Review

Description Rami Ylimaki 2009-12-08 06:44:09 UTC
The xkeyboard-config tree is missing keyboard definitions for Nokia's Maemo platform. This bug is about introducing those definitions without modifying or extending the current xkeyboard-config functionality. Only some Nokia keyboard definitions are added so that the resulting xkeyboard-config package can be used on the N900 phone. Possible functional modifications are introduced through other bugs (or alternatively not upstreamed at all).
Comment 1 Rami Ylimaki 2009-12-08 06:51:49 UTC
Created attachment 31834 [details] [review]
Maemo keyboard layout files

The patch contains only the layout files for three different Nokia keyboards used on the Maemo platform. The xkeyboard-config package is not modified to take the files into actual use yet.
Comment 2 Rami Ylimaki 2009-12-08 07:01:19 UTC
Created attachment 31835 [details] [review]
Enable Maemo keyboards layouts files.

This patch modifies xkeyboard-config so that the Maemo keyboard definitions can actually be used on the N900 phone.
Comment 3 Rami Ylimaki 2009-12-08 07:30:13 UTC
Adding Sergey into CC.

Sergey, the two attached patches add support for the Nokia N900 keyboard. Also the N810 and SU8W bluetooth keyboard layouts are included. I have tested these patches on top of v_1_7 tag and they seem to be working. However, I've only tested the N900 internal keyboard, I don't have access to N810 or SU8W currently.

This is what I have done for testing:
- clone the xkeyboard-config git repository and checkout v_1_7 tag
- apply latest debianization on top of that
- make slight modifications to debianization so that xkeyboard-config builds under scratchbox so that we can generate a package for N900
- test the resulting debian package on N900, different keyboard layouts were working fine

The current N900 xkeyboard-config package contains additionally a minor modification to allow using of specified keyboard model for multi layouts. I'll create another bug for this and we can discuss about that change later. It's not very important and we can leave that change out of these patches.

Could you check the patches and comment if there is something wrong with them? If they are OK, you can push them to the FDO repository.
Comment 4 Sergey V. Udaltsov 2009-12-10 16:48:44 UTC
Thanks a bunch, committed. The patches look nice and clean
Comment 5 Rami Ylimaki 2009-12-15 07:36:42 UTC
Marking this as verified as I've used the upstream xkeyboard-config on N900 successfully.


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.