<p>Matt Delco has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/28105">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">Kconfig: add option for convertible<br><br>This change adds a boolean open to flag if a device is a convertible.<br>This can be used to control SMBIOS enclosure type, and will be used in a<br>later change to control the presense of an ACPI tablet model device when<br>using a Chrome EC.<br><br>Change-Id: I2cc47d3b6233198a35ffb9844ef4a839328238d5<br>Signed-off-by: Matt Delco <delco@chromium.org><br>---<br>M src/Kconfig<br>M src/arch/x86/acpi.c<br>2 files changed, 10 insertions(+), 2 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/05/28105/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/Kconfig b/src/Kconfig</span><br><span>index bce3e32..0e40e2a 100644</span><br><span>--- a/src/Kconfig</span><br><span>+++ b/src/Kconfig</span><br><span>@@ -336,6 +336,11 @@</span><br><span>  default n</span><br><span>    bool</span><br><span> </span><br><span style="color: hsl(120, 100%, 40%);">+# controls chasis type (and tablet mode detection with Chrome EC)</span><br><span style="color: hsl(120, 100%, 40%);">+config SYSTEM_TYPE_CONVERTIBLE</span><br><span style="color: hsl(120, 100%, 40%);">+   default n</span><br><span style="color: hsl(120, 100%, 40%);">+     bool</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span> config CBFS_AUTOGEN_ATTRIBUTES</span><br><span>     default n</span><br><span>    bool</span><br><span>@@ -659,11 +664,13 @@</span><br><span>         hex</span><br><span>  depends on GENERATE_SMBIOS_TABLES</span><br><span>    default 0x09 if SYSTEM_TYPE_LAPTOP</span><br><span style="color: hsl(120, 100%, 40%);">+    default 0x1f if SYSTEM_TYPE_CONVERTIBLE</span><br><span>      default 0x03</span><br><span>         help</span><br><span>           System Enclosure or Chassis Types as defined in SMBIOS specification.</span><br><span>        The default value is SMBIOS_ENCLOSURE_DESKTOP (0x03) or</span><br><span style="color: hsl(0, 100%, 40%);">-         SMBIOS_ENCLOSURE_LAPTOP (0x09) if SYSTEM_TYPE_LAPTOP is set.</span><br><span style="color: hsl(120, 100%, 40%);">+          SMBIOS_ENCLOSURE_LAPTOP (0x09) if SYSTEM_TYPE_LAPTOP is set or</span><br><span style="color: hsl(120, 100%, 40%);">+        SMBIOS_ENCLOSURE_CONVERTIBLE (0x1f) if SYSTEM_TYPE_CONVERTIBLE is set</span><br><span> </span><br><span> endmenu</span><br><span> </span><br><span>diff --git a/src/arch/x86/acpi.c b/src/arch/x86/acpi.c</span><br><span>index 8d7579d..e294695 100644</span><br><span>--- a/src/arch/x86/acpi.c</span><br><span>+++ b/src/arch/x86/acpi.c</span><br><span>@@ -980,7 +980,8 @@</span><br><span>    fadt->x_dsdt_l = (unsigned long)dsdt;</span><br><span>     fadt->x_dsdt_h = 0;</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-      if (IS_ENABLED(CONFIG_SYSTEM_TYPE_LAPTOP))</span><br><span style="color: hsl(120, 100%, 40%);">+    if (IS_ENABLED(CONFIG_SYSTEM_TYPE_LAPTOP) ||</span><br><span style="color: hsl(120, 100%, 40%);">+      IS_ENABLED(CONFIG_SYSTEM_TYPE_CONVERTIBLE))</span><br><span>              fadt->preferred_pm_profile = PM_MOBILE;</span><br><span>   else</span><br><span>                 fadt->preferred_pm_profile = PM_DESKTOP;</span><br><span></span><br></pre><p>To view, visit <a href="https://review.coreboot.org/28105">change 28105</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/28105"/><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: I2cc47d3b6233198a35ffb9844ef4a839328238d5 </div>
<div style="display:none"> Gerrit-Change-Number: 28105 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Matt Delco <delco@chromium.org> </div>