Re: [PATCH v2 03/14] Bluetooth: qca: fix ROM version reading on WCN3998 chips

Bartosz Golaszewski
2026-01-06T12:38:39Z

On Tue, Jan 6, 2026 at 2:01 AM Dmitry Baryshkov
<dmitry.baryshkov@oss.qualcomm.com> wrote:
>
> WCN3998 uses a bit different format for rom version:
>
> [ 5.479978] Bluetooth: hci0: setting up wcn399x
> [ 5.633763] Bluetooth: hci0: QCA Product ID :0x0000000a
> [ 5.645350] Bluetooth: hci0: QCA SOC Version :0x40010224
> [ 5.650906] Bluetooth: hci0: QCA ROM Version :0x00001001
> [ 5.665173] Bluetooth: hci0: QCA Patch Version:0x00006699
> [ 5.679356] Bluetooth: hci0: QCA controller version 0x02241001
> [ 5.691109] Bluetooth: hci0: QCA Downloading qca/crbtfw21.tlv
> [ 6.680102] Bluetooth: hci0: QCA Downloading qca/crnv21.bin
> [ 6.842948] Bluetooth: hci0: QCA setup on UART is completed
>
> Fixes: 523760b7ff88 ("Bluetooth: hci_qca: Added support for WCN3998")
> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>
> ---
> drivers/bluetooth/btqca.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/drivers/bluetooth/btqca.c b/drivers/bluetooth/btqca.c
> index 7c958d6065be..86a48d009d1b 100644
> --- a/drivers/bluetooth/btqca.c
> +++ b/drivers/bluetooth/btqca.c
> @@ -804,6 +804,8 @@ int qca_uart_setup(struct hci_dev *hdev, uint8_t baudrate,
> */
> if (soc_type == QCA_WCN3988)
> rom_ver = ((soc_ver & 0x00000f00) >> 0x05) | (soc_ver & 0x0000000f);
> + else if (soc_type == QCA_WCN3998)
> + rom_ver = ((soc_ver & 0x0000f000) >> 0x07) | (soc_ver & 0x0000000f);
> else
> rom_ver = ((soc_ver & 0x00000f00) >> 0x04) | (soc_ver & 0x0000000f);
>
>
> --
> 2.47.3
>

Reviewed-by: Bartosz Golaszewski <bartosz.golaszewski@oss.qualcomm.com>