2024-04-22 16:12:10

by quic_zijuhu

[permalink] [raw]
Subject: Re: RFC for Qualcomm Bluetooth patches

On 4/22/2024 11:30 PM, quic_zijuhu wrote:
> On 4/11/2024 9:58 PM, Von Dentz, Luiz wrote:
>> Hi,
>>
>> How about resending them?
>>
>> Also we are getting pretty bad feedback from QCA drivers:
>>
>> https://lore.kernel.org/linux-bluetooth/[email protected]/
>>
>> Do you guys ever test your drivers with more traditional Linux distros that actually runs bluetoothd daemon?
>> ________________________________
>> From: quic_zijuhu <[email protected]>
>> Sent: Wednesday, April 10, 2024 11:02 PM
>> To: Von Dentz, Luiz <[email protected]>; [email protected] <[email protected]>
>> Subject: RFC for Qualcomm Bluetooth patches
>>
>> Hi Luiz, marcel
>>
>> could you have comments for several Qualcomm BT patches under below link.
>> https://patchwork.kernel.org/project/bluetooth/list/
>>
>>
>> A)
>> tile: [v3] Bluetooth: qca: Fix crash when use tool btattach for QCA_ROME
>>
>> the original v1 fix was firstly posted by me at 2024-01-11, and Johan Hovold <[email protected]> recently also
>> have a similar fix with title "Bluetooth: qca: fix NULL-deref on non-serdev setup"
>>
>> B)
>> Bluetooth: qca: Add tool btattach support for more QCA soc types
>> [v1] tools/btattach: Add support for all QCA soc_types
>>
>> these patches make tool btattach support more Qualcomm BT controllers.
>>
>> C)
>> [v2,RESEND] Bluetooth: btusb: Fix triggering coredump implementation for QCA
>>
>> this fix is for QCA controller with USB interface. the fix for UART interface has been merged
>> by below commit:
>> Fixes: 6abf9dd26bb1 ("Bluetooth: qca: Fix triggering coredump implementation")
>>
> Hi Luiz,marcel
> let me list my changes for QCA controllers with priority from high to
> low for you to code review and pick up. they are listed by tile
> following change link
>
> 1)
> Bluetooth: qca: Support downloading board id specific NVM for WCN7850
> https://patchwork.kernel.org/project/bluetooth/patch/[email protected]/
>
> 2) the issue you mentioned are fixed by below changes
> [v5,1/2] Bluetooth: qca: Fix BT enable failure for QCA_QCA6390
> https://patchwork.kernel.org/project/bluetooth/patch/[email protected]/
>
> [v5,2/2] Bluetooth: qca: Fix BT enable failure for QCA_QCA6390 after
> disable then warm reboot
> https://patchwork.kernel.org/project/bluetooth/patch/[email protected]/
>
> 3) i also have mentioned below fix as shown in last mail.
> [v2,RESEND] Bluetooth: qca: Fix crash when btattach BT controller QCA_ROME
> https://patchwork.kernel.org/project/bluetooth/patch/[email protected]/
>
> 4) below changes will improve code quality.
> [v1,RESEND] Bluetooth: Remove 3 repeated macro definitions
> https://patchwork.kernel.org/project/bluetooth/patch/[email protected]/
>
> [v1,RESEND] Bluetooth: hci_conn: Remove a redundant check for HFP offload
> https://patchwork.kernel.org/project/bluetooth/patch/[email protected]/
>
> [v1,RESEND] Bluetooth: btusb: Correct timeout macro argument used to
> receive control message
> https://patchwork.kernel.org/project/bluetooth/patch/[email protected]/
these changes which will improve code quality have been merged.

Hi Luiz, marcel,

hope you will take time to code review functionality 1) and bug fixes 2).

thanks.