<p>Marshall Dawson has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/21748">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">amd/stoneyridge: Fix pm_read16 argument size<br><br>Make pm_read16() consistent with the other PM register access functions.<br><br>Change-Id: Iba017b8090ed07d8684cc7f396a3e9a942b3ad95<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, 2 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/48/21748/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 fdb0995..054b36a 100644<br>--- a/src/soc/amd/stoneyridge/include/soc/southbridge.h<br>+++ b/src/soc/amd/stoneyridge/include/soc/southbridge.h<br>@@ -194,7 +194,7 @@<br> void lpc_wideio_512_window(uint16_t base);<br> void lpc_wideio_16_window(uint16_t base);<br> u8 pm_read8(u8 reg);<br>-u16 pm_read16(u16 reg);<br>+u16 pm_read16(u8 reg);<br> void pm_write8(u8 reg, u8 value);<br> void pm_write16(u8 reg, u16 value);<br> int s3_load_nvram_early(int size, u32 *old_dword, int nvram_pos);<br>diff --git a/src/soc/amd/stoneyridge/southbridge.c b/src/soc/amd/stoneyridge/southbridge.c<br>index 1ca742f..655a15a 100644<br>--- a/src/soc/amd/stoneyridge/southbridge.c<br>+++ b/src/soc/amd/stoneyridge/southbridge.c<br>@@ -55,7 +55,7 @@<br>     write16((void *)(PM_MMIO_BASE + reg), value);<br> }<br> <br>-u16 pm_read16(u16 reg)<br>+u16 pm_read16(u8 reg)<br> {<br>     return read16((void *)(PM_MMIO_BASE + reg));<br> }<br></pre><p>To view, visit <a href="https://review.coreboot.org/21748">change 21748</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/21748"/><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: Iba017b8090ed07d8684cc7f396a3e9a942b3ad95 </div>
<div style="display:none"> Gerrit-Change-Number: 21748 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Marshall Dawson <marshalldawson3rd@gmail.com> </div>