Re: Fw: [PATCH 2/2] Bluetooth: hci_qca: QCC2072 enablement

[Vivek Sahu]

> From: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxxxxxxxx>
> Sent: 17 December 2025 19:59
> To: Vivek Sahu <vivesahu@xxxxxxxxxxxxxxxx>
> Cc: Marcel Holtmann <marcel@xxxxxxxxxxxx>; Luiz Augusto von Dentz <luiz.dentz@xxxxxxxxx>; Rob Herring <robh@xxxxxxxxxx>; Krzysztof Kozlowski <krzk+dt@xxxxxxxxxx>; Conor Dooley <conor+dt@xxxxxxxxxx>; Bartosz Golaszewski <brgl@xxxxxxxx>; Balakrishna Godavarthi (QUIC) <quic_bgodavar@xxxxxxxxxxx>; Rocky Liao (QUIC) <quic_rjliao@xxxxxxxxxxx>; Mohammed Sameer Mulla (QUIC) <quic_mohamull@xxxxxxxxxxx>; Harish Bandi (QUIC) <quic_hbandi@xxxxxxxxxxx>; linux-bluetooth@xxxxxxxxxxxxxxx <linux-bluetooth@xxxxxxxxxxxxxxx>; devicetree@xxxxxxxxxxxxxxx <devicetree@xxxxxxxxxxxxxxx>; linux-kernel@xxxxxxxxxxxxxxx <linux-kernel@xxxxxxxxxxxxxxx>; linux-arm-msm@xxxxxxxxxxxxxxx <linux-arm-msm@xxxxxxxxxxxxxxx>
> Subject: Re: [PATCH 2/2] Bluetooth: hci_qca: QCC2072 enablement
>
> On Wed, Dec 17, 2025 at 04:58:50PM +0530, Vivek Kumar Sahu wrote:
> > Adding support for BT SoC QCC2072.
> > Set appropriate configurations for BT UART
> > transport.
> Read Documentation/process/submitting-patches.rst


I'll make the commit message more clear in the next patch set of this 
commit.


> > Signed-off-by: Vivek Kumar Sahu <vivesahu@xxxxxxxxxxxxxxxx>
> > ---
> >   drivers/bluetooth/btqca.c   |  8 ++++++++
> >   drivers/bluetooth/btqca.h   |  1 +
> >   drivers/bluetooth/hci_qca.c | 17 +++++++++++++++++
> >   3 files changed, 26 insertions(+)
> >
> > diff --git a/drivers/bluetooth/btqca.c b/drivers/bluetooth/btqca.c
> > index 7c958d6065be..7eb095db4a1d 100644
> > --- a/drivers/bluetooth/btqca.c
> > +++ b/drivers/bluetooth/btqca.c
> > @@ -854,6 +854,10 @@ int qca_uart_setup(struct hci_dev *hdev, uint8_t baudrate,
> >                         snprintf(config.fwname, sizeof(config.fwname),
> >                                  "qca/hmtbtfw%02x.tlv", rom_ver);
> >                         break;
> > +             case QCA_QCC2072:
> Please keep the file sorted. Find a correct place to insert your changes
> rather than randomly sticking them to the end. This applies to _all_ the
> changes you've made here.


I'll address this in the next patch set of this commit.

> > +                     snprintf(config.fwname, sizeof(config.fwname),
> > +                              "qca/ornbtfw%02x.tlv", rom_ver);
> I hope to see the firmware being submitted to linux-firmware.


"YES", firmware is being submitted.


> > +                     break;
> >                 default:
> >                         snprintf(config.fwname, sizeof(config.fwname),
> >                                  "qca/rampatch_%08x.bin", soc_ver);
> > @@ -929,6 +933,10 @@ int qca_uart_setup(struct hci_dev *hdev, uint8_t baudrate,
> >                         qca_get_nvm_name_by_board(config.fwname, sizeof(config.fwname),
> >                                  "hmtnv", soc_type, ver, rom_ver, boardid);
> >                         break;
> > +             case QCA_QCC2072:
> > +                     snprintf(config.fwname, sizeof(config.fwname),
> > +                              "qca/ornnv%02x.bin", rom_ver);
> No board-specific NVMEM dumps?


"NO", for this BT SoC there is no board specific NVM.


> > +                     break;
> >                 default:
> >                         snprintf(config.fwname, sizeof(config.fwname),
> >                                  "qca/nvm_%08x.bin", soc_ver);
> --
> With best wishes
> Dmitry