From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jouke Witteveen Subject: Re: 2nd Fan quirk for Thinkpad P50 causes spurios touchpad/trackpoint events on ThinkPad L570 Date: Tue, 23 Oct 2018 10:16:22 +0200 Message-ID: References: <36111ed4-621c-47a8-bc6a-66cf756a0ac6@ristioja.ee> <20181023035007.iy5vy2z3vuto2d6s@khazad-dum.debian.net> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20181023035007.iy5vy2z3vuto2d6s-ZGHd14iZgfaRjzvQDGKj+xxZW9W5cXbT@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: ibm-acpi-devel-bounces-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org To: hmh-N3TV7GIv+o9fyO9Q7EP/yw@public.gmane.org Cc: agk-mA9ux5SlULTR7s880joybQ@public.gmane.org, pali.rohar-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org, dmitry.torokhov-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org, platform-driver-x86-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, ibm-acpi-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org, linux-input-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, jaak-89mTbI93R4uuvFJfX82//w@public.gmane.org, andriy.shevchenko-VuQAYsv1563Yd54FQh9/CA@public.gmane.org List-Id: platform-driver-x86.vger.kernel.org On Tue, Oct 23, 2018 at 6:06 AM Henrique de Moraes Holschuh wrote: > > On Mon, 15 Oct 2018, Dmitry Torokhov wrote: > > On Sat, Oct 13, 2018 at 6:32 AM Jaak Ristioja wrote: > > > On 27.08.2018 19:22, Jaak Ristioja wrote: > > > > Upgrading Linux from 4.16 to 4.17, a ThinkPad L570 started receiving > > > > spurious input events, mostly right mouse button click events, but also > > > > cursor jumps. > > > > > > > > I have not attempted to understand whether these events come from the > > > > trackpoint or touchpad or some other driver, but I managed to bisect > > > > this issue to commit a986c75a7df0 titled "platform/x86: thinkpad_acpi: > > > > Add 2nd Fan Support for Thinkpad P50" by Alexander Kappner. > > > > > > > > Apparently the quirk mitigation is applied when the BIOS version begins > > > > with N1. The BIOS version on the L570 in question is N1XET57W (1.35 ) > > > > which is probably why this commit causes the described problems on the > > > > ThinkPad L570. How exactly? - I don't know. > > > > > > > > The issue did not reproduce when running some stable 4.17 and 4.18 > > > > kernels with commit a986c75a7df0 reverted. > > > > > > > > Please fix this for future kernels. Thanks! :) > > > > > > Ping. Do you need any additional information? > > > > Sounds like we need tighter check for the quirk, maybe based on > > DMI/Board name? Can we revert the offending commit for now? > > Apparently it has to be fixed properly, because if I understood the > issue correctly, every two-character quirk will trigger on *any* > three-character model that starts with those two characters. This should be (and hopefully is) incorrect. In the tpacpi_check_quirks function, equality between the quirk bios and the actual model bios is tested. For two-character models there are three nonzero bytes, for two character models there are only two. Therefore, a two-character quirk should not trigger on any three-character model. > We could revert the three-character quirk support, but this would cause > regressions as well. I doubt this would fix anything. Maybe the bios id for the L570 that is used by the driver is not the one Jaak Ristioja reported? The detected string is reported in the dmesg output. That said, I do wonder why the fan quirks redefine all matching macros (TPACPI_FAN_Q*) and don't use the generic ones (TPACPI_Q*). This looks like an easy thing to clean up. Regards, - Jouke > -- > Henrique Holschuh > > > _______________________________________________ > ibm-acpi-devel mailing list > ibm-acpi-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org > https://lists.sourceforge.net/lists/listinfo/ibm-acpi-devel