<p>Duncan Laurie has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/c/coreboot/+/30061">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">soc/intel/cannonlake: Fix I2C clock input<br><br>The input clock for the I2C controllers was set at 133MHz but should<br>really be 216MHz according to the kernel:<br><br>https://patchwork.kernel.org/patch/10408729/<br>"Intel Cannon Lake PCH has much higher 216 MHz input clock to LPSS I2C<br>than Sunrisepoint which uses 120 MHz. Preliminary information was that<br>both share the same clock rate but actual silicon implements elevated<br>rate for better support for 3.4 MHz high-speed I2C."<br><br>This change was tested on a sarien board where an I2C trackpad that was<br>measuring ~700MHz on I2C and is now measuring ~380MHz.<br><br>Change-Id: I792d1f013da5538a2b8157e2f99b754ca7b6bf70<br>Signed-off-by: Duncan Laurie <dlaurie@google.com><br>---<br>M src/soc/intel/cannonlake/Kconfig<br>1 file changed, 1 insertion(+), 1 deletion(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/61/30061/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/soc/intel/cannonlake/Kconfig b/src/soc/intel/cannonlake/Kconfig</span><br><span>index 78c6dfe..9e007b6 100644</span><br><span>--- a/src/soc/intel/cannonlake/Kconfig</span><br><span>+++ b/src/soc/intel/cannonlake/Kconfig</span><br><span>@@ -197,7 +197,7 @@</span><br><span> </span><br><span> config DRIVERS_I2C_DESIGNWARE_CLOCK_MHZ</span><br><span>         int</span><br><span style="color: hsl(0, 100%, 40%);">-     default 133</span><br><span style="color: hsl(120, 100%, 40%);">+   default 216</span><br><span> </span><br><span> config SOC_INTEL_COMMON_BLOCK_GSPI_MAX</span><br><span>    int</span><br><span></span><br></pre><p>To view, visit <a href="https://review.coreboot.org/c/coreboot/+/30061">change 30061</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://review.coreboot.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://review.coreboot.org/c/coreboot/+/30061"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: coreboot </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I792d1f013da5538a2b8157e2f99b754ca7b6bf70 </div>
<div style="display:none"> Gerrit-Change-Number: 30061 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Duncan Laurie <dlaurie@chromium.org> </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>