[coreboot] [PATCH] v3: make header files self-contained

Carl-Daniel Hailfinger c-d.hailfinger.devel.2006 at gmx.net
Mon Feb 18 18:43:05 CET 2008


A lot of the v3 header files require other header files to be #included 
before they can be #included. That is completely counter-intuitive. Add 
necessary #includes to the header files themselves.

Fix a few cases where nonexisting files were #included.

Signed-off-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006 at gmx.net>

Index: LinuxBIOSv3-includecleanup/include/device/smbus.h
===================================================================
--- LinuxBIOSv3-includecleanup/include/device/smbus.h	(revision 609)
+++ LinuxBIOSv3-includecleanup/include/device/smbus.h	(working copy)
@@ -18,7 +18,6 @@
 #ifndef DEVICE_SMBUS_H
 #define DEVICE_SMBUS_H
 
-#include <stdint.h>
 #include <device/device.h>
 #include <device/path.h>
 #include <device/smbus_def.h>
Index: LinuxBIOSv3-includecleanup/include/device/pcix.h
===================================================================
--- LinuxBIOSv3-includecleanup/include/device/pcix.h	(revision 609)
+++ LinuxBIOSv3-includecleanup/include/device/pcix.h	(working copy)
@@ -18,6 +18,8 @@
 #ifndef DEVICE_PCIX_H
 #define DEVICE_PCIX_H
 
+#include <device/device.h>
+
 /* (c) 2005 Linux Networx GPL see COPYING for details */
 
 unsigned int pcix_scan_bus(struct bus *bus, 
Index: LinuxBIOSv3-includecleanup/include/device/hypertransport.h
===================================================================
--- LinuxBIOSv3-includecleanup/include/device/hypertransport.h	(revision 609)
+++ LinuxBIOSv3-includecleanup/include/device/hypertransport.h	(working copy)
@@ -18,6 +18,7 @@
 #ifndef DEVICE_HYPERTRANSPORT_H
 #define DEVICE_HYPERTRANSPORT_H
 
+#include <device/device.h>
 #include <device/hypertransport_def.h>
 
 unsigned int hypertransport_scan_chain(struct bus *bus, 
Index: LinuxBIOSv3-includecleanup/include/device/agp.h
===================================================================
--- LinuxBIOSv3-includecleanup/include/device/agp.h	(revision 609)
+++ LinuxBIOSv3-includecleanup/include/device/agp.h	(working copy)
@@ -19,6 +19,8 @@
 #ifndef DEVICE_AGP_H
 #define DEVICE_AGP_H
 
+#include <device/device.h>
+
 unsigned int agp_scan_bus(struct bus *bus, 
 	unsigned min_devfn, unsigned max_devfn, unsigned int max);
 unsigned int agp_scan_bridge(struct device *dev, unsigned int max);
Index: LinuxBIOSv3-includecleanup/include/device/pcie.h
===================================================================
--- LinuxBIOSv3-includecleanup/include/device/pcie.h	(revision 609)
+++ LinuxBIOSv3-includecleanup/include/device/pcie.h	(working copy)
@@ -20,6 +20,8 @@
 #ifndef DEVICE_PCIE_H
 #define DEVICE_PCIE_H
 
+#include <device/device.h>
+
 unsigned int pcie_scan_bus(struct bus *bus, 
 	unsigned min_devfn, unsigned max_devfn, unsigned int max);
 unsigned int pcie_scan_bridge(struct device *dev, unsigned int max);
Index: LinuxBIOSv3-includecleanup/include/device/cardbus.h
===================================================================
--- LinuxBIOSv3-includecleanup/include/device/cardbus.h	(revision 609)
+++ LinuxBIOSv3-includecleanup/include/device/cardbus.h	(working copy)
@@ -19,6 +19,8 @@
 #ifndef DEVICE_CARDBUS_H
 #define DEVICE_CARDBUS_H
 
+#include <device/device.h>
+
 void cardbus_read_resources(struct device *dev);
 unsigned int cardbus_scan_bus(struct bus *bus, 
 	unsigned min_devfn, unsigned max_devfn, unsigned int max);
Index: LinuxBIOSv3-includecleanup/include/device/pci_rom.h
===================================================================
--- LinuxBIOSv3-includecleanup/include/device/pci_rom.h	(revision 609)
+++ LinuxBIOSv3-includecleanup/include/device/pci_rom.h	(working copy)
@@ -19,6 +19,7 @@
 
 #include <types.h>
 #include <byteorder.h>
+#include <device/device.h>
 
 #define PCI_ROM_HDR 0xAA55
 #define PCI_DATA_HDR (u32) ( ('R' << 24) | ('I' << 16) | ('C' << 8) | 'P' )
Index: LinuxBIOSv3-includecleanup/include/tables.h
===================================================================
--- LinuxBIOSv3-includecleanup/include/tables.h	(revision 609)
+++ LinuxBIOSv3-includecleanup/include/tables.h	(working copy)
@@ -23,6 +23,8 @@
 #ifndef TABLES_H
 #define TABLES_H
 
+#include <types.h>
+
 /*
  * Table management structs and prototypes for coreboot.
  *
Index: LinuxBIOSv3-includecleanup/include/post_code.h
===================================================================
--- LinuxBIOSv3-includecleanup/include/post_code.h	(revision 609)
+++ LinuxBIOSv3-includecleanup/include/post_code.h	(working copy)
@@ -21,6 +21,7 @@
 #ifndef POST_CODE_H
 #define POST_CODE_H
 
+#include <types.h>
 #include <shared.h>
 
 SHARED(post_code, void, u8 value);
Index: LinuxBIOSv3-includecleanup/include/spd.h
===================================================================
--- LinuxBIOSv3-includecleanup/include/spd.h	(revision 609)
+++ LinuxBIOSv3-includecleanup/include/spd.h	(working copy)
@@ -36,6 +36,8 @@
 #ifndef _SPD_H_
 #define _SPD_H_
 
+#include <types.h>
+
 /* Byte numbers. */
 #define SPD_NUM_MANUFACTURER_BYTES          0  /* Number of bytes used by module manufacturer */
 #define SPD_TOTAL_SPD_MEMORY_SIZE           1  /* Total SPD memory size */
Index: LinuxBIOSv3-includecleanup/include/arch/x86/pci_ops.h
===================================================================
--- LinuxBIOSv3-includecleanup/include/arch/x86/pci_ops.h	(revision 609)
+++ LinuxBIOSv3-includecleanup/include/arch/x86/pci_ops.h	(working copy)
@@ -17,6 +17,8 @@
 #ifndef ARCH_X86_PCI_OPS_H
 #define ARCH_X86_PCI_OPS_H
 
+#include <device/device.h>
+
 extern const struct pci_bus_operations pci_cf8_conf1;
 extern const struct pci_bus_operations pci_cf8_conf2;
 
Index: LinuxBIOSv3-includecleanup/include/arch/x86/msr.h
===================================================================
--- LinuxBIOSv3-includecleanup/include/arch/x86/msr.h	(revision 609)
+++ LinuxBIOSv3-includecleanup/include/arch/x86/msr.h	(working copy)
@@ -20,6 +20,8 @@
 #ifndef CPU_X86_MSR_H
 #define CPU_X86_MSR_H
 
+#include <types.h>
+
 /* standard MSR operations, everyone has written these one hundred times */
 struct msr {
 	u32 lo;
Index: LinuxBIOSv3-includecleanup/include/arch/x86/pirq_routing.h
===================================================================
--- LinuxBIOSv3-includecleanup/include/arch/x86/pirq_routing.h	(revision 609)
+++ LinuxBIOSv3-includecleanup/include/arch/x86/pirq_routing.h	(working copy)
@@ -21,6 +21,8 @@
 #ifndef ARCH_PIRQ_ROUTING_H
 #define ARCH_PIRQ_ROUTING_H
 
+#include <types.h>
+
 #define PIRQ_SIGNATURE	(('$' << 0) + ('P' << 8) + ('I' << 16) + ('R' << 24))
 #define PIRQ_VERSION 0x0100
 
Index: LinuxBIOSv3-includecleanup/device/smbus_ops.c
===================================================================
--- LinuxBIOSv3-includecleanup/device/smbus_ops.c	(revision 609)
+++ LinuxBIOSv3-includecleanup/device/smbus_ops.c	(working copy)
@@ -20,7 +20,6 @@
  */
 
 #include <console.h>
-#include <stdint.h>
 #include <device/device.h>
 #include <device/path.h>
 #include <device/smbus.h>
Index: LinuxBIOSv3-includecleanup/northbridge/intel/i440bxemulation/i440bx.h
===================================================================
--- LinuxBIOSv3-includecleanup/northbridge/intel/i440bxemulation/i440bx.h	(revision 609)
+++ LinuxBIOSv3-includecleanup/northbridge/intel/i440bxemulation/i440bx.h	(working copy)
@@ -21,6 +21,8 @@
 #ifndef NORTHBRIDGE_INTEL_I440BXEMULATION_I440BX_H
 #define NORTHBRIDGE_INTEL_I440BXEMULATION_I440BX_H
 
+#include <device/device.h>
+
 /*
  * Datasheet:
  *   - Name: Intel 440BX AGPset: 82443BX Host Bridge/Controller


-- 
http://www.hailfinger.org/





More information about the coreboot mailing list