Hi, On 04/17/2014 05:35 PM, Dmitry Torokhov wrote: > Hi Hans, > > On Thu, Apr 17, 2014 at 01:41:43PM +0200, Hans de Goede wrote: >> We expect that all the Haswell series will need such quirks, sigh. > > Given this statement do we really want this to be handled in kernel? I know this answer won't make you happy, but short term: Yes, we are getting many many bugreports about this, ie: https://bugzilla.redhat.com/show_bug.cgi?id=1060885 https://bugzilla.redhat.com/show_bug.cgi?id=1068716 https://bugzilla.redhat.com/show_bug.cgi?id=1085582 https://bugzilla.redhat.com/show_bug.cgi?id=1085697 https://bugzilla.redhat.com/show_bug.cgi?id=1086227 And by extending the *already present* quirk table we can get this issue resolved quickly, and also resolve it for people running older kernels through the various stable series. > Maybe we simply want udev to fix up the limits with EVIOSABS(), Ah, I did not know that it is possible to fixup the min/max values from user space, that is good to know. > similarly to how we adjust keymaps for laptops? We're currently looking into various ways to make this less painful, specifically for most laptops the problem seems to be the min value and not the max value. And the troublesome min value is the synaptics driver default, not the one we get from the firmware. The problem is we never ask the firmware because even though it has the "I can report min values" capability bit, its "maximum understood request" number is too low, so one of our 2 checks for getting the min value is failing. If we remove that check some models do give us a proper range (but not all, ie the T440s is still wrong). We're currently trying to figure out if it will be safe for all models to remove the "maximum understood request" number check. That should ie remove the quirk for the x240 and possible others. An other option to make this better is to switch the quirks to using pnp-ids, ie the L440 and L540 share the same pnp-id. Once you've merged the firmware_id patches I can take a shot at simplifying the quirk table that way. Downside is that we then probably need to put the firmware_id patches in the various stable kernels. Note that even if we end up moving this to userspace, then we still need the firmware_id, because I believe any userspace solution should be using pnp-ids too. TL;DR: It is complicated and for now we would like to continue with the quirks as we've done sofar. We are aware that this is undesirable from a maintenance pov and are looking into making this better. Regards, Hans -- To unsubscribe from this list: send the line "unsubscribe stable" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html