wifi: rtw89: coex: Add Bluetooth version report version 7
authorChing-Te Ku <ku920601@realtek.com>
Tue, 23 Apr 2024 13:04:58 +0000 (21:04 +0800)
committerPing-Ke Shih <pkshih@realtek.com>
Mon, 29 Apr 2024 00:52:43 +0000 (08:52 +0800)
The report is reported from Bluetooth, it shows the current Bluetooth
driver & firmware version code. Wi-Fi & Bluetooth need to use compatible
version. The version 7 report adjust the structure variables order.

Signed-off-by: Ching-Te Ku <ku920601@realtek.com>
Signed-off-by: Ping-Ke Shih <pkshih@realtek.com>
Link: https://msgid.link/20240423130502.32682-5-pkshih@realtek.com
drivers/net/wireless/realtek/rtw89/coex.c
drivers/net/wireless/realtek/rtw89/core.h

index f4ac9354f7ab5fa09ea24703a195f7a37f3259d9..15e04440888dd1129a6815c19567ce3133d35ced 100644 (file)
@@ -1132,29 +1132,36 @@ static void _update_bt_report(struct rtw89_dev *rtwdev, u8 rpt_type, u8 *pfinfo)
        struct rtw89_btc_bt_info *bt = &btc->cx.bt;
        struct rtw89_btc_bt_link_info *bt_linfo = &bt->link_info;
        struct rtw89_btc_bt_a2dp_desc *a2dp = &bt_linfo->a2dp_desc;
-       struct rtw89_btc_fbtc_btver *pver = NULL;
-       struct rtw89_btc_fbtc_btscan_v1 *pscan_v1;
-       struct rtw89_btc_fbtc_btscan_v2 *pscan_v2;
-       struct rtw89_btc_fbtc_btscan_v7 *pscan_v7;
-       struct rtw89_btc_fbtc_btafh *pafh_v1 = NULL;
+       union  rtw89_btc_fbtc_btver *pver = &btc->fwinfo.rpt_fbtc_btver.finfo;
        struct rtw89_btc_fbtc_btafh_v2 *pafh_v2 = NULL;
        struct rtw89_btc_fbtc_btafh_v7 *pafh_v7 = NULL;
        struct rtw89_btc_fbtc_btdevinfo *pdev = NULL;
+       struct rtw89_btc_fbtc_btafh *pafh_v1 = NULL;
+       struct rtw89_btc_fbtc_btscan_v1 *pscan_v1;
+       struct rtw89_btc_fbtc_btscan_v2 *pscan_v2;
+       struct rtw89_btc_fbtc_btscan_v7 *pscan_v7;
        bool scan_update = true;
        int i;
 
-       pver = (struct rtw89_btc_fbtc_btver *)pfinfo;
-       pdev = (struct rtw89_btc_fbtc_btdevinfo *)pfinfo;
-
        rtw89_debug(rtwdev, RTW89_DBG_BTC,
                    "[BTC], %s(): rpt_type:%d\n",
                    __func__, rpt_type);
 
        switch (rpt_type) {
        case BTC_RPT_TYPE_BT_VER:
-               bt->ver_info.fw = le32_to_cpu(pver->fw_ver);
-               bt->ver_info.fw_coex = le32_get_bits(pver->coex_ver, GENMASK(7, 0));
-               bt->feature = le32_to_cpu(pver->feature);
+               if (ver->fcxbtver == 7) {
+                       pver->v7 = *(struct rtw89_btc_fbtc_btver_v7 *)pfinfo;
+                       bt->ver_info.fw = le32_to_cpu(pver->v7.fw_ver);
+                       bt->ver_info.fw_coex = le32_get_bits(pver->v7.coex_ver,
+                                                            GENMASK(7, 0));
+                       bt->feature = le32_to_cpu(pver->v7.feature);
+               } else {
+                       pver->v1 = *(struct rtw89_btc_fbtc_btver_v1 *)pfinfo;
+                       bt->ver_info.fw = le32_to_cpu(pver->v1.fw_ver);
+                       bt->ver_info.fw_coex = le32_get_bits(pver->v1.coex_ver,
+                                                            GENMASK(7, 0));
+                       bt->feature = le32_to_cpu(pver->v1.feature);
+               }
                break;
        case BTC_RPT_TYPE_BT_SCAN:
                if (ver->fcxbtscan == 1) {
@@ -1218,6 +1225,7 @@ static void _update_bt_report(struct rtw89_dev *rtwdev, u8 rpt_type, u8 *pfinfo)
                }
                break;
        case BTC_RPT_TYPE_BT_DEVICE:
+               pdev = (struct rtw89_btc_fbtc_btdevinfo *)pfinfo;
                a2dp->device_name = le32_to_cpu(pdev->dev_name);
                a2dp->vendor_id = le16_to_cpu(pdev->vendor_id);
                a2dp->flush_time = le32_to_cpu(pdev->flush_time);
@@ -1438,8 +1446,13 @@ static u32 _chk_btc_report(struct rtw89_dev *rtwdev,
                break;
        case BTC_RPT_TYPE_BT_VER:
                pcinfo = &pfwinfo->rpt_fbtc_btver.cinfo;
-               pfinfo = &pfwinfo->rpt_fbtc_btver.finfo;
-               pcinfo->req_len = sizeof(pfwinfo->rpt_fbtc_btver.finfo);
+               if (ver->fcxbtver == 1) {
+                       pfinfo = &pfwinfo->rpt_fbtc_btver.finfo.v1;
+                       pcinfo->req_len = sizeof(pfwinfo->rpt_fbtc_btver.finfo.v1);
+               } else if (ver->fcxbtver == 7) {
+                       pfinfo = &pfwinfo->rpt_fbtc_btver.finfo.v7;
+                       pcinfo->req_len = sizeof(pfwinfo->rpt_fbtc_btver.finfo.v7);
+               }
                pcinfo->req_fver = ver->fcxbtver;
                break;
        case BTC_RPT_TYPE_BT_SCAN:
index 78600e7fbad495e3b3cecdab7a36d078ae5199f1..4007377d29ba518d9e8d42b0d798ccca2b78b09f 100644 (file)
@@ -2705,7 +2705,7 @@ union rtw89_btc_fbtc_cynullsta_info {
        struct rtw89_btc_fbtc_cynullsta_v7 v7;
 };
 
-struct rtw89_btc_fbtc_btver {
+struct rtw89_btc_fbtc_btver_v1 {
        u8 fver; /* btc_ver::fcxbtver */
        u8 rsvd;
        __le16 rsvd2;
@@ -2714,6 +2714,22 @@ struct rtw89_btc_fbtc_btver {
        __le32 feature;
 } __packed;
 
+struct rtw89_btc_fbtc_btver_v7 {
+       u8 fver;
+       u8 rsvd0;
+       u8 rsvd1;
+       u8 rsvd2;
+
+       __le32 coex_ver; /*bit[15:8]->shared, bit[7:0]->non-shared */
+       __le32 fw_ver;
+       __le32 feature;
+} __packed;
+
+union rtw89_btc_fbtc_btver {
+       struct rtw89_btc_fbtc_btver_v1 v1;
+       struct rtw89_btc_fbtc_btver_v7 v7;
+} __packed;
+
 struct rtw89_btc_fbtc_btafh {
        u8 fver; /* btc_ver::fcxbtafh */
        u8 rsvd;
@@ -2971,7 +2987,7 @@ struct rtw89_btc_rpt_fbtc_gpio_dbg {
 
 struct rtw89_btc_rpt_fbtc_btver {
        struct rtw89_btc_rpt_cmn_info cinfo; /* common info, by driver */
-       struct rtw89_btc_fbtc_btver finfo; /* info from fw */
+       union rtw89_btc_fbtc_btver finfo; /* info from fw */
 };
 
 struct rtw89_btc_rpt_fbtc_btscan {