[PATCH] __attribute__ placement fixes
authorVinay K Nallamothu <vinay.nallamothu@gsecone.com>
Sun, 1 May 2005 15:59:09 +0000 (08:59 -0700)
committerLinus Torvalds <torvalds@ppc970.osdl.org>
Sun, 1 May 2005 15:59:09 +0000 (08:59 -0700)
The variable attributes "packed" and "align" when used with struct, should
have the following order:

struct ... {...} __attribute__((packed)) var;

This patch fixes few instances where the variable and attributes are placed
the other way around and had no effect.

Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
drivers/net/gt96100eth.h
include/asm-m68knommu/MC68328.h
include/asm-m68knommu/MC68EZ328.h
include/asm-m68knommu/MC68VZ328.h

index 2f4bfd4dacbee3dc86220718c4c4aa2d8f013a7e..395869c5ed3edc4e4751377615ea8a1f2db2806a 100644 (file)
@@ -214,7 +214,7 @@ typedef struct {
        u32 cmdstat;
        u32 next;
        u32 buff_ptr;
-} gt96100_td_t __attribute__ ((packed));
+} __attribute__ ((packed)) gt96100_td_t;
 
 typedef struct {
 #ifdef DESC_BE
@@ -227,7 +227,7 @@ typedef struct {
        u32 cmdstat;
        u32 next;
        u32 buff_ptr;
-} gt96100_rd_t __attribute__ ((packed));
+} __attribute__ ((packed)) gt96100_rd_t;
 
 
 /* Values for the Tx command-status descriptor entry. */
index 4f5a9845f5be8c8493f8dc632ec992b3ccb65c72..a337e56d09bf9118eea7004ee9ba21921b02faca 100644 (file)
@@ -993,7 +993,7 @@ typedef volatile struct {
   volatile unsigned short int pad1;
   volatile unsigned short int pad2;
   volatile unsigned short int pad3;
-} m68328_uart __attribute__((packed));
+} __attribute__((packed)) m68328_uart;
 
 
 /**********
index 801933da4c7028b8514d2dfd7c8a6178ba4eef4c..69b7f9139e5efd0e873fefd82114108643442355 100644 (file)
@@ -815,7 +815,7 @@ typedef volatile struct {
   volatile unsigned short int nipr;
   volatile unsigned short int pad1;
   volatile unsigned short int pad2;
-} m68328_uart __attribute__((packed));
+} __attribute__((packed)) m68328_uart;
 
 
 /**********
index df74322f37ed82f79bc25b2a5d158e79d93dd05a..2b9bf626a0a5abf1454139ae50098e539405a38e 100644 (file)
@@ -909,7 +909,7 @@ typedef struct {
   volatile unsigned short int nipr;
   volatile unsigned short int hmark;
   volatile unsigned short int unused;
-} m68328_uart __attribute__((packed));
+} __attribute__((packed)) m68328_uart;