Re: [PATCH v1 1/2] Bluetooth: btnxpuart: Add msleep() after changing the baudrate

[Neeraj Sanjay Kale]

Hi Luiz,

Please do not merge this patch series.

I have sent out 2 different patches instead titled:
1) [PATCH v2] Bluetooth: btnxpuart: Add an error message if FW dump trigger fails
2) [PATCH v1] Bluetooth: btnxpuart: Revert baudrate change in nxp_shutdown

Thanks,
Neeraj


>
> Hello:
>
> This series was applied to bluetooth/bluetooth-next.git (master) by Luiz
> Augusto von Dentz <luiz.von.dentz@xxxxxxxxx>:
>
> On Thu, 27 Mar 2025 23:55:22 +0530 you wrote:
> > This adds a 100 millisec sleep after change baudrate vendor command.
> >
> > It is observed that when the baudrate change command changes the
> > baudrate from 3000000 to 115200, any immediate HCI command returns an
> > error -22 (Device Busy).
> >
> > Adding a small delay after the change baudrate command complete event
> > is received helps fix the issue.
> >
> > [...]
>
> Here is the summary with links:
>   - [v1,1/2] Bluetooth: btnxpuart: Add msleep() after changing the baudrate
>     (no matching commit)
>   - [v1,2/2] Bluetooth: btnxpuart: Add an error message if FW dump trigger
> fails
>
> https://git.kern/
> el.org%2Fbluetooth%2Fbluetooth-
> next%2Fc%2F061e4972c48c&data=05%7C02%7Cneeraj.sanjaykale%40nxp.co
> m%7C08b8a55d25e04b812fd008dd72ecea34%7C686ea1d3bc2b4c6fa92cd99c
> 5c301635%7C0%7C0%7C638793084075362630%7CUnknown%7CTWFpbGZsb
> 3d8eyJFbXB0eU1hcGkiOnRydWUsIlYiOiIwLjAuMDAwMCIsIlAiOiJXaW4zMiIsIkF
> OIjoiTWFpbCIsIldUIjoyfQ%3D%3D%7C0%7C%7C%7C&sdata=8u4Uw6T820eP6
> mAFNCXU1xuUFpXYUB32nrYKuj8th%2BQ%3D&reserved=0
>
> You are awesome, thank you!
> --
> Deet-doot-dot, I am a bot.
> https://korg.do/
> cs.kernel.org%2Fpatchwork%2Fpwbot.html&data=05%7C02%7Cneeraj.sanjay
> kale%40nxp.com%7C08b8a55d25e04b812fd008dd72ecea34%7C686ea1d3bc2
> b4c6fa92cd99c5c301635%7C0%7C0%7C638793084075397504%7CUnknown%
> 7CTWFpbGZsb3d8eyJFbXB0eU1hcGkiOnRydWUsIlYiOiIwLjAuMDAwMCIsIlAiOi
> JXaW4zMiIsIkFOIjoiTWFpbCIsIldUIjoyfQ%3D%3D%7C0%7C%7C%7C&sdata=th
> Hgu6Dw8RWjMWGzjv7C%2F4JPJzBwwC7RzDUmor8bq4M%3D&reserved=0
>