lkml.org 
[lkml]   [2019]   [Dec]   [29]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 5.4 143/434] rtw88: coex: Set 4 slot mode for A2DP
    Date
    From: Ping-Ke Shih <pkshih@realtek.com>

    [ Upstream commit 12078aae453556a88fb46777b7cc5fc97f867b7c ]

    With shallow buffer size, certain BT devices have active
    A2DP flow control to fill buffer frequently. If the slot
    is not at BT side, data can't be sent successfully to BT
    devices, and will cause audio glitch.

    To resolve this issue, this commit splits TUs into 4-slots
    instead of 2-slot for all of the A2DP related coexistence
    strategies. That makes BT have higher opportunity to fill
    the A2DP buffer in time, and the audio quality could be
    more stable and smooth.

    Signed-off-by: Ping-Ke Shih <pkshih@realtek.com>
    Signed-off-by: Yan-Hsuan Chuang <yhchuang@realtek.com>
    Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    drivers/net/wireless/realtek/rtw88/coex.c | 24 ++++++++++++++---------
    1 file changed, 15 insertions(+), 9 deletions(-)

    diff --git a/drivers/net/wireless/realtek/rtw88/coex.c b/drivers/net/wireless/realtek/rtw88/coex.c
    index 793b40bdbf7c..3e95ad198912 100644
    --- a/drivers/net/wireless/realtek/rtw88/coex.c
    +++ b/drivers/net/wireless/realtek/rtw88/coex.c
    @@ -1308,6 +1308,7 @@ static void rtw_coex_action_bt_inquiry(struct rtw_dev *rtwdev)
    struct rtw_chip_info *chip = rtwdev->chip;
    bool wl_hi_pri = false;
    u8 table_case, tdma_case;
    + u32 slot_type = 0;

    if (coex_stat->wl_linkscan_proc || coex_stat->wl_hi_pri_task1 ||
    coex_stat->wl_hi_pri_task2)
    @@ -1318,14 +1319,16 @@ static void rtw_coex_action_bt_inquiry(struct rtw_dev *rtwdev)
    if (wl_hi_pri) {
    table_case = 15;
    if (coex_stat->bt_a2dp_exist &&
    - !coex_stat->bt_pan_exist)
    + !coex_stat->bt_pan_exist) {
    + slot_type = TDMA_4SLOT;
    tdma_case = 11;
    - else if (coex_stat->wl_hi_pri_task1)
    + } else if (coex_stat->wl_hi_pri_task1) {
    tdma_case = 6;
    - else if (!coex_stat->bt_page)
    + } else if (!coex_stat->bt_page) {
    tdma_case = 8;
    - else
    + } else {
    tdma_case = 9;
    + }
    } else if (coex_stat->wl_connected) {
    table_case = 10;
    tdma_case = 10;
    @@ -1361,7 +1364,7 @@ static void rtw_coex_action_bt_inquiry(struct rtw_dev *rtwdev)
    rtw_coex_set_ant_path(rtwdev, false, COEX_SET_ANT_2G);
    rtw_coex_set_rf_para(rtwdev, chip->wl_rf_para_rx[0]);
    rtw_coex_table(rtwdev, table_case);
    - rtw_coex_tdma(rtwdev, false, tdma_case);
    + rtw_coex_tdma(rtwdev, false, tdma_case | slot_type);
    }

    static void rtw_coex_action_bt_hfp(struct rtw_dev *rtwdev)
    @@ -1475,13 +1478,13 @@ static void rtw_coex_action_bt_a2dp(struct rtw_dev *rtwdev)

    if (efuse->share_ant) {
    /* Shared-Ant */
    + slot_type = TDMA_4SLOT;
    +
    if (coex_stat->wl_gl_busy && coex_stat->wl_noisy_level == 0)
    table_case = 10;
    else
    table_case = 9;

    - slot_type = TDMA_4SLOT;
    -
    if (coex_stat->wl_gl_busy)
    tdma_case = 13;
    else
    @@ -1585,13 +1588,14 @@ static void rtw_coex_action_bt_a2dp_hid(struct rtw_dev *rtwdev)

    if (efuse->share_ant) {
    /* Shared-Ant */
    + slot_type = TDMA_4SLOT;
    +
    if (coex_stat->bt_ble_exist)
    table_case = 26;
    else
    table_case = 9;

    if (coex_stat->wl_gl_busy) {
    - slot_type = TDMA_4SLOT;
    tdma_case = 13;
    } else {
    tdma_case = 14;
    @@ -1794,10 +1798,12 @@ static void rtw_coex_action_wl_linkscan(struct rtw_dev *rtwdev)
    struct rtw_efuse *efuse = &rtwdev->efuse;
    struct rtw_chip_info *chip = rtwdev->chip;
    u8 table_case, tdma_case;
    + u32 slot_type = 0;

    if (efuse->share_ant) {
    /* Shared-Ant */
    if (coex_stat->bt_a2dp_exist) {
    + slot_type = TDMA_4SLOT;
    table_case = 9;
    tdma_case = 11;
    } else {
    @@ -1818,7 +1824,7 @@ static void rtw_coex_action_wl_linkscan(struct rtw_dev *rtwdev)
    rtw_coex_set_ant_path(rtwdev, true, COEX_SET_ANT_2G);
    rtw_coex_set_rf_para(rtwdev, chip->wl_rf_para_rx[0]);
    rtw_coex_table(rtwdev, table_case);
    - rtw_coex_tdma(rtwdev, false, tdma_case);
    + rtw_coex_tdma(rtwdev, false, tdma_case | slot_type);
    }

    static void rtw_coex_action_wl_not_connected(struct rtw_dev *rtwdev)
    --
    2.20.1


    \
     
     \ /
      Last update: 2019-12-29 19:08    [W:2.469 / U:0.456 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site