diff options
author | Allan Stephens <allan.stephens@windriver.com> | 2011-05-25 13:28:27 -0400 |
---|---|---|
committer | Paul Gortmaker <paul.gortmaker@windriver.com> | 2011-06-24 16:18:15 -0400 |
commit | 7eb878ed8e0eae67269439bfd82234f9ba52ffe4 (patch) | |
tree | 31df81dc4f4a2afec5a3e026102ee6ab7dc6caa7 /net | |
parent | 74d33b32deaa9ec864d6db3255b3a17a459f75fe (diff) |
tipc: Eliminate redundant masking in message header routines
Gets rid of unnecessary masking in two routines that set TIPC message
header fields. (The msg_set_bits() routine already takes care of
masking the new value to the correct size.)
Signed-off-by: Allan Stephens <allan.stephens@windriver.com>
Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com>
Diffstat (limited to 'net')
-rw-r--r-- | net/tipc/msg.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/net/tipc/msg.h b/net/tipc/msg.h index 11b74dc253f5..a58975ced222 100644 --- a/net/tipc/msg.h +++ b/net/tipc/msg.h | |||
@@ -615,7 +615,7 @@ static inline u32 msg_link_selector(struct tipc_msg *m) | |||
615 | 615 | ||
616 | static inline void msg_set_link_selector(struct tipc_msg *m, u32 n) | 616 | static inline void msg_set_link_selector(struct tipc_msg *m, u32 n) |
617 | { | 617 | { |
618 | msg_set_bits(m, 4, 0, 1, (n & 1)); | 618 | msg_set_bits(m, 4, 0, 1, n); |
619 | } | 619 | } |
620 | 620 | ||
621 | /* | 621 | /* |
@@ -639,7 +639,7 @@ static inline u32 msg_probe(struct tipc_msg *m) | |||
639 | 639 | ||
640 | static inline void msg_set_probe(struct tipc_msg *m, u32 val) | 640 | static inline void msg_set_probe(struct tipc_msg *m, u32 val) |
641 | { | 641 | { |
642 | msg_set_bits(m, 5, 0, 1, (val & 1)); | 642 | msg_set_bits(m, 5, 0, 1, val); |
643 | } | 643 | } |
644 | 644 | ||
645 | static inline char msg_net_plane(struct tipc_msg *m) | 645 | static inline char msg_net_plane(struct tipc_msg *m) |