On 2020-05-20 13:23:32 [+0800], yhchuang@xxxxxxxxxxx wrote: > From: Tzu-En Huang <tehuang@xxxxxxxxxxx> > > To let 8821c connect to change to different 2.4G and 5G channels, > this commit implements the callback function for set channel. > Set the required chip setting for 8821c, which includes mac, bb and > rf related settings. 8821c is capable of 2.4G and 5G. Implement rtw_chip_ops::set_channel() to set 2G and 5G channels. This includes MAC, BB and RF related settings. > Signed-off-by: Tzu-En Huang <tehuang@xxxxxxxxxxx> > Signed-off-by: Yan-Hsuan Chuang <yhchuang@xxxxxxxxxxx> > diff --git a/drivers/net/wireless/realtek/rtw88/rtw8821c.c b/drivers/net/wireless/realtek/rtw88/rtw8821c.c > index df95a095df73..e3b52caf9ab3 100644 > --- a/drivers/net/wireless/realtek/rtw88/rtw8821c.c > +++ b/drivers/net/wireless/realtek/rtw88/rtw8821c.c > @@ -94,6 +94,9 @@ static void rtw8821c_phy_set_param(struct rtw_dev *rtwdev) > > /* post init after header files config */ > rtw_write32_set(rtwdev, REG_RXPSEL, BIT_RX_PSEL_RST); > + rtwdev->chip->ch_param[0] = rtw_read32_mask(rtwdev, REG_TXSF2, MASKDWORD); > + rtwdev->chip->ch_param[1] = rtw_read32_mask(rtwdev, REG_TXSF6, MASKDWORD); > + rtwdev->chip->ch_param[2] = rtw_read32_mask(rtwdev, 0xaac, MASKDWORD); The first two are the same as in 8822b. Would you mind providing a define for 0xaac? > rtw_phy_init(rtwdev); > } Sebastian