On Sat, 2018-03-17 at 22:26 +0800, Kevin Lo wrote: > Correct comment. Set bit 3 and bit 4 of 0x0005 register (REG_APS_FSMCO + 1) > to 0 which means disable WL suspend, not enable WL suspend. > > Signed-off-by: Kevin Lo <kevlo@xxxxxxxxx> Acked-by: Ping-Ke Shih <pkshih@xxxxxxxxxxx> > --- > diff --git a/drivers/net/wireless/realtek/rtlwifi/rtl8188ee/pwrseq.h > b/drivers/net/wireless/realtek/rtlwifi/rtl8188ee/pwrseq.h > index f2d9c6116e5c..8379a3e5198c 100644 > --- a/drivers/net/wireless/realtek/rtlwifi/rtl8188ee/pwrseq.h > +++ b/drivers/net/wireless/realtek/rtlwifi/rtl8188ee/pwrseq.h > @@ -142,7 +142,7 @@ > /*wait power state to suspend*/}, \ > {0x0005, PWR_CUT_ALL_MSK, PWR_FAB_ALL_MSK, PWR_INTF_ALL_MSK, \ > PWR_BASEADDR_MAC, PWR_CMD_WRITE, BIT(3) | BIT(4), 0 \ > - /*0x04[12:11] = 2b'01enable WL suspend*/}, > + /*0x04[12:11] = 2b'00 disable WL suspend*/}, > [...] > diff --git a/drivers/net/wireless/realtek/rtlwifi/rtl8192ee/pwrseq.h > b/drivers/net/wireless/realtek/rtlwifi/rtl8192ee/pwrseq.h > index 781eeaa6af49..c570801508cc 100644 > --- a/drivers/net/wireless/realtek/rtlwifi/rtl8192ee/pwrseq.h > +++ b/drivers/net/wireless/realtek/rtlwifi/rtl8192ee/pwrseq.h > @@ -134,7 +134,7 @@ > /*wait power state to suspend*/ \ > {0x0086, PWR_CUT_ALL_MSK, PWR_FAB_ALL_MSK, PWR_INTF_SDIO_MSK, \ > PWR_BASEADDR_SDIO , PWR_CMD_POLLING, BIT(1), BIT(1)}, \ > - /*0x04[12:11] = 2b'01enable WL suspend*/ \ > + /*0x04[12:11] = 2b'00 disable WL suspend*/ \ > {0x0005, PWR_CUT_ALL_MSK, PWR_FAB_ALL_MSK, PWR_INTF_ALL_MSK, \ > PWR_BASEADDR_MAC , PWR_CMD_WRITE, BIT(3) | BIT(4), 0}, > For further reviewers, please pay attention to comments exist above or below the statements.