From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 6F155C77B6E for ; Thu, 13 Apr 2023 20:48:35 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229822AbjDMUse (ORCPT ); Thu, 13 Apr 2023 16:48:34 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:38528 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229493AbjDMUsc (ORCPT ); Thu, 13 Apr 2023 16:48:32 -0400 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id F090A9EC9 for ; Thu, 13 Apr 2023 13:47:38 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1681418858; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=HtJyH8SGOogFlfOnGodd9IaFOBzxhFrcUEkKoY2za9c=; b=GEREALTdlTJqAGp/425wv0pa2wSW+Zt0pnue45caHyq13q5T5daPAueNcnllqmjO2mPCwx xzTs7fcxa7MV11bDe9C4BVUkC+9UPhoT4+uZJollvWFS/DgYpP6JIOEMjy8RMLc0e+ZvwK wM8AnEj7Xg75XW/4g4ELkYHV1YvWmhw= Received: from mail-qt1-f199.google.com (mail-qt1-f199.google.com [209.85.160.199]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-563-0WeMEmBQNF-1-cPnp8bLPA-1; Thu, 13 Apr 2023 16:47:34 -0400 X-MC-Unique: 0WeMEmBQNF-1-cPnp8bLPA-1 Received: by mail-qt1-f199.google.com with SMTP id c6-20020ac87d86000000b003e4df699997so10008040qtd.20 for ; Thu, 13 Apr 2023 13:47:34 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1681418854; x=1684010854; h=mime-version:user-agent:content-transfer-encoding:organization :references:in-reply-to:date:cc:to:from:subject:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=HtJyH8SGOogFlfOnGodd9IaFOBzxhFrcUEkKoY2za9c=; b=HUrKXT2k62PzVjh02hLoBBoMjVguSEnZvsO6R78bmv/3PZK+aBBg3fpyqQw6+2IaTK QCdi7sN3AZnhlBf/UrHu5mKO2cmJ2VOq+HJV9Even3GtRlMEE2ghbPUqUtpoJWAHtKaA QSPO7pzAgqzh4NI1Zslex7VB2cTEhKcn8/6iUYDWHZ1+MjH1qu9NSYEuJ9UcZNCyksFs qNLCK9LdH2kMbKvTs5GOiCVTJB22cERFh/WlcmJRziDLHn5uEu5xkiwtHMsbuh3T+BGL iZicFJifOHoZo+kXFhNAuCcsE3MQQwNES61pWAppXJbHOlrSQJVHsdQYTOkYIadFzyva CboQ== X-Gm-Message-State: AAQBX9f+JtE6NGIgO9CrM+zXpMyrUnGzVXsxXYNqaVzXF2FquWZdA+sq 4H9pfDNjdId6hbTk0he3JJI+ki7G7fNncXLRp4Pbzw2r3vKhrNH0my7JEReHiHBVHUiWdB7tSNC D3YKDQVn5hGmPp+H0rjsg7E36 X-Received: by 2002:a05:6214:3016:b0:5ab:56d4:dc43 with SMTP id ke22-20020a056214301600b005ab56d4dc43mr5587079qvb.7.1681418854259; Thu, 13 Apr 2023 13:47:34 -0700 (PDT) X-Google-Smtp-Source: AKy350a8sRNTuKwFoQtKvkf4jEyAXIx03sqtsM/4yGDcBMha7WAuFQVZmn5TalW41FaoMN1GUCoIdA== X-Received: by 2002:a05:6214:3016:b0:5ab:56d4:dc43 with SMTP id ke22-20020a056214301600b005ab56d4dc43mr5587055qvb.7.1681418853990; Thu, 13 Apr 2023 13:47:33 -0700 (PDT) Received: from ?IPv6:2600:4040:5c62:8200::feb? ([2600:4040:5c62:8200::feb]) by smtp.gmail.com with ESMTPSA id dr3-20020a05621408e300b005ef44499177sm668792qvb.116.2023.04.13.13.47.33 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 13 Apr 2023 13:47:33 -0700 (PDT) Message-ID: <063c8f77c216ffac463532023009124542d54c19.camel@redhat.com> Subject: Re: [PATCH] Input: synaptics - disable intertouch for Lenovo L440 From: Lyude Paul To: Jonathan Denose , LKML Cc: Dmitry Torokhov , Wolfram Sang , linux-input@vger.kernel.org, Jonathan Denose , Aman Dhoot , Mark Pearson , Andrew Duggan Date: Thu, 13 Apr 2023 16:47:32 -0400 In-Reply-To: <20230412175311.1.Ieb687047a5b75c7b7ee5dd258207ef5ca9a3b728@changeid> References: <20230412175311.1.Ieb687047a5b75c7b7ee5dd258207ef5ca9a3b728@changeid> Organization: Red Hat Inc. Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable User-Agent: Evolution 3.44.4 (3.44.4-2.fc36) MIME-Version: 1.0 Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org This patch looks fine to me, I'm a bit curious though whether the folks at Synaptics have any idea of some other workaround we might be able to do rat= her than disabling intertouch? Added Andrew Duggan to CC to ask. If we don't get any response from them after a while, feel free to consider this: Reviewed-by: Lyude Paul On Wed, 2023-04-12 at 17:54 -0500, Jonathan Denose wrote: > When intertouch is enabled for the L440 a (deep)sleep/resume > cycle causes the touchpad driver to hang which causes the > touchpad to become unresponsive. Disable intertouch resolves > this issue and the touchpad is fine after resume from sleep. >=20 > Additionally, when the PNP id for the L440 is only removed > from the topbuttonpad_pnp_ids list, a message is logged to > enable psmouse.synaptics_intertouch, which would cause the > sleep/resume issue again. By removing the PNP id from > topbutton_pnp_ids and then adding it to the > forcepad_pnp_ids array, intertouch is disabled and the > message is not logged. >=20 > Signed-off-by: Jonathan Denose > --- >=20 > drivers/input/mouse/synaptics.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) >=20 > diff --git a/drivers/input/mouse/synaptics.c b/drivers/input/mouse/synapt= ics.c > index fa021af8506e4..77a4f58128e84 100644 > --- a/drivers/input/mouse/synaptics.c > +++ b/drivers/input/mouse/synaptics.c > @@ -150,7 +150,6 @@ static const char * const topbuttonpad_pnp_ids[] =3D = { > "LEN2001", /* Edge E431 */ > "LEN2002", /* Edge E531 */ > "LEN2003", > - "LEN2004", /* L440 */ > "LEN2005", > "LEN2006", /* Edge E440/E540 */ > "LEN2007", > @@ -198,6 +197,7 @@ static const char * const smbus_pnp_ids[] =3D { > static const char * const forcepad_pnp_ids[] =3D { > "SYN300D", > "SYN3014", > + "LEN2004", /* L440 */ > NULL > }; > =20 > @@ -1769,6 +1769,8 @@ static int synaptics_create_intertouch(struct psmou= se *psmouse, > .flags =3D I2C_CLIENT_HOST_NOTIFY, > }; > =20 > + psmouse_dbg(psmouse, "topbuttonpad is: %s\n", topbuttonpad ? "true" : "= false"); > + > return psmouse_smbus_init(psmouse, &intertouch_board, > &pdata, sizeof(pdata), true, > leave_breadcrumbs); --=20 Cheers, Lyude Paul (she/her) Software Engineer at Red Hat