Commit dbb1a307 authored by Andrew Morton's avatar Andrew Morton Committed by Linus Torvalds

[PATCH] fix test_and_change_bit comment

From: Paul Jackson <pj@sgi.com>

I've read over the code in each case, built and ran a test case for i386 in
particular, and studied the other uses and definitions of
test_and_change_bit().  Everything I see recommends this change.

- Fix test_and_change_bit() comment: returns old value, not new one.
parent ed459323
...@@ -169,7 +169,7 @@ extern inline int __test_and_clear_bit(int nr, void *addr) ...@@ -169,7 +169,7 @@ extern inline int __test_and_clear_bit(int nr, void *addr)
return retval; return retval;
} }
/** /**
* test_and_change_bit - Change a bit and return its new value * test_and_change_bit - Change a bit and return its old value
* @nr: Bit to change * @nr: Bit to change
* @addr: Address to count from * @addr: Address to count from
* *
......
...@@ -212,7 +212,7 @@ static __inline__ int __test_and_change_bit(int nr, volatile unsigned long *addr ...@@ -212,7 +212,7 @@ static __inline__ int __test_and_change_bit(int nr, volatile unsigned long *addr
} }
/** /**
* test_and_change_bit - Change a bit and return its new value * test_and_change_bit - Change a bit and return its old value
* @nr: Bit to change * @nr: Bit to change
* @addr: Address to count from * @addr: Address to count from
* *
......
...@@ -236,7 +236,7 @@ __test_and_clear_bit(int nr, volatile void * addr) ...@@ -236,7 +236,7 @@ __test_and_clear_bit(int nr, volatile void * addr)
} }
/** /**
* test_and_change_bit - Change a bit and return its new value * test_and_change_bit - Change a bit and return its old value
* @nr: Bit to set * @nr: Bit to set
* @addr: Address to count from * @addr: Address to count from
* *
......
...@@ -296,7 +296,7 @@ static inline int __test_and_clear_bit(unsigned long nr, ...@@ -296,7 +296,7 @@ static inline int __test_and_clear_bit(unsigned long nr,
} }
/* /*
* test_and_change_bit - Change a bit and return its new value * test_and_change_bit - Change a bit and return its old value
* @nr: Bit to change * @nr: Bit to change
* @addr: Address to count from * @addr: Address to count from
* *
...@@ -567,7 +567,7 @@ static inline int __test_and_clear_bit(unsigned long nr, ...@@ -567,7 +567,7 @@ static inline int __test_and_clear_bit(unsigned long nr,
} }
/* /*
* test_and_change_bit - Change a bit and return its new value * test_and_change_bit - Change a bit and return its old value
* @nr: Bit to change * @nr: Bit to change
* @addr: Address to count from * @addr: Address to count from
* *
......
...@@ -204,7 +204,7 @@ static __inline__ int __test_and_change_bit(int nr, volatile void * addr) ...@@ -204,7 +204,7 @@ static __inline__ int __test_and_change_bit(int nr, volatile void * addr)
} }
/** /**
* test_and_change_bit - Change a bit and return its new value * test_and_change_bit - Change a bit and return its old value
* @nr: Bit to change * @nr: Bit to change
* @addr: Address to count from * @addr: Address to count from
* *
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment