On Fri, Dec 8, 2017 at 4:04 PM, Darren Hart <dvhart@xxxxxxxxxxxxx> wrote: > On Thu, Dec 07, 2017 at 11:40:23AM +0800, Alex Hung wrote: >> HEBC method reports capabilities of 5 button array but Wacom >> MobileStudio Pro does not have this control method. A DMI quirk >> was created to enable 5 button array for this system. > > Jason, have you been able to verify this patch with testing? > I tested the attachment at [1], which produces an identical copy of intel-hid.c as applying this patch to the platform-drivers-x86/review-dvhart branch of git://git.infradead.org/users/dvhart/linux-platform-drivers-x86.git. Assuming that's the correct code, feel free to attach the following: Tested-by: Jason Gerecke <jason.gerecke@xxxxxxxxx> [1]: https://bugzilla.kernel.org/attachment.cgi?id=261033 Jason --- Now instead of four in the eights place / you’ve got three, ‘Cause you added one / (That is to say, eight) to the two, / But you can’t take seven from three, / So you look at the sixty-fours.... >> [V2] Update DMI strings from shipping systems > > Please place patch versioning detail below the --- in the future > >> Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=197991 >> >> Reported-by: Jason Gerecke <jason.gerecke@xxxxxxxxx> >> Signed-off-by: Alex Hung <alex.hung@xxxxxxxxxxxxx> >> --- > > here (see submitting patches docs for details) > > > -- > Darren Hart > VMware Open Source Technology Center