<p style="white-space: pre-wrap; word-wrap: break-word;">Can you please split this change into two CLs:<br>1. For driver support<br>2. For mainboard changes</p><p><a href="https://review.coreboot.org/27172">View Change</a></p><p>10 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/27172/1//COMMIT_MSG">Commit Message:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/27172/1//COMMIT_MSG@7">Patch Set #1, Line 7:</a> <code style="font-family:monospace,monospace">oogle/poppy</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">mb/google/poppy/variants/nocturne</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/27172/1/src/drivers/i2c/sx9310/Kconfig">File src/drivers/i2c/sx9310/Kconfig:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/27172/1/src/drivers/i2c/sx9310/Kconfig@3">Patch Set #1, Line 3:</a> <code style="font-family:monospace,monospace">       default n</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">depends on HAVE_ACPI_TABLES</p><p style="white-space: pre-wrap; word-wrap: break-word;">Then you don't have to add #ifdef for HAVE_ACPI_TABLES in .c file</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/27172/1/src/drivers/i2c/sx9310/chip.h">File src/drivers/i2c/sx9310/chip.h:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/27172/1/src/drivers/i2c/sx9310/chip.h@4">Patch Set #1, Line 4:</a> <code style="font-family:monospace,monospace">Inc</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">LLC</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/27172/1/src/drivers/i2c/sx9310/registers.h">File src/drivers/i2c/sx9310/registers.h:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/27172/1/src/drivers/i2c/sx9310/registers.h@4">Patch Set #1, Line 4:</a> <code style="font-family:monospace,monospace">Inc</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">LLC</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/27172/1/src/drivers/i2c/sx9310/sx9310.c">File src/drivers/i2c/sx9310/sx9310.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/27172/1/src/drivers/i2c/sx9310/sx9310.c@4">Patch Set #1, Line 4:</a> <code style="font-family:monospace,monospace">Inc</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">LLC</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/27172/1/src/drivers/i2c/sx9310/sx9310.c@4">Patch Set #1, Line 4:</a> <code style="font-family:monospace,monospace">2016</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">2018</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/27172/1/src/drivers/i2c/sx9310/sx9310.c@18">Patch Set #1, Line 18:</a> <code style="font-family:monospace,monospace">#include <console/console.h></code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Is this required?</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/27172/1/src/drivers/i2c/sx9310/sx9310.c@22">Patch Set #1, Line 22:</a> <code style="font-family:monospace,monospace">#include <gpio.h></code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Is this required?</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/27172/1/src/drivers/i2c/sx9310/sx9310.c@43">Patch Set #1, Line 43:</a> <code style="font-family:monospace,monospace">= NULL</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">No need to set this to NULL. dsd is set on line 64.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/27172/1/src/drivers/i2c/sx9310/sx9310.c@78">Patch Set #1, Line 78:</a> <code style="font-family:monospace,monospace">%sD%03.3X</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">This should be 4-character long.</p></li></ul></li></ul><p>To view, visit <a href="https://review.coreboot.org/27172">change 27172</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/27172"/><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: comment </div>
<div style="display:none"> Gerrit-Change-Id: I7358ee34df873098a86d692cc8a909b0ec5023a8 </div>
<div style="display:none"> Gerrit-Change-Number: 27172 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: Enrico Granata <egranata@chromium.org> </div>
<div style="display:none"> Gerrit-Reviewer: Duncan Laurie <dlaurie@chromium.org> </div>
<div style="display:none"> Gerrit-Reviewer: Furquan Shaikh <furquan@google.com> </div>
<div style="display:none"> Gerrit-Reviewer: Gwendal Grignou <gwendal@chromium.org> </div>
<div style="display:none"> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 20 Jun 2018 19:13:57 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>