Re: [PATCH 0/4] HID: wacom: fixes for next

From: Benjamin Tissoires
Date: Mon Jan 23 2017 - 03:02:50 EST


On Jan 20 2017 or thereabouts, Jason Gerecke wrote:
> Patches 1/3 look reasonable to me, though I've not run into the bugs
> they aim to fix. For those:
>
> Acked-by: Jason Gerecke <jason.gerecke@xxxxxxxxx>
>
> As for patch 4, I have some additional reservations about hiding the
> message... We can discuss that further in its thread.
>

Yeah, I am fine dropping patch 4. Thanks for the review/tests Jason!

Cheers,
Benjamin

> 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....
>
>
>
> On Fri, Jan 20, 2017 at 7:20 AM, Benjamin Tissoires
> <benjamin.tissoires@xxxxxxxxxx> wrote:
> > Hi,
> >
> > well, these are non critical but nonetheless interesting fixes for
> > Wacom I was working on last week. I was waiting for the current v4.10
> > fix to be sent by Jason before pushing those out.
> >
> > Cheers,
> > Benjamin
> >
> > Benjamin Tissoires (4):
> > HID: wacom: release the resources before leaving despite devm
> > HID: wacom: remove warning while disconnecting devices
> > HID: wacom: do not attempt to switch mode while in probe
> > HID: wacom: do not shout an error on LED control
> >
> > drivers/hid/wacom.h | 1 +
> > drivers/hid/wacom_sys.c | 42 ++++++++++++++++++++++++++++++------------
> > 2 files changed, 31 insertions(+), 12 deletions(-)
> >
> > --
> > 2.9.3
> >