<p>Martin Roth has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/20747">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">src/northbridge: Add guards on all header files<br><br>Change-Id: I93b939478615f22f2c078b1efb7999ad4f3a4c28<br>Signed-off-by: Martin Roth <martinroth@google.com><br>---<br>M src/northbridge/amd/amdfam10/ht_config.h<br>M src/northbridge/via/cn700/cn700.h<br>M src/northbridge/via/cx700/registers.h<br>M src/northbridge/via/vx900/chip.h<br>4 files changed, 20 insertions(+), 0 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/47/20747/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/src/northbridge/amd/amdfam10/ht_config.h b/src/northbridge/amd/amdfam10/ht_config.h<br>index bcfcf44..75626fa 100644<br>--- a/src/northbridge/amd/amdfam10/ht_config.h<br>+++ b/src/northbridge/amd/amdfam10/ht_config.h<br>@@ -13,6 +13,9 @@<br>  * GNU General Public License for more details.<br>  */<br> <br>+#ifndef __AMDFAM10_HT_CONFIG_H__<br>+#define __AMDFAM10_HT_CONFIG_H__<br>+<br> typedef struct amdfam10_sysconf_t sys_info_conf_t;<br> <br> /* FIXME */<br>@@ -47,3 +50,5 @@<br>                             u32 io_min, u32 io_max);<br> <br> void set_mmio_addr_reg(u32 nodeid, u32 linkn, u32 reg, u32 index, u32 mmio_min, u32 mmio_max, u32 nodes);<br>+<br>+#endif<br>diff --git a/src/northbridge/via/cn700/cn700.h b/src/northbridge/via/cn700/cn700.h<br>index 453b7f5..a5e8c1c 100644<br>--- a/src/northbridge/via/cn700/cn700.h<br>+++ b/src/northbridge/via/cn700/cn700.h<br>@@ -14,6 +14,9 @@<br>  * GNU General Public License for more details.<br>  */<br> <br>+#ifndef __VIA_CN700_H__<br>+#define __VIA_CN700_H__<br>+<br> /* VGA stuff */<br> #define SR_INDEX              0x3c4<br> #define SR_DATA                 0x3c5<br>@@ -43,3 +46,5 @@<br> #define RAM_COMMAND_PRECHARGE        0x2<br> #define RAM_COMMAND_MRS           0x3<br> #define RAM_COMMAND_CBR           0x4<br>+<br>+#endif<br>diff --git a/src/northbridge/via/cx700/registers.h b/src/northbridge/via/cx700/registers.h<br>index 02ec02a..f8b475b 100644<br>--- a/src/northbridge/via/cx700/registers.h<br>+++ b/src/northbridge/via/cx700/registers.h<br>@@ -13,6 +13,9 @@<br>  * GNU General Public License for more details.<br>  */<br> <br>+#ifndef __CX700_REGISTERS_H__<br>+#define __CX700_REGISTERS_H__<br>+<br> /* CX700 has 48 bytes of scratch registers in D0F4 starting at Reg. 0xd0 */<br> #define   SCRATCH_REG_BASE                0xd0<br> #define  SCRATCH_RANK_0                  0xd0<br>@@ -38,3 +41,5 @@<br> #define DDRII_333     0x2<br> #define DDRII_266 0x1<br> #define DDRII_200 0x0<br>+<br>+#endif<br>diff --git a/src/northbridge/via/vx900/chip.h b/src/northbridge/via/vx900/chip.h<br>index 3ba7583..0a2857d 100644<br>--- a/src/northbridge/via/vx900/chip.h<br>+++ b/src/northbridge/via/vx900/chip.h<br>@@ -14,6 +14,9 @@<br>  * GNU General Public License for more details.<br>  */<br> <br>+#ifndef __VX900_CHIP_H__<br>+#define __VX900_CHIP_H__<br>+<br> struct northbridge_via_vx900_config {<br>       /**<br>    * \brief PCIe Lane[3:0] Function Select<br>@@ -47,3 +50,5 @@<br>    */<br>   char ext_int_route_to_pirq;<br> };<br>+<br>+#endif<br></pre><p>To view, visit <a href="https://review.coreboot.org/20747">change 20747</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/20747"/><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: I93b939478615f22f2c078b1efb7999ad4f3a4c28 </div>
<div style="display:none"> Gerrit-Change-Number: 20747 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Martin Roth <martinroth@google.com> </div>