<p>Marshall Dawson has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/21615">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">amd/stoneyridge: Clean up include files in northbridge.h<br><br>Add an extra include file to northbridge.c for completeness.  cpu/msr.h<br>is already included in cpu/amd/mtrr.h which allows the file to build<br>properly.<br><br>Remove include files that are no longer required for the file.<br><br>Change-Id: I3e5ab39fd0640d2983fc5b7b202fb65d42c5ce3d<br>Signed-off-by: Marshall Dawson <marshalldawson3rd@gmail.com><br>---<br>M src/soc/amd/stoneyridge/northbridge.c<br>1 file changed, 1 insertion(+), 13 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/15/21615/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/src/soc/amd/stoneyridge/northbridge.c b/src/soc/amd/stoneyridge/northbridge.c<br>index ae2cdbb..07ea10f 100644<br>--- a/src/soc/amd/stoneyridge/northbridge.c<br>+++ b/src/soc/amd/stoneyridge/northbridge.c<br>@@ -23,11 +23,10 @@<br> #include <cpu/amd/amdfam15.h><br> #include <cpu/amd/mtrr.h><br> #include <cpu/cpu.h><br>+#include <cpu/x86/msr.h><br> #include <device/device.h><br> #include <device/pci.h><br> #include <device/pci_ids.h><br>-#include <device/hypertransport.h><br>-#include <lib.h><br> #include <agesawrapper.h><br> #include <agesawrapper_call.h><br> #include <soc/northbridge.h><br>@@ -35,17 +34,6 @@<br> #include <stdint.h><br> #include <stdlib.h><br> #include <string.h><br>-<br>-/*<br>- * AMD vendorcode files. Place at the end so coreboot defaults and maintained<br>- * and not set by vendorcode<br>- */<br>-#include <AGESA.h><br>-#include <cpuRegisters.h><br>-#include <FieldAccessors.h><br>-#include <Options.h><br>-#include <Porting.h><br>-#include <Topology.h><br> <br> typedef struct dram_base_mask {<br>    u32 base; /* [47:27] at [28:8] */<br></pre><p>To view, visit <a href="https://review.coreboot.org/21615">change 21615</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/21615"/><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: I3e5ab39fd0640d2983fc5b7b202fb65d42c5ce3d </div>
<div style="display:none"> Gerrit-Change-Number: 21615 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Marshall Dawson <marshalldawson3rd@gmail.com> </div>