<p>Furquan Shaikh would like Duncan Laurie to <strong>review</strong> this change.</p><p><a href="https://review.coreboot.org/25846">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">vbnv: Add function to read UDC enable flag<br><br>BUG=b:78577893<br>BRANCH=poppy<br><br>Change-Id: Ifd1e9b0781ffee242d695b72287632bc944a50c7<br>Signed-off-by: Duncan Laurie <dlaurie@google.com><br>Signed-off-by: Furquan Shaikh <furquan@google.com><br>---<br>M src/security/vboot/vbnv.c<br>M src/security/vboot/vbnv.h<br>M src/security/vboot/vbnv_layout.h<br>3 files changed, 12 insertions(+), 0 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/46/25846/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/security/vboot/vbnv.c b/src/security/vboot/vbnv.c</span><br><span>index 0fecacd..96507c0 100644</span><br><span>--- a/src/security/vboot/vbnv.c</span><br><span>+++ b/src/security/vboot/vbnv.c</span><br><span>@@ -147,6 +147,13 @@</span><br><span>        return (vbnv_data(BOOT_OFFSET) & BOOT_OPROM_NEEDED) ? 1 : 0;</span><br><span> }</span><br><span> </span><br><span style="color: hsl(120, 100%, 40%);">+/* Read the UDC enable flag from VBNV. */</span><br><span style="color: hsl(120, 100%, 40%);">+int vbnv_udc_enable_flag(void)</span><br><span style="color: hsl(120, 100%, 40%);">+{</span><br><span style="color: hsl(120, 100%, 40%);">+       vbnv_setup();</span><br><span style="color: hsl(120, 100%, 40%);">+ return (vbnv_data(DEV_FLAGS_OFFSET) & DEV_ENABLE_UDC) ? 1 : 0;</span><br><span style="color: hsl(120, 100%, 40%);">+}</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span> void vbnv_init(uint8_t *vbnv_copy)</span><br><span> {</span><br><span>   if (IS_ENABLED(CONFIG_VBOOT_VBNV_CMOS))</span><br><span>diff --git a/src/security/vboot/vbnv.h b/src/security/vboot/vbnv.h</span><br><span>index 0288d0d..00d6443 100644</span><br><span>--- a/src/security/vboot/vbnv.h</span><br><span>+++ b/src/security/vboot/vbnv.h</span><br><span>@@ -26,6 +26,10 @@</span><br><span> int get_recovery_mode_from_vbnv(void);</span><br><span> void set_recovery_mode_into_vbnv(int recovery_reason);</span><br><span> int vboot_wants_oprom(void);</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+/* Read the UDC enable flag from VBNV. */</span><br><span style="color: hsl(120, 100%, 40%);">+int vbnv_udc_enable_flag(void);</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span> /* Initialize and read vbnv. This is used in the main vboot logic path. */</span><br><span> void vbnv_init(uint8_t *vbnv_copy);</span><br><span> /* Reset vbnv snapshot to a known state. */</span><br><span>diff --git a/src/security/vboot/vbnv_layout.h b/src/security/vboot/vbnv_layout.h</span><br><span>index 1dc01c9..a9326e4 100644</span><br><span>--- a/src/security/vboot/vbnv_layout.h</span><br><span>+++ b/src/security/vboot/vbnv_layout.h</span><br><span>@@ -40,6 +40,7 @@</span><br><span> #define DEV_FLAGS_OFFSET             4</span><br><span> #define DEV_BOOT_USB_MASK               0x01</span><br><span> #define DEV_BOOT_SIGNED_ONLY_MASK       0x02</span><br><span style="color: hsl(120, 100%, 40%);">+#define DEV_ENABLE_UDC                  0x40</span><br><span> </span><br><span> #define MISC_FLAGS_OFFSET            8</span><br><span> #define MISC_FLAGS_BATTERY_CUTOFF_MASK  0x08</span><br><span></span><br></pre><p>To view, visit <a href="https://review.coreboot.org/25846">change 25846</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/25846"/><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: Ifd1e9b0781ffee242d695b72287632bc944a50c7 </div>
<div style="display:none"> Gerrit-Change-Number: 25846 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Furquan Shaikh <furquan@google.com> </div>
<div style="display:none"> Gerrit-Reviewer: Duncan Laurie <dlaurie@chromium.org> </div>