asm-generic/atomic.h: fix type used in atomic_clear_mask
authorMike Frysinger <vapier@gentoo.org>
Tue, 26 Jul 2011 23:09:10 +0000 (16:09 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Tue, 26 Jul 2011 23:49:47 +0000 (16:49 -0700)
The atomic helpers are supposed to take an atomic_t pointer, not a random
unsigned long pointer.  So convert atomic_clear_mask over.

While we're here, also add some nice documentation to the func.

Signed-off-by: Mike Frysinger <vapier@gentoo.org>
Cc: Arun Sharma <asharma@fb.com>
Cc: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
include/asm-generic/atomic.h

index 2a23da03365e7636682bc9d500bcc7f4efefe07f..4e8ecf0693e984162d939e21e1aa7e4bc20bebaa 100644 (file)
@@ -140,13 +140,20 @@ static inline int __atomic_add_unless(atomic_t *v, int a, int u)
   return c;
 }
 
-static inline void atomic_clear_mask(unsigned long mask, unsigned long *addr)
+/**
+ * atomic_clear_mask - Atomically clear bits in atomic variable
+ * @mask: Mask of the bits to be cleared
+ * @v: pointer of type atomic_t
+ *
+ * Atomically clears the bits set in @mask from @v
+ */
+static inline void atomic_clear_mask(unsigned long mask, atomic_t *v)
 {
        unsigned long flags;
 
        mask = ~mask;
        raw_local_irq_save(flags); /* Don't trace it in a irqsoff handler */
-       *addr &= mask;
+       v->counter &= mask;
        raw_local_irq_restore(flags);
 }