Message ID | 20250312083847.7364-1-sy2239101@buaa.edu.cn (mailing list archive) |
---|---|
State | Awaiting Upstream |
Delegated to: | Netdev Maintainers |
Headers | show |
Series | Bluetooth: HCI: Fix value of HCI_ERROR_UNSUPPORTED_REMOTE_FEATURE | expand |
[Cc: +Jonas Dreßler] Dear Si-Jie, Welcome to the mailing list, and thank you for your patch! Am 12.03.25 um 09:38 schrieb Si-Jie Bai: > HCI_ERROR_UNSUPPORTED_REMOTE_FEATURE is actually 0x1a not 0x1e: > > BLUETOOTH CORE SPECIFICATION Version 5.3 | Vol 1, Part F > page 371: The above length is 66 characters (< 72), and fits in one line. > 0x1A Unsupported Remote Feature > Maybe add: The value was probably changed by mistake, when defining the macro. Please add a Fixes: tag. It should be: Fixes: 79c0868ad65a ("Bluetooth: hci_event: Use HCI error defines instead of magic values") > Signed-off-by: Si-Jie Bai <sy2239101@buaa.edu.cn> > --- > include/net/bluetooth/hci.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/include/net/bluetooth/hci.h b/include/net/bluetooth/hci.h > index 0d51970d8..3ec915738 100644 > --- a/include/net/bluetooth/hci.h > +++ b/include/net/bluetooth/hci.h > @@ -683,7 +683,7 @@ enum { > #define HCI_ERROR_REMOTE_POWER_OFF 0x15 > #define HCI_ERROR_LOCAL_HOST_TERM 0x16 > #define HCI_ERROR_PAIRING_NOT_ALLOWED 0x18 > -#define HCI_ERROR_UNSUPPORTED_REMOTE_FEATURE 0x1e > +#define HCI_ERROR_UNSUPPORTED_REMOTE_FEATURE 0x1a > #define HCI_ERROR_INVALID_LL_PARAMS 0x1e > #define HCI_ERROR_UNSPECIFIED 0x1f > #define HCI_ERROR_ADVERTISING_TIMEOUT 0x3c With the above fixes, you can add: Reviewed-by: Paul Menzel <pmenzel@molgen.mpg.de> Kind regards, Paul
Hi Si-Jie, On Wed, Mar 12, 2025 at 4:39 AM Si-Jie Bai <sy2239101@buaa.edu.cn> wrote: > > HCI_ERROR_UNSUPPORTED_REMOTE_FEATURE is actually 0x1a not 0x1e: > > BLUETOOTH CORE SPECIFICATION Version 5.3 | Vol 1, Part F > page 371: > > 0x1A Unsupported Remote Feature > > Signed-off-by: Si-Jie Bai <sy2239101@buaa.edu.cn> > --- > include/net/bluetooth/hci.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/include/net/bluetooth/hci.h b/include/net/bluetooth/hci.h > index 0d51970d8..3ec915738 100644 > --- a/include/net/bluetooth/hci.h > +++ b/include/net/bluetooth/hci.h > @@ -683,7 +683,7 @@ enum { > #define HCI_ERROR_REMOTE_POWER_OFF 0x15 > #define HCI_ERROR_LOCAL_HOST_TERM 0x16 > #define HCI_ERROR_PAIRING_NOT_ALLOWED 0x18 > -#define HCI_ERROR_UNSUPPORTED_REMOTE_FEATURE 0x1e > +#define HCI_ERROR_UNSUPPORTED_REMOTE_FEATURE 0x1a > #define HCI_ERROR_INVALID_LL_PARAMS 0x1e > #define HCI_ERROR_UNSPECIFIED 0x1f > #define HCI_ERROR_ADVERTISING_TIMEOUT 0x3c > -- > 2.25.1 I tooked the following one over this since it better explains the issue and had the Fixes tag: https://patchwork.kernel.org/project/bluetooth/patch/20250312190943.152482-1-arkadiusz.bokowy@gmail.com/
Hello: This patch was applied to bluetooth/bluetooth-next.git (master) by Luiz Augusto von Dentz <luiz.von.dentz@intel.com>: On Wed, 12 Mar 2025 16:38:47 +0800 you wrote: > HCI_ERROR_UNSUPPORTED_REMOTE_FEATURE is actually 0x1a not 0x1e: > > BLUETOOTH CORE SPECIFICATION Version 5.3 | Vol 1, Part F > page 371: > > 0x1A Unsupported Remote Feature > > [...] Here is the summary with links: - Bluetooth: HCI: Fix value of HCI_ERROR_UNSUPPORTED_REMOTE_FEATURE https://git.kernel.org/bluetooth/bluetooth-next/c/8d35126f69f2 You are awesome, thank you!
diff --git a/include/net/bluetooth/hci.h b/include/net/bluetooth/hci.h index 0d51970d8..3ec915738 100644 --- a/include/net/bluetooth/hci.h +++ b/include/net/bluetooth/hci.h @@ -683,7 +683,7 @@ enum { #define HCI_ERROR_REMOTE_POWER_OFF 0x15 #define HCI_ERROR_LOCAL_HOST_TERM 0x16 #define HCI_ERROR_PAIRING_NOT_ALLOWED 0x18 -#define HCI_ERROR_UNSUPPORTED_REMOTE_FEATURE 0x1e +#define HCI_ERROR_UNSUPPORTED_REMOTE_FEATURE 0x1a #define HCI_ERROR_INVALID_LL_PARAMS 0x1e #define HCI_ERROR_UNSPECIFIED 0x1f #define HCI_ERROR_ADVERTISING_TIMEOUT 0x3c
HCI_ERROR_UNSUPPORTED_REMOTE_FEATURE is actually 0x1a not 0x1e: BLUETOOTH CORE SPECIFICATION Version 5.3 | Vol 1, Part F page 371: 0x1A Unsupported Remote Feature Signed-off-by: Si-Jie Bai <sy2239101@buaa.edu.cn> --- include/net/bluetooth/hci.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)