Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1693464 > unrolled thread

[PATCH] bluetooth: document config options

Back to article view | Back to linux.kernel

Contents

  [PATCH] bluetooth: document config options Pavel Machek <pavel@ucw.cz> - 2017-07-21 10:00 +0200
    Re: [PATCH] bluetooth: document config options Marcel Holtmann <marcel@holtmann.org> - 2017-07-21 12:00 +0200
      Re: [PATCH] bluetooth: document config options Pavel Machek <pavel@ucw.cz> - 2017-07-21 12:50 +0200
        Re: [PATCH] bluetooth: document config options Marcel Holtmann <marcel@holtmann.org> - 2017-07-21 13:00 +0200

#1693464 — [PATCH] bluetooth: document config options

[Multipart message — attachments visible in raw view] — view raw

Kernel config options should include useful help text; I had to look
up the terms on wikipedia.

Signed-off-by: Pavel Machek <pavel@ucw.cz>

diff --git a/net/bluetooth/Kconfig b/net/bluetooth/Kconfig
index 68f951b..133c8a6 100644
--- a/net/bluetooth/Kconfig
+++ b/net/bluetooth/Kconfig
@@ -45,6 +45,8 @@ config BT_BREDR
 	bool "Bluetooth Classic (BR/EDR) features"
 	depends on BT
 	default y
+	help
+	  Support Bluetooth version 1 and 2 connections.
 
 source "net/bluetooth/rfcomm/Kconfig"
 
@@ -58,11 +60,17 @@ config BT_HS
 	bool "Bluetooth High Speed (HS) features"
 	depends on BT_BREDR
 	default y
+	help
+	  Bluetooth 3 introduces high-speed mode where bluetooth endpoints
+	  negotiate fast connection over WIFI. This controls its support.
 
 config BT_LE
 	bool "Bluetooth Low Energy (LE) features"
 	depends on BT
 	default y
+	help
+	  Bluetooth 4 introduces special low-energy protocol, designed
+	  for simple devices. This controls its support.
 
 config BT_6LOWPAN
 	tristate "Bluetooth 6LoWPAN support"

-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

[toc] | [next] | [standalone]

#1693552

Hi Pavel,

> Kernel config options should include useful help text; I had to look
> up the terms on wikipedia.
> 
> Signed-off-by: Pavel Machek <pavel@ucw.cz>
> 
> diff --git a/net/bluetooth/Kconfig b/net/bluetooth/Kconfig
> index 68f951b..133c8a6 100644
> --- a/net/bluetooth/Kconfig
> +++ b/net/bluetooth/Kconfig
> @@ -45,6 +45,8 @@ config BT_BREDR
> 	bool "Bluetooth Classic (BR/EDR) features"
> 	depends on BT
> 	default y
> +	help
> +	  Support Bluetooth version 1 and 2 connections.

so this is actually not correct. Version 5.0 is also Bluetooth BR/EDR.

I am fine if use the terms "Basic Rate" and "Enhanced Data Rate" in the description, but the version numbers are kinda not how this works. So some alternative wording needs to be used:

"Bluetooth Classic includes support for Basic Rate (BR) available with Bluetooth version 1.0b or later and support for Enhanced Data Rate (EDR) available with Bluetooth version 2.0 or later."

> source "net/bluetooth/rfcomm/Kconfig"
> 
> @@ -58,11 +60,17 @@ config BT_HS
> 	bool "Bluetooth High Speed (HS) features"
> 	depends on BT_BREDR
> 	default y
> +	help
> +	  Bluetooth 3 introduces high-speed mode where bluetooth endpoints
> +	  negotiate fast connection over WIFI. This controls its support.

Again, while Bluetooth version 3.0 introduces HS support, it is not what is selecting this.

I think here we can write something like this:

"Bluetooth High Speed includes support for off-loading Bluetooth connections via 802.11 physical layer available with Bluetooth version 3.0 or later."

> 
> config BT_LE
> 	bool "Bluetooth Low Energy (LE) features"
> 	depends on BT
> 	default y
> +	help
> +	  Bluetooth 4 introduces special low-energy protocol, designed
> +	  for simple devices. This controls its support.

I think here it is also better to have an alternative text:

"Bluetooth Low Energy includes support low-energy physical layer available with Bluetooth version 4.0 or later."

Regards

Marcel

[toc] | [prev] | [next] | [standalone]

#1693583

[Multipart message — attachments visible in raw view] — view raw

Hi!

> > Kernel config options should include useful help text; I had to look
> > up the terms on wikipedia.
> > 
> > Signed-off-by: Pavel Machek <pavel@ucw.cz>
> > 
> > diff --git a/net/bluetooth/Kconfig b/net/bluetooth/Kconfig
> > index 68f951b..133c8a6 100644
> > --- a/net/bluetooth/Kconfig
> > +++ b/net/bluetooth/Kconfig
> > @@ -45,6 +45,8 @@ config BT_BREDR
> > 	bool "Bluetooth Classic (BR/EDR) features"
> > 	depends on BT
> > 	default y
> > +	help
> > +	  Support Bluetooth version 1 and 2 connections.
> 
> so this is actually not correct. Version 5.0 is also Bluetooth BR/EDR.
> 
> I am fine if use the terms "Basic Rate" and "Enhanced Data Rate" in the description, but the version numbers are kinda not how this works. So some alternative wording needs to be used:
> 
> "Bluetooth Classic includes support for Basic Rate (BR) available with Bluetooth version 1.0b or later and support for Enhanced Data Rate (EDR) available with Bluetooth version 2.0 or later."
> 

Ok, works for me.

> > +	help
> > +	  Bluetooth 3 introduces high-speed mode where bluetooth endpoints
> > +	  negotiate fast connection over WIFI. This controls its support.
> 
> Again, while Bluetooth version 3.0 introduces HS support, it is not what is selecting this.
> 
> I think here we can write something like this:
> 
> "Bluetooth High Speed includes support for off-loading Bluetooth connections via 802.11 physical layer available with Bluetooth version 3.0 or later."
>

Ok.

> > config BT_LE
> > 	bool "Bluetooth Low Energy (LE) features"
> > 	depends on BT
> > 	default y
> > +	help
> > +	  Bluetooth 4 introduces special low-energy protocol, designed
> > +	  for simple devices. This controls its support.
> 
> I think here it is also better to have an alternative text:
> 
> "Bluetooth Low Energy includes support low-energy physical layer available with Bluetooth version 4.0 or later."
> 

Ok. Do I understand it correctly that Bluetooth LE basically has
nothing to do with bluetooth, and that Bluetooth 2 hardware will not
be able to detect / communicate with Bluetooth LE hardware?

									Pavel
-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

[toc] | [prev] | [next] | [standalone]

#1693589

Hi Pavel,

>>> Kernel config options should include useful help text; I had to look
>>> up the terms on wikipedia.
>>> 
>>> Signed-off-by: Pavel Machek <pavel@ucw.cz>
>>> 
>>> diff --git a/net/bluetooth/Kconfig b/net/bluetooth/Kconfig
>>> index 68f951b..133c8a6 100644
>>> --- a/net/bluetooth/Kconfig
>>> +++ b/net/bluetooth/Kconfig
>>> @@ -45,6 +45,8 @@ config BT_BREDR
>>> 	bool "Bluetooth Classic (BR/EDR) features"
>>> 	depends on BT
>>> 	default y
>>> +	help
>>> +	  Support Bluetooth version 1 and 2 connections.
>> 
>> so this is actually not correct. Version 5.0 is also Bluetooth BR/EDR.
>> 
>> I am fine if use the terms "Basic Rate" and "Enhanced Data Rate" in the description, but the version numbers are kinda not how this works. So some alternative wording needs to be used:
>> 
>> "Bluetooth Classic includes support for Basic Rate (BR) available with Bluetooth version 1.0b or later and support for Enhanced Data Rate (EDR) available with Bluetooth version 2.0 or later."
>> 
> 
> Ok, works for me.
> 
>>> +	help
>>> +	  Bluetooth 3 introduces high-speed mode where bluetooth endpoints
>>> +	  negotiate fast connection over WIFI. This controls its support.
>> 
>> Again, while Bluetooth version 3.0 introduces HS support, it is not what is selecting this.
>> 
>> I think here we can write something like this:
>> 
>> "Bluetooth High Speed includes support for off-loading Bluetooth connections via 802.11 physical layer available with Bluetooth version 3.0 or later."
>> 
> 
> Ok.
> 
>>> config BT_LE
>>> 	bool "Bluetooth Low Energy (LE) features"
>>> 	depends on BT
>>> 	default y
>>> +	help
>>> +	  Bluetooth 4 introduces special low-energy protocol, designed
>>> +	  for simple devices. This controls its support.
>> 
>> I think here it is also better to have an alternative text:
>> 
>> "Bluetooth Low Energy includes support low-energy physical layer available with Bluetooth version 4.0 or later."
>> 
> 
> Ok. Do I understand it correctly that Bluetooth LE basically has
> nothing to do with bluetooth, and that Bluetooth 2 hardware will not
> be able to detect / communicate with Bluetooth LE hardware?

if you have LE only on one side and BR/EDR only on the other side, then they can not communicate with each other. You need at least one side with dual-mode BR/EDR/LE support.

Regards

Marcel

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web