Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754748AbdCGHqG (ORCPT ); Tue, 7 Mar 2017 02:46:06 -0500 Received: from emcscan.emc.com.tw ([192.72.220.5]:59324 "EHLO emcscan.emc.com.tw" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754355AbdCGHp6 (ORCPT ); Tue, 7 Mar 2017 02:45:58 -0500 X-Greylist: delayed 2375 seconds by postgrey-1.27 at vger.kernel.org; Tue, 07 Mar 2017 02:44:25 EST From: =?big5?B?ufmxUrph?= To: "'Dmitry Torokhov'" Cc: , , "'Matjaz Hegedic'" , =?big5?B?tsClQMRQILhnsno=?= , References: <1488679982-9825-1-git-send-email-matjaz.hegedic@gmail.com> <20170306195442.GC30488@dtor-ws> In-Reply-To: <20170306195442.GC30488@dtor-ws> Subject: RE: [PATCH] Input: elan_i2c - add ASUS EeeBook X205TA special touchpad fw Date: Tue, 7 Mar 2017 15:05:21 +0800 Message-ID: <00b601d29711$302c1800$90844800$@emc.com.tw> MIME-Version: 1.0 Content-Type: text/plain; charset="big5" Content-Transfer-Encoding: 7bit X-Mailer: Microsoft Outlook 14.0 Thread-Index: AQLzHLA+3YqrWeTwtyCd6vG6WDV/owKrlECxnzKHNIA= Content-Language: zh-tw Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2164 Lines: 76 Hi Dmitry -----Original Message----- From: Dmitry Torokhov [mailto:dmitry.torokhov@gmail.com] Sent: Tuesday, March 07, 2017 3:55 AM To: KT Liao Cc: linux-input@vger.kernel.org; linux-kernel@vger.kernel.org; Matjaz Hegedic Subject: Re: [PATCH] Input: elan_i2c - add ASUS EeeBook X205TA special touchpad fw On Sun, Mar 05, 2017 at 03:13:02AM +0100, Matjaz Hegedic wrote: > EeeBook X205TA is yet another ASUS device with a special touchpad > firmware that needs to be accounted for during initialization, or else > the touchpad will go into an invalid state upon suspend/resume. > Adding the appropriate ic_type and product_id check fixes the problem. KT, does this look reasonable? Are there more ASUS models that need such handling? [KT] : I just discuss it with FW team. We can't confirm it right now because it's an old product. And the solution focus on power-on issue, not suspend/resume. I will let you know once we figure it out. Our FW has modified, the issue should not happen on new models. Thanks KT > > Signed-off-by: Matjaz Hegedic > --- > drivers/input/mouse/elan_i2c_core.c | 22 ++++++++++++---------- > 1 file changed, 12 insertions(+), 10 deletions(-) > > diff --git a/drivers/input/mouse/elan_i2c_core.c > b/drivers/input/mouse/elan_i2c_core.c > index 2c7d287..dde3ad7 100644 > --- a/drivers/input/mouse/elan_i2c_core.c > +++ b/drivers/input/mouse/elan_i2c_core.c > @@ -218,17 +218,19 @@ static int elan_query_product(struct > elan_tp_data *data) > > static int elan_check_ASUS_special_fw(struct elan_tp_data *data) { > - if (data->ic_type != 0x0E) > - return false; > - > - switch (data->product_id) { > - case 0x05 ... 0x07: > - case 0x09: > - case 0x13: > - return true; > - default: > - return false; > + if (data->ic_type == 0x0E) { > + switch (data->product_id) { > + case 0x05 ... 0x07: > + case 0x09: > + case 0x13: > + return true; > + } > } > + /* ASUS EeeBook X205TA */ > + else if (data->ic_type == 0x8 && data->product_id == 0x26) > + return true; > + > + return false; > } > > static int __elan_initialize(struct elan_tp_data *data) > -- > 2.7.4 > Thanks. -- Dmitry