[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-ID: <1658383473-32188-1-git-send-email-quic_zijuhu@quicinc.com>
Date: Thu, 21 Jul 2022 14:04:29 +0800
From: Zijun Hu <quic_zijuhu@...cinc.com>
To: <marcel@...tmann.org>, <johan.hedberg@...il.com>,
<luiz.dentz@...il.com>, <davem@...emloft.net>,
<edumazet@...gle.com>, <kuba@...nel.org>, <pabeni@...hat.com>,
<luiz.von.dentz@...el.com>, <swyterzone@...il.com>,
<quic_zijuhu@...cinc.com>
CC: <linux-kernel@...r.kernel.org>, <linux-bluetooth@...r.kernel.org>,
<netdev@...r.kernel.org>
Subject: [PATCH v2 0/4] Bluetooth: Remove HCI_QUIRK_BROKEN_ERR_DATA_REPORTING
This patch series remove bluetooth HCI_QUIRK_BROKEN_ERR_DATA_REPORTING
the quirk was introduced by 'commit cde1a8a99287 ("Bluetooth: btusb: Fix
and detect most of the Chinese Bluetooth controllers")' to mark HCI
commands HCI_Read|Write_Default_Erroneous_Data_Reporting broken within BT
device driver, but the reason why these two HCI commands are broken is
that feature "Erroneous Data Reporting" is not enabled by firmware, so BT
core driver can addtionally check feature bit "Erroneous Data Reporting"
instead of the quirk to decide if these two HCI commands work fine.
BLUETOOTH CORE SPECIFICATION Version 5.3 | Vol 2, Part C | page 587
This feature indicates whether the device is able to support the
Packet_Status_Flag and the HCI commands HCI_Write_Default_-
Erroneous_Data_Reporting and HCI_Read_Default_Erroneous_-
Data_Reporting.
Only QCA and fake CSR btusb device driver set the quirk currently since
the feature "Erroneous Data Reporting" are not enabled by their firmware
so we also remove the quirk from their device driver.
Changes since v1:
- split changes to solve build error between patches
- optimize commit messages
Zijun Hu (4):
Bluetooth: hci_sync: Check LMP feature bit instead of quirk
Bluetooth: btusb: Remove HCI_QUIRK_BROKEN_ERR_DATA_REPORTING for QCA
Bluetooth: btusb: Remove HCI_QUIRK_BROKEN_ERR_DATA_REPORTING for fake
CSR
Bluetooth: hci_sync: Remove HCI_QUIRK_BROKEN_ERR_DATA_REPORTING
drivers/bluetooth/btusb.c | 2 --
include/net/bluetooth/hci.h | 12 +-----------
net/bluetooth/hci_sync.c | 7 ++-----
3 files changed, 3 insertions(+), 18 deletions(-)
--
2.7.4
Powered by blists - more mailing lists