<p>Daisuke Nojiri has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/21489">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">vboot: Add config to enable EC EFS support<br><br>This patch makes coreboot set VBSD_EC_EFS flag if CONFIG_VBOOT_EC_EFS<br>is set. Depthcharge/Vboot uses this flag to determine whether it can<br>update EC when it's already running a RW copy.<br><br>BUG=b:65028930<br>BRANCH=none<br>CQ-DEPEND=CL:616346<br>TEST=Verify soft sync runs successfully on Fizz.<br><br>Change-Id: Ic51ddd4819262162a8d8519461c4ace57ee01cb5<br>Signed-off-by: Daisuke Nojiri <dnojiri@chromium.org><br>---<br>M src/vboot/Kconfig<br>M src/vboot/vboot_handoff.c<br>2 files changed, 15 insertions(+), 3 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/89/21489/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/src/vboot/Kconfig b/src/vboot/Kconfig<br>index 8c8dffe..14490d6 100644<br>--- a/src/vboot/Kconfig<br>+++ b/src/vboot/Kconfig<br>@@ -167,6 +167,15 @@<br>        Whether the EC (or PD) is slow to update and needs to display a<br>       screen that informs the user the update is happening.<br> <br>+config VBOOT_EC_EFS<br>+     bool "Run software sync for EFS EC"<br>+        default n<br>+    depends on VBOOT_EC_SOFTWARE_SYNC<br>+    help<br>+   CrosEC can support EFS: Early Firmware Selection. If it's enabled,<br>+       software sync need to also support it. This switch enables EFS<br>+       support for software sync.<br>+<br> config VBOOT_PHYSICAL_DEV_SWITCH<br>    bool<br>  default n<br>diff --git a/src/vboot/vboot_handoff.c b/src/vboot/vboot_handoff.c<br>index 64c8ca6..974fe78 100644<br>--- a/src/vboot/vboot_handoff.c<br>+++ b/src/vboot/vboot_handoff.c<br>@@ -83,12 +83,15 @@<br>   /* TODO: Set these in depthcharge */<br>  if (!IS_ENABLED(CONFIG_VBOOT_PHYSICAL_DEV_SWITCH))<br>            vb_sd->flags |= VBSD_HONOR_VIRT_DEV_SWITCH;<br>-       if (IS_ENABLED(CONFIG_VBOOT_EC_SOFTWARE_SYNC))<br>+       if (IS_ENABLED(CONFIG_VBOOT_EC_SOFTWARE_SYNC)) {<br>              vb_sd->flags |= VBSD_EC_SOFTWARE_SYNC;<br>+            if (IS_ENABLED(CONFIG_VBOOT_EC_SLOW_UPDATE))<br>+                 vb_sd->flags |= VBSD_EC_SLOW_UPDATE;<br>+              if (IS_ENABLED(CONFIG_VBOOT_EC_EFS))<br>+                 vb_sd->flags |= VBSD_EC_EFS;<br>+      }<br>     if (!IS_ENABLED(CONFIG_VBOOT_PHYSICAL_REC_SWITCH))<br>            vb_sd->flags |= VBSD_BOOT_REC_SWITCH_VIRTUAL;<br>-     if (IS_ENABLED(CONFIG_VBOOT_EC_SLOW_UPDATE))<br>-         vb_sd->flags |= VBSD_EC_SLOW_UPDATE;<br>       if (IS_ENABLED(CONFIG_VBOOT_OPROM_MATTERS)) {<br>                 vb_sd->flags |= VBSD_OPROM_MATTERS;<br>                /*<br></pre><p>To view, visit <a href="https://review.coreboot.org/21489">change 21489</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/21489"/><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: Ic51ddd4819262162a8d8519461c4ace57ee01cb5 </div>
<div style="display:none"> Gerrit-Change-Number: 21489 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Daisuke Nojiri <dnojiri@chromium.org> </div>