<p>Marshall Dawson has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/22642">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">soc/amd/common: Add S3 region to Kconfig<br><br>Add the symbols back to Kconfig and make the S3 base and offset<br>depend on HAVE_ACPI_RESUME.  These will form the "backup" region<br>for storing S3 data.  The values are typical for AMD platforms and<br>should allow adequate room for rotation (the Stoney Ridge NV size<br>is 0x4c1 currently).<br><br>Note that the primary choice for the S3 save location will be<br>eventually be determined by the FMAP.<br><br>Change-Id: I67846e47d364ecfc01934e879e3299813fcda1b3<br>Signed-off-by: Marshall Dawson <marshalldawson3rd@gmail.com><br>---<br>M src/soc/amd/common/Kconfig<br>1 file changed, 20 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/42/22642/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/src/soc/amd/common/Kconfig b/src/soc/amd/common/Kconfig<br>index debedac..33713f9 100644<br>--- a/src/soc/amd/common/Kconfig<br>+++ b/src/soc/amd/common/Kconfig<br>@@ -11,4 +11,24 @@<br> <br> source "src/soc/amd/common/block/Kconfig"<br> <br>+config S3_DATA_POS<br>+     hex<br>+  depends on HAVE_ACPI_RESUME<br>+  default 0xffff0000<br>+   help<br>+   For designs not using a flash map, this is the physical address where<br>+        S3 resume data is stored in the SPI flash.  Place this on a 4KB<br>+      boundary for best results. The designer must ensure the region does<br>+          not interfere with anything else in the image.<br>+<br>+config S3_DATA_SIZE<br>+    int<br>+  depends on HAVE_ACPI_RESUME<br>+  default 32768<br>+        help<br>+   For designs not using a flash map, this is the total size of the<br>+     region to be used for S3 resume data in the SPI flash.  Make this<br>+    a multiple of 4KB for best results.  The designer must ensure the<br>+    region does not interfere with anything else in the image.<br>+<br> endif # SOC_AMD_COMMON<br></pre><p>To view, visit <a href="https://review.coreboot.org/22642">change 22642</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/22642"/><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: I67846e47d364ecfc01934e879e3299813fcda1b3 </div>
<div style="display:none"> Gerrit-Change-Number: 22642 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Marshall Dawson <marshalldawson3rd@gmail.com> </div>