On Wed, 13 May 2020, Scott Shumate wrote:

Hi Roderick,

The official DS3 has a Report Count(19) instead of Report Count(13) in the
exact same offset.  I have no idea what the silicon vendor for these dongles
was thinking but it's suspicious that the official count of 19 (0x13) turned
into 13 (0xd) in the knock-off.  It makes you wonder if the engineers confused
the decimal/hex numbers.

As buggy as all of these third-party devices are, I'm afraid relying on the
HID parser to get it right is only going to worse over time.  I do like your
idea of having each device register themselves.  It would be nice to have each
device provide a callback to decode its own report rather than handle a bunch
of special conditions and quirks in a unified report decoding function.  The
drawback of course is that its going to be a little more effort to maintain.

I've added Cc: stable and Fixes: tag, and applied.

Thanks,

-- 
Jiri Kosina
SUSE Labs