<p><a href="https://review.coreboot.org/25741">View Change</a></p><p>5 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/25741/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/25741/1//COMMIT_MSG@9">Patch Set #1, Line 9:</a> <code style="font-family:monospace,monospace">copy of yorp settings</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Did you compare yorp and bip GPIOs to see if there are any differences at all? If not, it might make sense to reuse the baseboard gpio table. We can fork off if there are differences.</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/25741/1/src/mainboard/google/octopus/variants/bip/gpio.c">File src/mainboard/google/octopus/variants/bip/gpio.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/25741/1/src/mainboard/google/octopus/variants/bip/gpio.c@257">Patch Set #1, Line 257:</a> <code style="font-family:monospace,monospace">__attribute__((weak)) </code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Why weak in variant?</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/25741/1/src/mainboard/google/octopus/variants/bip/gpio.c@283">Patch Set #1, Line 283:</a> <code style="font-family:monospace,monospace">__attribute__((weak))</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Why weak in variant?</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/25741/1/src/mainboard/google/octopus/variants/bip/gpio.c@290">Patch Set #1, Line 290:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><pre style="font-family: monospace,monospace; white-space: pre-wrap;">/* GPIO settings before entering sleep. */<br>static const struct pad_config sleep_gpio_table[] = {<br>};<br><br>const struct pad_config *__attribute__((weak))<br>variant_sleep_gpio_table(size_t *num)<br>{<br>       *num = ARRAY_SIZE(sleep_gpio_table);<br>  return sleep_gpio_table;<br>}<br></pre></blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">No need to add this right now. It can be done when required.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/25741/1/src/mainboard/google/octopus/variants/bip/gpio.c@301">Patch Set #1, Line 301:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><pre style="font-family: monospace,monospace; white-space: pre-wrap;">static const struct cros_gpio cros_gpios[] = {<br> CROS_GPIO_WP_AH(PAD_SCC(GPIO_PCH_WP), GPIO_COMM_SCC_NAME),<br>};<br><br>const struct cros_gpio *__attribute__((weak)) variant_cros_gpios(size_t *num)<br>{<br>    *num = ARRAY_SIZE(cros_gpios);<br>        return cros_gpios;<br>}<br></pre></blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">This will most likely be same across variants. We can add it later if there are differences.</p></li></ul></li></ul><p>To view, visit <a href="https://review.coreboot.org/25741">change 25741</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/25741"/><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: Icc205f576691427d78c9159dfdbced87e41a0517 </div>
<div style="display:none"> Gerrit-Change-Number: 25741 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Shamile Khan <shamile.khan@intel.com> </div>
<div style="display:none"> Gerrit-Reviewer: Furquan Shaikh <furquan@google.com> </div>
<div style="display:none"> Gerrit-Reviewer: Hannah Williams <hannah.williams@intel.com> </div>
<div style="display:none"> Gerrit-Reviewer: Justin TerAvest <teravest@chromium.org> </div>
<div style="display:none"> Gerrit-Reviewer: Shaunak Saha <shaunak.saha@intel.com> </div>
<div style="display:none"> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 19 Apr 2018 20:33:54 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>