<p>Yang Jiaxun has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/21944">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">mainboard/lenovo: Add support for the Lenovo ThinkPad X230i<br><br>The ThinkPad x230i have a similar main broad with ThinkPad x230 but it has some different CPUs (SandyBridge i3, SandyBridge Celeron, IvyBridge i3) and the VGA_BIOS_ID differs.<br>This commit is to support this by using x230's mainboard directory, add a choice for x230i's different CPUs type and setting the VGA_BIOS_ID in Kconfig.<br><br>Change-Id: Icbdbc7a08145c4a2dc54a7fb5299cb104dfe6cb9<br>Signed-off-by: Yang Jiaxun <yjx@flygoat.com><br>---<br>M src/mainboard/lenovo/x230/Kconfig<br>M src/mainboard/lenovo/x230/Kconfig.name<br>2 files changed, 26 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/44/21944/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/src/mainboard/lenovo/x230/Kconfig b/src/mainboard/lenovo/x230/Kconfig<br>index 39af0e3..357ecfa 100644<br>--- a/src/mainboard/lenovo/x230/Kconfig<br>+++ b/src/mainboard/lenovo/x230/Kconfig<br>@@ -1,4 +1,4 @@<br>-if BOARD_LENOVO_X230<br>+if BOARD_LENOVO_X230 || BOARD_LENOVO_X230I<br> <br> config BOARD_SPECIFIC_OPTIONS # dummy<br>      def_bool y<br>@@ -24,6 +24,24 @@<br> <br>     # Workaround for EC/KBC IRQ1.<br>         select SERIRQ_CONTINUOUS_MODE<br>+<br>+choice<br>+<br>+ depends on BOARD_LENOVO_X230I<br>+<br>+     prompt "X230i CPU TYPE"<br>+<br>+ config BROAD_HAVE_SANDYBRIDGE_I3_CPU<br>+         bool "X230i with SandyBridge i3 CPU"<br>+               default y<br>+    <br>+     config BROAD_HAVE_SANDYBRIDGE_CELERON_CPU<br>+            bool "X230i with SandyBridge Celeron CPU"<br>+<br>+       config BROAD_HAVE_IVYBRIDGE_I3_CPU<br>+           bool "X230i with IvyBridge i3 CPU"<br>+<br>+endchoice<br> <br> config HAVE_IFD_BIN<br>  bool<br>@@ -55,10 +73,14 @@<br> <br> config VGA_BIOS_FILE<br>   string<br>+       default "pci8086,0116.rom" if BROAD_HAVE_SANDYBRIDGE_I3_CPU<br>+        default "pci8086,0106.rom" if BROAD_HAVE_SANDYBRIDGE_CELERON_CPU<br>    default "pci8086,0166.rom"<br> <br> config VGA_BIOS_ID<br>  string<br>+       default "8086,0116" if BROAD_HAVE_SANDYBRIDGE_I3_CPU<br>+       default "8086,0106" if BROAD_HAVE_SANDYBRIDGE_CELERON_CPU<br>   default "8086,0166"<br> <br> config MAINBOARD_PCI_SUBSYSTEM_VENDOR_ID<br>diff --git a/src/mainboard/lenovo/x230/Kconfig.name b/src/mainboard/lenovo/x230/Kconfig.name<br>index d207653..a571598 100644<br>--- a/src/mainboard/lenovo/x230/Kconfig.name<br>+++ b/src/mainboard/lenovo/x230/Kconfig.name<br>@@ -1,2 +1,5 @@<br> config BOARD_LENOVO_X230<br>      bool "ThinkPad X230"<br>+<br>+config BOARD_LENOVO_X230I<br>+        bool "ThinkPad X230i"<br></pre><p>To view, visit <a href="https://review.coreboot.org/21944">change 21944</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/21944"/><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: Icbdbc7a08145c4a2dc54a7fb5299cb104dfe6cb9 </div>
<div style="display:none"> Gerrit-Change-Number: 21944 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Yang Jiaxun <yjx@flygoat.com> </div>