<p>Marshall Dawson has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/22725">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">amd/stoneyridge: Remove unused S3 NVRAM save/restore<br><br>Remove the BiosRam read and write functions that were brought over from<br>the hudson source. The functionality will be superseded later with new<br>general-purpose functions.<br><br>Change-Id: Ib80c66b838fdbdd388a392b4fedaac36bf0bbb0c<br>Signed-off-by: Marshall Dawson <marshalldawson3rd@gmail.com><br>---<br>M src/soc/amd/stoneyridge/include/soc/southbridge.h<br>M src/soc/amd/stoneyridge/southbridge.c<br>2 files changed, 0 insertions(+), 33 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/25/22725/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/src/soc/amd/stoneyridge/include/soc/southbridge.h b/src/soc/amd/stoneyridge/include/soc/southbridge.h<br>index bdd5895..afe8e3c 100644<br>--- a/src/soc/amd/stoneyridge/include/soc/southbridge.h<br>+++ b/src/soc/amd/stoneyridge/include/soc/southbridge.h<br>@@ -311,8 +311,6 @@<br> void biosram_write32(uint8_t offset, uint32_t value);<br> uint16_t pm_acpi_pm_cnt_blk(void);<br> uint16_t pm_acpi_pm_evt_blk(void);<br>-int s3_load_nvram_early(int size, u32 *old_dword, int nvram_pos);<br>-int s3_save_nvram_early(u32 dword, int size, int nvram_pos);<br> void bootblock_fch_early_init(void);<br> <br> #endif /* __STONEYRIDGE_H__ */<br>diff --git a/src/soc/amd/stoneyridge/southbridge.c b/src/soc/amd/stoneyridge/southbridge.c<br>index 5010e9b..49a69e1 100644<br>--- a/src/soc/amd/stoneyridge/southbridge.c<br>+++ b/src/soc/amd/stoneyridge/southbridge.c<br>@@ -245,37 +245,6 @@<br> sb_lpc_decode();<br> }<br> <br>-int s3_save_nvram_early(u32 dword, int size, int nvram_pos)<br>-{<br>- int i;<br>- printk(BIOS_DEBUG, "Writing %x of size %d to nvram pos: %d\n",<br>- dword, size, nvram_pos);<br>-<br>- for (i = 0; i < size; i++) {<br>- outb(nvram_pos, BIOSRAM_INDEX);<br>- outb((dword >> (8 * i)) & 0xff, BIOSRAM_DATA);<br>- nvram_pos++;<br>- }<br>-<br>- return nvram_pos;<br>-}<br>-<br>-int s3_load_nvram_early(int size, u32 *old_dword, int nvram_pos)<br>-{<br>- u32 data = *old_dword;<br>- int i;<br>- for (i = 0; i < size; i++) {<br>- outb(nvram_pos, BIOSRAM_INDEX);<br>- data &= ~(0xff << (i * 8));<br>- data |= inb(BIOSRAM_DATA) << (i * 8);<br>- nvram_pos++;<br>- }<br>- *old_dword = data;<br>- printk(BIOS_DEBUG, "Loading %x of size %d to nvram pos:%d\n",<br>- *old_dword, size, nvram_pos-size);<br>- return nvram_pos;<br>-}<br>-<br> void sb_enable(device_t dev)<br> {<br> printk(BIOS_DEBUG, "%s\n", __func__);<br></pre><p>To view, visit <a href="https://review.coreboot.org/22725">change 22725</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/22725"/><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: Ib80c66b838fdbdd388a392b4fedaac36bf0bbb0c </div>
<div style="display:none"> Gerrit-Change-Number: 22725 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Marshall Dawson <marshalldawson3rd@gmail.com> </div>