HID: wacom: Improve generic name generation
authorJason Gerecke <killertofu@gmail.com>
Mon, 24 Jul 2017 16:46:19 +0000 (09:46 -0700)
committerJiri Kosina <jkosina@suse.cz>
Thu, 27 Jul 2017 13:14:29 +0000 (15:14 +0200)
commit09dc28acaec74d7467c7c9b81dc8676e5bc957ce
treec777f7f0ad2e883917b4539745e8978b28b7e0ab
parentfc2237a724a9e448599076d7d23497f51e2f7441
HID: wacom: Improve generic name generation

The 'wacom_update_name' function is responsible for producing names for
the input device nodes based on the hardware device name. Commit f2209d4
added the ability to strip off prefixes like "Wacom Co.,Ltd." where the
prefix was immediately (and redundantly) followed by "Wacom". The
2nd-generation Intuos Pro 2 has such a prefix, but with a small error
(the period and comma are swapped) that prevents the existing code from
matching it. We're loath to extend the number of cases out endlessly and
so instead try to be smarter about name generation.

We observe that the cause of the redundant prefixes is HID combining the
manufacturer and product strings of USB devices together. By using the
original product name (with "Wacom" prefixed, if it does not already
exist in the string) we can bypass the gyrations to find and remove
redundant prefixes. Other devices either don't have a manufacturer string
that needs to be removed (Bluetooth, uhid) or should have their name
generated from scratch (I2C).

Signed-off-by: Jason Gerecke <jason.gerecke@wacom.com>
Acked-By: Benjamin Tissoires <benjamin.tissoires@redhat.com>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>
drivers/hid/wacom_sys.c