diff options
author | Amit Beka <amit.beka@intel.com> | 2012-02-05 17:40:47 -0500 |
---|---|---|
committer | Wey-Yi Guy <wey-yi.w.guy@intel.com> | 2012-02-17 13:55:00 -0500 |
commit | 858b7c74dda1d748a687a523e83cf7d1449c70f3 (patch) | |
tree | 4303849e0bd88de67147831c99a93bc9079ca399 | |
parent | 637d7925910df46bb29a0260dd2fec68b50a811b (diff) |
iwlwifi: fixes to testmode indirect access
Fixed casting of buffer addressing, and added size to
the read method, like in __iwl_read_prph.
Signed-off-by: Amit Beka <amit.beka@intel.com>
Signed-off-by: Wey-Yi Guy <wey-yi.w.guy@intel.com>
-rw-r--r-- | drivers/net/wireless/iwlwifi/iwl-testmode.c | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/drivers/net/wireless/iwlwifi/iwl-testmode.c b/drivers/net/wireless/iwlwifi/iwl-testmode.c index 5f00a3712707..ba5c63a023a2 100644 --- a/drivers/net/wireless/iwlwifi/iwl-testmode.c +++ b/drivers/net/wireless/iwlwifi/iwl-testmode.c | |||
@@ -799,9 +799,10 @@ static int iwl_testmode_indirect_read(struct iwl_priv *priv, u32 addr, u32 size) | |||
799 | addr < IWL_TM_ABS_PRPH_START + PRPH_END) { | 799 | addr < IWL_TM_ABS_PRPH_START + PRPH_END) { |
800 | spin_lock_irqsave(&trans->reg_lock, flags); | 800 | spin_lock_irqsave(&trans->reg_lock, flags); |
801 | iwl_grab_nic_access(trans); | 801 | iwl_grab_nic_access(trans); |
802 | iwl_write32(trans, HBUS_TARG_PRPH_RADDR, addr); | 802 | iwl_write32(trans, HBUS_TARG_PRPH_RADDR, |
803 | addr | (3 << 24)); | ||
803 | for (i = 0; i < size; i += 4) | 804 | for (i = 0; i < size; i += 4) |
804 | priv->testmode_mem.buff_addr[i] = | 805 | *(u32 *)(priv->testmode_mem.buff_addr + i) = |
805 | iwl_read32(trans, HBUS_TARG_PRPH_RDAT); | 806 | iwl_read32(trans, HBUS_TARG_PRPH_RDAT); |
806 | iwl_release_nic_access(trans); | 807 | iwl_release_nic_access(trans); |
807 | spin_unlock_irqrestore(&trans->reg_lock, flags); | 808 | spin_unlock_irqrestore(&trans->reg_lock, flags); |
@@ -833,7 +834,8 @@ static int iwl_testmode_indirect_write(struct iwl_priv *priv, u32 addr, | |||
833 | spin_lock_irqsave(&trans->reg_lock, flags); | 834 | spin_lock_irqsave(&trans->reg_lock, flags); |
834 | iwl_grab_nic_access(trans); | 835 | iwl_grab_nic_access(trans); |
835 | iwl_write32(trans, HBUS_TARG_PRPH_WADDR, | 836 | iwl_write32(trans, HBUS_TARG_PRPH_WADDR, |
836 | (addr & 0x0000FFFF) | (size << 24)); | 837 | (addr & 0x0000FFFF) | |
838 | ((size - 1) << 24)); | ||
837 | iwl_write32(trans, HBUS_TARG_PRPH_WDAT, val); | 839 | iwl_write32(trans, HBUS_TARG_PRPH_WDAT, val); |
838 | iwl_release_nic_access(trans); | 840 | iwl_release_nic_access(trans); |
839 | /* needed after consecutive writes w/o read */ | 841 | /* needed after consecutive writes w/o read */ |
@@ -844,7 +846,7 @@ static int iwl_testmode_indirect_write(struct iwl_priv *priv, u32 addr, | |||
844 | return -EINVAL; | 846 | return -EINVAL; |
845 | for (i = 0; i < size; i += 4) | 847 | for (i = 0; i < size; i += 4) |
846 | iwl_write_prph(trans, addr+i, | 848 | iwl_write_prph(trans, addr+i, |
847 | *(u32 *)buf+i); | 849 | *(u32 *)(buf+i)); |
848 | } | 850 | } |
849 | } else if (iwlagn_hw_valid_rtc_data_addr(addr) || | 851 | } else if (iwlagn_hw_valid_rtc_data_addr(addr) || |
850 | (IWLAGN_RTC_INST_LOWER_BOUND <= addr && | 852 | (IWLAGN_RTC_INST_LOWER_BOUND <= addr && |