diff options
author | Rusty Russell <rusty@rustcorp.com.au> | 2011-12-19 08:56:45 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2011-12-19 22:27:29 -0500 |
commit | 3db1cd5c05f35fb43eb134df6f321de4e63141f2 (patch) | |
tree | 960039f3f4f0a524b37e94434624da154859bc64 /drivers/net/wireless/b43/xmit.c | |
parent | a8e510f682fe6d7671c11887e07c55f86caaf3c1 (diff) |
net: fix assignment of 0/1 to bool variables.
DaveM said:
Please, this kind of stuff rots forever and not using bool properly
drives me crazy.
Joe Perches <joe@perches.com> gave me the spatch script:
@@
bool b;
@@
-b = 0
+b = false
@@
bool b;
@@
-b = 1
+b = true
I merely installed coccinelle, read the documentation and took credit.
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/wireless/b43/xmit.c')
-rw-r--r-- | drivers/net/wireless/b43/xmit.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/net/wireless/b43/xmit.c b/drivers/net/wireless/b43/xmit.c index 5f77cbe0b6aa..2c5367884b3f 100644 --- a/drivers/net/wireless/b43/xmit.c +++ b/drivers/net/wireless/b43/xmit.c | |||
@@ -874,7 +874,7 @@ bool b43_fill_txstatus_report(struct b43_wldev *dev, | |||
874 | struct ieee80211_tx_info *report, | 874 | struct ieee80211_tx_info *report, |
875 | const struct b43_txstatus *status) | 875 | const struct b43_txstatus *status) |
876 | { | 876 | { |
877 | bool frame_success = 1; | 877 | bool frame_success = true; |
878 | int retry_limit; | 878 | int retry_limit; |
879 | 879 | ||
880 | /* preserve the confiured retry limit before clearing the status | 880 | /* preserve the confiured retry limit before clearing the status |
@@ -890,7 +890,7 @@ bool b43_fill_txstatus_report(struct b43_wldev *dev, | |||
890 | /* The frame was not ACKed... */ | 890 | /* The frame was not ACKed... */ |
891 | if (!(report->flags & IEEE80211_TX_CTL_NO_ACK)) { | 891 | if (!(report->flags & IEEE80211_TX_CTL_NO_ACK)) { |
892 | /* ...but we expected an ACK. */ | 892 | /* ...but we expected an ACK. */ |
893 | frame_success = 0; | 893 | frame_success = false; |
894 | } | 894 | } |
895 | } | 895 | } |
896 | if (status->frame_count == 0) { | 896 | if (status->frame_count == 0) { |