Hi Joseph,
> Add the btandroid.c file to support Android BQR commands.
>
> This module may be referenced by btusb, btrtl, and hci_qca when a
> Bluetooth controller supports the Android Bluetooth Quality Report.
>
> Reported-by: kernel test robot <[email protected]>
>
> Reviewed-by: Miao-chen Chou <[email protected]>
> Signed-off-by: Joseph Hwang <[email protected]>
> ---
>
> Changes in v3:
> - Fix the auto build test ERROR
> "undefined symbol: btandroid_set_quality_report" that occurred
> with some kernel configs.
> - Note that the mgmt-tester "Read Exp Feature - Success" failed.
> But on my test device, the same test passed. Please kindly let me
> know what may be going wrong. These patches do not actually
> modify read/set experimental features.
> - As to CheckPatch failed. No need to modify the MAINTAINERS file.
> Thanks.
>
> Changes in v2:
> - Fix the titles of patches 2/3 and 3/3 and reduce their lengths.
>
> drivers/bluetooth/Kconfig | 5 ++
> drivers/bluetooth/Makefile | 1 +
> drivers/bluetooth/btandroid.c | 106 ++++++++++++++++++++++++++++++++++
> drivers/bluetooth/btandroid.h | 10 ++++
> 4 files changed, 122 insertions(+)
> create mode 100644 drivers/bluetooth/btandroid.c
> create mode 100644 drivers/bluetooth/btandroid.h
I am confused now. Did you read my review comments? I do _not_ want it this way; so please read my previous response and don’t send the exact same patch again.
Regards
Marcel
Hi Marcel:
I am very sorry that I missed your review comment! I have used
net/bluetooth/aosp.c to replace drivers/bluetooth/btandroid.c, and
enable the aosp capability by hci_set_aosp_capable() in the drivers.
The new patch subject is
[v4,1/4] Bluetooth: aosp: Support AOSP Bluetooth Quality Report
(https://patchwork.kernel.org/project/bluetooth/patch/20210926150657.v4.1.Iaa4a0269e51d8e8d8784a6ac8e05899b49a1377d@changeid/)
Please take a look.
Thanks and regards,
Joseph
On Mon, Sep 13, 2021 at 10:32 PM Marcel Holtmann <[email protected]> wrote:
>
> Hi Joseph,
>
> > Add the btandroid.c file to support Android BQR commands.
> >
> > This module may be referenced by btusb, btrtl, and hci_qca when a
> > Bluetooth controller supports the Android Bluetooth Quality Report.
> >
> > Reported-by: kernel test robot <[email protected]>
> >
> > Reviewed-by: Miao-chen Chou <[email protected]>
> > Signed-off-by: Joseph Hwang <[email protected]>
> > ---
> >
> > Changes in v3:
> > - Fix the auto build test ERROR
> > "undefined symbol: btandroid_set_quality_report" that occurred
> > with some kernel configs.
> > - Note that the mgmt-tester "Read Exp Feature - Success" failed.
> > But on my test device, the same test passed. Please kindly let me
> > know what may be going wrong. These patches do not actually
> > modify read/set experimental features.
> > - As to CheckPatch failed. No need to modify the MAINTAINERS file.
> > Thanks.
> >
> > Changes in v2:
> > - Fix the titles of patches 2/3 and 3/3 and reduce their lengths.
> >
> > drivers/bluetooth/Kconfig | 5 ++
> > drivers/bluetooth/Makefile | 1 +
> > drivers/bluetooth/btandroid.c | 106 ++++++++++++++++++++++++++++++++++
> > drivers/bluetooth/btandroid.h | 10 ++++
> > 4 files changed, 122 insertions(+)
> > create mode 100644 drivers/bluetooth/btandroid.c
> > create mode 100644 drivers/bluetooth/btandroid.h
>
> I am confused now. Did you read my review comments? I do _not_ want it this way; so please read my previous response and don’t send the exact same patch again.
>
> Regards
>
> Marcel
>
--
Joseph Shyh-In Hwang
Email: [email protected]