diff options
author | Michael Ellerman <michael@ellerman.id.au> | 2006-03-23 23:33:03 +1100 |
---|---|---|
committer | Paul Mackerras <paulus@samba.org> | 2006-03-27 14:48:12 +1100 |
commit | dd4d7bfad635dddc56b74dab1894ef01c8c836e1 (patch) | |
tree | 4ce50568ffa94cd01803c57881d59c8bacda6dbe /include | |
parent | e3f94b85f98a346c5eb0ac0d9539b71cb7057143 (diff) | |
download | blackbird-op-linux-dd4d7bfad635dddc56b74dab1894ef01c8c836e1.tar.gz blackbird-op-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 'include')
-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); |