linux-stable.git/drivers/hid/hid-input.c, branch v5.0

HID: input: use the Resolution Multiplier for high-resolution scrolling

2018-12-07T15:27:13+00:00

Windows uses a magic number of 120 for a wheel click. High-resolution
scroll wheels are supposed to use a fraction of 120 to signal smaller
scroll steps. This is implemented by the Resolution Multiplier in the
device itself.

If the multiplier is present in the report descriptor, set it to the
logical max and then use the resolution multiplier to calculate the
high-resolution events. This is the recommendation by Microsoft, see
http://msdn.microsoft.com/en-us/windows/hardware/gg487477.aspx

Note that all mice encountered so far have a logical min/max of 0/1, so
it's a binary "yes or no" to high-res scrolling anyway.

To make userspace simpler, always enable the REL_WHEEL_HI_RES bit. Where
the device doesn't support high-resolution scrolling, the value for the
high-res data will simply be a multiple of 120 every time. For userspace,
if REL_WHEEL_HI_RES is available that is the one to be used.

Potential side-effect: a device with a Resolution Multiplier applying to
other Input items will have those items set to the logical max as well.
This cannot easily be worked around but it is doubtful such devices exist.

Signed-off-by: Peter Hutterer 
Verified-by: Harry Cutts 
Signed-off-by: Benjamin Tissoires

Revert "HID: input: Create a utility class for counting scroll events"

2018-11-22T07:57:38+00:00

This reverts commit 1ff2e1a44e02d4bdbb9be67c7d9acc240a67141f.

It turns out the current API is not that compatible with
some Microsoft mice, so better start again from scratch.

Signed-off-by: Benjamin Tissoires 
Acked-by: Harry Cutts 
Acked-by: Dmitry Torokhov 
Acked-by: Jiri Kosina

Revert "HID: input: simplify/fix high-res scroll event handling"

2018-11-22T07:56:04+00:00

This reverts commit 044ee890286153a1aefb40cb8b6659921aecb38b.

It turns out the current API is not that compatible with
some Microsoft mice, so better start again from scratch.

Signed-off-by: Benjamin Tissoires 
Acked-by: Harry Cutts 
Acked-by: Dmitry Torokhov 
Acked-by: Jiri Kosina

HID: input: Ignore battery reported by Symbol DS4308

2018-11-12T09:08:46+00:00

The Motorola/Zebra Symbol DS4308-HD is a handheld USB barcode scanner
which does not have a battery, but reports one anyway that always has
capacity 2.

Let's apply the IGNORE quirk to prevent it from being treated like a
power supply so that userspaces don't get confused that this
accessory is almost out of power and warn the user that they need to charge
their wired barcode scanner.

Reported here: https://bugs.chromium.org/p/chromium/issues/detail?id=804720

Signed-off-by: Benson Leung 
Reviewed-by: Benjamin Tissoires 
Signed-off-by: Benjamin Tissoires

HID: input: simplify/fix high-res scroll event handling

2018-10-29T18:25:24+00:00

Commit 1ff2e1a44e02 ("HID: input: Create a utility class for counting
scroll events") created the helper function

    hid_scroll_counter_handle_scroll()

to handle high-res scroll events and also expose them as regular wheel
events.

But the resulting algorithm was unstable, and causes scrolling to be
very unreliable.  When you hit the half-way mark of the highres
multiplier, small highres movements will incorrectly translate into big
traditional wheel movements, causing odd jitters.

Simplify the code and make the output stable.

NOTE! I'm pretty sure this will need further tweaking.  But this at
least turns a unusable mouse wheel on my Logitech MX Anywhere 2S into
a usable one.

Cc: Jiri Kosina 
Cc: Harry Cutts 
Cc: Benjamin Tissoires 
Cc: Peter Hutterer 
Signed-off-by: Linus Torvalds

Merge branch 'for-4.20/logitech-highres' into for-linus

2018-10-23T11:35:22+00:00

High-resolution support for hid-logitech

Merge branch 'for-4.20/core' into for-linus

2018-10-23T11:17:27+00:00

Fixes and new features for driver core. Highlights:

- maximum global item tag report size gets increased to 256
- improved INPUT_PROP reporting for Digitizer devices

HID: input: Set INPUT_PROP_-property for HID_UP_DIGITIZERS

2018-09-05T14:59:24+00:00

Some system may want to know if a detected digitizer device is either an
integrated or an external device.  In order to distinguish such condition,
setting either INPUT_PROP_DIRECT or INPUT_PROP_POINTER is required,
checking the member, "application", in "hid_field" structure.

Signed-off-by: Tatsunosuke Tobita 
Reviewed-by: Ping Cheng 
Signed-off-by: Jiri Kosina

HID: input: Create a utility class for counting scroll events

2018-09-05T08:12:07+00:00

To avoid code duplication, this class counts high-resolution scroll
movements and emits the legacy low-resolution events when appropriate.
Drivers should be able to create one instance for each scroll wheel that
they need to handle.

Signed-off-by: Harry Cutts 
Reviewed-by: Benjamin Tissoires 
Signed-off-by: Jiri Kosina

HID: input: do not append a suffix if the name already has it

2018-09-04T19:32:24+00:00

Or it creates some weird input names like:
"MI Dongle MI Wireless Mouse Mouse"

Signed-off-by: Benjamin Tissoires 
Signed-off-by: Jiri Kosina