]> git.baikalelectronics.ru Git - kernel.git/commit
HID: zeroing of bytes in output fields is bogus
authorJiri Kosina <jkosina@suse.cz>
Sat, 10 Mar 2007 21:45:45 +0000 (22:45 +0100)
committerJiri Kosina <jkosina@suse.cz>
Mon, 12 Mar 2007 13:55:26 +0000 (14:55 +0100)
commit7b5c1be58cf96ec570eac5b5cc316603cf8e303a
tree1965c3abc44bdff17d75aee85f603f555e1731c9
parent730286ef6bdd156e2aac71b6662b02447d0cafcb
HID: zeroing of bytes in output fields is bogus

This patch removes bogus zeroing of unused bits in output reports,
introduced in Simon's patch in commit 660184db.
According to the specification, any sane device should not care
about values of unused bits.

What is worse, the zeroing is done in a way which is broken and
might clear certain bits in output reports which are actually
_used_ - a device that has multiple fields with one value of
the size 1 bit each might serve as an example of why this is
bogus - the second call of hid_output_report() would clear the
first bit of report, which has already been set up previously.

This patch will break LEDs on SpaceNavigator, because this device
is broken and takes into account the bits which it shouldn't touch.
The quirk for this particular device will be provided in a separate
patch.

Signed-off-by: Jiri Kosina <jkosina@suse.cz>
drivers/hid/hid-core.c