From: Satyam Sharma <[EMAIL PROTECTED]> [5/8] i386: bitops: Contain warnings fallout from the death of volatiles
The wrappers below included from all over tree re-used "volatile" just because the bitops used them. With them killed, almost every file ends up crying about: warning: passing argument 2 of 'set_bit' discards qualifiers from pointer target type Silence these bogus warnings by killing volatile casts a second time. Signed-off-by: Satyam Sharma <[EMAIL PROTECTED]> Cc: David Howells <[EMAIL PROTECTED]> Cc: Nick Piggin <[EMAIL PROTECTED]> --- [ Cscope tells us there are other such wrappers that didn't show up in my test .config -- I'll update them shortly. Also, I should probably merge this patch with the previous one. Otherwise git-bisecters who hit the window between these two patches will be flooded with bogus warnings and would probably want to kill me :-) ] include/linux/cpumask.h | 4 ++-- include/linux/nodemask.h | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/include/linux/cpumask.h b/include/linux/cpumask.h index 23f5514..49f6ed4 100644 --- a/include/linux/cpumask.h +++ b/include/linux/cpumask.h @@ -89,13 +89,13 @@ typedef struct { DECLARE_BITMAP(bits, NR_CPUS); } cpumask_t; extern cpumask_t _unused_cpumask_arg_; #define cpu_set(cpu, dst) __cpu_set((cpu), &(dst)) -static inline void __cpu_set(int cpu, volatile cpumask_t *dstp) +static inline void __cpu_set(int cpu, cpumask_t *dstp) { set_bit(cpu, dstp->bits); } #define cpu_clear(cpu, dst) __cpu_clear((cpu), &(dst)) -static inline void __cpu_clear(int cpu, volatile cpumask_t *dstp) +static inline void __cpu_clear(int cpu, cpumask_t *dstp) { clear_bit(cpu, dstp->bits); } diff --git a/include/linux/nodemask.h b/include/linux/nodemask.h index 52c54a5..81ba056 100644 --- a/include/linux/nodemask.h +++ b/include/linux/nodemask.h @@ -89,13 +89,13 @@ typedef struct { DECLARE_BITMAP(bits, MAX_NUMNODES); } nodemask_t; extern nodemask_t _unused_nodemask_arg_; #define node_set(node, dst) __node_set((node), &(dst)) -static inline void __node_set(int node, volatile nodemask_t *dstp) +static inline void __node_set(int node, nodemask_t *dstp) { set_bit(node, dstp->bits); } #define node_clear(node, dst) __node_clear((node), &(dst)) -static inline void __node_clear(int node, volatile nodemask_t *dstp) +static inline void __node_clear(int node, nodemask_t *dstp) { clear_bit(node, dstp->bits); } - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/