diff mbox

[1/2] Revert "rtlwifi: rtl8188ee: enable MSI interrupts mode"

Message ID 1398308924-15312-1-git-send-email-adam.lee@canonical.com (mailing list archive)
State Not Applicable, archived
Headers show

Commit Message

Adam Lee April 24, 2014, 3:08 a.m. UTC
This reverts commit 2a54eb5e1476426ee639bbfbe179b52342a0d82c.

94010fa0dd07e8b904e7c6b6589f15573008ab15 introduced MSI interrupts mode
support, which seemed safe enough with RTL8188EE and RTL8723BE as
RealTek's testing results, but some users reported their RTL8188EE
modules could not connect to any wireless network after the MSI mode was
enabled by Ubuntu 14.04.

So, let's fallback to pin-based mode until rtlwifi's MSI support get
good compatibility.

BugLink: https://bugs.launchpad.net/bugs/1310512
Signed-off-by: Adam Lee <adam.lee@canonical.com>
---
 drivers/net/wireless/rtlwifi/rtl8188ee/sw.c | 1 -
 1 file changed, 1 deletion(-)

Comments

Larry Finger April 24, 2014, 6:25 a.m. UTC | #1
On 04/23/2014 10:08 PM, Adam Lee wrote:
> This reverts commit 2a54eb5e1476426ee639bbfbe179b52342a0d82c.
>
> 94010fa0dd07e8b904e7c6b6589f15573008ab15 introduced MSI interrupts mode
> support, which seemed safe enough with RTL8188EE and RTL8723BE as
> RealTek's testing results, but some users reported their RTL8188EE
> modules could not connect to any wireless network after the MSI mode was
> enabled by Ubuntu 14.04.
>
> So, let's fallback to pin-based mode until rtlwifi's MSI support get
> good compatibility.
>
> BugLink: https://bugs.launchpad.net/bugs/1310512
> Signed-off-by: Adam Lee <adam.lee@canonical.com>

ACK for both of these patches. I thought I had tested these patches when they 
first came through, but obviously I did not. I do not know which Realtek 
engineers you consulted, but they failed to supply you with the MSI handler. The 
Realsil group that actually does the coding for the PCI-based devices have 
supplied such a routine along with the driver for the RTL8192EE devices. With 
the handler, MSI does work. After I integrate that handler into rtlwifi, I will 
resubmit the patches turning on MSI for rtl8188ee and rtl8723ae.

Larry

> ---
>   drivers/net/wireless/rtlwifi/rtl8188ee/sw.c | 1 -
>   1 file changed, 1 deletion(-)
>
> diff --git a/drivers/net/wireless/rtlwifi/rtl8188ee/sw.c b/drivers/net/wireless/rtlwifi/rtl8188ee/sw.c
> index 1b4101b..347af1e 100644
> --- a/drivers/net/wireless/rtlwifi/rtl8188ee/sw.c
> +++ b/drivers/net/wireless/rtlwifi/rtl8188ee/sw.c
> @@ -93,7 +93,6 @@ int rtl88e_init_sw_vars(struct ieee80211_hw *hw)
>   	u8 tid;
>
>   	rtl8188ee_bt_reg_init(hw);
> -	rtlpci->msi_support = true;
>
>   	rtlpriv->dm.dm_initialgain_enable = 1;
>   	rtlpriv->dm.dm_flag = 0;
>

--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Sergei Shtylyov April 24, 2014, 12:04 p.m. UTC | #2
Hello.

On 24-04-2014 7:08, Adam Lee wrote:

> This reverts commit 2a54eb5e1476426ee639bbfbe179b52342a0d82c.

> 94010fa0dd07e8b904e7c6b6589f15573008ab15 introduced MSI interrupts mode

    Please also specify that commit's summary line in parens.

> support, which seemed safe enough with RTL8188EE and RTL8723BE as
> RealTek's testing results, but some users reported their RTL8188EE
> modules could not connect to any wireless network after the MSI mode was
> enabled by Ubuntu 14.04.

> So, let's fallback to pin-based mode until rtlwifi's MSI support get
> good compatibility.

> BugLink: https://bugs.launchpad.net/bugs/1310512
> Signed-off-by: Adam Lee <adam.lee@canonical.com>

WBR, Sergei


--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/drivers/net/wireless/rtlwifi/rtl8188ee/sw.c b/drivers/net/wireless/rtlwifi/rtl8188ee/sw.c
index 1b4101b..347af1e 100644
--- a/drivers/net/wireless/rtlwifi/rtl8188ee/sw.c
+++ b/drivers/net/wireless/rtlwifi/rtl8188ee/sw.c
@@ -93,7 +93,6 @@  int rtl88e_init_sw_vars(struct ieee80211_hw *hw)
 	u8 tid;
 
 	rtl8188ee_bt_reg_init(hw);
-	rtlpci->msi_support = true;
 
 	rtlpriv->dm.dm_initialgain_enable = 1;
 	rtlpriv->dm.dm_flag = 0;