From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752829AbeEOKUn (ORCPT ); Tue, 15 May 2018 06:20:43 -0400 Received: from mga18.intel.com ([134.134.136.126]:40959 "EHLO mga18.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752343AbeEOKUl (ORCPT ); Tue, 15 May 2018 06:20:41 -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,403,1520924400"; d="scan'208";a="55333502" Subject: Re: [BUG] i2c-hid: ELAN Touchpad does not work on ASUS X580GD To: Chris Chiu Cc: Daniel Drake , Jian-Hong Pan , Jiri Kosina , Benjamin Tissoires , Jani Nikula , Hans de Goede , 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: From: Jarkko Nikula Message-ID: <7728da79-8a7a-b87d-d09c-b36978b3032e@linux.intel.com> Date: Tue, 15 May 2018 13:20:58 +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 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. -- Jarkko