* [PATCH 0/3i 3.18] Fix more problems with rtlwifi
@ 2014-11-06 1:10 Larry Finger
2014-11-06 1:10 ` [PATCH 1/3 3.18] rtlwifi: Fix setting of tx descriptor for new trx flow Larry Finger
` (2 more replies)
0 siblings, 3 replies; 9+ messages in thread
From: Larry Finger @ 2014-11-06 1:10 UTC (permalink / raw)
To: linville; +Cc: linux-wireless, Larry Finger, netdev
This set of patches fix some additional problems found for rtlwifi,
rtl8192se, and rtl8192ee.
It is certainly possible that rtlwifi is getting too large. For that reason,
my changes for 3.19 will be restricted to identifying common routines, and
moving such code from the individual drivers into driver rtlwifi.
Signed-off-by: Larry Finger <Larry.Finger@lwfinger.net>
Larry Finger (3):
rtlwifi: Fix setting of tx descriptor for new trx flow
rtlwifi: Fix errors in descriptor manipulation
rtlwifi: rtl8192se: Fix connection problems
drivers/net/wireless/rtlwifi/pci.c | 16 ++--
drivers/net/wireless/rtlwifi/rtl8192se/hw.c | 129 +++++++++++++--------------
drivers/net/wireless/rtlwifi/rtl8192se/phy.c | 8 +-
drivers/net/wireless/rtlwifi/rtl8192se/sw.c | 4 +
drivers/net/wireless/rtlwifi/rtl8192se/trx.c | 23 +++++
drivers/net/wireless/rtlwifi/rtl8192se/trx.h | 4 +
6 files changed, 110 insertions(+), 74 deletions(-)
--
2.1.2
^ permalink raw reply [flat|nested] 9+ messages in thread* [PATCH 1/3 3.18] rtlwifi: Fix setting of tx descriptor for new trx flow 2014-11-06 1:10 [PATCH 0/3i 3.18] Fix more problems with rtlwifi Larry Finger @ 2014-11-06 1:10 ` Larry Finger 2014-11-06 1:10 ` [PATCH 2/3 3.18] rtlwifi: Fix errors in descriptor manipulation Larry Finger 2014-11-06 1:10 ` [PATCH 3/3 3.18] rtlwifi: rtl8192se: Fix connection problems Larry Finger 2 siblings, 0 replies; 9+ messages in thread From: Larry Finger @ 2014-11-06 1:10 UTC (permalink / raw) To: linville; +Cc: linux-wireless, Larry Finger, netdev Device RTL8192EE uses a new form of trx flow. This fix sets up the descriptors correctly. Signed-off-by: Larry Finger <Larry.Finger@lwfinger.net> --- drivers/net/wireless/rtlwifi/pci.c | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/drivers/net/wireless/rtlwifi/pci.c b/drivers/net/wireless/rtlwifi/pci.c index 25daa87..116f746 100644 --- a/drivers/net/wireless/rtlwifi/pci.c +++ b/drivers/net/wireless/rtlwifi/pci.c @@ -1127,9 +1127,14 @@ static void _rtl_pci_prepare_bcn_tasklet(struct ieee80211_hw *hw) __skb_queue_tail(&ring->queue, pskb); - rtlpriv->cfg->ops->set_desc(hw, (u8 *)pdesc, true, HW_DESC_OWN, - &temp_one); - + if (rtlpriv->use_new_trx_flow) { + temp_one = 4; + rtlpriv->cfg->ops->set_desc(hw, (u8 *)pbuffer_desc, true, + HW_DESC_OWN, (u8 *)&temp_one); + } else { + rtlpriv->cfg->ops->set_desc(hw, (u8 *)pdesc, true, HW_DESC_OWN, + &temp_one); + } return; } -- 2.1.2 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* [PATCH 2/3 3.18] rtlwifi: Fix errors in descriptor manipulation 2014-11-06 1:10 [PATCH 0/3i 3.18] Fix more problems with rtlwifi Larry Finger 2014-11-06 1:10 ` [PATCH 1/3 3.18] rtlwifi: Fix setting of tx descriptor for new trx flow Larry Finger @ 2014-11-06 1:10 ` Larry Finger 2014-11-06 1:10 ` [PATCH 3/3 3.18] rtlwifi: rtl8192se: Fix connection problems Larry Finger 2 siblings, 0 replies; 9+ messages in thread From: Larry Finger @ 2014-11-06 1:10 UTC (permalink / raw) To: linville; +Cc: linux-wireless, Larry Finger, netdev There are typos in the handling of the descriptor pointers where the wrong descriptor is referenced. There is also an error in which the pointer is incremented twice. Signed-off-by: Larry Finger <Larry.Finger@lwfinger.net> --- drivers/net/wireless/rtlwifi/pci.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/net/wireless/rtlwifi/pci.c b/drivers/net/wireless/rtlwifi/pci.c index 116f746..6d2b628 100644 --- a/drivers/net/wireless/rtlwifi/pci.c +++ b/drivers/net/wireless/rtlwifi/pci.c @@ -1375,9 +1375,9 @@ static void _rtl_pci_free_tx_ring(struct ieee80211_hw *hw, ring->desc = NULL; if (rtlpriv->use_new_trx_flow) { pci_free_consistent(rtlpci->pdev, - sizeof(*ring->desc) * ring->entries, + sizeof(*ring->buffer_desc) * ring->entries, ring->buffer_desc, ring->buffer_desc_dma); - ring->desc = NULL; + ring->buffer_desc = NULL; } } @@ -1548,7 +1548,6 @@ int rtl_pci_reset_trx_ring(struct ieee80211_hw *hw) true, HW_DESC_TXBUFF_ADDR), skb->len, PCI_DMA_TODEVICE); - ring->idx = (ring->idx + 1) % ring->entries; kfree_skb(skb); ring->idx = (ring->idx + 1) % ring->entries; } -- 2.1.2 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* [PATCH 3/3 3.18] rtlwifi: rtl8192se: Fix connection problems 2014-11-06 1:10 [PATCH 0/3i 3.18] Fix more problems with rtlwifi Larry Finger 2014-11-06 1:10 ` [PATCH 1/3 3.18] rtlwifi: Fix setting of tx descriptor for new trx flow Larry Finger 2014-11-06 1:10 ` [PATCH 2/3 3.18] rtlwifi: Fix errors in descriptor manipulation Larry Finger @ 2014-11-06 1:10 ` Larry Finger 2014-11-06 20:45 ` John W. Linville 2 siblings, 1 reply; 9+ messages in thread From: Larry Finger @ 2014-11-06 1:10 UTC (permalink / raw) To: linville; +Cc: linux-wireless, Larry Finger, netdev Changes in the vendor driver were added to rtlwifi, but some updates to rtl8192se were missed. Signed-off-by: Larry Finger <Larry.Finger@lwfinger.net> --- drivers/net/wireless/rtlwifi/rtl8192se/hw.c | 129 +++++++++++++-------------- drivers/net/wireless/rtlwifi/rtl8192se/phy.c | 8 +- drivers/net/wireless/rtlwifi/rtl8192se/sw.c | 4 + drivers/net/wireless/rtlwifi/rtl8192se/trx.c | 23 +++++ drivers/net/wireless/rtlwifi/rtl8192se/trx.h | 4 + 5 files changed, 100 insertions(+), 68 deletions(-) diff --git a/drivers/net/wireless/rtlwifi/rtl8192se/hw.c b/drivers/net/wireless/rtlwifi/rtl8192se/hw.c index 00e0670..4626203 100644 --- a/drivers/net/wireless/rtlwifi/rtl8192se/hw.c +++ b/drivers/net/wireless/rtlwifi/rtl8192se/hw.c @@ -1170,27 +1170,32 @@ static int _rtl92se_set_media_status(struct ieee80211_hw *hw, { struct rtl_priv *rtlpriv = rtl_priv(hw); u8 bt_msr = rtl_read_byte(rtlpriv, MSR); + enum led_ctl_mode ledaction = LED_CTL_NO_LINK; u32 temp; + u8 mode = MSR_NOLINK; + bt_msr &= ~MSR_LINK_MASK; switch (type) { case NL80211_IFTYPE_UNSPECIFIED: - bt_msr |= (MSR_LINK_NONE << MSR_LINK_SHIFT); + mode = MSR_NOLINK; RT_TRACE(rtlpriv, COMP_INIT, DBG_TRACE, "Set Network type to NO LINK!\n"); break; case NL80211_IFTYPE_ADHOC: - bt_msr |= (MSR_LINK_ADHOC << MSR_LINK_SHIFT); + mode = MSR_ADHOC; RT_TRACE(rtlpriv, COMP_INIT, DBG_TRACE, "Set Network type to Ad Hoc!\n"); break; case NL80211_IFTYPE_STATION: - bt_msr |= (MSR_LINK_MANAGED << MSR_LINK_SHIFT); + mode = MSR_INFRA; + ledaction = LED_CTL_LINK; RT_TRACE(rtlpriv, COMP_INIT, DBG_TRACE, "Set Network type to STA!\n"); break; case NL80211_IFTYPE_AP: - bt_msr |= (MSR_LINK_MASTER << MSR_LINK_SHIFT); + mode = MSR_AP; + ledaction = LED_CTL_LINK; RT_TRACE(rtlpriv, COMP_INIT, DBG_TRACE, "Set Network type to AP!\n"); break; @@ -1201,7 +1206,17 @@ static int _rtl92se_set_media_status(struct ieee80211_hw *hw, } - rtl_write_byte(rtlpriv, (MSR), bt_msr); + /* MSR_INFRA == Link in infrastructure network; + * MSR_ADHOC == Link in ad hoc network; + * Therefore, check link state is necessary. + * + * MSR_AP == AP mode; link state is not cared here. + */ + if (mode != MSR_AP && rtlpriv->mac80211.link_state < MAC80211_LINKED) { + mode = MSR_NOLINK; + ledaction = LED_CTL_NO_LINK; +} + rtl_write_byte(rtlpriv, (MSR), bt_msr | mode); temp = rtl_read_dword(rtlpriv, TCR); rtl_write_dword(rtlpriv, TCR, temp & (~BIT(8))); @@ -1262,6 +1277,7 @@ void rtl92se_enable_interrupt(struct ieee80211_hw *hw) rtl_write_dword(rtlpriv, INTA_MASK, rtlpci->irq_mask[0]); /* Support Bit 32-37(Assign as Bit 0-5) interrupt setting now */ rtl_write_dword(rtlpriv, INTA_MASK + 4, rtlpci->irq_mask[1] & 0x3F); + rtlpci->irq_enabled = true; } void rtl92se_disable_interrupt(struct ieee80211_hw *hw) @@ -1276,8 +1292,7 @@ void rtl92se_disable_interrupt(struct ieee80211_hw *hw) rtlpci = rtl_pcidev(rtl_pcipriv(hw)); rtl_write_dword(rtlpriv, INTA_MASK, 0); rtl_write_dword(rtlpriv, INTA_MASK + 4, 0); - - synchronize_irq(rtlpci->pdev->irq); + rtlpci->irq_enabled = false; } static u8 _rtl92s_set_sysclk(struct ieee80211_hw *hw, u8 data) @@ -2035,9 +2050,9 @@ static void rtl92se_update_hal_rate_table(struct ieee80211_hw *hw, u32 ratr_value; u8 ratr_index = 0; u8 nmode = mac->ht_enable; - u8 mimo_ps = IEEE80211_SMPS_OFF; u16 shortgi_rate = 0; u32 tmp_ratr_value = 0; + u32 ratr_mask; u8 curtxbw_40mhz = mac->bw_40; u8 curshortgi_40mhz = (sta->ht_cap.cap & IEEE80211_HT_CAP_SGI_40) ? 1 : 0; @@ -2063,26 +2078,21 @@ static void rtl92se_update_hal_rate_table(struct ieee80211_hw *hw, case WIRELESS_MODE_N_24G: case WIRELESS_MODE_N_5G: nmode = 1; - if (mimo_ps == IEEE80211_SMPS_STATIC) { - ratr_value &= 0x0007F005; - } else { - u32 ratr_mask; - if (get_rf_type(rtlphy) == RF_1T2R || - get_rf_type(rtlphy) == RF_1T1R) { - if (curtxbw_40mhz) - ratr_mask = 0x000ff015; - else - ratr_mask = 0x000ff005; - } else { - if (curtxbw_40mhz) - ratr_mask = 0x0f0ff015; - else - ratr_mask = 0x0f0ff005; - } - - ratr_value &= ratr_mask; + if (get_rf_type(rtlphy) == RF_1T2R || + get_rf_type(rtlphy) == RF_1T1R) { + if (curtxbw_40mhz) + ratr_mask = 0x000ff015; + else + ratr_mask = 0x000ff005; + } else { + if (curtxbw_40mhz) + ratr_mask = 0x0f0ff015; + else + ratr_mask = 0x0f0ff005; } + + ratr_value &= ratr_mask; break; default: if (rtlphy->rf_type == RF_1T2R) @@ -2137,7 +2147,8 @@ static void rtl92se_update_hal_rate_mask(struct ieee80211_hw *hw, struct rtl_sta_info *sta_entry = NULL; u32 ratr_bitmap; u8 ratr_index = 0; - u8 curtxbw_40mhz = (sta->bandwidth >= IEEE80211_STA_RX_BW_40) ? 1 : 0; + u8 curtxbw_40mhz = (sta->ht_cap.cap & IEEE80211_HT_CAP_SUP_WIDTH_20_40) + ? 1 : 0; u8 curshortgi_40mhz = (sta->ht_cap.cap & IEEE80211_HT_CAP_SGI_40) ? 1 : 0; u8 curshortgi_20mhz = (sta->ht_cap.cap & IEEE80211_HT_CAP_SGI_20) ? @@ -2148,9 +2159,7 @@ static void rtl92se_update_hal_rate_mask(struct ieee80211_hw *hw, u8 shortgi_rate = 0; u32 mask = 0; u32 band = 0; - bool bmulticast = false; u8 macid = 0; - u8 mimo_ps = IEEE80211_SMPS_OFF; sta_entry = (struct rtl_sta_info *) sta->drv_priv; wirelessmode = sta_entry->wireless_mode; @@ -2198,41 +2207,32 @@ static void rtl92se_update_hal_rate_mask(struct ieee80211_hw *hw, band |= (WIRELESS_11N | WIRELESS_11G | WIRELESS_11B); ratr_index = RATR_INX_WIRELESS_NGB; - if (mimo_ps == IEEE80211_SMPS_STATIC) { - if (rssi_level == 1) - ratr_bitmap &= 0x00070000; - else if (rssi_level == 2) - ratr_bitmap &= 0x0007f000; - else - ratr_bitmap &= 0x0007f005; + if (rtlphy->rf_type == RF_1T2R || + rtlphy->rf_type == RF_1T1R) { + if (rssi_level == 1) { + ratr_bitmap &= 0x000f0000; + } else if (rssi_level == 3) { + ratr_bitmap &= 0x000fc000; + } else if (rssi_level == 5) { + ratr_bitmap &= 0x000ff000; + } else { + if (curtxbw_40mhz) + ratr_bitmap &= 0x000ff015; + else + ratr_bitmap &= 0x000ff005; + } } else { - if (rtlphy->rf_type == RF_1T2R || - rtlphy->rf_type == RF_1T1R) { - if (rssi_level == 1) { - ratr_bitmap &= 0x000f0000; - } else if (rssi_level == 3) { - ratr_bitmap &= 0x000fc000; - } else if (rssi_level == 5) { - ratr_bitmap &= 0x000ff000; - } else { - if (curtxbw_40mhz) - ratr_bitmap &= 0x000ff015; - else - ratr_bitmap &= 0x000ff005; - } + if (rssi_level == 1) { + ratr_bitmap &= 0x0f8f0000; + } else if (rssi_level == 3) { + ratr_bitmap &= 0x0f8fc000; + } else if (rssi_level == 5) { + ratr_bitmap &= 0x0f8ff000; } else { - if (rssi_level == 1) { - ratr_bitmap &= 0x0f8f0000; - } else if (rssi_level == 3) { - ratr_bitmap &= 0x0f8fc000; - } else if (rssi_level == 5) { - ratr_bitmap &= 0x0f8ff000; - } else { - if (curtxbw_40mhz) - ratr_bitmap &= 0x0f8ff015; - else - ratr_bitmap &= 0x0f8ff005; - } + if (curtxbw_40mhz) + ratr_bitmap &= 0x0f8ff015; + else + ratr_bitmap &= 0x0f8ff005; } } @@ -2275,15 +2275,12 @@ static void rtl92se_update_hal_rate_mask(struct ieee80211_hw *hw, rtl_write_byte(rtlpriv, SG_RATE, shortgi_rate); } - mask |= (bmulticast ? 1 : 0) << 9 | (macid & 0x1f) << 4 | (band & 0xf); + mask |= (macid & 0x1f) << 4 | (band & 0xf); RT_TRACE(rtlpriv, COMP_RATR, DBG_TRACE, "mask = %x, bitmap = %x\n", mask, ratr_bitmap); rtl_write_dword(rtlpriv, 0x2c4, ratr_bitmap); rtl_write_dword(rtlpriv, WFM5, (FW_RA_UPDATE_MASK | (mask << 8))); - - if (macid != 0) - sta_entry->ratr_index = ratr_index; } void rtl92se_update_hal_rate_tbl(struct ieee80211_hw *hw, diff --git a/drivers/net/wireless/rtlwifi/rtl8192se/phy.c b/drivers/net/wireless/rtlwifi/rtl8192se/phy.c index 77c5b5f..e382cef 100644 --- a/drivers/net/wireless/rtlwifi/rtl8192se/phy.c +++ b/drivers/net/wireless/rtlwifi/rtl8192se/phy.c @@ -399,6 +399,11 @@ static bool _rtl92s_phy_sw_chnl_step_by_step(struct ieee80211_hw *hw, case 2: currentcmd = &postcommoncmd[*step]; break; + default: + RT_TRACE(rtlpriv, COMP_ERR, DBG_LOUD, + "Invalid 'stage' = %d, Check it!\n", + *stage); + return true; } if (currentcmd->cmdid == CMDID_END) { @@ -602,7 +607,7 @@ bool rtl92s_phy_set_rf_power_state(struct ieee80211_hw *hw, } case ERFSLEEP: if (ppsc->rfpwr_state == ERFOFF) - return false; + break; for (queue_id = 0, i = 0; queue_id < RTL_PCI_MAX_TX_QUEUE_COUNT;) { @@ -1064,7 +1069,6 @@ bool rtl92s_phy_bb_config(struct ieee80211_hw *hw) /* Check BB/RF confiuration setting. */ /* We only need to configure RF which is turned on. */ path1 = (u8)(rtl92s_phy_query_bb_reg(hw, RFPGA0_TXINFO, 0xf)); - mdelay(10); path2 = (u8)(rtl92s_phy_query_bb_reg(hw, ROFDM0_TRXPATHENABLE, 0xf)); pathmap = path1 | path2; diff --git a/drivers/net/wireless/rtlwifi/rtl8192se/sw.c b/drivers/net/wireless/rtlwifi/rtl8192se/sw.c index aadba29..3c4238e 100644 --- a/drivers/net/wireless/rtlwifi/rtl8192se/sw.c +++ b/drivers/net/wireless/rtlwifi/rtl8192se/sw.c @@ -269,6 +269,7 @@ static struct rtl_hal_ops rtl8192se_hal_ops = { .led_control = rtl92se_led_control, .set_desc = rtl92se_set_desc, .get_desc = rtl92se_get_desc, + .is_tx_desc_closed = rtl92se_is_tx_desc_closed, .tx_polling = rtl92se_tx_polling, .enable_hw_sec = rtl92se_enable_hw_security_config, .set_key = rtl92se_set_key, @@ -278,6 +279,7 @@ static struct rtl_hal_ops rtl8192se_hal_ops = { .get_rfreg = rtl92s_phy_query_rf_reg, .set_rfreg = rtl92s_phy_set_rf_reg, .get_btc_status = rtl_btc_status_false, + .rx_command_packet = rtl92se_rx_command_packet, }; static struct rtl_mod_params rtl92se_mod_params = { @@ -306,6 +308,8 @@ static struct rtl_hal_cfg rtl92se_hal_cfg = { .maps[MAC_RCR_ACRC32] = RCR_ACRC32, .maps[MAC_RCR_ACF] = RCR_ACF, .maps[MAC_RCR_AAP] = RCR_AAP, + .maps[MAC_HIMR] = INTA_MASK, + .maps[MAC_HIMRE] = INTA_MASK + 4, .maps[EFUSE_TEST] = REG_EFUSE_TEST, .maps[EFUSE_CTRL] = REG_EFUSE_CTRL, diff --git a/drivers/net/wireless/rtlwifi/rtl8192se/trx.c b/drivers/net/wireless/rtlwifi/rtl8192se/trx.c index 672fd3b..2014b18 100644 --- a/drivers/net/wireless/rtlwifi/rtl8192se/trx.c +++ b/drivers/net/wireless/rtlwifi/rtl8192se/trx.c @@ -652,8 +652,31 @@ u32 rtl92se_get_desc(u8 *desc, bool istx, u8 desc_name) return ret; } +bool rtl92se_is_tx_desc_closed(struct ieee80211_hw *hw, u8 hw_queue, u16 index) +{ + struct rtl_pci *rtlpci = rtl_pcidev(rtl_pcipriv(hw)); + struct rtl8192_tx_ring *ring = &rtlpci->tx_ring[hw_queue]; + u8 *entry = (u8 *)(&ring->desc[ring->idx]); + u8 own = (u8)rtl92se_get_desc(entry, true, HW_DESC_OWN); + + /* beacon packet will only use the first + * descriptor iby default, and the own bit may not + * be cleared by the hardware + */ + if (own) + return false; + return true; +} + void rtl92se_tx_polling(struct ieee80211_hw *hw, u8 hw_queue) { struct rtl_priv *rtlpriv = rtl_priv(hw); rtl_write_word(rtlpriv, TP_POLL, BIT(0) << (hw_queue)); } + +u32 rtl92se_rx_command_packet(struct ieee80211_hw *hw, + struct rtl_stats status, + struct sk_buff *skb) +{ + return 0; +} diff --git a/drivers/net/wireless/rtlwifi/rtl8192se/trx.h b/drivers/net/wireless/rtlwifi/rtl8192se/trx.h index 5a13f17..bd9f4bf 100644 --- a/drivers/net/wireless/rtlwifi/rtl8192se/trx.h +++ b/drivers/net/wireless/rtlwifi/rtl8192se/trx.h @@ -43,6 +43,10 @@ bool rtl92se_rx_query_desc(struct ieee80211_hw *hw, struct rtl_stats *stats, void rtl92se_set_desc(struct ieee80211_hw *hw, u8 *pdesc, bool istx, u8 desc_name, u8 *val); u32 rtl92se_get_desc(u8 *pdesc, bool istx, u8 desc_name); +bool rtl92se_is_tx_desc_closed(struct ieee80211_hw *hw, u8 hw_queue, u16 index); void rtl92se_tx_polling(struct ieee80211_hw *hw, u8 hw_queue); +u32 rtl92se_rx_command_packet(struct ieee80211_hw *hw, + struct rtl_stats status, + struct sk_buff *skb); #endif -- 2.1.2 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH 3/3 3.18] rtlwifi: rtl8192se: Fix connection problems @ 2014-11-06 20:45 ` John W. Linville 0 siblings, 0 replies; 9+ messages in thread From: John W. Linville @ 2014-11-06 20:45 UTC (permalink / raw) To: Larry Finger; +Cc: linux-wireless, netdev On Wed, Nov 05, 2014 at 07:10:54PM -0600, Larry Finger wrote: > Changes in the vendor driver were added to rtlwifi, but some updates > to rtl8192se were missed. > > Signed-off-by: Larry Finger <Larry.Finger@lwfinger.net> > --- > drivers/net/wireless/rtlwifi/rtl8192se/hw.c | 129 +++++++++++++-------------- > drivers/net/wireless/rtlwifi/rtl8192se/phy.c | 8 +- > drivers/net/wireless/rtlwifi/rtl8192se/sw.c | 4 + > drivers/net/wireless/rtlwifi/rtl8192se/trx.c | 23 +++++ > drivers/net/wireless/rtlwifi/rtl8192se/trx.h | 4 + > 5 files changed, 100 insertions(+), 68 deletions(-) This looks a bit big for a fix. Could this be broken-up a bit more? Perhaps you could enhance the changelog a bit more? > diff --git a/drivers/net/wireless/rtlwifi/rtl8192se/hw.c b/drivers/net/wireless/rtlwifi/rtl8192se/hw.c > index 00e0670..4626203 100644 > --- a/drivers/net/wireless/rtlwifi/rtl8192se/hw.c > +++ b/drivers/net/wireless/rtlwifi/rtl8192se/hw.c > @@ -1170,27 +1170,32 @@ static int _rtl92se_set_media_status(struct ieee80211_hw *hw, > { > struct rtl_priv *rtlpriv = rtl_priv(hw); > u8 bt_msr = rtl_read_byte(rtlpriv, MSR); > + enum led_ctl_mode ledaction = LED_CTL_NO_LINK; > u32 temp; > + u8 mode = MSR_NOLINK; > + > bt_msr &= ~MSR_LINK_MASK; > > switch (type) { > case NL80211_IFTYPE_UNSPECIFIED: > - bt_msr |= (MSR_LINK_NONE << MSR_LINK_SHIFT); > + mode = MSR_NOLINK; > RT_TRACE(rtlpriv, COMP_INIT, DBG_TRACE, > "Set Network type to NO LINK!\n"); > break; > case NL80211_IFTYPE_ADHOC: > - bt_msr |= (MSR_LINK_ADHOC << MSR_LINK_SHIFT); > + mode = MSR_ADHOC; > RT_TRACE(rtlpriv, COMP_INIT, DBG_TRACE, > "Set Network type to Ad Hoc!\n"); > break; > case NL80211_IFTYPE_STATION: > - bt_msr |= (MSR_LINK_MANAGED << MSR_LINK_SHIFT); > + mode = MSR_INFRA; > + ledaction = LED_CTL_LINK; > RT_TRACE(rtlpriv, COMP_INIT, DBG_TRACE, > "Set Network type to STA!\n"); > break; > case NL80211_IFTYPE_AP: > - bt_msr |= (MSR_LINK_MASTER << MSR_LINK_SHIFT); > + mode = MSR_AP; > + ledaction = LED_CTL_LINK; > RT_TRACE(rtlpriv, COMP_INIT, DBG_TRACE, > "Set Network type to AP!\n"); > break; > @@ -1201,7 +1206,17 @@ static int _rtl92se_set_media_status(struct ieee80211_hw *hw, > > } > > - rtl_write_byte(rtlpriv, (MSR), bt_msr); > + /* MSR_INFRA == Link in infrastructure network; > + * MSR_ADHOC == Link in ad hoc network; > + * Therefore, check link state is necessary. > + * > + * MSR_AP == AP mode; link state is not cared here. > + */ > + if (mode != MSR_AP && rtlpriv->mac80211.link_state < MAC80211_LINKED) { > + mode = MSR_NOLINK; > + ledaction = LED_CTL_NO_LINK; > +} > + rtl_write_byte(rtlpriv, (MSR), bt_msr | mode); > > temp = rtl_read_dword(rtlpriv, TCR); > rtl_write_dword(rtlpriv, TCR, temp & (~BIT(8))); > @@ -1262,6 +1277,7 @@ void rtl92se_enable_interrupt(struct ieee80211_hw *hw) > rtl_write_dword(rtlpriv, INTA_MASK, rtlpci->irq_mask[0]); > /* Support Bit 32-37(Assign as Bit 0-5) interrupt setting now */ > rtl_write_dword(rtlpriv, INTA_MASK + 4, rtlpci->irq_mask[1] & 0x3F); > + rtlpci->irq_enabled = true; > } > > void rtl92se_disable_interrupt(struct ieee80211_hw *hw) > @@ -1276,8 +1292,7 @@ void rtl92se_disable_interrupt(struct ieee80211_hw *hw) > rtlpci = rtl_pcidev(rtl_pcipriv(hw)); > rtl_write_dword(rtlpriv, INTA_MASK, 0); > rtl_write_dword(rtlpriv, INTA_MASK + 4, 0); > - > - synchronize_irq(rtlpci->pdev->irq); > + rtlpci->irq_enabled = false; > } > > static u8 _rtl92s_set_sysclk(struct ieee80211_hw *hw, u8 data) > @@ -2035,9 +2050,9 @@ static void rtl92se_update_hal_rate_table(struct ieee80211_hw *hw, > u32 ratr_value; > u8 ratr_index = 0; > u8 nmode = mac->ht_enable; > - u8 mimo_ps = IEEE80211_SMPS_OFF; > u16 shortgi_rate = 0; > u32 tmp_ratr_value = 0; > + u32 ratr_mask; > u8 curtxbw_40mhz = mac->bw_40; > u8 curshortgi_40mhz = (sta->ht_cap.cap & IEEE80211_HT_CAP_SGI_40) ? > 1 : 0; > @@ -2063,26 +2078,21 @@ static void rtl92se_update_hal_rate_table(struct ieee80211_hw *hw, > case WIRELESS_MODE_N_24G: > case WIRELESS_MODE_N_5G: > nmode = 1; > - if (mimo_ps == IEEE80211_SMPS_STATIC) { > - ratr_value &= 0x0007F005; > - } else { > - u32 ratr_mask; > > - if (get_rf_type(rtlphy) == RF_1T2R || > - get_rf_type(rtlphy) == RF_1T1R) { > - if (curtxbw_40mhz) > - ratr_mask = 0x000ff015; > - else > - ratr_mask = 0x000ff005; > - } else { > - if (curtxbw_40mhz) > - ratr_mask = 0x0f0ff015; > - else > - ratr_mask = 0x0f0ff005; > - } > - > - ratr_value &= ratr_mask; > + if (get_rf_type(rtlphy) == RF_1T2R || > + get_rf_type(rtlphy) == RF_1T1R) { > + if (curtxbw_40mhz) > + ratr_mask = 0x000ff015; > + else > + ratr_mask = 0x000ff005; > + } else { > + if (curtxbw_40mhz) > + ratr_mask = 0x0f0ff015; > + else > + ratr_mask = 0x0f0ff005; > } > + > + ratr_value &= ratr_mask; > break; > default: > if (rtlphy->rf_type == RF_1T2R) > @@ -2137,7 +2147,8 @@ static void rtl92se_update_hal_rate_mask(struct ieee80211_hw *hw, > struct rtl_sta_info *sta_entry = NULL; > u32 ratr_bitmap; > u8 ratr_index = 0; > - u8 curtxbw_40mhz = (sta->bandwidth >= IEEE80211_STA_RX_BW_40) ? 1 : 0; > + u8 curtxbw_40mhz = (sta->ht_cap.cap & IEEE80211_HT_CAP_SUP_WIDTH_20_40) > + ? 1 : 0; > u8 curshortgi_40mhz = (sta->ht_cap.cap & IEEE80211_HT_CAP_SGI_40) ? > 1 : 0; > u8 curshortgi_20mhz = (sta->ht_cap.cap & IEEE80211_HT_CAP_SGI_20) ? > @@ -2148,9 +2159,7 @@ static void rtl92se_update_hal_rate_mask(struct ieee80211_hw *hw, > u8 shortgi_rate = 0; > u32 mask = 0; > u32 band = 0; > - bool bmulticast = false; > u8 macid = 0; > - u8 mimo_ps = IEEE80211_SMPS_OFF; > > sta_entry = (struct rtl_sta_info *) sta->drv_priv; > wirelessmode = sta_entry->wireless_mode; > @@ -2198,41 +2207,32 @@ static void rtl92se_update_hal_rate_mask(struct ieee80211_hw *hw, > band |= (WIRELESS_11N | WIRELESS_11G | WIRELESS_11B); > ratr_index = RATR_INX_WIRELESS_NGB; > > - if (mimo_ps == IEEE80211_SMPS_STATIC) { > - if (rssi_level == 1) > - ratr_bitmap &= 0x00070000; > - else if (rssi_level == 2) > - ratr_bitmap &= 0x0007f000; > - else > - ratr_bitmap &= 0x0007f005; > + if (rtlphy->rf_type == RF_1T2R || > + rtlphy->rf_type == RF_1T1R) { > + if (rssi_level == 1) { > + ratr_bitmap &= 0x000f0000; > + } else if (rssi_level == 3) { > + ratr_bitmap &= 0x000fc000; > + } else if (rssi_level == 5) { > + ratr_bitmap &= 0x000ff000; > + } else { > + if (curtxbw_40mhz) > + ratr_bitmap &= 0x000ff015; > + else > + ratr_bitmap &= 0x000ff005; > + } > } else { > - if (rtlphy->rf_type == RF_1T2R || > - rtlphy->rf_type == RF_1T1R) { > - if (rssi_level == 1) { > - ratr_bitmap &= 0x000f0000; > - } else if (rssi_level == 3) { > - ratr_bitmap &= 0x000fc000; > - } else if (rssi_level == 5) { > - ratr_bitmap &= 0x000ff000; > - } else { > - if (curtxbw_40mhz) > - ratr_bitmap &= 0x000ff015; > - else > - ratr_bitmap &= 0x000ff005; > - } > + if (rssi_level == 1) { > + ratr_bitmap &= 0x0f8f0000; > + } else if (rssi_level == 3) { > + ratr_bitmap &= 0x0f8fc000; > + } else if (rssi_level == 5) { > + ratr_bitmap &= 0x0f8ff000; > } else { > - if (rssi_level == 1) { > - ratr_bitmap &= 0x0f8f0000; > - } else if (rssi_level == 3) { > - ratr_bitmap &= 0x0f8fc000; > - } else if (rssi_level == 5) { > - ratr_bitmap &= 0x0f8ff000; > - } else { > - if (curtxbw_40mhz) > - ratr_bitmap &= 0x0f8ff015; > - else > - ratr_bitmap &= 0x0f8ff005; > - } > + if (curtxbw_40mhz) > + ratr_bitmap &= 0x0f8ff015; > + else > + ratr_bitmap &= 0x0f8ff005; > } > } > > @@ -2275,15 +2275,12 @@ static void rtl92se_update_hal_rate_mask(struct ieee80211_hw *hw, > rtl_write_byte(rtlpriv, SG_RATE, shortgi_rate); > } > > - mask |= (bmulticast ? 1 : 0) << 9 | (macid & 0x1f) << 4 | (band & 0xf); > + mask |= (macid & 0x1f) << 4 | (band & 0xf); > > RT_TRACE(rtlpriv, COMP_RATR, DBG_TRACE, "mask = %x, bitmap = %x\n", > mask, ratr_bitmap); > rtl_write_dword(rtlpriv, 0x2c4, ratr_bitmap); > rtl_write_dword(rtlpriv, WFM5, (FW_RA_UPDATE_MASK | (mask << 8))); > - > - if (macid != 0) > - sta_entry->ratr_index = ratr_index; > } > > void rtl92se_update_hal_rate_tbl(struct ieee80211_hw *hw, > diff --git a/drivers/net/wireless/rtlwifi/rtl8192se/phy.c b/drivers/net/wireless/rtlwifi/rtl8192se/phy.c > index 77c5b5f..e382cef 100644 > --- a/drivers/net/wireless/rtlwifi/rtl8192se/phy.c > +++ b/drivers/net/wireless/rtlwifi/rtl8192se/phy.c > @@ -399,6 +399,11 @@ static bool _rtl92s_phy_sw_chnl_step_by_step(struct ieee80211_hw *hw, > case 2: > currentcmd = &postcommoncmd[*step]; > break; > + default: > + RT_TRACE(rtlpriv, COMP_ERR, DBG_LOUD, > + "Invalid 'stage' = %d, Check it!\n", > + *stage); > + return true; > } > > if (currentcmd->cmdid == CMDID_END) { > @@ -602,7 +607,7 @@ bool rtl92s_phy_set_rf_power_state(struct ieee80211_hw *hw, > } > case ERFSLEEP: > if (ppsc->rfpwr_state == ERFOFF) > - return false; > + break; > > for (queue_id = 0, i = 0; > queue_id < RTL_PCI_MAX_TX_QUEUE_COUNT;) { > @@ -1064,7 +1069,6 @@ bool rtl92s_phy_bb_config(struct ieee80211_hw *hw) > /* Check BB/RF confiuration setting. */ > /* We only need to configure RF which is turned on. */ > path1 = (u8)(rtl92s_phy_query_bb_reg(hw, RFPGA0_TXINFO, 0xf)); > - mdelay(10); > path2 = (u8)(rtl92s_phy_query_bb_reg(hw, ROFDM0_TRXPATHENABLE, 0xf)); > pathmap = path1 | path2; > > diff --git a/drivers/net/wireless/rtlwifi/rtl8192se/sw.c b/drivers/net/wireless/rtlwifi/rtl8192se/sw.c > index aadba29..3c4238e 100644 > --- a/drivers/net/wireless/rtlwifi/rtl8192se/sw.c > +++ b/drivers/net/wireless/rtlwifi/rtl8192se/sw.c > @@ -269,6 +269,7 @@ static struct rtl_hal_ops rtl8192se_hal_ops = { > .led_control = rtl92se_led_control, > .set_desc = rtl92se_set_desc, > .get_desc = rtl92se_get_desc, > + .is_tx_desc_closed = rtl92se_is_tx_desc_closed, > .tx_polling = rtl92se_tx_polling, > .enable_hw_sec = rtl92se_enable_hw_security_config, > .set_key = rtl92se_set_key, > @@ -278,6 +279,7 @@ static struct rtl_hal_ops rtl8192se_hal_ops = { > .get_rfreg = rtl92s_phy_query_rf_reg, > .set_rfreg = rtl92s_phy_set_rf_reg, > .get_btc_status = rtl_btc_status_false, > + .rx_command_packet = rtl92se_rx_command_packet, > }; > > static struct rtl_mod_params rtl92se_mod_params = { > @@ -306,6 +308,8 @@ static struct rtl_hal_cfg rtl92se_hal_cfg = { > .maps[MAC_RCR_ACRC32] = RCR_ACRC32, > .maps[MAC_RCR_ACF] = RCR_ACF, > .maps[MAC_RCR_AAP] = RCR_AAP, > + .maps[MAC_HIMR] = INTA_MASK, > + .maps[MAC_HIMRE] = INTA_MASK + 4, > > .maps[EFUSE_TEST] = REG_EFUSE_TEST, > .maps[EFUSE_CTRL] = REG_EFUSE_CTRL, > diff --git a/drivers/net/wireless/rtlwifi/rtl8192se/trx.c b/drivers/net/wireless/rtlwifi/rtl8192se/trx.c > index 672fd3b..2014b18 100644 > --- a/drivers/net/wireless/rtlwifi/rtl8192se/trx.c > +++ b/drivers/net/wireless/rtlwifi/rtl8192se/trx.c > @@ -652,8 +652,31 @@ u32 rtl92se_get_desc(u8 *desc, bool istx, u8 desc_name) > return ret; > } > > +bool rtl92se_is_tx_desc_closed(struct ieee80211_hw *hw, u8 hw_queue, u16 index) > +{ > + struct rtl_pci *rtlpci = rtl_pcidev(rtl_pcipriv(hw)); > + struct rtl8192_tx_ring *ring = &rtlpci->tx_ring[hw_queue]; > + u8 *entry = (u8 *)(&ring->desc[ring->idx]); > + u8 own = (u8)rtl92se_get_desc(entry, true, HW_DESC_OWN); > + > + /* beacon packet will only use the first > + * descriptor iby default, and the own bit may not > + * be cleared by the hardware > + */ > + if (own) > + return false; > + return true; > +} > + > void rtl92se_tx_polling(struct ieee80211_hw *hw, u8 hw_queue) > { > struct rtl_priv *rtlpriv = rtl_priv(hw); > rtl_write_word(rtlpriv, TP_POLL, BIT(0) << (hw_queue)); > } > + > +u32 rtl92se_rx_command_packet(struct ieee80211_hw *hw, > + struct rtl_stats status, > + struct sk_buff *skb) > +{ > + return 0; > +} > diff --git a/drivers/net/wireless/rtlwifi/rtl8192se/trx.h b/drivers/net/wireless/rtlwifi/rtl8192se/trx.h > index 5a13f17..bd9f4bf 100644 > --- a/drivers/net/wireless/rtlwifi/rtl8192se/trx.h > +++ b/drivers/net/wireless/rtlwifi/rtl8192se/trx.h > @@ -43,6 +43,10 @@ bool rtl92se_rx_query_desc(struct ieee80211_hw *hw, struct rtl_stats *stats, > void rtl92se_set_desc(struct ieee80211_hw *hw, u8 *pdesc, bool istx, > u8 desc_name, u8 *val); > u32 rtl92se_get_desc(u8 *pdesc, bool istx, u8 desc_name); > +bool rtl92se_is_tx_desc_closed(struct ieee80211_hw *hw, u8 hw_queue, u16 index); > void rtl92se_tx_polling(struct ieee80211_hw *hw, u8 hw_queue); > +u32 rtl92se_rx_command_packet(struct ieee80211_hw *hw, > + struct rtl_stats status, > + struct sk_buff *skb); > > #endif > -- > 2.1.2 > > -- John W. Linville Someday the world will need a hero, and you linville@tuxdriver.com might be all we have. Be ready. ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 3/3 3.18] rtlwifi: rtl8192se: Fix connection problems @ 2014-11-06 20:45 ` John W. Linville 0 siblings, 0 replies; 9+ messages in thread From: John W. Linville @ 2014-11-06 20:45 UTC (permalink / raw) To: Larry Finger Cc: linux-wireless-u79uwXL29TY76Z2rM5mHXA, netdev-u79uwXL29TY76Z2rM5mHXA On Wed, Nov 05, 2014 at 07:10:54PM -0600, Larry Finger wrote: > Changes in the vendor driver were added to rtlwifi, but some updates > to rtl8192se were missed. > > Signed-off-by: Larry Finger <Larry.Finger-tQ5ms3gMjBLk1uMJSBkQmQ@public.gmane.org> > --- > drivers/net/wireless/rtlwifi/rtl8192se/hw.c | 129 +++++++++++++-------------- > drivers/net/wireless/rtlwifi/rtl8192se/phy.c | 8 +- > drivers/net/wireless/rtlwifi/rtl8192se/sw.c | 4 + > drivers/net/wireless/rtlwifi/rtl8192se/trx.c | 23 +++++ > drivers/net/wireless/rtlwifi/rtl8192se/trx.h | 4 + > 5 files changed, 100 insertions(+), 68 deletions(-) This looks a bit big for a fix. Could this be broken-up a bit more? Perhaps you could enhance the changelog a bit more? > diff --git a/drivers/net/wireless/rtlwifi/rtl8192se/hw.c b/drivers/net/wireless/rtlwifi/rtl8192se/hw.c > index 00e0670..4626203 100644 > --- a/drivers/net/wireless/rtlwifi/rtl8192se/hw.c > +++ b/drivers/net/wireless/rtlwifi/rtl8192se/hw.c > @@ -1170,27 +1170,32 @@ static int _rtl92se_set_media_status(struct ieee80211_hw *hw, > { > struct rtl_priv *rtlpriv = rtl_priv(hw); > u8 bt_msr = rtl_read_byte(rtlpriv, MSR); > + enum led_ctl_mode ledaction = LED_CTL_NO_LINK; > u32 temp; > + u8 mode = MSR_NOLINK; > + > bt_msr &= ~MSR_LINK_MASK; > > switch (type) { > case NL80211_IFTYPE_UNSPECIFIED: > - bt_msr |= (MSR_LINK_NONE << MSR_LINK_SHIFT); > + mode = MSR_NOLINK; > RT_TRACE(rtlpriv, COMP_INIT, DBG_TRACE, > "Set Network type to NO LINK!\n"); > break; > case NL80211_IFTYPE_ADHOC: > - bt_msr |= (MSR_LINK_ADHOC << MSR_LINK_SHIFT); > + mode = MSR_ADHOC; > RT_TRACE(rtlpriv, COMP_INIT, DBG_TRACE, > "Set Network type to Ad Hoc!\n"); > break; > case NL80211_IFTYPE_STATION: > - bt_msr |= (MSR_LINK_MANAGED << MSR_LINK_SHIFT); > + mode = MSR_INFRA; > + ledaction = LED_CTL_LINK; > RT_TRACE(rtlpriv, COMP_INIT, DBG_TRACE, > "Set Network type to STA!\n"); > break; > case NL80211_IFTYPE_AP: > - bt_msr |= (MSR_LINK_MASTER << MSR_LINK_SHIFT); > + mode = MSR_AP; > + ledaction = LED_CTL_LINK; > RT_TRACE(rtlpriv, COMP_INIT, DBG_TRACE, > "Set Network type to AP!\n"); > break; > @@ -1201,7 +1206,17 @@ static int _rtl92se_set_media_status(struct ieee80211_hw *hw, > > } > > - rtl_write_byte(rtlpriv, (MSR), bt_msr); > + /* MSR_INFRA == Link in infrastructure network; > + * MSR_ADHOC == Link in ad hoc network; > + * Therefore, check link state is necessary. > + * > + * MSR_AP == AP mode; link state is not cared here. > + */ > + if (mode != MSR_AP && rtlpriv->mac80211.link_state < MAC80211_LINKED) { > + mode = MSR_NOLINK; > + ledaction = LED_CTL_NO_LINK; > +} > + rtl_write_byte(rtlpriv, (MSR), bt_msr | mode); > > temp = rtl_read_dword(rtlpriv, TCR); > rtl_write_dword(rtlpriv, TCR, temp & (~BIT(8))); > @@ -1262,6 +1277,7 @@ void rtl92se_enable_interrupt(struct ieee80211_hw *hw) > rtl_write_dword(rtlpriv, INTA_MASK, rtlpci->irq_mask[0]); > /* Support Bit 32-37(Assign as Bit 0-5) interrupt setting now */ > rtl_write_dword(rtlpriv, INTA_MASK + 4, rtlpci->irq_mask[1] & 0x3F); > + rtlpci->irq_enabled = true; > } > > void rtl92se_disable_interrupt(struct ieee80211_hw *hw) > @@ -1276,8 +1292,7 @@ void rtl92se_disable_interrupt(struct ieee80211_hw *hw) > rtlpci = rtl_pcidev(rtl_pcipriv(hw)); > rtl_write_dword(rtlpriv, INTA_MASK, 0); > rtl_write_dword(rtlpriv, INTA_MASK + 4, 0); > - > - synchronize_irq(rtlpci->pdev->irq); > + rtlpci->irq_enabled = false; > } > > static u8 _rtl92s_set_sysclk(struct ieee80211_hw *hw, u8 data) > @@ -2035,9 +2050,9 @@ static void rtl92se_update_hal_rate_table(struct ieee80211_hw *hw, > u32 ratr_value; > u8 ratr_index = 0; > u8 nmode = mac->ht_enable; > - u8 mimo_ps = IEEE80211_SMPS_OFF; > u16 shortgi_rate = 0; > u32 tmp_ratr_value = 0; > + u32 ratr_mask; > u8 curtxbw_40mhz = mac->bw_40; > u8 curshortgi_40mhz = (sta->ht_cap.cap & IEEE80211_HT_CAP_SGI_40) ? > 1 : 0; > @@ -2063,26 +2078,21 @@ static void rtl92se_update_hal_rate_table(struct ieee80211_hw *hw, > case WIRELESS_MODE_N_24G: > case WIRELESS_MODE_N_5G: > nmode = 1; > - if (mimo_ps == IEEE80211_SMPS_STATIC) { > - ratr_value &= 0x0007F005; > - } else { > - u32 ratr_mask; > > - if (get_rf_type(rtlphy) == RF_1T2R || > - get_rf_type(rtlphy) == RF_1T1R) { > - if (curtxbw_40mhz) > - ratr_mask = 0x000ff015; > - else > - ratr_mask = 0x000ff005; > - } else { > - if (curtxbw_40mhz) > - ratr_mask = 0x0f0ff015; > - else > - ratr_mask = 0x0f0ff005; > - } > - > - ratr_value &= ratr_mask; > + if (get_rf_type(rtlphy) == RF_1T2R || > + get_rf_type(rtlphy) == RF_1T1R) { > + if (curtxbw_40mhz) > + ratr_mask = 0x000ff015; > + else > + ratr_mask = 0x000ff005; > + } else { > + if (curtxbw_40mhz) > + ratr_mask = 0x0f0ff015; > + else > + ratr_mask = 0x0f0ff005; > } > + > + ratr_value &= ratr_mask; > break; > default: > if (rtlphy->rf_type == RF_1T2R) > @@ -2137,7 +2147,8 @@ static void rtl92se_update_hal_rate_mask(struct ieee80211_hw *hw, > struct rtl_sta_info *sta_entry = NULL; > u32 ratr_bitmap; > u8 ratr_index = 0; > - u8 curtxbw_40mhz = (sta->bandwidth >= IEEE80211_STA_RX_BW_40) ? 1 : 0; > + u8 curtxbw_40mhz = (sta->ht_cap.cap & IEEE80211_HT_CAP_SUP_WIDTH_20_40) > + ? 1 : 0; > u8 curshortgi_40mhz = (sta->ht_cap.cap & IEEE80211_HT_CAP_SGI_40) ? > 1 : 0; > u8 curshortgi_20mhz = (sta->ht_cap.cap & IEEE80211_HT_CAP_SGI_20) ? > @@ -2148,9 +2159,7 @@ static void rtl92se_update_hal_rate_mask(struct ieee80211_hw *hw, > u8 shortgi_rate = 0; > u32 mask = 0; > u32 band = 0; > - bool bmulticast = false; > u8 macid = 0; > - u8 mimo_ps = IEEE80211_SMPS_OFF; > > sta_entry = (struct rtl_sta_info *) sta->drv_priv; > wirelessmode = sta_entry->wireless_mode; > @@ -2198,41 +2207,32 @@ static void rtl92se_update_hal_rate_mask(struct ieee80211_hw *hw, > band |= (WIRELESS_11N | WIRELESS_11G | WIRELESS_11B); > ratr_index = RATR_INX_WIRELESS_NGB; > > - if (mimo_ps == IEEE80211_SMPS_STATIC) { > - if (rssi_level == 1) > - ratr_bitmap &= 0x00070000; > - else if (rssi_level == 2) > - ratr_bitmap &= 0x0007f000; > - else > - ratr_bitmap &= 0x0007f005; > + if (rtlphy->rf_type == RF_1T2R || > + rtlphy->rf_type == RF_1T1R) { > + if (rssi_level == 1) { > + ratr_bitmap &= 0x000f0000; > + } else if (rssi_level == 3) { > + ratr_bitmap &= 0x000fc000; > + } else if (rssi_level == 5) { > + ratr_bitmap &= 0x000ff000; > + } else { > + if (curtxbw_40mhz) > + ratr_bitmap &= 0x000ff015; > + else > + ratr_bitmap &= 0x000ff005; > + } > } else { > - if (rtlphy->rf_type == RF_1T2R || > - rtlphy->rf_type == RF_1T1R) { > - if (rssi_level == 1) { > - ratr_bitmap &= 0x000f0000; > - } else if (rssi_level == 3) { > - ratr_bitmap &= 0x000fc000; > - } else if (rssi_level == 5) { > - ratr_bitmap &= 0x000ff000; > - } else { > - if (curtxbw_40mhz) > - ratr_bitmap &= 0x000ff015; > - else > - ratr_bitmap &= 0x000ff005; > - } > + if (rssi_level == 1) { > + ratr_bitmap &= 0x0f8f0000; > + } else if (rssi_level == 3) { > + ratr_bitmap &= 0x0f8fc000; > + } else if (rssi_level == 5) { > + ratr_bitmap &= 0x0f8ff000; > } else { > - if (rssi_level == 1) { > - ratr_bitmap &= 0x0f8f0000; > - } else if (rssi_level == 3) { > - ratr_bitmap &= 0x0f8fc000; > - } else if (rssi_level == 5) { > - ratr_bitmap &= 0x0f8ff000; > - } else { > - if (curtxbw_40mhz) > - ratr_bitmap &= 0x0f8ff015; > - else > - ratr_bitmap &= 0x0f8ff005; > - } > + if (curtxbw_40mhz) > + ratr_bitmap &= 0x0f8ff015; > + else > + ratr_bitmap &= 0x0f8ff005; > } > } > > @@ -2275,15 +2275,12 @@ static void rtl92se_update_hal_rate_mask(struct ieee80211_hw *hw, > rtl_write_byte(rtlpriv, SG_RATE, shortgi_rate); > } > > - mask |= (bmulticast ? 1 : 0) << 9 | (macid & 0x1f) << 4 | (band & 0xf); > + mask |= (macid & 0x1f) << 4 | (band & 0xf); > > RT_TRACE(rtlpriv, COMP_RATR, DBG_TRACE, "mask = %x, bitmap = %x\n", > mask, ratr_bitmap); > rtl_write_dword(rtlpriv, 0x2c4, ratr_bitmap); > rtl_write_dword(rtlpriv, WFM5, (FW_RA_UPDATE_MASK | (mask << 8))); > - > - if (macid != 0) > - sta_entry->ratr_index = ratr_index; > } > > void rtl92se_update_hal_rate_tbl(struct ieee80211_hw *hw, > diff --git a/drivers/net/wireless/rtlwifi/rtl8192se/phy.c b/drivers/net/wireless/rtlwifi/rtl8192se/phy.c > index 77c5b5f..e382cef 100644 > --- a/drivers/net/wireless/rtlwifi/rtl8192se/phy.c > +++ b/drivers/net/wireless/rtlwifi/rtl8192se/phy.c > @@ -399,6 +399,11 @@ static bool _rtl92s_phy_sw_chnl_step_by_step(struct ieee80211_hw *hw, > case 2: > currentcmd = &postcommoncmd[*step]; > break; > + default: > + RT_TRACE(rtlpriv, COMP_ERR, DBG_LOUD, > + "Invalid 'stage' = %d, Check it!\n", > + *stage); > + return true; > } > > if (currentcmd->cmdid == CMDID_END) { > @@ -602,7 +607,7 @@ bool rtl92s_phy_set_rf_power_state(struct ieee80211_hw *hw, > } > case ERFSLEEP: > if (ppsc->rfpwr_state == ERFOFF) > - return false; > + break; > > for (queue_id = 0, i = 0; > queue_id < RTL_PCI_MAX_TX_QUEUE_COUNT;) { > @@ -1064,7 +1069,6 @@ bool rtl92s_phy_bb_config(struct ieee80211_hw *hw) > /* Check BB/RF confiuration setting. */ > /* We only need to configure RF which is turned on. */ > path1 = (u8)(rtl92s_phy_query_bb_reg(hw, RFPGA0_TXINFO, 0xf)); > - mdelay(10); > path2 = (u8)(rtl92s_phy_query_bb_reg(hw, ROFDM0_TRXPATHENABLE, 0xf)); > pathmap = path1 | path2; > > diff --git a/drivers/net/wireless/rtlwifi/rtl8192se/sw.c b/drivers/net/wireless/rtlwifi/rtl8192se/sw.c > index aadba29..3c4238e 100644 > --- a/drivers/net/wireless/rtlwifi/rtl8192se/sw.c > +++ b/drivers/net/wireless/rtlwifi/rtl8192se/sw.c > @@ -269,6 +269,7 @@ static struct rtl_hal_ops rtl8192se_hal_ops = { > .led_control = rtl92se_led_control, > .set_desc = rtl92se_set_desc, > .get_desc = rtl92se_get_desc, > + .is_tx_desc_closed = rtl92se_is_tx_desc_closed, > .tx_polling = rtl92se_tx_polling, > .enable_hw_sec = rtl92se_enable_hw_security_config, > .set_key = rtl92se_set_key, > @@ -278,6 +279,7 @@ static struct rtl_hal_ops rtl8192se_hal_ops = { > .get_rfreg = rtl92s_phy_query_rf_reg, > .set_rfreg = rtl92s_phy_set_rf_reg, > .get_btc_status = rtl_btc_status_false, > + .rx_command_packet = rtl92se_rx_command_packet, > }; > > static struct rtl_mod_params rtl92se_mod_params = { > @@ -306,6 +308,8 @@ static struct rtl_hal_cfg rtl92se_hal_cfg = { > .maps[MAC_RCR_ACRC32] = RCR_ACRC32, > .maps[MAC_RCR_ACF] = RCR_ACF, > .maps[MAC_RCR_AAP] = RCR_AAP, > + .maps[MAC_HIMR] = INTA_MASK, > + .maps[MAC_HIMRE] = INTA_MASK + 4, > > .maps[EFUSE_TEST] = REG_EFUSE_TEST, > .maps[EFUSE_CTRL] = REG_EFUSE_CTRL, > diff --git a/drivers/net/wireless/rtlwifi/rtl8192se/trx.c b/drivers/net/wireless/rtlwifi/rtl8192se/trx.c > index 672fd3b..2014b18 100644 > --- a/drivers/net/wireless/rtlwifi/rtl8192se/trx.c > +++ b/drivers/net/wireless/rtlwifi/rtl8192se/trx.c > @@ -652,8 +652,31 @@ u32 rtl92se_get_desc(u8 *desc, bool istx, u8 desc_name) > return ret; > } > > +bool rtl92se_is_tx_desc_closed(struct ieee80211_hw *hw, u8 hw_queue, u16 index) > +{ > + struct rtl_pci *rtlpci = rtl_pcidev(rtl_pcipriv(hw)); > + struct rtl8192_tx_ring *ring = &rtlpci->tx_ring[hw_queue]; > + u8 *entry = (u8 *)(&ring->desc[ring->idx]); > + u8 own = (u8)rtl92se_get_desc(entry, true, HW_DESC_OWN); > + > + /* beacon packet will only use the first > + * descriptor iby default, and the own bit may not > + * be cleared by the hardware > + */ > + if (own) > + return false; > + return true; > +} > + > void rtl92se_tx_polling(struct ieee80211_hw *hw, u8 hw_queue) > { > struct rtl_priv *rtlpriv = rtl_priv(hw); > rtl_write_word(rtlpriv, TP_POLL, BIT(0) << (hw_queue)); > } > + > +u32 rtl92se_rx_command_packet(struct ieee80211_hw *hw, > + struct rtl_stats status, > + struct sk_buff *skb) > +{ > + return 0; > +} > diff --git a/drivers/net/wireless/rtlwifi/rtl8192se/trx.h b/drivers/net/wireless/rtlwifi/rtl8192se/trx.h > index 5a13f17..bd9f4bf 100644 > --- a/drivers/net/wireless/rtlwifi/rtl8192se/trx.h > +++ b/drivers/net/wireless/rtlwifi/rtl8192se/trx.h > @@ -43,6 +43,10 @@ bool rtl92se_rx_query_desc(struct ieee80211_hw *hw, struct rtl_stats *stats, > void rtl92se_set_desc(struct ieee80211_hw *hw, u8 *pdesc, bool istx, > u8 desc_name, u8 *val); > u32 rtl92se_get_desc(u8 *pdesc, bool istx, u8 desc_name); > +bool rtl92se_is_tx_desc_closed(struct ieee80211_hw *hw, u8 hw_queue, u16 index); > void rtl92se_tx_polling(struct ieee80211_hw *hw, u8 hw_queue); > +u32 rtl92se_rx_command_packet(struct ieee80211_hw *hw, > + struct rtl_stats status, > + struct sk_buff *skb); > > #endif > -- > 2.1.2 > > -- John W. Linville Someday the world will need a hero, and you linville-2XuSBdqkA4R54TAoqtyWWQ@public.gmane.org might be all we have. Be ready. -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 3/3 3.18] rtlwifi: rtl8192se: Fix connection problems 2014-11-06 20:45 ` John W. Linville (?) @ 2014-11-06 22:46 ` Larry Finger 2014-11-07 15:19 ` John W. Linville -1 siblings, 1 reply; 9+ messages in thread From: Larry Finger @ 2014-11-06 22:46 UTC (permalink / raw) To: John W. Linville; +Cc: linux-wireless, netdev On 11/06/2014 02:45 PM, John W. Linville wrote: > On Wed, Nov 05, 2014 at 07:10:54PM -0600, Larry Finger wrote: >> Changes in the vendor driver were added to rtlwifi, but some updates >> to rtl8192se were missed. >> >> Signed-off-by: Larry Finger <Larry.Finger@lwfinger.net> >> --- >> drivers/net/wireless/rtlwifi/rtl8192se/hw.c | 129 +++++++++++++-------------- >> drivers/net/wireless/rtlwifi/rtl8192se/phy.c | 8 +- >> drivers/net/wireless/rtlwifi/rtl8192se/sw.c | 4 + >> drivers/net/wireless/rtlwifi/rtl8192se/trx.c | 23 +++++ >> drivers/net/wireless/rtlwifi/rtl8192se/trx.h | 4 + >> 5 files changed, 100 insertions(+), 68 deletions(-) > > This looks a bit big for a fix. Could this be broken-up a bit more? > Perhaps you could enhance the changelog a bit more? I had used a scatter-gun approach to finding the missing parts, and I did not take the time to see which changes were crucial, and which were not. Now that I knew what it took, I was able to eliminate a lot of the patch that can be deferred for 3.19. The commit message and the changelog for the new version are Changes in the vendor driver were added to rtlwifi, but some updates to rtl8192se were missed, and the driver could neither scan nor connect. There are other changes that will enhance performance, but this minimal set fixes the basic functionality. drivers/net/wireless/rtlwifi/pci.c | 3 ++- drivers/net/wireless/rtlwifi/rtl8192se/hw.c | 7 +++++-- drivers/net/wireless/rtlwifi/rtl8192se/phy.c | 2 ++ drivers/net/wireless/rtlwifi/rtl8192se/sw.c | 16 ++++++++++++++++ 4 files changed, 25 insertions(+), 3 deletions(-) Will that be OK? I need to do more testing, but V2 of the patch should be ready for submission by tomorrow. Thanks, Larry ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 3/3 3.18] rtlwifi: rtl8192se: Fix connection problems @ 2014-11-07 15:19 ` John W. Linville 0 siblings, 0 replies; 9+ messages in thread From: John W. Linville @ 2014-11-07 15:19 UTC (permalink / raw) To: Larry Finger; +Cc: linux-wireless, netdev On Thu, Nov 06, 2014 at 04:46:42PM -0600, Larry Finger wrote: > On 11/06/2014 02:45 PM, John W. Linville wrote: > >On Wed, Nov 05, 2014 at 07:10:54PM -0600, Larry Finger wrote: > >>Changes in the vendor driver were added to rtlwifi, but some updates > >>to rtl8192se were missed. > >> > >>Signed-off-by: Larry Finger <Larry.Finger@lwfinger.net> > >>--- > >> drivers/net/wireless/rtlwifi/rtl8192se/hw.c | 129 +++++++++++++-------------- > >> drivers/net/wireless/rtlwifi/rtl8192se/phy.c | 8 +- > >> drivers/net/wireless/rtlwifi/rtl8192se/sw.c | 4 + > >> drivers/net/wireless/rtlwifi/rtl8192se/trx.c | 23 +++++ > >> drivers/net/wireless/rtlwifi/rtl8192se/trx.h | 4 + > >> 5 files changed, 100 insertions(+), 68 deletions(-) > > > >This looks a bit big for a fix. Could this be broken-up a bit more? > >Perhaps you could enhance the changelog a bit more? > > I had used a scatter-gun approach to finding the missing parts, and I did > not take the time to see which changes were crucial, and which were not. > > Now that I knew what it took, I was able to eliminate a lot of the patch > that can be deferred for 3.19. > > The commit message and the changelog for the new version are > > Changes in the vendor driver were added to rtlwifi, but some updates > to rtl8192se were missed, and the driver could neither scan nor connect. > There are other changes that will enhance performance, but this minimal > set fixes the basic functionality. > > drivers/net/wireless/rtlwifi/pci.c | 3 ++- > drivers/net/wireless/rtlwifi/rtl8192se/hw.c | 7 +++++-- > drivers/net/wireless/rtlwifi/rtl8192se/phy.c | 2 ++ > drivers/net/wireless/rtlwifi/rtl8192se/sw.c | 16 ++++++++++++++++ > 4 files changed, 25 insertions(+), 3 deletions(-) > > Will that be OK? I need to do more testing, but V2 of the patch should be > ready for submission by tomorrow. That sounds better, at least. :-) Thanks, Larry! John -- John W. Linville Someday the world will need a hero, and you linville@tuxdriver.com might be all we have. Be ready. ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 3/3 3.18] rtlwifi: rtl8192se: Fix connection problems @ 2014-11-07 15:19 ` John W. Linville 0 siblings, 0 replies; 9+ messages in thread From: John W. Linville @ 2014-11-07 15:19 UTC (permalink / raw) To: Larry Finger Cc: linux-wireless-u79uwXL29TY76Z2rM5mHXA, netdev-u79uwXL29TY76Z2rM5mHXA On Thu, Nov 06, 2014 at 04:46:42PM -0600, Larry Finger wrote: > On 11/06/2014 02:45 PM, John W. Linville wrote: > >On Wed, Nov 05, 2014 at 07:10:54PM -0600, Larry Finger wrote: > >>Changes in the vendor driver were added to rtlwifi, but some updates > >>to rtl8192se were missed. > >> > >>Signed-off-by: Larry Finger <Larry.Finger-tQ5ms3gMjBLk1uMJSBkQmQ@public.gmane.org> > >>--- > >> drivers/net/wireless/rtlwifi/rtl8192se/hw.c | 129 +++++++++++++-------------- > >> drivers/net/wireless/rtlwifi/rtl8192se/phy.c | 8 +- > >> drivers/net/wireless/rtlwifi/rtl8192se/sw.c | 4 + > >> drivers/net/wireless/rtlwifi/rtl8192se/trx.c | 23 +++++ > >> drivers/net/wireless/rtlwifi/rtl8192se/trx.h | 4 + > >> 5 files changed, 100 insertions(+), 68 deletions(-) > > > >This looks a bit big for a fix. Could this be broken-up a bit more? > >Perhaps you could enhance the changelog a bit more? > > I had used a scatter-gun approach to finding the missing parts, and I did > not take the time to see which changes were crucial, and which were not. > > Now that I knew what it took, I was able to eliminate a lot of the patch > that can be deferred for 3.19. > > The commit message and the changelog for the new version are > > Changes in the vendor driver were added to rtlwifi, but some updates > to rtl8192se were missed, and the driver could neither scan nor connect. > There are other changes that will enhance performance, but this minimal > set fixes the basic functionality. > > drivers/net/wireless/rtlwifi/pci.c | 3 ++- > drivers/net/wireless/rtlwifi/rtl8192se/hw.c | 7 +++++-- > drivers/net/wireless/rtlwifi/rtl8192se/phy.c | 2 ++ > drivers/net/wireless/rtlwifi/rtl8192se/sw.c | 16 ++++++++++++++++ > 4 files changed, 25 insertions(+), 3 deletions(-) > > Will that be OK? I need to do more testing, but V2 of the patch should be > ready for submission by tomorrow. That sounds better, at least. :-) Thanks, Larry! John -- John W. Linville Someday the world will need a hero, and you linville-2XuSBdqkA4R54TAoqtyWWQ@public.gmane.org might be all we have. Be ready. -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2014-11-07 15:30 UTC | newest] Thread overview: 9+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2014-11-06 1:10 [PATCH 0/3i 3.18] Fix more problems with rtlwifi Larry Finger 2014-11-06 1:10 ` [PATCH 1/3 3.18] rtlwifi: Fix setting of tx descriptor for new trx flow Larry Finger 2014-11-06 1:10 ` [PATCH 2/3 3.18] rtlwifi: Fix errors in descriptor manipulation Larry Finger 2014-11-06 1:10 ` [PATCH 3/3 3.18] rtlwifi: rtl8192se: Fix connection problems Larry Finger 2014-11-06 20:45 ` John W. Linville 2014-11-06 20:45 ` John W. Linville 2014-11-06 22:46 ` Larry Finger 2014-11-07 15:19 ` John W. Linville 2014-11-07 15:19 ` John W. Linville
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.