<p>Maulik V Vaghela <strong>uploaded patch set #5</strong> to this change.</p><p><a href="https://review.coreboot.org/26030">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">soc/intel/cannonlake: Use common i2c support for CNL PCH<br><br>In previous patch, common i2c support was added for soc which uses same<br>configurations as cannonlake pch. For cannonlake soc, we'll switch to common<br>i2c support and remove code from soc folder. This will allow us to<br>reduce redundant copy of common code across multiple soc.<br><br>When common i2c support is used, soc needs to implement one function for<br>i2c which returns soc specific configuration for the IP. This function<br>is implemented inside chip_config.c file which will have all the chip<br>specific information.<br><br>BUG=none<br>BRANCH=none<br>TEST=code compiles with different configurations. No changes in code<br>logic.<br><br>Change-Id: I95b3d1b2b0dfd5fb49859ddf5c42fee44b16dd8c<br>Signed-off-by: Maulik V Vaghela <maulik.v.vaghela@intel.com><br>---<br>M src/soc/intel/cannonlake/Kconfig<br>M src/soc/intel/cannonlake/Makefile.inc<br>M src/soc/intel/cannonlake/chip_config.c<br>D src/soc/intel/cannonlake/i2c.c<br>4 files changed, 23 insertions(+), 87 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/30/26030/5</pre><p>To view, visit <a href="https://review.coreboot.org/26030">change 26030</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/26030"/><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-MessageType: newpatchset </div>
<div style="display:none"> Gerrit-Change-Id: I95b3d1b2b0dfd5fb49859ddf5c42fee44b16dd8c </div>
<div style="display:none"> Gerrit-Change-Number: 26030 </div>
<div style="display:none"> Gerrit-PatchSet: 5 </div>
<div style="display:none"> Gerrit-Owner: Maulik V Vaghela <maulik.v.vaghela@intel.com> </div>
<div style="display:none"> Gerrit-Reviewer: Balaji Manigandan <balaji.manigandan@intel.com> </div>
<div style="display:none"> Gerrit-Reviewer: Subrata Banik <subrata.banik@intel.com> </div>
<div style="display:none"> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> </div>