RomanNum3ral
/
Solaar
mirror of https://github.com/pwr-Solaar/Solaar.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

hid10: update flags description for notifications 

The Logitech HID++ 1.0 documentation actually names the fields for devices and
receiver. Clarify that and explain why enabling all of the bits is a bad idea.
This commit is contained in:
Peter Wu 2013-05-19 10:30:41 +02:00
parent 29d0c07164
commit f0542923d7
1 changed files with 8 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
lib/logitech/unifying_receiver/hidpp10.py
View File

@ -39,7 +39,14 @@ POWER_SWITCH_LOCATION = _NamedInts(
left_edge=0x0B, left_edge=0x0B,
bottom_edge=0x0C) bottom_edge=0x0C)
# Some flags are used both by devices and receivers, the Logitech documentation makes no difference # Some flags are used both by devices and receivers. The Logitech documentation
# mentions that the first and last (third) byte are used for devices while the
# second is used for the receiver. In practise, the second byte is also used for
# some device-specific notifications (keyboard illumination level). Do not
# simply set all notification bits if the software does not support it. For
# example, enabling keyboard_sleep_raw makes the Sleep key a no-operation unless
# the software is updated to handle that event.
# Observations:
# - wireless and software present were seen on receivers, reserved_r1b4 as well # - wireless and software present were seen on receivers, reserved_r1b4 as well
# - the rest work only on devices as far as we can tell right now # - the rest work only on devices as far as we can tell right now
# In the future would be useful to have separate enums for receiver and device notification flags, # In the future would be useful to have separate enums for receiver and device notification flags,