Skip to contentSkip to conversation

mac80211: remove Hi16, Lo16 helpers

PATCH In review Harvey Harrison 2 participants 2 messages Mar 8, 2008 – Mar 12, 2008 mac80211

lore
Share & tools
lore
#0 lore raw

PATCH 4/5 mac80211: remove Hi16, Lo16 helpers

They were only used in one place, just open-code it there.

---
 net/mac80211/tkip.c |   19 ++-----------------
 1 files changed, 2 insertions(+), 17 deletions(-)
net/mac80211/tkip.c +2 -17
index 9a36e83..264e392 100644
@@ -65,31 +65,16 @@ static inline u8 Hi8(u16 v)
 	return v >> 8;
 }
 
-
 static inline u8 Lo8(u16 v)
 {
 	return v & 0xff;
 }
 
-
-static inline u16 Hi16(u32 v)
-{
-	return v >> 16;
-}
-
-
-static inline u16 Lo16(u32 v)
-{
-	return v & 0xffff;
-}
-
 static u16 tkip_S(u16 val)
 {
 	return tkip_sbox[val & 0xff] ^ swab16(tkip_sbox[val >> 8]);
 }
 
-
-
 /* P1K := Phase1(TA, TK, TSC)
  * TA = transmitter address (48 bits)
  * TK = dot11DefaultKeyValue or dot11KeyMappingValue (128 bits)
@@ -102,8 +87,8 @@ static void tkip_mixing_phase1(const u8 *ta, const u8 *tk, u32 tsc_IV32,
 	int i, j;
 	__le16 *leptr = (__le16 *)ta;
 
-	p1k[0] = Lo16(tsc_IV32);
-	p1k[1] = Hi16(tsc_IV32);
+	p1k[0] = tsc_IV32 & 0xffff;
+	p1k[1] = tsc_IV32 >> 16;
 	p1k[2] = le16_to_cpup(leptr++);
 	p1k[3] = le16_to_cpup(leptr++);
 	p1k[4] = le16_to_cpup(leptr++);
Signature · 1 line
1.5.4.GIT
maintainer
#1 lore raw
Quoted 1 line · Harvey Harrison

They were only used in one place, just open-code it there.

This depends on the previous patches removing a few uses, right?

johannes