diff options
| author | Michael Ellerman <michael@ellerman.id.au> | 2006-03-23 13:33:03 +0100 |
|---|---|---|
| committer | Paul Mackerras <paulus@samba.org> | 2006-03-27 05:48:12 +0200 |
| commit | dd4d7bfad635dddc56b74dab1894ef01c8c836e1 (patch) | |
| tree | 4ce50568ffa94cd01803c57881d59c8bacda6dbe /include/asm-powerpc/firmware.h | |
| parent | [PATCH] powerpc: Make BUG_ON & WARN_ON play nice with compile-time optimisations (diff) | |
| download | linux-dd4d7bfad635dddc56b74dab1894ef01c8c836e1.tar.xz linux-dd4d7bfad635dddc56b74dab1894ef01c8c836e1.zip | |
[PATCH] powerpc: Change firmware_has_feature() to a macro
So that we can use firmware_has_feature() in a BUG_ON() and have the compiler
elide the code entirely if the feature can never be set, change
firmware_has_feature to a macro. Unfortunate, but necessary at least until
GCC bug #26724 is fixed.
Signed-off-by: Michael Ellerman <michael@ellerman.id.au>
Signed-off-by: Paul Mackerras <paulus@samba.org>
Diffstat (limited to '')
| -rw-r--r-- | include/asm-powerpc/firmware.h | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/include/asm-powerpc/firmware.h b/include/asm-powerpc/firmware.h index ce3788224ed0..03c2fdff021b 100644 --- a/include/asm-powerpc/firmware.h +++ b/include/asm-powerpc/firmware.h @@ -84,11 +84,9 @@ enum { */ extern unsigned long ppc64_firmware_features; -static inline unsigned long firmware_has_feature(unsigned long feature) -{ - return (FW_FEATURE_ALWAYS & feature) || - (FW_FEATURE_POSSIBLE & ppc64_firmware_features & feature); -} +#define firmware_has_feature(feature) \ + ((FW_FEATURE_ALWAYS & (feature)) || \ + (FW_FEATURE_POSSIBLE & ppc64_firmware_features & (feature))) extern void system_reset_fwnmi(void); extern void machine_check_fwnmi(void); |