<p>Rizwan Qureshi has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/20993">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">soc/intel/skylake: Fix gpio pad reset config discrepency<br><br>The pad reset config field values for RSMRST differ between<br>GPP (Primary well GPIOs) and GPD (Deep Sleep Well GPIOs) pads.<br>Currently same value (11b) is being used for both. To eliminate<br>the confusion add separate macros for each config.<br><br>BUG=b:64386481<br>TEST=Check if the pad config is repserved across reset when RSMRST is selected<br><br>Change-Id: Ib5c2b90a5af022ceb7312ff56fa8b4cc31ef8b8c<br>Signed-off-by: Rizwan Qureshi <rizwan.qureshi@intel.com><br>---<br>M src/soc/intel/skylake/include/soc/gpio_defs.h<br>1 file changed, 2 insertions(+), 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/93/20993/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/src/soc/intel/skylake/include/soc/gpio_defs.h b/src/soc/intel/skylake/include/soc/gpio_defs.h<br>index 34461f2..90c54c5 100644<br>--- a/src/soc/intel/skylake/include/soc/gpio_defs.h<br>+++ b/src/soc/intel/skylake/include/soc/gpio_defs.h<br>@@ -224,7 +224,8 @@<br> #define  PADRSTCFG_DSW_PWROK       0<br> #define  PADRSTCFG_DEEP             1<br> #define  PADRSTCFG_PLTRST   2<br>-#define  PADRSTCFG_RSMRST   3<br>+#define  PADRSTCFG_GPP_RSMRST       0<br>+#define  PADRSTCFG_GPD_RSMRST       3<br>     /* RXPADSTSEL - raw signal or internal state */<br> #define  RXPADSTSEL_SHIFT     29<br> #define  RXPADSTSEL_MASK   0x1<br></pre><p>To view, visit <a href="https://review.coreboot.org/20993">change 20993</a>. To unsubscribe, 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/20993"/><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: newchange </div>
<div style="display:none"> Gerrit-Change-Id: Ib5c2b90a5af022ceb7312ff56fa8b4cc31ef8b8c </div>
<div style="display:none"> Gerrit-Change-Number: 20993 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Rizwan Qureshi <rizwan.qureshi@intel.com> </div>