> From: Chris Chiu > > On Thu, Sep 12, 2019 at 2:39 PM <yhchuang@xxxxxxxxxxx> wrote: > > > > From: Yan-Hsuan Chuang <yhchuang@xxxxxxxxxxx> > > > > Report chains and chain_signal in ieee80211_rx_status. > > It is useful for program such as tcpdump to see if the > > antennas are well connected/placed. > > > > 8822C is able to receive CCK rates with 2 antennas, while > > 8822B can only use 1 antenna path to receive CCK rates. > > > > Signed-off-by: Yan-Hsuan Chuang <yhchuang@xxxxxxxxxxx> > > --- > > drivers/net/wireless/realtek/rtw88/rtw8822b.c | 1 + > > drivers/net/wireless/realtek/rtw88/rtw8822c.c | 6 +++--- > > drivers/net/wireless/realtek/rtw88/rx.c | 5 +++++ > > 3 files changed, 9 insertions(+), 3 deletions(-) > > > > diff --git a/drivers/net/wireless/realtek/rtw88/rtw8822b.c > b/drivers/net/wireless/realtek/rtw88/rtw8822b.c > > index 568033a..0861195 100644 > > --- a/drivers/net/wireless/realtek/rtw88/rtw8822b.c > > +++ b/drivers/net/wireless/realtek/rtw88/rtw8822b.c > > @@ -766,6 +766,7 @@ static void query_phy_status_page0(struct rtw_dev > *rtwdev, u8 *phy_status, > > s8 min_rx_power = -120; > > u8 pwdb = GET_PHY_STAT_P0_PWDB(phy_status); > > > > + /* 8822B uses only 1 antenna to RX CCK rates */ > > pkt_stat->rx_power[RF_PATH_A] = pwdb - 110; > > It only uses PATH_A for receiving CCK rates packets? The comment seems > not clear enough. > Yes, it can only use PATH_A, so we don't have PWDB_[A|B] for 8822B. Yan-Hsuan