Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Subject: Re: [PATCH v2] Input: elantech - fix x_max/y_max values
To:     jingle <jingle.wu@emc.com.tw>,
        'Dmitry Torokhov' <dmitry.torokhov@gmail.com>
Cc:     'Benjamin Tissoires' <benjamin.tissoires@redhat.com>,
        "'Dave.Wang'" <dave.wang@emc.com.tw>, linux-kernel@vger.kernel.org,
        linux-input@vger.kernel.org,
        'Kai-Heng Feng' <kai.heng.feng@canonical.com>,
        'Enrico Weigelt' <info@metux.net>,
        'Allison Randal' <allison@lohutok.net>,
        'Aaron Ma' <aaron.ma@canonical.com>,
        =?UTF-8?B?J+S5iemahi1waG9lbml4Jw==?= <phoenix@emc.com.tw>
References: <PR2PR03MB517975370D754EEF47E062BFE4EB0@PR2PR03MB5179.eurprd03.prod.outlook.com>
 <CAO-hwJ+eua8-fNj9o8CxjFyM0i1+R9ZwPMroF9uLUH6NVu3Tdg@mail.gmail.com>
 <PR2PR03MB51799BF709B3975A08F139F4E4EB0@PR2PR03MB5179.eurprd03.prod.outlook.com>
 <AM6PR03MB5170F5FC30556BEF89C775C2E4EB0@AM6PR03MB5170.eurprd03.prod.outlook.com>
 <AM6PR03MB5170D26019747F5090FE74B5E4C50@AM6PR03MB5170.eurprd03.prod.outlook.com>
 <AM6PR03MB51702AF00C6BA06BBC9F517EE4C50@AM6PR03MB5170.eurprd03.prod.outlook.com>
 <AM6PR03MB51705B32136F527B99CCD907E4C10@AM6PR03MB5170.eurprd03.prod.outlook.com>
 <20200409200746.GP75430@dtor-ws> <001b01d61223$3b1e5fd0$b15b1f70$@emc.com.tw>
From:   Bernd Edlinger <bernd.edlinger@hotmail.de>
Message-ID: <AM8PR10MB4708F28B8B8ED4A9B2D843CFE4219@AM8PR10MB4708.EURPRD10.PROD.OUTLOOK.COM>
Date:   Sat, 29 May 2021 17:17:51 +0200
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101
 Thunderbird/60.6.1
In-Reply-To: <001b01d61223$3b1e5fd0$b15b1f70$@emc.com.tw>
Content-Type: text/plain; charset=gbk
Content-Language: en-US
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-MessageSentRepresentingType: 1
Received: from [192.168.1.101] (84.57.61.94) by AM0PR04CA0058.eurprd04.prod.outlook.com (2603:10a6:208:1::35) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4173.20 via Frontend Transport; Sat, 29 May 2021 15:17:52 +0000
X-MS-PublicTrafficType: Email
X-IncomingHeaderCount: 48
X-EOPAttributedMessage: 0
X-MS-Office365-Filtering-Correlation-Id: 939e2c92-09ae-4c6f-ad99-08d922b4ee0a
X-MS-TrafficTypeDiagnostic: VE1EUR01HT104:
X-Microsoft-Antispam: BCL:0;
X-Microsoft-Antispam-Message-Info: KsMaawi22sAlVCxAZ8dzi6glMumFCq0+21YQIG9n1D6IJ97+4YWgM93pFCM1pH0q6xBensexXaSK89zRGIZFrx32XWCqV6pOOBU3APOW+ZJJ8bG6TWmUH25S3NYGPEZD60LxTPiXBmkO/6sxnGEqFA5AfF3ZRW961qAHEF4SD1j0yb/jrTK1bZivBIEYUpd6LBc9zVXRXXaLPca23d78W6ZTztRbFkpIAT7Yck89upPd/+iFdWwvkQ0C1IHwzrgc/Wf3ZwlUqq7/haAaqQjfplluUiZ2t3c1LJN3NlTmLE6q2EacF0XKLG3MP0IPPRLY+Dit1C8892Ma296OZ7sK8SH5IFzO4hVDRf8CgfpoL9IHOkEL7lTOwH5+PUCx7ztNiadri8vEoiDuenFq56xyAQ==
X-MS-Exchange-AntiSpam-MessageData: 0LZsK1zYUoXLTFaZs0fqOMEvKJc9ZZlsQ5UKO9HVG5MjbZsqNBe9S8VBrQ7jza8XJVRFpJRB8ZyrUKHhFb2NVcvqZYJ7UWy30FhmNrRFRXU9zXJcExJYBprJJvThTikw9Q3MfwoBZssiGD2CA+Pk8Q==
X-OriginatorOrg: outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 939e2c92-09ae-4c6f-ad99-08d922b4ee0a
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 29 May 2021 15:17:53.5837
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 84df9e7f-e9f6-40af-b435-aaaaaaaaaaaa
X-MS-Exchange-CrossTenant-AuthSource: VE1EUR01FT005.eop-EUR01.prod.protection.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Anonymous
X-MS-Exchange-CrossTenant-FromEntityHeader: Internet
X-MS-Exchange-CrossTenant-RMS-PersistedConsumerOrg: 00000000-0000-0000-0000-000000000000
X-MS-Exchange-Transport-CrossTenantHeadersStamped: VE1EUR01HT104
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hi Jingle,

apologies for not responding earlier, but better late than never...

Okay, if I understood your advice correctly, you want me to
elantech_set_absolute_mode from elantech_query_info
for hardware version 2 and 3, before sending the ETP_FW_ID_QUERY
command, while hardware version 4 devices will not need that.

Initially when I wrote this patch I was concerned that there might
be a way how the firmware might be in a different state after doing
this when elantech_setup_smbus is called later.

But meanwhile I convinced myself that this can never happen, since
the elantech_setup_smbus seems to be unreachable for version < 4
devices.

I'll send a new version of the patch later today, if everything works
as expected.


Thanks,
Bernd.


On 4/14/20 8:09 AM, jingle wrote:
> Hi Dmitry, Bernd:
> 
> We have checked the related function internal. 
> It is right to switch the touchpad into absolute mode first to get valid
> results from the ID query call in hardware version 2 and 3.
> 
> Thanks
> jingle
> 
> -----Original Message-----
> From: Dmitry Torokhov [mailto:dmitry.torokhov@gmail.com] 
> Sent: Friday, April 10, 2020 4:08 AM
> To: Bernd Edlinger
> Cc: Benjamin Tissoires; Dave.Wang; jingle; linux-kernel@vger.kernel.org;
> linux-input@vger.kernel.org; Kai-Heng Feng; Enrico Weigelt; Allison Randal;
> Aaron Ma
> Subject: Re: [PATCH v2] Input: elantech - fix x_max/y_max values
> 
> Hi Bernd,
> 
> On Thu, Apr 09, 2020 at 07:38:17PM +0200, Bernd Edlinger wrote:
>> Are you there?
>>
>> Should I re-post the v2 patch, was that dropped somehow?
>>
>> If I don't hear anything I'll assume I just repost, probably as 
>> unchanged v3, right?
> 
> I do not think we ever got confirmation from Dave or Jingle; from my POV I
> really dislike arbitrary mangling of the data.
> 
> I think you are right that the issue is with order of calls, and we need to
> switch the touchpad into absolute mode to get valid results from the ID
> query call. Dave, Jingle, any impot here?
> 
> Benjamin, Kai-Feng, do you know if we try to switch to absolute mode to
> begin with, will it cause issues with SMBus mode?
> 
> Thanks.
> 
>>
>> Thanks,
>> Bernd.
>>
>>
>> On 4/5/20 7:54 AM, Bernd Edlinger wrote:
>>> And, furthermore, there is one thing I find really confusing,
>>>
>>> I do not see the message v2 quoted below which had an updated commit 
>>> message, due to that Benjamin Tissoires request.
>>>
>>> It was from my point of view sent on 2/27/20 11:03 PM, but I cannot 
>>> find it neither on spinics, nor on marc.info.
>>> That is funny.
>>>
>>> Did it reach you guys at all?
>>> Or should I re-send it just in case?
>>>
>>>
>>> Thanks
>>> Bernd.
>>>
>>> On 4/5/20 7:26 AM, Bernd Edlinger wrote:
>>>> Ping...
>>>>
>>>> This patch works fine for me since several weeks, without it I 
>>>> would not be able to use my laptop any more.
>>>>
>>>> Could you please accept this patch?
>>>>
>>>>
>>>> Thanks
>>>> Bernd.
>>>>
>>>>
>>>> On 2/27/20 11:03 PM, Bernd Edlinger wrote:
>>>>> Since commit 37548659bb22 ("Input: elantech - query the min/max 
>>>>> information beforehand too") moved the querying of the x_max/y_max 
>>>>> values from elantech_set_input_params to elantech_query_info, the 
>>>>> returned x_max/y_max values are different than before, at least 
>>>>> for some firmware versions.
>>>>>
>>>>> The reason is likely that this is now done before 
>>>>> elantech_set_absolute_mode does run.  So it may happen that the 
>>>>> returned values are exactly half of what they used to be, which 
>>>>> makes input_report_abs in PS/2 mode report ABS_X values which 
>>>>> exceed the x_max value, which is very annoying since the mouse 
>>>>> stops to move then, and ABS_Y value become negative, which is benign.
>>>>>
>>>>> This was observed with a MSI GX70 laptop:
>>>>>
>>>>> elantech: assuming hardware version 3 (with firmware version 
>>>>> 0x250f01)
>>>>> elantech: Synaptics capabilities query result 0x18, 0x17, 0x0b.
>>>>> elantech: Elan sample query result 05, 0e, 00
>>>>> input: ETPS/2 Elantech Touchpad as /devices/platform/i8042/serio...
>>>>>
>>>>> Correct this by doubling the returned x_max and y_max value for 
>>>>> this specific firmware version.
>>>>>
>>>>> Fixes: 37548659bb22 ("Input: elantech - query the min/max 
>>>>> information beforehand too")
>>>>>
>>>>> Signed-off-by: Bernd Edlinger <bernd.edlinger@hotmail.de>
>>>>> ---
>>>>>  drivers/input/mouse/elantech.c | 4 ++++
>>>>>  1 file changed, 4 insertions(+)
>>>>>
>>>>> diff --git a/drivers/input/mouse/elantech.c 
>>>>> b/drivers/input/mouse/elantech.c index 2d8434b..3399db8 100644
>>>>> --- a/drivers/input/mouse/elantech.c
>>>>> +++ b/drivers/input/mouse/elantech.c
>>>>> @@ -1720,6 +1720,10 @@ static int elantech_query_info(struct 
>>>>> psmouse *psmouse,
>>>>>  
>>>>>  		info->x_max = (0x0f & param[0]) << 8 | param[1];
>>>>>  		info->y_max = (0xf0 & param[0]) << 4 | param[2];
>>>>> +		if (info->fw_version == 0x250f01) {
>>>>> +			info->x_max <<= 1;
>>>>> +			info->y_max <<= 1;
>>>>> +		}
>>>>>  		break;
>>>>>  
>>>>>  	case 4:
>>>>>
> 
> --
> Dmitry
>