diff options
author | Bryan O'Sullivan <bos@serpentine.com> | 2006-01-17 20:00:05 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-01-17 20:18:25 -0500 |
commit | c6b3feaf5774508ab5a955a81e45ffefa3b12eaa (patch) | |
tree | f2b275db2e425b1441f9a1ab40a377a232c2bc38 | |
parent | 2664b25051f7ab96b22b199aa2f5ef6a949a4296 (diff) |
[PATCH] Fix sparse parse error in lppaca.h
sparse can't parse a struct definition in include/asm-powerpc/lppaca.h,
even though gcc can accept it. The form looks like this:
struct __attribute__((whatever)) foo { };
An equivalent that both gcc and sparse can handle is
struct foo { } __attribute__((whatever));
This is the only definition of this type in the tree, and fixing it is
easier than fixing sparse.
Signed-off-by: Bryan O'Sullivan <bos@serpentine.com>
[ Side note: fixing sparse wouldn't be hard, but the "attribute at the
end" version is the canonical one, and the one that makes sense. So
let's just fix the kernel instead. Luc Van Oostenryck already sent
out a sparse patch to the sparse mailing list in case anybody cares.
-- Linus ]
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
-rw-r--r-- | include/asm-powerpc/lppaca.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/include/asm-powerpc/lppaca.h b/include/asm-powerpc/lppaca.h index cd9f11f1ef14..4dc514aabfe7 100644 --- a/include/asm-powerpc/lppaca.h +++ b/include/asm-powerpc/lppaca.h | |||
@@ -31,7 +31,7 @@ | |||
31 | 31 | ||
32 | /* The Hypervisor barfs if the lppaca crosses a page boundary. A 1k | 32 | /* The Hypervisor barfs if the lppaca crosses a page boundary. A 1k |
33 | * alignment is sufficient to prevent this */ | 33 | * alignment is sufficient to prevent this */ |
34 | struct __attribute__((__aligned__(0x400))) lppaca { | 34 | struct lppaca { |
35 | //============================================================================= | 35 | //============================================================================= |
36 | // CACHE_LINE_1 0x0000 - 0x007F Contains read-only data | 36 | // CACHE_LINE_1 0x0000 - 0x007F Contains read-only data |
37 | // NOTE: The xDynXyz fields are fields that will be dynamically changed by | 37 | // NOTE: The xDynXyz fields are fields that will be dynamically changed by |
@@ -129,7 +129,7 @@ struct __attribute__((__aligned__(0x400))) lppaca { | |||
129 | // CACHE_LINE_4-5 0x0100 - 0x01FF Contains PMC interrupt data | 129 | // CACHE_LINE_4-5 0x0100 - 0x01FF Contains PMC interrupt data |
130 | //============================================================================= | 130 | //============================================================================= |
131 | u8 pmc_save_area[256]; // PMC interrupt Area x00-xFF | 131 | u8 pmc_save_area[256]; // PMC interrupt Area x00-xFF |
132 | }; | 132 | } __attribute__((__aligned__(0x400))); |
133 | 133 | ||
134 | extern struct lppaca lppaca[]; | 134 | extern struct lppaca lppaca[]; |
135 | 135 | ||