Hi Luiz and all: I am QC bt host guys, today I sync up with qc btfw team and conclude following result: 1: as I mentioned before, qca2066 and wcn6855 are the same chip family, and they can share the same btfw patch, and distinguish by using different nvm based on board id. 2: I upload the btfw previously for qca2066 , you can refer to https://git.kernel.org/pub/scm/linux/kernel/git/firmware/linux-firmware.git/commit/?id=2ba1beaae0c649ce8a50baecc8df9e81cd524e65 , and also wcn6855 can share to use btfw patch and nvm with qca2066. 3:steev patch does not include read board id part logical , I think it is better combine the steev patch and my patch into one. Regards. Tim -----Original Message----- From: Luiz Augusto von Dentz <luiz.dentz@xxxxxxxxx> Sent: Friday, February 10, 2023 5:38 AM To: Tim Jiang (QUIC) <quic_tjiang@xxxxxxxxxxx> Cc: Steev Klimaszewski <steev@xxxxxxxx> Subject: Re: [PATCH v4 2/4] Bluetooth: hci_qca: Add support for QTI Bluetooth chip wcn6855 Hi Tim, On Thu, Feb 9, 2023 at 12:45 AM Tim Jiang (QUIC) <quic_tjiang@xxxxxxxxxxx> wrote: > > Hi steev: > > 6855 and 2066 are the same chip, the difference is 6855 use 2 ANT and 2066 use 3 ANT , but we have many nvm configs for 2066 and 6855 , that is the same , you can refer to I ever add support for 6855 usb . Could you please review Steev changes, also if you could please refactor the code regarding the soc_type, making the necessary field as part of the data structure should make it a lot easier to add support for new configurations without having to change the code in multiple places. > Regards. > Tim > > -- Luiz Augusto von Dentz