Re: [PATCH] Bluetooth: btrtl: Avoid loading the config file on security chips

[Paul Menzel]

Dear Max,


Thank you for your reply, and thank you for sending out v2.

Am 05.11.25 um 10:19 schrieb Max Chou:

[…]

> > -----Original Message-----
> > From: Paul Menzel [mailto:pmenzel@xxxxxxxxxxxxx]
> > Sent: Wednesday, November 5, 2025 3:58 PM

> > Am 05.11.25 um 07:37 schrieb Max Chou:
> > > For chips with security enabled, it's only possible to load firmware
> > > with a valid signature pattern.
> >
> > How can security be enabled?
> >
> > What is currently logged? An error?
>
> The security chips will be shipped to specific brand customers and
> are managed by an eFuse, which is programmed during manufacturing.> Currently, loading the config file causes initialization to fail on 
security chips.
> The security chips can only load firmware files that contain a valid signature.
>
> > Please go into the changes. What is the vendor command 0xAD over 0x0D?
>
> Actually, the current value is incorrect. The correct value for the
> parameters of the vendor command is 0xAD.
Understood. You should have mentioned in the commit message of v2, and 
maybe explain how that typo happened, and where that command is documented.

> > > - Example log for a security chip.
> > >
> > > Bluetooth: hci0: RTL: examining hci_ver=0c hci_rev=000a
> > >     lmp_ver=0c lmp_subver=8922
> > > Bluetooth: hci0: RTL: rom_version status=0 version=1
> > > Bluetooth: hci0: RTL: loading rtl_bt/rtl8922au_fw.bin
> > > Bluetooth: hci0: RTL: cfg_sz 0, total sz 71301
> > > Bluetooth: hci0: RTL: fw version 0x41c0c905
> > >
> > > - Example log for a normal chip.
> > >
> > > Bluetooth: hci0: RTL: examining hci_ver=0c hci_rev=000a
> > >     lmp_ver=0c lmp_subver=8922
> > > Bluetooth: hci0: RTL: rom_version status=0 version=1
> > > Bluetooth: hci0: RTL: loading rtl_bt/rtl8922au_fw.bin
> > > Bluetooth: hci0: RTL: loading rtl_bt/rtl8922au_config.bin
> > > Bluetooth: hci0: RTL: cfg_sz 6, total sz 71307
> > > Bluetooth: hci0: RTL: fw version 0x41c0c905
> > >
> > > Tested-by: Hilda Wu <hildawu@xxxxxxxxxxx>
> > > Signed-off-by: Nial Ni <niall_ni@xxxxxxxxxxxxxx>
> > > Signed-off-by: Max Chou <max.chou@xxxxxxxxxxx>
> > > ---
> > >    drivers/bluetooth/btrtl.c | 24 +++++++++++++-----------
> > >    1 file changed, 13 insertions(+), 11 deletions(-)

[…]


Kind regards,

Paul