<p>Martin Roth has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/20700">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">soc/dmp/vortex86: Fix CMOS read<br><br>The array of CMOS values that was passed into the read routine was<br>never getting updated.  GCC 7.1 gives a warning on this:<br><br>error:  may be used uninitialized in this function<br><br>Change-Id: I2f7c9b6455761a38598467b001efb0603fd14c32<br>Signed-off-by: Martin Roth <martinroth@google.com><br>---<br>M src/soc/dmp/vortex86ex/southbridge.c<br>1 file changed, 3 insertions(+), 3 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/00/20700/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/src/soc/dmp/vortex86ex/southbridge.c b/src/soc/dmp/vortex86ex/southbridge.c<br>index 451aea6..9306034 100644<br>--- a/src/soc/dmp/vortex86ex/southbridge.c<br>+++ b/src/soc/dmp/vortex86ex/southbridge.c<br>@@ -457,14 +457,14 @@<br> {<br>    /* Read RTC twice and check update-in-progress flag, to make<br>   * sure RTC is correct */<br>-    u8 rtc_old[7], rtc_new[7];<br>+   rtc_new[7];<br>   while (get_rtc_update_in_progress()) ;<br>        unsafe_read_cmos_rtc(rtc_new);<br>        do {<br>-         memcpy(rtc_old, rtc_new, 7);<br>+         memcpy(rtc, rtc_new, 7);<br>              while (get_rtc_update_in_progress()) ;<br>                unsafe_read_cmos_rtc(rtc_new);<br>-       } while (memcmp(rtc_new, rtc_old, 7) != 0);<br>+  } while (memcmp(rtc_new, rtc, 7) != 0);<br> }<br> <br> /*<br></pre><p>To view, visit <a href="https://review.coreboot.org/20700">change 20700</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/20700"/><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: I2f7c9b6455761a38598467b001efb0603fd14c32 </div>
<div style="display:none"> Gerrit-Change-Number: 20700 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Martin Roth <martinroth@google.com> </div>