<p>Nico Huber <strong>uploaded patch set #2</strong> to this change.</p><p><a href="https://review.coreboot.org/c/coreboot/+/29680">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">Kconfig: Unify power-after-failure options<br><br>The newest and most useful incarnation was hiding in soc/intel/common/.<br>We move it into the Mainboard menu and extend it with various flags to<br>be selected to control the default and which options are visible. Also<br>add a new `int` config MAINBOARD_POWER_FAILURE_STATE that moves the<br>boolean to int conversion into Kconfig:<br>  0 - S5<br>  1 - S0<br>  2 - previous state<br><br>This patch focuses on the Kconfig code. The C code could be unified as<br>well, e.g. starting with a common enum and safe wrapper around the<br>get_option() call.<br><br>TEST=Did what-jenkins-does with and without this commit and compared<br>     binaries. Nothing changed for the default configurations.<br><br>Change-Id: I61259f864c8a8cfc7099cc2699059f972fa056c0<br>Signed-off-by: Nico Huber <nico.h@gmx.de><br>---<br>M src/mainboard/Kconfig<br>M src/mainboard/asrock/h81m-hds/Kconfig<br>M src/mainboard/asus/kcma-d8/Kconfig<br>M src/mainboard/asus/kfsn4-dre/Kconfig<br>M src/mainboard/asus/kgpe-d16/Kconfig<br>M src/mainboard/msi/ms9652_fam10/Kconfig<br>M src/mainboard/samsung/lumpy/Kconfig<br>M src/mainboard/samsung/stumpy/Kconfig<br>M src/soc/intel/broadwell/Kconfig<br>M src/soc/intel/broadwell/lpc.c<br>M src/soc/intel/broadwell/smihandler.c<br>M src/soc/intel/common/block/include/intelblocks/pmclib.h<br>M src/soc/intel/common/block/pmc/Kconfig<br>M src/soc/intel/common/block/pmc/pmclib.c<br>M src/southbridge/amd/agesa/hudson/sm.c<br>M src/southbridge/amd/amd8111/Kconfig<br>M src/southbridge/amd/amd8111/acpi.c<br>M src/southbridge/amd/pi/hudson/sm.c<br>M src/southbridge/amd/sb700/Kconfig<br>M src/southbridge/amd/sb700/sm.c<br>M src/southbridge/amd/sb800/sm.c<br>M src/southbridge/intel/bd82x6x/lpc.c<br>M src/southbridge/intel/common/Kconfig<br>M src/southbridge/intel/common/pmutil.h<br>M src/southbridge/intel/common/smihandler.c<br>M src/southbridge/intel/fsp_bd82x6x/Kconfig<br>M src/southbridge/intel/fsp_bd82x6x/lpc.c<br>M src/southbridge/intel/fsp_bd82x6x/pch.h<br>M src/southbridge/intel/fsp_bd82x6x/smihandler.c<br>M src/southbridge/intel/fsp_i89xx/Kconfig<br>M src/southbridge/intel/fsp_i89xx/lpc.c<br>M src/southbridge/intel/fsp_i89xx/pch.h<br>M src/southbridge/intel/fsp_i89xx/smihandler.c<br>M src/southbridge/intel/fsp_rangeley/soc.h<br>M src/southbridge/intel/i82801dx/Kconfig<br>M src/southbridge/intel/i82801dx/i82801dx.h<br>M src/southbridge/intel/i82801dx/lpc.c<br>M src/southbridge/intel/i82801dx/smihandler.c<br>M src/southbridge/intel/i82801gx/Kconfig<br>M src/southbridge/intel/i82801gx/i82801gx.h<br>M src/southbridge/intel/i82801gx/lpc.c<br>M src/southbridge/intel/i82801gx/smihandler.c<br>M src/southbridge/intel/i82801ix/i82801ix.h<br>M src/southbridge/intel/i82801ix/lpc.c<br>M src/southbridge/intel/i82801jx/Kconfig<br>M src/southbridge/intel/i82801jx/i82801jx.h<br>M src/southbridge/intel/i82801jx/lpc.c<br>M src/southbridge/intel/ibexpeak/Kconfig<br>M src/southbridge/intel/ibexpeak/lpc.c<br>M src/southbridge/intel/ibexpeak/pch.h<br>M src/southbridge/intel/ibexpeak/smihandler.c<br>M src/southbridge/intel/lynxpoint/Kconfig<br>M src/southbridge/intel/lynxpoint/lpc.c<br>M src/southbridge/intel/lynxpoint/pch.h<br>M src/southbridge/intel/lynxpoint/smihandler.c<br>M src/southbridge/nvidia/ck804/Kconfig<br>M src/southbridge/nvidia/ck804/lpc.c<br>M src/southbridge/nvidia/mcp55/Kconfig<br>M src/southbridge/nvidia/mcp55/lpc.c<br>M src/superio/nuvoton/nct5572d/Kconfig<br>M src/superio/nuvoton/nct5572d/superio.c<br>M src/superio/winbond/w83667hg-a/Kconfig<br>M src/superio/winbond/w83667hg-a/superio.c<br>63 files changed, 129 insertions(+), 176 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/80/29680/2</pre><p>To view, visit <a href="https://review.coreboot.org/c/coreboot/+/29680">change 29680</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/c/coreboot/+/29680"/><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-Change-Id: I61259f864c8a8cfc7099cc2699059f972fa056c0 </div>
<div style="display:none"> Gerrit-Change-Number: 29680 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: Nico Huber <nico.h@gmx.de> </div>
<div style="display:none"> Gerrit-Reviewer: HAOUAS Elyes <ehaouas@noos.fr> </div>
<div style="display:none"> Gerrit-Reviewer: Nico Huber <nico.h@gmx.de> </div>
<div style="display:none"> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> </div>
<div style="display:none"> Gerrit-CC: Angel Pons <th3fanbus@gmail.com> </div>
<div style="display:none"> Gerrit-MessageType: newpatchset </div>