diff options
author | Vinay K Nallamothu <vinay.nallamothu@gsecone.com> | 2005-05-01 11:59:09 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@ppc970.osdl.org> | 2005-05-01 11:59:09 -0400 |
commit | f78fc874f42f63a460bcebc2aeb98db526280d1c (patch) | |
tree | ad78acf82f36bb14e8a9ae0980d350d6f5cf704e | |
parent | e49332bd12e92da2df6d002f857ec62675ba2648 (diff) |
[PATCH] __attribute__ placement fixes
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>
-rw-r--r-- | drivers/net/gt96100eth.h | 4 | ||||
-rw-r--r-- | include/asm-m68knommu/MC68328.h | 2 | ||||
-rw-r--r-- | include/asm-m68knommu/MC68EZ328.h | 2 | ||||
-rw-r--r-- | include/asm-m68knommu/MC68VZ328.h | 2 |
4 files changed, 5 insertions, 5 deletions
diff --git a/drivers/net/gt96100eth.h b/drivers/net/gt96100eth.h index 2f4bfd4dacbe..395869c5ed3e 100644 --- a/drivers/net/gt96100eth.h +++ b/drivers/net/gt96100eth.h | |||
@@ -214,7 +214,7 @@ typedef struct { | |||
214 | u32 cmdstat; | 214 | u32 cmdstat; |
215 | u32 next; | 215 | u32 next; |
216 | u32 buff_ptr; | 216 | u32 buff_ptr; |
217 | } gt96100_td_t __attribute__ ((packed)); | 217 | } __attribute__ ((packed)) gt96100_td_t; |
218 | 218 | ||
219 | typedef struct { | 219 | typedef struct { |
220 | #ifdef DESC_BE | 220 | #ifdef DESC_BE |
@@ -227,7 +227,7 @@ typedef struct { | |||
227 | u32 cmdstat; | 227 | u32 cmdstat; |
228 | u32 next; | 228 | u32 next; |
229 | u32 buff_ptr; | 229 | u32 buff_ptr; |
230 | } gt96100_rd_t __attribute__ ((packed)); | 230 | } __attribute__ ((packed)) gt96100_rd_t; |
231 | 231 | ||
232 | 232 | ||
233 | /* Values for the Tx command-status descriptor entry. */ | 233 | /* Values for the Tx command-status descriptor entry. */ |
diff --git a/include/asm-m68knommu/MC68328.h b/include/asm-m68knommu/MC68328.h index 4f5a9845f5be..a337e56d09bf 100644 --- a/include/asm-m68knommu/MC68328.h +++ b/include/asm-m68knommu/MC68328.h | |||
@@ -993,7 +993,7 @@ typedef volatile struct { | |||
993 | volatile unsigned short int pad1; | 993 | volatile unsigned short int pad1; |
994 | volatile unsigned short int pad2; | 994 | volatile unsigned short int pad2; |
995 | volatile unsigned short int pad3; | 995 | volatile unsigned short int pad3; |
996 | } m68328_uart __attribute__((packed)); | 996 | } __attribute__((packed)) m68328_uart; |
997 | 997 | ||
998 | 998 | ||
999 | /********** | 999 | /********** |
diff --git a/include/asm-m68knommu/MC68EZ328.h b/include/asm-m68knommu/MC68EZ328.h index 801933da4c70..69b7f9139e5e 100644 --- a/include/asm-m68knommu/MC68EZ328.h +++ b/include/asm-m68knommu/MC68EZ328.h | |||
@@ -815,7 +815,7 @@ typedef volatile struct { | |||
815 | volatile unsigned short int nipr; | 815 | volatile unsigned short int nipr; |
816 | volatile unsigned short int pad1; | 816 | volatile unsigned short int pad1; |
817 | volatile unsigned short int pad2; | 817 | volatile unsigned short int pad2; |
818 | } m68328_uart __attribute__((packed)); | 818 | } __attribute__((packed)) m68328_uart; |
819 | 819 | ||
820 | 820 | ||
821 | /********** | 821 | /********** |
diff --git a/include/asm-m68knommu/MC68VZ328.h b/include/asm-m68knommu/MC68VZ328.h index df74322f37ed..2b9bf626a0a5 100644 --- a/include/asm-m68knommu/MC68VZ328.h +++ b/include/asm-m68knommu/MC68VZ328.h | |||
@@ -909,7 +909,7 @@ typedef struct { | |||
909 | volatile unsigned short int nipr; | 909 | volatile unsigned short int nipr; |
910 | volatile unsigned short int hmark; | 910 | volatile unsigned short int hmark; |
911 | volatile unsigned short int unused; | 911 | volatile unsigned short int unused; |
912 | } m68328_uart __attribute__((packed)); | 912 | } __attribute__((packed)) m68328_uart; |
913 | 913 | ||
914 | 914 | ||
915 | 915 | ||