Skip to content

Commit

Permalink
iwlegacy: mark poll bit as unlocked function
Browse files Browse the repository at this point in the history
We do not take reg_lock during poll bit, so mark it such
using underscore.

Signed-off-by: Stanislaw Gruszka <sgruszka@redhat.com>
  • Loading branch information
Stanislaw Gruszka committed Nov 15, 2011
1 parent 841b2cc commit 142b343
Show file tree
Hide file tree
Showing 6 changed files with 8 additions and 9 deletions.
4 changes: 2 additions & 2 deletions drivers/net/wireless/iwlegacy/iwl-3945.c
Original file line number Diff line number Diff line change
Expand Up @@ -757,7 +757,7 @@ static void il3945_set_pwr_vmain(struct il_priv *il)
APMG_PS_CTRL_VAL_PWR_SRC_VAUX,
~APMG_PS_CTRL_MSK_PWR_SRC);
il_poll_bit(il, CSR_GPIO_IN,
_il_poll_bit(il, CSR_GPIO_IN,
CSR_GPIO_IN_VAL_VAUX_PWR_SRC,
CSR_GPIO_IN_BIT_AUX_POWER, 5000);
}
Expand All @@ -767,7 +767,7 @@ static void il3945_set_pwr_vmain(struct il_priv *il)
APMG_PS_CTRL_VAL_PWR_SRC_VMAIN,
~APMG_PS_CTRL_MSK_PWR_SRC);

il_poll_bit(il, CSR_GPIO_IN, CSR_GPIO_IN_VAL_VMAIN_PWR_SRC,
_il_poll_bit(il, CSR_GPIO_IN, CSR_GPIO_IN_VAL_VMAIN_PWR_SRC,
CSR_GPIO_IN_BIT_AUX_POWER, 5000); /* uS */
}

Expand Down
2 changes: 1 addition & 1 deletion drivers/net/wireless/iwlegacy/iwl-4965-eeprom.c
Original file line number Diff line number Diff line change
Expand Up @@ -98,7 +98,7 @@ int il4965_eeprom_acquire_semaphore(struct il_priv *il)
CSR_HW_IF_CONFIG_REG_BIT_EEPROM_OWN_SEM);

/* See if we got it */
ret = il_poll_bit(il, CSR_HW_IF_CONFIG_REG,
ret = _il_poll_bit(il, CSR_HW_IF_CONFIG_REG,
CSR_HW_IF_CONFIG_REG_BIT_EEPROM_OWN_SEM,
CSR_HW_IF_CONFIG_REG_BIT_EEPROM_OWN_SEM,
EEPROM_SEM_TIMEOUT);
Expand Down
4 changes: 2 additions & 2 deletions drivers/net/wireless/iwlegacy/iwl-core.c
Original file line number Diff line number Diff line change
Expand Up @@ -960,7 +960,7 @@ static int il_apm_stop_master(struct il_priv *il)
/* stop device's busmaster DMA activity */
il_set_bit(il, CSR_RESET, CSR_RESET_REG_FLAG_STOP_MASTER);

ret = il_poll_bit(il, CSR_RESET, CSR_RESET_REG_FLAG_MASTER_DISABLED,
ret = _il_poll_bit(il, CSR_RESET, CSR_RESET_REG_FLAG_MASTER_DISABLED,
CSR_RESET_REG_FLAG_MASTER_DISABLED, 100);
if (ret)
IL_WARN("Master Disable Timed Out, 100 usec\n");
Expand Down Expand Up @@ -1072,7 +1072,7 @@ int il_apm_init(struct il_priv *il)
* device-internal resources is supported, e.g. il_write_prph()
* and accesses to uCode SRAM.
*/
ret = il_poll_bit(il, CSR_GP_CNTRL,
ret = _il_poll_bit(il, CSR_GP_CNTRL,
CSR_GP_CNTRL_REG_FLAG_MAC_CLOCK_READY,
CSR_GP_CNTRL_REG_FLAG_MAC_CLOCK_READY, 25000);
if (ret < 0) {
Expand Down
2 changes: 1 addition & 1 deletion drivers/net/wireless/iwlegacy/iwl-eeprom.c
Original file line number Diff line number Diff line change
Expand Up @@ -226,7 +226,7 @@ int il_eeprom_init(struct il_priv *il)
_il_wr(il, CSR_EEPROM_REG,
CSR_EEPROM_REG_MSK_ADDR & (addr << 1));

ret = il_poll_bit(il, CSR_EEPROM_REG,
ret = _il_poll_bit(il, CSR_EEPROM_REG,
CSR_EEPROM_REG_READ_VALID_MSK,
CSR_EEPROM_REG_READ_VALID_MSK,
IL_EEPROM_ACCESS_TIMEOUT);
Expand Down
1 change: 0 additions & 1 deletion drivers/net/wireless/iwlegacy/iwl-io.h
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,6 @@ _il_poll_bit(struct il_priv *il, u32 addr,

return -ETIMEDOUT;
}
#define il_poll_bit(p, a, b, m, t) _il_poll_bit(p, a, b, m, t)

static inline void _il_set_bit(struct il_priv *il, u32 reg, u32 mask)
{
Expand Down
4 changes: 2 additions & 2 deletions drivers/net/wireless/iwlegacy/iwl4965-base.c
Original file line number Diff line number Diff line change
Expand Up @@ -1907,7 +1907,7 @@ static int il4965_set_hw_ready(struct il_priv *il)
CSR_HW_IF_CONFIG_REG_BIT_NIC_READY);

/* See if we got it */
ret = il_poll_bit(il, CSR_HW_IF_CONFIG_REG,
ret = _il_poll_bit(il, CSR_HW_IF_CONFIG_REG,
CSR_HW_IF_CONFIG_REG_BIT_NIC_READY,
CSR_HW_IF_CONFIG_REG_BIT_NIC_READY,
HW_READY_TIMEOUT);
Expand Down Expand Up @@ -1935,7 +1935,7 @@ static int il4965_prepare_card_hw(struct il_priv *il)
il_set_bit(il, CSR_HW_IF_CONFIG_REG,
CSR_HW_IF_CONFIG_REG_PREPARE);

ret = il_poll_bit(il, CSR_HW_IF_CONFIG_REG,
ret = _il_poll_bit(il, CSR_HW_IF_CONFIG_REG,
~CSR_HW_IF_CONFIG_REG_BIT_NIC_PREPARE_DONE,
CSR_HW_IF_CONFIG_REG_BIT_NIC_PREPARE_DONE, 150000);

Expand Down

0 comments on commit 142b343

Please sign in to comment.