iwlegacy: rename priv to il

Make code shorter.

Signed-off-by: Stanislaw Gruszka <sgruszka@redhat.com>

1 files changed, 162 insertions, 162 deletions

diff --git a/drivers/net/wireless/iwlegacy/iwl-io.h b/drivers/net/wireless/iwlegacy/iwl-io.h
index ebeb6e2aa6e7..42d241fd6ac9 100644
--- a/drivers/net/wireless/iwlegacy/iwl-io.h
+++ b/drivers/net/wireless/iwlegacy/iwl-io.h
@@ -62,72 +62,72 @@
  *
  */
 
-static inline void _il_write8(struct il_priv *priv, u32 ofs, u8 val)
+static inline void _il_write8(struct il_priv *il, u32 ofs, u8 val)
 {
-	iowrite8(val, priv->hw_base + ofs);
+	iowrite8(val, il->hw_base + ofs);
 }
 
 #ifdef CONFIG_IWLWIFI_LEGACY_DEBUG
 static inline void
-__il_write8(const char *f, u32 l, struct il_priv *priv,
+__il_write8(const char *f, u32 l, struct il_priv *il,
 				 u32 ofs, u8 val)
 {
-	IL_DEBUG_IO(priv, "write8(0x%08X, 0x%02X) - %s %d\n", ofs, val, f, l);
-	_il_write8(priv, ofs, val);
+	IL_DEBUG_IO(il, "write8(0x%08X, 0x%02X) - %s %d\n", ofs, val, f, l);
+	_il_write8(il, ofs, val);
 }
-#define il_write8(priv, ofs, val) \
-	__il_write8(__FILE__, __LINE__, priv, ofs, val)
+#define il_write8(il, ofs, val) \
+	__il_write8(__FILE__, __LINE__, il, ofs, val)
 #else
-#define il_write8(priv, ofs, val) _il_write8(priv, ofs, val)
+#define il_write8(il, ofs, val) _il_write8(il, ofs, val)
 #endif
 
 
-static inline void _il_write32(struct il_priv *priv, u32 ofs, u32 val)
+static inline void _il_write32(struct il_priv *il, u32 ofs, u32 val)
 {
-	iowrite32(val, priv->hw_base + ofs);
+	iowrite32(val, il->hw_base + ofs);
 }
 
 #ifdef CONFIG_IWLWIFI_LEGACY_DEBUG
 static inline void
-__il_write32(const char *f, u32 l, struct il_priv *priv,
+__il_write32(const char *f, u32 l, struct il_priv *il,
 				 u32 ofs, u32 val)
 {
-	IL_DEBUG_IO(priv, "write32(0x%08X, 0x%08X) - %s %d\n", ofs, val, f, l);
-	_il_write32(priv, ofs, val);
+	IL_DEBUG_IO(il, "write32(0x%08X, 0x%08X) - %s %d\n", ofs, val, f, l);
+	_il_write32(il, ofs, val);
 }
-#define il_write32(priv, ofs, val) \
-	__il_write32(__FILE__, __LINE__, priv, ofs, val)
+#define il_write32(il, ofs, val) \
+	__il_write32(__FILE__, __LINE__, il, ofs, val)
 #else
-#define il_write32(priv, ofs, val) _il_write32(priv, ofs, val)
+#define il_write32(il, ofs, val) _il_write32(il, ofs, val)
 #endif
 
-static inline u32 _il_read32(struct il_priv *priv, u32 ofs)
+static inline u32 _il_read32(struct il_priv *il, u32 ofs)
 {
-	u32 val = ioread32(priv->hw_base + ofs);
+	u32 val = ioread32(il->hw_base + ofs);
 	return val;
 }
 
 #ifdef CONFIG_IWLWIFI_LEGACY_DEBUG
 static inline u32
-__il_read32(char *f, u32 l, struct il_priv *priv, u32 ofs)
+__il_read32(char *f, u32 l, struct il_priv *il, u32 ofs)
 {
-	IL_DEBUG_IO(priv, "read_direct32(0x%08X) - %s %d\n", ofs, f, l);
-	return _il_read32(priv, ofs);
+	IL_DEBUG_IO(il, "read_direct32(0x%08X) - %s %d\n", ofs, f, l);
+	return _il_read32(il, ofs);
 }
-#define il_read32(priv, ofs) __il_read32(__FILE__, __LINE__, priv, ofs)
+#define il_read32(il, ofs) __il_read32(__FILE__, __LINE__, il, ofs)
 #else
 #define il_read32(p, o) _il_read32(p, o)
 #endif
 
 #define IL_POLL_INTERVAL 10	/* microseconds */
 static inline int
-_il_poll_bit(struct il_priv *priv, u32 addr,
+_il_poll_bit(struct il_priv *il, u32 addr,
 				u32 bits, u32 mask, int timeout)
 {
 	int t = 0;
 
 	do {
-		if ((_il_read32(priv, addr) & mask) == (bits & mask))
+		if ((_il_read32(il, addr) & mask) == (bits & mask))
 			return t;
 		udelay(IL_POLL_INTERVAL);
 		t += IL_POLL_INTERVAL;
@@ -137,34 +137,34 @@ _il_poll_bit(struct il_priv *priv, u32 addr,
 }
 #ifdef CONFIG_IWLWIFI_LEGACY_DEBUG
 static inline int __il_poll_bit(const char *f, u32 l,
-				 struct il_priv *priv, u32 addr,
+				 struct il_priv *il, u32 addr,
 				 u32 bits, u32 mask, int timeout)
 {
-	int ret = _il_poll_bit(priv, addr, bits, mask, timeout);
-	IL_DEBUG_IO(priv, "poll_bit(0x%08X, 0x%08X, 0x%08X) - %s- %s %d\n",
+	int ret = _il_poll_bit(il, addr, bits, mask, timeout);
+	IL_DEBUG_IO(il, "poll_bit(0x%08X, 0x%08X, 0x%08X) - %s- %s %d\n",
 		     addr, bits, mask,
 		     unlikely(ret  == -ETIMEDOUT) ? "timeout" : "", f, l);
 	return ret;
 }
-#define il_poll_bit(priv, addr, bits, mask, timeout) \
-	__il_poll_bit(__FILE__, __LINE__, priv, addr, \
+#define il_poll_bit(il, addr, bits, mask, timeout) \
+	__il_poll_bit(__FILE__, __LINE__, il, addr, \
 	bits, mask, timeout)
 #else
 #define il_poll_bit(p, a, b, m, t) _il_poll_bit(p, a, b, m, t)
 #endif
 
-static inline void _il_set_bit(struct il_priv *priv, u32 reg, u32 mask)
+static inline void _il_set_bit(struct il_priv *il, u32 reg, u32 mask)
 {
-	_il_write32(priv, reg, _il_read32(priv, reg) | mask);
+	_il_write32(il, reg, _il_read32(il, reg) | mask);
 }
 #ifdef CONFIG_IWLWIFI_LEGACY_DEBUG
 static inline void __il_set_bit(const char *f, u32 l,
-				 struct il_priv *priv, u32 reg, u32 mask)
+				 struct il_priv *il, u32 reg, u32 mask)
 {
-	u32 val = _il_read32(priv, reg) | mask;
-	IL_DEBUG_IO(priv, "set_bit(0x%08X, 0x%08X) = 0x%08X\n", reg,
+	u32 val = _il_read32(il, reg) | mask;
+	IL_DEBUG_IO(il, "set_bit(0x%08X, 0x%08X) = 0x%08X\n", reg,
 							mask, val);
-	_il_write32(priv, reg, val);
+	_il_write32(il, reg, val);
 }
 static inline void il_set_bit(struct il_priv *p, u32 r, u32 m)
 {
@@ -186,18 +186,18 @@ static inline void il_set_bit(struct il_priv *p, u32 r, u32 m)
 #endif
 
 static inline void
-_il_clear_bit(struct il_priv *priv, u32 reg, u32 mask)
+_il_clear_bit(struct il_priv *il, u32 reg, u32 mask)
 {
-	_il_write32(priv, reg, _il_read32(priv, reg) & ~mask);
+	_il_write32(il, reg, _il_read32(il, reg) & ~mask);
 }
 #ifdef CONFIG_IWLWIFI_LEGACY_DEBUG
 static inline void
 __il_clear_bit(const char *f, u32 l,
-				   struct il_priv *priv, u32 reg, u32 mask)
+				   struct il_priv *il, u32 reg, u32 mask)
 {
-	u32 val = _il_read32(priv, reg) & ~mask;
-	IL_DEBUG_IO(priv, "clear_bit(0x%08X, 0x%08X) = 0x%08X\n", reg, mask, val);
-	_il_write32(priv, reg, val);
+	u32 val = _il_read32(il, reg) & ~mask;
+	IL_DEBUG_IO(il, "clear_bit(0x%08X, 0x%08X) = 0x%08X\n", reg, mask, val);
+	_il_write32(il, reg, val);
 }
 static inline void il_clear_bit(struct il_priv *p, u32 r, u32 m)
 {
@@ -218,13 +218,13 @@ static inline void il_clear_bit(struct il_priv *p, u32 r, u32 m)
 }
 #endif
 
-static inline int _il_grab_nic_access(struct il_priv *priv)
+static inline int _il_grab_nic_access(struct il_priv *il)
 {
 	int ret;
 	u32 val;
 
 	/* this bit wakes up the NIC */
-	_il_set_bit(priv, CSR_GP_CNTRL,
+	_il_set_bit(il, CSR_GP_CNTRL,
 				CSR_GP_CNTRL_REG_FLAG_MAC_ACCESS_REQ);
 
 	/*
@@ -244,15 +244,15 @@ static inline int _il_grab_nic_access(struct il_priv *priv)
 	 * good idea before accessing 3945/4965 SRAM (e.g. reading Event Log).
 	 *
 	 */
-	ret = _il_poll_bit(priv, CSR_GP_CNTRL,
+	ret = _il_poll_bit(il, CSR_GP_CNTRL,
 			   CSR_GP_CNTRL_REG_VAL_MAC_ACCESS_EN,
 			   (CSR_GP_CNTRL_REG_FLAG_MAC_CLOCK_READY |
 			    CSR_GP_CNTRL_REG_FLAG_GOING_TO_SLEEP), 15000);
 	if (ret < 0) {
-		val = _il_read32(priv, CSR_GP_CNTRL);
-		IL_ERR(priv,
+		val = _il_read32(il, CSR_GP_CNTRL);
+		IL_ERR(il,
 			"MAC is in deep sleep!.  CSR_GP_CNTRL = 0x%08X\n", val);
-		_il_write32(priv, CSR_RESET,
+		_il_write32(il, CSR_RESET,
 				CSR_RESET_REG_FLAG_FORCE_NMI);
 		return -EIO;
 	}
@@ -262,117 +262,117 @@ static inline int _il_grab_nic_access(struct il_priv *priv)
 
 #ifdef CONFIG_IWLWIFI_LEGACY_DEBUG
 static inline int __il_grab_nic_access(const char *f, u32 l,
-					       struct il_priv *priv)
+					       struct il_priv *il)
 {
-	IL_DEBUG_IO(priv, "grabbing nic access - %s %d\n", f, l);
-	return _il_grab_nic_access(priv);
+	IL_DEBUG_IO(il, "grabbing nic access - %s %d\n", f, l);
+	return _il_grab_nic_access(il);
 }
-#define il_grab_nic_access(priv) \
-	__il_grab_nic_access(__FILE__, __LINE__, priv)
+#define il_grab_nic_access(il) \
+	__il_grab_nic_access(__FILE__, __LINE__, il)
 #else
-#define il_grab_nic_access(priv) \
-	_il_grab_nic_access(priv)
+#define il_grab_nic_access(il) \
+	_il_grab_nic_access(il)
 #endif
 
-static inline void _il_release_nic_access(struct il_priv *priv)
+static inline void _il_release_nic_access(struct il_priv *il)
 {
-	_il_clear_bit(priv, CSR_GP_CNTRL,
+	_il_clear_bit(il, CSR_GP_CNTRL,
 			CSR_GP_CNTRL_REG_FLAG_MAC_ACCESS_REQ);
 }
 #ifdef CONFIG_IWLWIFI_LEGACY_DEBUG
 static inline void __il_release_nic_access(const char *f, u32 l,
-					    struct il_priv *priv)
+					    struct il_priv *il)
 {
 
-	IL_DEBUG_IO(priv, "releasing nic access - %s %d\n", f, l);
-	_il_release_nic_access(priv);
+	IL_DEBUG_IO(il, "releasing nic access - %s %d\n", f, l);
+	_il_release_nic_access(il);
 }
-#define il_release_nic_access(priv) \
-	__il_release_nic_access(__FILE__, __LINE__, priv)
+#define il_release_nic_access(il) \
+	__il_release_nic_access(__FILE__, __LINE__, il)
 #else
-#define il_release_nic_access(priv) \
-	_il_release_nic_access(priv)
+#define il_release_nic_access(il) \
+	_il_release_nic_access(il)
 #endif
 
-static inline u32 _il_read_direct32(struct il_priv *priv, u32 reg)
+static inline u32 _il_read_direct32(struct il_priv *il, u32 reg)
 {
-	return _il_read32(priv, reg);
+	return _il_read32(il, reg);
 }
 #ifdef CONFIG_IWLWIFI_LEGACY_DEBUG
 static inline u32 __il_read_direct32(const char *f, u32 l,
-					struct il_priv *priv, u32 reg)
+					struct il_priv *il, u32 reg)
 {
-	u32 value = _il_read_direct32(priv, reg);
-	IL_DEBUG_IO(priv,
+	u32 value = _il_read_direct32(il, reg);
+	IL_DEBUG_IO(il,
 			"read_direct32(0x%4X) = 0x%08x - %s %d\n", reg, value,
 		     f, l);
 	return value;
 }
-static inline u32 il_read_direct32(struct il_priv *priv, u32 reg)
+static inline u32 il_read_direct32(struct il_priv *il, u32 reg)
 {
 	u32 value;
 	unsigned long reg_flags;
 
-	spin_lock_irqsave(&priv->reg_lock, reg_flags);
-	il_grab_nic_access(priv);
-	value = __il_read_direct32(__FILE__, __LINE__, priv, reg);
-	il_release_nic_access(priv);
-	spin_unlock_irqrestore(&priv->reg_lock, reg_flags);
+	spin_lock_irqsave(&il->reg_lock, reg_flags);
+	il_grab_nic_access(il);
+	value = __il_read_direct32(__FILE__, __LINE__, il, reg);
+	il_release_nic_access(il);
+	spin_unlock_irqrestore(&il->reg_lock, reg_flags);
 	return value;
 }
 
 #else
-static inline u32 il_read_direct32(struct il_priv *priv, u32 reg)
+static inline u32 il_read_direct32(struct il_priv *il, u32 reg)
 {
 	u32 value;
 	unsigned long reg_flags;
 
-	spin_lock_irqsave(&priv->reg_lock, reg_flags);
-	il_grab_nic_access(priv);
-	value = _il_read_direct32(priv, reg);
-	il_release_nic_access(priv);
-	spin_unlock_irqrestore(&priv->reg_lock, reg_flags);
+	spin_lock_irqsave(&il->reg_lock, reg_flags);
+	il_grab_nic_access(il);
+	value = _il_read_direct32(il, reg);
+	il_release_nic_access(il);
+	spin_unlock_irqrestore(&il->reg_lock, reg_flags);
 	return value;
 
 }
 #endif
 
-static inline void _il_write_direct32(struct il_priv *priv,
+static inline void _il_write_direct32(struct il_priv *il,
 					 u32 reg, u32 value)
 {
-	_il_write32(priv, reg, value);
+	_il_write32(il, reg, value);
 }
 static inline void
-il_write_direct32(struct il_priv *priv, u32 reg, u32 value)
+il_write_direct32(struct il_priv *il, u32 reg, u32 value)
 {
 	unsigned long reg_flags;
 
-	spin_lock_irqsave(&priv->reg_lock, reg_flags);
-	if (!il_grab_nic_access(priv)) {
-		_il_write_direct32(priv, reg, value);
-		il_release_nic_access(priv);
+	spin_lock_irqsave(&il->reg_lock, reg_flags);
+	if (!il_grab_nic_access(il)) {
+		_il_write_direct32(il, reg, value);
+		il_release_nic_access(il);
 	}
-	spin_unlock_irqrestore(&priv->reg_lock, reg_flags);
+	spin_unlock_irqrestore(&il->reg_lock, reg_flags);
 }
 
-static inline void il_write_reg_buf(struct il_priv *priv,
+static inline void il_write_reg_buf(struct il_priv *il,
 					       u32 reg, u32 len, u32 *values)
 {
 	u32 count = sizeof(u32);
 
-	if ((priv != NULL) && (values != NULL)) {
+	if ((il != NULL) && (values != NULL)) {
 		for (; 0 < len; len -= count, reg += count, values++)
-			il_write_direct32(priv, reg, *values);
+			il_write_direct32(il, reg, *values);
 	}
 }
 
-static inline int _il_poll_direct_bit(struct il_priv *priv, u32 addr,
+static inline int _il_poll_direct_bit(struct il_priv *il, u32 addr,
 				       u32 mask, int timeout)
 {
 	int t = 0;
 
 	do {
-		if ((il_read_direct32(priv, addr) & mask) == mask)
+		if ((il_read_direct32(il, addr) & mask) == mask)
 			return t;
 		udelay(IL_POLL_INTERVAL);
 		t += IL_POLL_INTERVAL;
@@ -383,159 +383,159 @@ static inline int _il_poll_direct_bit(struct il_priv *priv, u32 addr,
 
 #ifdef CONFIG_IWLWIFI_LEGACY_DEBUG
 static inline int __il_poll_direct_bit(const char *f, u32 l,
-					    struct il_priv *priv,
+					    struct il_priv *il,
 					    u32 addr, u32 mask, int timeout)
 {
-	int ret  = _il_poll_direct_bit(priv, addr, mask, timeout);
+	int ret  = _il_poll_direct_bit(il, addr, mask, timeout);
 
 	if (unlikely(ret == -ETIMEDOUT))
-		IL_DEBUG_IO(priv, "poll_direct_bit(0x%08X, 0x%08X) - "
+		IL_DEBUG_IO(il, "poll_direct_bit(0x%08X, 0x%08X) - "
 			     "timedout - %s %d\n", addr, mask, f, l);
 	else
-		IL_DEBUG_IO(priv, "poll_direct_bit(0x%08X, 0x%08X) = 0x%08X "
+		IL_DEBUG_IO(il, "poll_direct_bit(0x%08X, 0x%08X) = 0x%08X "
 			     "- %s %d\n", addr, mask, ret, f, l);
 	return ret;
 }
-#define il_poll_direct_bit(priv, addr, mask, timeout) \
-__il_poll_direct_bit(__FILE__, __LINE__, priv, addr, mask, timeout)
+#define il_poll_direct_bit(il, addr, mask, timeout) \
+__il_poll_direct_bit(__FILE__, __LINE__, il, addr, mask, timeout)
 #else
 #define il_poll_direct_bit _il_poll_direct_bit
 #endif
 
-static inline u32 _il_read_prph(struct il_priv *priv, u32 reg)
+static inline u32 _il_read_prph(struct il_priv *il, u32 reg)
 {
-	_il_write_direct32(priv, HBUS_TARG_PRPH_RADDR, reg | (3 << 24));
+	_il_write_direct32(il, HBUS_TARG_PRPH_RADDR, reg | (3 << 24));
 	rmb();
-	return _il_read_direct32(priv, HBUS_TARG_PRPH_RDAT);
+	return _il_read_direct32(il, HBUS_TARG_PRPH_RDAT);
 }
-static inline u32 il_read_prph(struct il_priv *priv, u32 reg)
+static inline u32 il_read_prph(struct il_priv *il, u32 reg)
 {
 	unsigned long reg_flags;
 	u32 val;
 
-	spin_lock_irqsave(&priv->reg_lock, reg_flags);
-	il_grab_nic_access(priv);
-	val = _il_read_prph(priv, reg);
-	il_release_nic_access(priv);
-	spin_unlock_irqrestore(&priv->reg_lock, reg_flags);
+	spin_lock_irqsave(&il->reg_lock, reg_flags);
+	il_grab_nic_access(il);
+	val = _il_read_prph(il, reg);
+	il_release_nic_access(il);
+	spin_unlock_irqrestore(&il->reg_lock, reg_flags);
 	return val;
 }
 
-static inline void _il_write_prph(struct il_priv *priv,
+static inline void _il_write_prph(struct il_priv *il,
 					     u32 addr, u32 val)
 {
-	_il_write_direct32(priv, HBUS_TARG_PRPH_WADDR,
+	_il_write_direct32(il, HBUS_TARG_PRPH_WADDR,
 			      ((addr & 0x0000FFFF) | (3 << 24)));
 	wmb();
-	_il_write_direct32(priv, HBUS_TARG_PRPH_WDAT, val);
+	_il_write_direct32(il, HBUS_TARG_PRPH_WDAT, val);
 }
 
 static inline void
-il_write_prph(struct il_priv *priv, u32 addr, u32 val)
+il_write_prph(struct il_priv *il, u32 addr, u32 val)
 {
 	unsigned long reg_flags;
 
-	spin_lock_irqsave(&priv->reg_lock, reg_flags);
-	if (!il_grab_nic_access(priv)) {
-		_il_write_prph(priv, addr, val);
-		il_release_nic_access(priv);
+	spin_lock_irqsave(&il->reg_lock, reg_flags);
+	if (!il_grab_nic_access(il)) {
+		_il_write_prph(il, addr, val);
+		il_release_nic_access(il);
 	}
-	spin_unlock_irqrestore(&priv->reg_lock, reg_flags);
+	spin_unlock_irqrestore(&il->reg_lock, reg_flags);
 }
 
-#define _il_set_bits_prph(priv, reg, mask) \
-_il_write_prph(priv, reg, (_il_read_prph(priv, reg) | mask))
+#define _il_set_bits_prph(il, reg, mask) \
+_il_write_prph(il, reg, (_il_read_prph(il, reg) | mask))
 
 static inline void
-il_set_bits_prph(struct il_priv *priv, u32 reg, u32 mask)
+il_set_bits_prph(struct il_priv *il, u32 reg, u32 mask)
 {
 	unsigned long reg_flags;
 
-	spin_lock_irqsave(&priv->reg_lock, reg_flags);
-	il_grab_nic_access(priv);
-	_il_set_bits_prph(priv, reg, mask);
-	il_release_nic_access(priv);
-	spin_unlock_irqrestore(&priv->reg_lock, reg_flags);
+	spin_lock_irqsave(&il->reg_lock, reg_flags);
+	il_grab_nic_access(il);
+	_il_set_bits_prph(il, reg, mask);
+	il_release_nic_access(il);
+	spin_unlock_irqrestore(&il->reg_lock, reg_flags);
 }
 
-#define _il_set_bits_mask_prph(priv, reg, bits, mask) \
-_il_write_prph(priv, reg,				\
-		 ((_il_read_prph(priv, reg) & mask) | bits))
+#define _il_set_bits_mask_prph(il, reg, bits, mask) \
+_il_write_prph(il, reg,				\
+		 ((_il_read_prph(il, reg) & mask) | bits))
 
-static inline void il_set_bits_mask_prph(struct il_priv *priv, u32 reg,
+static inline void il_set_bits_mask_prph(struct il_priv *il, u32 reg,
 				u32 bits, u32 mask)
 {
 	unsigned long reg_flags;
 
-	spin_lock_irqsave(&priv->reg_lock, reg_flags);
-	il_grab_nic_access(priv);
-	_il_set_bits_mask_prph(priv, reg, bits, mask);
-	il_release_nic_access(priv);
-	spin_unlock_irqrestore(&priv->reg_lock, reg_flags);
+	spin_lock_irqsave(&il->reg_lock, reg_flags);
+	il_grab_nic_access(il);
+	_il_set_bits_mask_prph(il, reg, bits, mask);
+	il_release_nic_access(il);
+	spin_unlock_irqrestore(&il->reg_lock, reg_flags);
 }
 
 static inline void il_clear_bits_prph(struct il_priv
-						 *priv, u32 reg, u32 mask)
+						 *il, u32 reg, u32 mask)
 {
 	unsigned long reg_flags;
 	u32 val;
 
-	spin_lock_irqsave(&priv->reg_lock, reg_flags);
-	il_grab_nic_access(priv);
-	val = _il_read_prph(priv, reg);
-	_il_write_prph(priv, reg, (val & ~mask));
-	il_release_nic_access(priv);
-	spin_unlock_irqrestore(&priv->reg_lock, reg_flags);
+	spin_lock_irqsave(&il->reg_lock, reg_flags);
+	il_grab_nic_access(il);
+	val = _il_read_prph(il, reg);
+	_il_write_prph(il, reg, (val & ~mask));
+	il_release_nic_access(il);
+	spin_unlock_irqrestore(&il->reg_lock, reg_flags);
 }
 
-static inline u32 il_read_targ_mem(struct il_priv *priv, u32 addr)
+static inline u32 il_read_targ_mem(struct il_priv *il, u32 addr)
 {
 	unsigned long reg_flags;
 	u32 value;
 
-	spin_lock_irqsave(&priv->reg_lock, reg_flags);
-	il_grab_nic_access(priv);
+	spin_lock_irqsave(&il->reg_lock, reg_flags);
+	il_grab_nic_access(il);
 
-	_il_write_direct32(priv, HBUS_TARG_MEM_RADDR, addr);
+	_il_write_direct32(il, HBUS_TARG_MEM_RADDR, addr);
 	rmb();
-	value = _il_read_direct32(priv, HBUS_TARG_MEM_RDAT);
+	value = _il_read_direct32(il, HBUS_TARG_MEM_RDAT);
 
-	il_release_nic_access(priv);
-	spin_unlock_irqrestore(&priv->reg_lock, reg_flags);
+	il_release_nic_access(il);
+	spin_unlock_irqrestore(&il->reg_lock, reg_flags);
 	return value;
 }
 
 static inline void
-il_write_targ_mem(struct il_priv *priv, u32 addr, u32 val)
+il_write_targ_mem(struct il_priv *il, u32 addr, u32 val)
 {
 	unsigned long reg_flags;
 
-	spin_lock_irqsave(&priv->reg_lock, reg_flags);
-	if (!il_grab_nic_access(priv)) {
-		_il_write_direct32(priv, HBUS_TARG_MEM_WADDR, addr);
+	spin_lock_irqsave(&il->reg_lock, reg_flags);
+	if (!il_grab_nic_access(il)) {
+		_il_write_direct32(il, HBUS_TARG_MEM_WADDR, addr);
 		wmb();
-		_il_write_direct32(priv, HBUS_TARG_MEM_WDAT, val);
-		il_release_nic_access(priv);
+		_il_write_direct32(il, HBUS_TARG_MEM_WDAT, val);
+		il_release_nic_access(il);
 	}
-	spin_unlock_irqrestore(&priv->reg_lock, reg_flags);
+	spin_unlock_irqrestore(&il->reg_lock, reg_flags);
 }
 
 static inline void
-il_write_targ_mem_buf(struct il_priv *priv, u32 addr,
+il_write_targ_mem_buf(struct il_priv *il, u32 addr,
 					  u32 len, u32 *values)
 {
 	unsigned long reg_flags;
 
-	spin_lock_irqsave(&priv->reg_lock, reg_flags);
-	if (!il_grab_nic_access(priv)) {
-		_il_write_direct32(priv, HBUS_TARG_MEM_WADDR, addr);
+	spin_lock_irqsave(&il->reg_lock, reg_flags);
+	if (!il_grab_nic_access(il)) {
+		_il_write_direct32(il, HBUS_TARG_MEM_WADDR, addr);
 		wmb();
 		for (; 0 < len; len -= sizeof(u32), values++)
-			_il_write_direct32(priv,
+			_il_write_direct32(il,
 					HBUS_TARG_MEM_WDAT, *values);
 
-		il_release_nic_access(priv);
+		il_release_nic_access(il);
 	}
-	spin_unlock_irqrestore(&priv->reg_lock, reg_flags);
+	spin_unlock_irqrestore(&il->reg_lock, reg_flags);
 }
 #endif