Re: [PATCH 2/6] Bluetooth: Introduce HCI_ADVERTISING_INSTANCE setting and add AD flags

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Arman,

> This patch introduces the HCI_ADVERTISING_INSTANCE setting, which is set
> when an at least one advertising instance has been added using the
> "Add Advertising" mgmt command. This patch also adds macro definitions
> various advertising data fields which can be assigned using the
> "Add Advertising" command.
> 
> Signed-off-by: Arman Uguray <armansito@xxxxxxxxxxxx>
> ---
> include/net/bluetooth/hci.h | 14 ++++++++++++++
> 1 file changed, 14 insertions(+)
> 
> diff --git a/include/net/bluetooth/hci.h b/include/net/bluetooth/hci.h
> index 06e7eee..e400175 100644
> --- a/include/net/bluetooth/hci.h
> +++ b/include/net/bluetooth/hci.h
> @@ -227,6 +227,7 @@ enum {
> 	HCI_LE_ENABLED,
> 	HCI_ADVERTISING,
> 	HCI_ADVERTISING_CONNECTABLE,
> +	HCI_ADVERTISING_INSTANCE,
> 	HCI_CONNECTABLE,
> 	HCI_DISCOVERABLE,
> 	HCI_LIMITED_DISCOVERABLE,
> @@ -477,6 +478,19 @@ enum {
> #define LE_AD_SIM_LE_BREDR_CTRL	0x08 /* Simultaneous LE & BR/EDR Controller */
> #define LE_AD_SIM_LE_BREDR_HOST	0x10 /* Simultaneous LE & BR/EDR Host */
> 
> +/* Other Low Energy Advertising Data fielt types  */
> +#define LE_AD_FIELD_SOLICIT16		0x14 /* 16-bit service solic. UUIDs */
> +#define LE_AD_FIELD_SOLICIT32		0x1F /* 32-bit service solic. UUIDs */
> +#define LE_AD_FIELD_SOLICIT128		0x15 /* 128-bit service solic. UUIDs */
> +#define LE_AD_FIELD_SVC_DATA16		0x16 /* Service Data - 16-bit UUID */
> +#define LE_AD_FIELD_SVC_DATA32		0x20 /* Service Data - 32-bit UUID */
> +#define LE_AD_FIELD_SVC_DATA128		0x21 /* Service Data - 128-bit UUID */
> +#define LE_AD_FIELD_PUB_TRGT_ADDR	0x17 /* Public Target Address */
> +#define LE_AD_FIELD_RND_TRGT_ADDR	0x18 /* Random Target Address */
> +#define LE_AD_FIELD_APPEARANCE		0x19 /* Appearance */
> +#define LE_AD_FIELD_MANUF_DATA		0xFF /* Manufacturer Specific Data */

please use the EIR_ section for this and extend it. I know this will look odd initially, but since AD and EIR share the same types we actually opted for calling most things EIR and kept it like that.

Also I think the only one you need to add is the APPEARANCE one. All others will not be used inside the kernel and have to be provided by userspace in raw format. So leave them out.

Regards

Marcel

--
To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Bluez Devel]     [Linux Wireless Networking]     [Linux Wireless Personal Area Networking]     [Linux ATH6KL]     [Linux USB Devel]     [Linux Media Drivers]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux