Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   Neeraj Sanjay Kale <neeraj.sanjaykale@nxp.com>
To:     marcel@holtmann.org, johan.hedberg@gmail.com, luiz.dentz@gmail.com
Cc:     linux-kernel@vger.kernel.org, linux-bluetooth@vger.kernel.org,
        amitkumar.karwar@nxp.com, rohit.fule@nxp.com, sherry.sun@nxp.com,
        neeraj.sanjaykale@nxp.com
Subject: [PATCH v1] Bluetooth: btnxpuart: Enable flow control before checking boot signature
Date:   Wed, 19 Apr 2023 14:33:46 +0530
Message-Id: <20230419090346.1671130-1-neeraj.sanjaykale@nxp.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?UhmTHLozNEsCuZoXl+5w1irflA+0P74Y6UDtG9e4Sb9QGlqWFycp9DctkguB?=
 =?us-ascii?Q?DbOx5WN3VRtl0lAimdfrdcNEWHMGoduHvNPQ2j8086pKw/kMuWrEhMpg2cJD?=
 =?us-ascii?Q?NZTx+vBb2U9a1PRMbN+RKByLYUk36dX+3eGOZr5dgNB/9mvmQMoHmYSvlwFK?=
 =?us-ascii?Q?2roN78tO4elKfixibRqVbzVV9RP8aXByT+jb9d55DQO630HT1zIgk52wkDMC?=
 =?us-ascii?Q?sCFiDaAZJfYoPqPIqZRERLDYp4+DWEULSdFJyogdZTNQ9HoL2U4S0aaQJzLs?=
 =?us-ascii?Q?oc0/2o6Rr0oszl4tW+wLF+DmeCFGKE8c4gqFE49AxCAow7y5FyKdXNKHQVDv?=
 =?us-ascii?Q?AlhDd8+bzYXrfU/MAbj1Pok46isx/h10IyPj8d5nDk/WDzIO3DAbIr0U0Oqo?=
 =?us-ascii?Q?ZgvISf4R+TnT0LhQyFMjSFhj3T3B6Yss2CKH0Dm2U8K1iGnP2R6D+Kka3mAP?=
 =?us-ascii?Q?8dbW+3IKg3UibN0DqYwmVHwDsvMg1t3085HFoqAY3lhGoV1u4tmSZCPXH/uI?=
 =?us-ascii?Q?sR3LZGU/uzu8dQqD07mdgiLESWku+goeDfpqoaUAX/dZ30sZfV8UT3PtjVl1?=
 =?us-ascii?Q?aIJuBawXQ7pOlaNswf017B4+2MWMTevO4Hs5dk2TN64mXpwIi/5DKEB79IUe?=
 =?us-ascii?Q?fPpIWKXvvN0MiQTLCwZBE0QaeFXdmZiU0NZBS+hy1at/WiqtPbi9zi1Mt+Gl?=
 =?us-ascii?Q?czQbJkDyeHBHZafHM8l0UQedapJxk7qRx+q834rW/H4j+wgStq5AzHY0Z9dd?=
 =?us-ascii?Q?rSa/yDHfjQd+JqkWzSE7efraIuSjUtYJepVSnKRZGcqoTSh3UM5oJG9jEyWt?=
 =?us-ascii?Q?wkC+KPzKNXS0VFpk8ELBZ+Sth6yGan8bCzU7wnXvF/U/U4daz+6sP7RmicY6?=
 =?us-ascii?Q?MD1TFDqgWBOfbpjnc8R0wKBoEcI9jRfsV9Ppd2Gpei7633/lbeaelkMUDJF3?=
 =?us-ascii?Q?+rAwJFHcEvDMXYqCAjNILb53ZWEmvX1gWQm8gkDEArNpWRF6yvyupQMM+uhW?=
 =?us-ascii?Q?uheNX90wF2XS/5yPvPr58be9yT+ttL7gu/PTjAO2CnJC8d1k5hoNjWrE28U6?=
 =?us-ascii?Q?/SIKBloaBXH2dcRUTaPyfnO0Wn4dZi5t0ASawSUDtEeJgEDof0HSoHuVk/dI?=
 =?us-ascii?Q?j+123Qh5Ky3QFX3eB3TuCQgbjrdEC81YUJ9SQNB3JPZRUEIyZjdYzR5Y/BoB?=
 =?us-ascii?Q?QhXalZf2tN3NNaZxyDXPRSbeXXJPjirttKEVr+yezOXm4nsV35SHRvKngSga?=
 =?us-ascii?Q?LqcyDg/gab/F0110EquH85ilRasgwzXqubdQckbLlWd7fAdxS+xzgoXxrKsX?=
 =?us-ascii?Q?UHM6vHGmeTSW+NcHY+Hlbu/DavXFCHC2XuCr6ZT4xB0G0z1olNeKQBikyysP?=
 =?us-ascii?Q?qpOCIKXeh5uWryPRdMkzlAI7jTh5CwIx5pUT6PeHxPWn8Feou4YzYtci9bSq?=
 =?us-ascii?Q?RjPIS81FOQKzZ806KwnUQ8Ifc9Lcd+Wevr2Xy24BbcTjA/CsOhQAvs1IAcnc?=
 =?us-ascii?Q?0GG99biZdIfsJvLL/60ZgQ4f3QOVcA3JbmX2ng3R9UWPrLfULsOB160j1efY?=
 =?us-ascii?Q?wohxBuf8h3/gMHNbAjXkRDWJZE3CQoSAsEusfPRo0koP83tqxGWJJHWrhDcq?=
 =?us-ascii?Q?1w=3D=3D?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: f18687b8-cfd2-42b6-5ec4-08db40b506af
X-MS-Exchange-CrossTenant-AuthSource: DU2PR04MB8600.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 19 Apr 2023 09:04:05.7327
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: onds1504p/IEUko9mRRCAkLbFKzHb4DaeLMgA/p5veezFGD9s2Q5jjXR1f+3C/yBai3NVUtcIbwaE5sQIj9tYd/sqWmgUOcCPJ6PcueOITM=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PAWPR04MB9987
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE,
        RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS,T_SCC_BODY_TEXT_LINE,
        URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

This enables flow control before checking for bootloader signature and
deciding whether FW download is needed or not. In case of V1 bootloader
chips w8987 and w8997, it is observed that if WLAN FW is downloaded first
and power save is enabled in wlan core, bootloader signatures are not
emitted by the BT core when the chip is put to sleep. This results in BT
FW download timeout error. By enabling the flow control, the host enables
its RTS pin, and an interrupt in chip's UART peripheral causes the
bootloader to wakeup, enabling the bootloader signatures, which then helps
in downloading the bluetooth FW file.

Signed-off-by: Neeraj Sanjay Kale <neeraj.sanjaykale@nxp.com>
---
 drivers/bluetooth/btnxpuart.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/bluetooth/btnxpuart.c b/drivers/bluetooth/btnxpuart.c
index 93f3afc0c0c8..3ee41a0d45e7 100644
--- a/drivers/bluetooth/btnxpuart.c
+++ b/drivers/bluetooth/btnxpuart.c
@@ -967,7 +967,7 @@ static int nxp_set_ind_reset(struct hci_dev *hdev, void *data)
 static int nxp_check_boot_sign(struct btnxpuart_dev *nxpdev)
 {
 	serdev_device_set_baudrate(nxpdev->serdev, HCI_NXP_PRI_BAUDRATE);
-	serdev_device_set_flow_control(nxpdev->serdev, 0);
+	serdev_device_set_flow_control(nxpdev->serdev, 1);
 	set_bit(BTNXPUART_CHECK_BOOT_SIGNATURE, &nxpdev->tx_state);
 
 	return wait_event_interruptible_timeout(nxpdev->check_boot_sign_wait_q,
-- 
2.34.1