Received-SPF: pass (google.com: domain of linux-bluetooth+bounces-3521-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223;
Precedence: bulk
Mime-Version: 1.0
Content-Transfer-Encoding: quoted-printable
Content-Type: text/plain; charset=UTF-8
Date: Fri, 12 Apr 2024 16:23:59 +0200
Message-Id: <D0I7NRQQGOSJ.3JB82F1X4GIQM@fairphone.com>
From: "Luca Weiss" <luca.weiss@fairphone.com>
To: "Luca Weiss" <luca.weiss@fairphone.com>, "Marcel Holtmann"
 <marcel@holtmann.org>, "Johan Hedberg" <johan.hedberg@gmail.com>, "Luiz
 Augusto von Dentz" <luiz.dentz@gmail.com>, "David S. Miller"
 <davem@davemloft.net>, "Eric Dumazet" <edumazet@google.com>, "Jakub
 Kicinski" <kuba@kernel.org>, "Paolo Abeni" <pabeni@redhat.com>
Cc: <~postmarketos/upstreaming@lists.sr.ht>,
 <linux-bluetooth@vger.kernel.org>, <netdev@vger.kernel.org>,
 <linux-kernel@vger.kernel.org>, <phone-devel@vger.kernel.org>
Subject: Re: [PATCH] Bluetooth: Add more Bluetooth version defines
References: <20240216-bluetooth-defines-v1-1-6c39aacc66a8@fairphone.com>
In-Reply-To: <20240216-bluetooth-defines-v1-1-6c39aacc66a8@fairphone.com>

On Fri Feb 16, 2024 at 2:22 PM CET, Luca Weiss wrote:
> Add the various Bluetooth version identifiers found in the "Assigned
> Numbers" document[0] from the Bluetooth SIG.
>
> [0] https://www.bluetooth.com/specifications/assigned-numbers/

Hi all,

Is there any interest in this patch? Would be nice to get at least a
positive or negative reaction to it.

Regards
Luca

>
> Signed-off-by: Luca Weiss <luca.weiss@fairphone.com>
> ---
> To be clear, I don't have a use case for these extra defines myself but
> some time ago when working on Bluetooth I came across this and thought
> it would be interesting to have the list complete. No other motives.
> ---
>  include/net/bluetooth/bluetooth.h | 9 +++++++++
>  1 file changed, 9 insertions(+)
>
> diff --git a/include/net/bluetooth/bluetooth.h b/include/net/bluetooth/bl=
uetooth.h
> index 7ffa8c192c3f..818eb142eda3 100644
> --- a/include/net/bluetooth/bluetooth.h
> +++ b/include/net/bluetooth/bluetooth.h
> @@ -39,11 +39,20 @@
>  #endif
> =20
>  /* Bluetooth versions */
> +#define BLUETOOTH_VER_1_0B	0
>  #define BLUETOOTH_VER_1_1	1
>  #define BLUETOOTH_VER_1_2	2
>  #define BLUETOOTH_VER_2_0	3
>  #define BLUETOOTH_VER_2_1	4
> +#define BLUETOOTH_VER_3_0	5
>  #define BLUETOOTH_VER_4_0	6
> +#define BLUETOOTH_VER_4_1	7
> +#define BLUETOOTH_VER_4_2	8
> +#define BLUETOOTH_VER_5_0	9
> +#define BLUETOOTH_VER_5_1	10
> +#define BLUETOOTH_VER_5_2	11
> +#define BLUETOOTH_VER_5_3	12
> +#define BLUETOOTH_VER_5_4	13
> =20
>  /* Reserv for core and drivers use */
>  #define BT_SKB_RESERVE	8
>
> ---
> base-commit: 841c35169323cd833294798e58b9bf63fa4fa1de
> change-id: 20240216-bluetooth-defines-b810ce543191
>
> Best regards,