Use __unused instead of __used for static data structures.
authorMichael Brown <mcb30@etherboot.org>
Tue, 17 May 2005 12:16:28 +0000 (12:16 +0000)
committerMichael Brown <mcb30@etherboot.org>
Tue, 17 May 2005 12:16:28 +0000 (12:16 +0000)
Remove compiler.h defines from osdep.h

src/arch/i386/include/relocate.h
src/include/compiler.h
src/include/console.h
src/include/dev.h
src/include/init.h
src/include/osdep.h
src/include/tables.h

index fdc807f..5e482bd 100644 (file)
@@ -17,7 +17,7 @@ struct post_reloc_fn {
 /* Macro for creating a post-relocation function table entry */
 #define POST_RELOC_FN( order, post_reloc_func )                                      \
        static struct post_reloc_fn PREFIX_OBJECT(post_reloc_fn__)            \
-           __attribute__ (( used, __table_section(post_reloc_fn,order) )) = {\
+           __table ( post_reloc_fn, order ) = {                              \
                .post_reloc = post_reloc_func,                                \
        };
 
index 4912b42..4a7c48a 100644 (file)
@@ -84,9 +84,18 @@ __asm__ ( ".equ\tDEBUG_LEVEL, " DEBUG_SYMBOL_STR );
        } while (0)
 #endif
 
-#define PACKED __attribute__((packed))
-#define __unused __attribute__((unused))
-#define __used __attribute__((used))
+/*
+ * Commonly-used attributes.
+ *
+ * Note that __used can be used only for functions.  If you have a
+ * static variable declaration that you want to force to be included,
+ * use __unused.
+ *
+ */
+#define PACKED __attribute__ (( packed ))
+#define __unused __attribute__ (( unused ))
+#define __used __attribute__ (( used ))
+#define __aligned __attribute__ (( aligned ( 16 ) ))
 
 /*
  * To save space in the binary when multiple-driver images are
index 146fd9c..1decb9c 100644 (file)
@@ -18,8 +18,7 @@ struct console_driver {
        int ( *iskey ) ( void );
 };
 
-#define __console_driver \
-       __attribute__ (( used, __table_section ( console, 01 ) ))
+#define __console_driver __table ( console, 01 )
 
 /* Function prototypes */
 
index 0d0d4d8..d6e31da 100644 (file)
@@ -160,7 +160,7 @@ struct bus_driver {
        const char * ( *name_device ) ( struct bus_dev *bus_dev );
 };
 
-#define __bus_driver __attribute__ (( used, __table_section(bus_driver,01) ))
+#define __bus_driver __table ( bus_driver, 01 )
 
 /*
  * A structure fully describing the bus-independent parts of a
@@ -186,7 +186,7 @@ struct type_driver {
        int ( * load ) ( struct type_dev *type_dev, struct buffer *buffer );
 };
 
-#define __type_driver __attribute__ (( used, __table_section(type_driver,01) ))
+#define __type_driver __table ( type_driver, 01 )
 
 /*
  * A driver for a device.
@@ -203,8 +203,7 @@ struct device_driver {
                             struct bus_dev *bus_dev );
 };
 
-#define __device_driver \
-       __attribute__ (( used, __table_section(device_driver,01) ))
+#define __device_driver __table ( device_driver, 01 )
 
 #define DRIVER(_name,_type_driver,_bus_driver,_bus_info,                     \
               _probe,_disable)                                               \
index e28a434..0b341b8 100644 (file)
@@ -51,7 +51,7 @@ struct init_fn {
 /* Macro for creating an initialisation function table entry */
 #define INIT_FN( init_order, init_func, reset_func, exit_func )                      \
        static struct init_fn PREFIX_OBJECT(init_fn__)                        \
-           __attribute__ (( used, __table_section(init_fn,init_order) )) = { \
+           __table ( init_fn, init_order ) = {                               \
                .init = init_func,                                            \
                .reset = reset_func,                                          \
                .exit = exit_func,                                            \
index 121eaca..262c2d9 100644 (file)
@@ -1,10 +1,6 @@
 #ifndef        ETHERBOOT_OSDEP_H
 #define ETHERBOOT_OSDEP_H
 
-#define __unused __attribute__((unused))
-#define __aligned __attribute__((aligned(16)))
-#define PACKED __attribute__((packed))
-
 /* Optimization barrier */
 /* The "volatile" is due to gcc bugs */
 #define barrier() __asm__ __volatile__("": : :"memory")
index 4851446..26ad61d 100644 (file)
 #define __table_section_start(table) __table_section(table,00)
 #define __table_section_end(table) __table_section(table,99)
 
-#define __table(table,idx) __attribute__ (( __table_section(table,idx) ))
-#define __table_start(table) __attribute__ (( __table_section_start(table) ))
-#define __table_end(table) __attribute__ (( __table_section_end(table) ))
+#define __table(table,idx) \
+       __attribute__ (( unused, __table_section(table,idx) ))
+#define __table_start(table) \
+       __attribute__ (( unused, __table_section_start(table) ))
+#define __table_end(table) \
+       __attribute__ (( unused, __table_section_end(table) ))
 
 #endif /* TABLES_H */