From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932534AbeEHObD (ORCPT ); Tue, 8 May 2018 10:31:03 -0400 Received: from mga17.intel.com ([192.55.52.151]:40612 "EHLO mga17.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932334AbeEHObB (ORCPT ); Tue, 8 May 2018 10:31:01 -0400 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.49,378,1520924400"; d="scan'208";a="38333571" Subject: Re: [BUG] i2c-hid: ELAN Touchpad does not work on ASUS X580GD To: Chris Chiu , Jian-Hong Pan Cc: Jiri Kosina , Benjamin Tissoires , Jani Nikula , Hans de Goede , Dmitry Torokhov , Adrian Salido , Jason Gerecke , linux-input@vger.kernel.org, Andy Shevchenko , Mika Westerberg , Wolfram Sang , linux-i2c@vger.kernel.org, Linux Kernel , Daniel Drake , Linux Upstreaming Team References: From: Jarkko Nikula Message-ID: Date: Tue, 8 May 2018 17:31:16 +0300 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 On 05/07/2018 11:07 AM, Chris Chiu wrote: >> Besides, according to ASUS's input, they observed that after entering >> the OS, the frequency of SCL is too fast (about 600kHz) and the SCL High >> time is around 450ns, not following I2C spec which is 0.6us. >> https://pasteboard.co/HjzSWXd.png >> > Some update, we can make the touchpad work by simply modifying the > clk_rate of spt_i2c_info from 120000000 to 133000000 in intel-lpss-pci.c for > specific PCI ID 8086:a368 ~ a36a (CoffeeLake). Is the clock setting different > for the CoffeeLake series? > Hmm.. at quick look it looks like to be 133 MHz but I want to double check first from other specs. I'm also wondering why it results 600 kHz instead of expected 133/120 * 400 kHz = 443 kHz. Need to check timing parameters too. -- Jarkko