Received-SPF: pass (google.com: domain of linux-bluetooth+bounces-5076-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
Message-ID: <44e2840e-06b9-4900-bfe6-881d27081ff0@quicinc.com>
Date: Mon, 3 Jun 2024 19:17:50 +0530
Precedence: bulk
MIME-Version: 1.0
User-Agent: Mozilla Thunderbird
Subject: Re: FW: [PATCH] PENDING: Bluetooth: Increase settling time for
 baudrate change VSC
Content-Language: en-US
To: Marcel Holtmann <marcel@holtmann.org>,
        Luiz Augusto von Dentz
	<luiz.dentz@gmail.com>
References: <20240603125357.3035-1-quic_janathot@quicinc.com>
 <SA1PR02MB8694BA5D75CF5FF8C86C7274E1FF2@SA1PR02MB8694.namprd02.prod.outlook.com>
CC: <linux-bluetooth@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
        <quic_hbandi@quicinc.com>, <quic_mohamull@quicinc.com>
From: Janaki Ramaiah Thota <quic_janathot@quicinc.com>
In-Reply-To: <SA1PR02MB8694BA5D75CF5FF8C86C7274E1FF2@SA1PR02MB8694.namprd02.prod.outlook.com>
Content-Type: text/plain; charset="UTF-8"; format=flowed
Content-Transfer-Encoding: 7bit


> From: Janaki Ramaiah Thota (Temp) (QUIC) <quic_janathot@quicinc.com>
> Sent: Monday, June 3, 2024 6:24 PM
> To: Marcel Holtmann <marcel@holtmann.org>; Luiz Augusto von Dentz <luiz.dentz@gmail.com>
> Cc: Mohammed Sameer Mulla (QUIC) <quic_mohamull@quicinc.com>; Harish Bandi (QUIC) <quic_hbandi@quicinc.com>; linux-bluetooth@vger.kernel.org; linux-kernel@vger.kernel.org
> Subject: [PATCH] PENDING: Bluetooth: Increase settling time for baudrate change VSC
> 
> From: Harish Bandi <quic_hbandi@quicinc.com>
> 
> This change is done to align the settling time and synchronization for baudrate VSC for WCN6750.
> 
> In logging disabled builds and few devices baudrate change and flow control is taking time so increasing the wait time to controller and uart to handle baudrate change request properly.
> 
> Change-Id: If0631cb886e53817f963f075e626c89a791b7be4
> Signed-off-by: Harish Bandi <quic_hbandi@quicinc.com>
> ---
> 
> diff --git a/drivers/bluetooth/btqca.c b/drivers/bluetooth/btqca.c index eba9165..a59b35e 100644
> --- a/drivers/bluetooth/btqca.c
> +++ b/drivers/bluetooth/btqca.c
> @@ -600,7 +600,6 @@
>                  rom_ver = ((soc_ver & 0x00000f00) >> 0x04) | (soc_ver & 0x0000000f);
> 
>          if (soc_type == QCA_WCN6750) {
> -               msleep(100);
>                  qca_send_patch_config_cmd(hdev);
>          }
> 
> diff --git a/drivers/bluetooth/hci_qca.c b/drivers/bluetooth/hci_qca.c index d8abd27..0b41214 100644
> --- a/drivers/bluetooth/hci_qca.c
> +++ b/drivers/bluetooth/hci_qca.c
> @@ -1356,11 +1356,13 @@
>          case QCA_WCN3990:
>          case QCA_WCN3991:
>          case QCA_WCN3998:
> -       case QCA_WCN6750:
>          case QCA_WCN6855:
>          case QCA_WCN7850:
>                  usleep_range(1000, 10000);
>                  break;
> +       case QCA_WCN6750:
> +               msleep(30);
> +               break;
> 
>          default:
>                  msleep(300);
> @@ -1904,7 +1906,6 @@
>                  qca_set_speed(hu, QCA_INIT_SPEED);
>          }
> 
> -       usleep_range(1000, 20000);
>          /* Setup user speed if needed */
>          speed = qca_get_speed(hu, QCA_OPER_SPEED);
>          if (speed) {

Please ignore the above patch.

-Janaki Ram