Hi Jiri, On Tue, Nov 13, 2012 at 8:47 AM, Jiri Kosina <jkosina@xxxxxxx> wrote: > On Wed, 7 Nov 2012, Benjamin Tissoires wrote: > >> Hi Guys, >> >> here is the third version of this patchset. >> I think I included most of Henrik's comments. >> >> Happy reviewing :) >> >> Cheers, >> Benjamin >> >> v1 introduction: >> So, this is an update for supporting Win 8 multitouch devices in the kernel. >> As I wanted to reliably forward the resolution, I noticed a bug in the >> processing of the unit_exponent in the hid core layer. >> Thus the fixes for hid-core and hid-input. >> >> v2 changes: >> * added missing initial patch that prevents the series to be applied on top of Jiri's tree >> * update to include latest hid changes >> * taken into account Alan's patch: "hid: put the case in the right switch statement" >> >> v3 changes: >> * splitted "round return value of hidinput_calc_abs_res" in a separate patch >> * export snto32 in hid.h as we need to use it in hid-input.c >> * didn't change all drivers, but add a field in hid_usage instead >> * add quirk MT_QUIRK_IGNORE_DUPLICATES so that any device can rely on it >> * easier understandable support of hovering devices >> * changed scan time definition >> * applied new definition of scan time in hid-multitouch >> * some other few things. > > Benjamin, > > thanks for the patchset. > > I am fine with the HID-specific changes (and I have sent out my Acks to > the invidivual patches). Thanks :) > > For hid-multitouch changes, I'd like to have Reviewed-by: from Henrik as > well. Henrik, are you planning to review this patchset, please? > > Also the note from Dmitry on turning ABS_SCAN_TIME into > EV_MSC/MSC_TIMESTAMP should be addressed. Yes, sure. I'll do it today. > > Once this is finalized, I'd like to queue this for 3.8 in my tree. Thanks, Benjamin > > Thanks, > > -- > Jiri Kosina > SUSE Labs -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
