<p>Richard Spiegel has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/21973">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">soc/amd/stoneyridge/sata.c: Requested cleanup.<br><br>Execute cleanup requested at code review for Change-Id:<br>Ib88a868e654ad127be70ecc506f6b90b784f8d1b<br><br>BUG=b:62200375<br><br>Change-Id: I9058d9102fc8ea0bd03ea089ba98da4590dd3533<br>Signed-off-by: Richard Spiegel <richard.spiegel@silverbackltd.com><br>---<br>M src/soc/amd/stoneyridge/sata.c<br>1 file changed, 4 insertions(+), 4 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/73/21973/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/src/soc/amd/stoneyridge/sata.c b/src/soc/amd/stoneyridge/sata.c<br>index c1e5493..3361107 100644<br>--- a/src/soc/amd/stoneyridge/sata.c<br>+++ b/src/soc/amd/stoneyridge/sata.c<br>@@ -35,8 +35,10 @@<br>    #define SATA_CAPABILITIES_REG 0xfc<br>    #define CFG_CAP_SPM (1<<12)<br> <br>- volatile u32 *ahci_ptr = (u32 *)(pci_read_config32(dev,<br>-                              AHCI_BASE_ADDRESS_REG) & 0xffffff00);<br>+    u32 *ahci_ptr = (void *)(uintptr_t)ALIGN_DOWN(<br>+                                               pci_read_config32(dev,<br>+                                               AHCI_BASE_ADDRESS_REG),<br>+                                              256);<br>         u32 temp;<br> <br>  /* unlock the write-protect */<br>@@ -68,8 +70,6 @@<br> };<br> <br> static const unsigned short pci_device_ids[] = {<br>- PCI_DEVICE_ID_AMD_SB900_SATA,<br>-        PCI_DEVICE_ID_AMD_SB900_SATA_AHCI,<br>    PCI_DEVICE_ID_AMD_CZ_SATA,<br>    PCI_DEVICE_ID_AMD_CZ_SATA_AHCI,<br>       0<br></pre><p>To view, visit <a href="https://review.coreboot.org/21973">change 21973</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/21973"/><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: I9058d9102fc8ea0bd03ea089ba98da4590dd3533 </div>
<div style="display:none"> Gerrit-Change-Number: 21973 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Richard Spiegel <richard.spiegel@silverbackltd.com> </div>