From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751661AbeERHs2 (ORCPT ); Fri, 18 May 2018 03:48:28 -0400 Received: from mail-wm0-f67.google.com ([74.125.82.67]:34094 "EHLO mail-wm0-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751049AbeERHsZ (ORCPT ); Fri, 18 May 2018 03:48:25 -0400 X-Google-Smtp-Source: AB8JxZoeo5IbvILMI0zHqDO5Ytv/Fn0DQWJMszh0XrVoTUN8APtpdkA4AvnfbpoJ7UtVUn68aTJWcw== Subject: Re: [BUG] i2c-hid: ELAN Touchpad does not work on ASUS X580GD To: Jarkko Nikula , Chris Chiu Cc: Daniel Drake , Jian-Hong Pan , Jiri Kosina , Benjamin Tissoires , Jani Nikula , Dmitry Torokhov , Adrian Salido , Jason Gerecke , linux-input , Andy Shevchenko , Mika Westerberg , Wolfram Sang , linux-i2c@vger.kernel.org, Linux Kernel , Linux Upstreaming Team References: <7728da79-8a7a-b87d-d09c-b36978b3032e@linux.intel.com> From: Hans de Goede Message-ID: Date: Fri, 18 May 2018 09:48:22 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.7.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Jarkko, et.al., On 17-05-18 09:48, Jarkko Nikula wrote: > Hi > > On 05/15/2018 01:20 PM, Jarkko Nikula wrote: >> On 05/15/2018 06:22 AM, Chris Chiu wrote: >>> What if I change the 120MHz to 180MHz and then make sure that the I2C operates >>> in target FS mode frequency 400kHz via scope? Would there be any side effect? >>> Maybe some other busses frequency could be also affected and causing some other >>> component malfunction? >>> >> Should be safe. It is only clock rate information when registering a fixed clock with known rate in intel-lpss.c and i2c-designware uses that info when calculating the timing parameters. I.e. it doesn't change any internal clocks. >> >> I'm trying to find a contact who can confirm what is the expected rate of I2C input clock and is it common to all Cannon Lake HW. >> > I got confirmation that input clock is actually even higher 216 Mhz. > > While checking does it cover all of those CNL CNL-LP and CNL-H PCI IDs may I add your Jian-Hong, Chris and Daniel email addresses to Repored-by tags in a fix patch? Could it be the i2c input clock definition in drivers/mfd/intel-lpss-pci.c is also wrong for Apollo Lake (N3450) ? There are lots of people having various issues with i2c attached touchpads on Apollo Lake devices, this bug: https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1728244 Is sort of a collection bug for these. Various models laptops, lots of reporters. Note not sure thie is an i2c-designware issue, but it would be good to double check the input clock on Apollo Lake. I've checked the datasheet and the datasheet mentions 133MHz as "serial input clk" in the lpio_bxt_regs Registers Summary, which is also part of the LPSS, no clk is mentioned in the "Summary of DW_apb_i2c_mem_map_DW_apb_i2c_addr_block1 Registers". Regards, Hans