Bitterblue Smith <rtl8821cerfe2@xxxxxxxxx> wrote: > rtw_mac_power_switch() checks bit 8 of REG_SYS_STATUS1 to see if the > chip is powered on. This bit appears to be always on in the RTL8814AU, > so ignore it. > > Signed-off-by: Bitterblue Smith <rtl8821cerfe2@xxxxxxxxx> > --- > drivers/net/wireless/realtek/rtw88/mac.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/net/wireless/realtek/rtw88/mac.c b/drivers/net/wireless/realtek/rtw88/mac.c > index 63edf6461de8..5de16c75adf2 100644 > --- a/drivers/net/wireless/realtek/rtw88/mac.c > +++ b/drivers/net/wireless/realtek/rtw88/mac.c > @@ -291,7 +291,8 @@ static int rtw_mac_power_switch(struct rtw_dev *rtwdev, bool pwr_on) > if (rtw_read8(rtwdev, REG_CR) == 0xea) > cur_pwr = false; > else if (rtw_hci_type(rtwdev) == RTW_HCI_TYPE_USB && > - (rtw_read8(rtwdev, REG_SYS_STATUS1 + 1) & BIT(0))) > + chip->id != RTW_CHIP_TYPE_8814A && > + (rtw_read8(rtwdev, REG_SYS_STATUS1 + 1) & BIT(0))) The newly added codes are not aligned, and seemingly only need to add the line checking 8814A. > cur_pwr = false; > else > cur_pwr = true; > -- > 2.48.1
