* [PATCH BlueZ 0/1] client: add public-broadcast advertise helper @ 2026-03-17 14:26 raghava447 2026-03-17 14:26 ` [PATCH BlueZ 1/1] client: Add public-broadcast profile advertise command raghava447 2026-04-24 14:18 ` [PATCH BlueZ v2 0/2] client: Add public broadcast advertising support raghu447 0 siblings, 2 replies; 11+ messages in thread From: raghava447 @ 2026-03-17 14:26 UTC (permalink / raw) To: linux-bluetooth; +Cc: raghava447 This patch adds a bluetoothctl advertise helper for BLE Audio public broadcast profile advertising. This series adds a semantic helper: public-broadcast sq public-broadcast hq This matches with current commands: service 0x1856 0x02 0x00 service 0x1856 0x04 0x00 raghava447 (1): client: Add public-broadcast profile advertise command client/advertising.c | 70 ++++++++++++++++++++++++++++++++++++++++++++ client/advertising.h | 2 ++ client/main.c | 19 ++++++++++++ 3 files changed, 91 insertions(+) -- 2.53.0 ^ permalink raw reply [flat|nested] 11+ messages in thread
* [PATCH BlueZ 1/1] client: Add public-broadcast profile advertise command 2026-03-17 14:26 [PATCH BlueZ 0/1] client: add public-broadcast advertise helper raghava447 @ 2026-03-17 14:26 ` raghava447 2026-03-17 16:21 ` client: add public-broadcast advertise helper bluez.test.bot 2026-03-17 18:00 ` [PATCH BlueZ 1/1] client: Add public-broadcast profile advertise command Luiz Augusto von Dentz 2026-04-24 14:18 ` [PATCH BlueZ v2 0/2] client: Add public broadcast advertising support raghu447 1 sibling, 2 replies; 11+ messages in thread From: raghava447 @ 2026-03-17 14:26 UTC (permalink / raw) To: linux-bluetooth; +Cc: raghava447 Add an advertise submenu command for BLE Audio public broadcast profile advertising in bluetoothctl. The new "public-broadcast" command reuses the existing staged AD ServiceData field for UUID 0x1856. It supports: - public-broadcast sq - public-broadcast hq - public-broadcast The sq and hq values map to the existing raw service-data payloads: - sq -> 0x02 0x00 - hq -> 0x04 0x00 When called without arguments, the command reports the currently staged public-broadcast state if the staged AD service UUID is 0x1856. Else it reports that public broadcast is not set. --- client/advertising.c | 70 ++++++++++++++++++++++++++++++++++++++++++++ client/advertising.h | 2 ++ client/main.c | 19 ++++++++++++ 3 files changed, 91 insertions(+) diff --git a/client/advertising.c b/client/advertising.c index f9df1b855..30b89a404 100644 --- a/client/advertising.c +++ b/client/advertising.c @@ -20,6 +20,8 @@ #include <string.h> #include <errno.h> +#include "bluetooth/bluetooth.h" +#include "bluetooth/uuid.h" #include "gdbus/gdbus.h" #include "src/shared/util.h" #include "src/shared/shell.h" @@ -27,6 +29,7 @@ #define AD_PATH "/org/bluez/advertising" #define AD_IFACE "org.bluez.LEAdvertisement1" +#define AD_PUBLIC_BROADCAST_UUID "0x1856" struct ad_data { uint8_t data[245]; @@ -1005,6 +1008,73 @@ static void ad_clear_data(int type) memset(&ad.data[type], 0, sizeof(ad.data[type])); } +static bool ad_is_public_broadcast_uuid(const char *uuid) +{ + return uuid && !bt_uuid_strcmp(uuid, AD_PUBLIC_BROADCAST_UUID); +} + +static const char *ad_public_broadcast_state(void) +{ + if (!ad_is_public_broadcast_uuid(ad.service[AD_TYPE_AD].uuid)) + return NULL; + + if (ad.service[AD_TYPE_AD].data.len != 2) + return NULL; + + if (ad.service[AD_TYPE_AD].data.data[0] == 0x02 && + ad.service[AD_TYPE_AD].data.data[1] == 0x00) + return "sq"; + + if (ad.service[AD_TYPE_AD].data.data[0] == 0x04 && + ad.service[AD_TYPE_AD].data.data[1] == 0x00) + return "hq"; + + return NULL; +} + +void ad_advertise_public_broadcast(DBusConnection *conn, int argc, char *argv[]) +{ + struct ad_data data = { + .data = { 0x00, 0x00 }, + .len = 2, + }; + const char *state; + + if (argc < 2) { + state = ad_public_broadcast_state(); + if (state) + bt_shell_printf("Public Broadcast: %s\n", state); + else + bt_shell_printf("Public Broadcast not set\n"); + + return bt_shell_noninteractive_quit(EXIT_SUCCESS); + } + + if (!strlen(argv[1])) { + bt_shell_printf("Public broadcast value cannot be empty\n"); + return bt_shell_noninteractive_quit(EXIT_FAILURE); + } + + if (!strcasecmp(argv[1], "sq")) + data.data[0] = 0x02; + else if (!strcasecmp(argv[1], "hq")) + data.data[0] = 0x04; + else { + bt_shell_printf("Invalid argument: accepted values are sq or hq\n"); + return bt_shell_noninteractive_quit(EXIT_FAILURE); + } + + ad_clear_service(AD_TYPE_AD); + + ad.service[AD_TYPE_AD].uuid = g_strdup(AD_PUBLIC_BROADCAST_UUID); + ad.service[AD_TYPE_AD].data = data; + + g_dbus_emit_property_changed(conn, AD_PATH, AD_IFACE, + prop_names.service[AD_TYPE_AD]); + + return bt_shell_noninteractive_quit(EXIT_SUCCESS); +} + void ad_advertise_data(DBusConnection *conn, int type, int argc, char *argv[]) { char *endptr = NULL; diff --git a/client/advertising.h b/client/advertising.h index 9d124c7af..8c3fd041b 100644 --- a/client/advertising.h +++ b/client/advertising.h @@ -34,6 +34,8 @@ void ad_advertise_local_appearance(DBusConnection *conn, long int *value); void ad_advertise_duration(DBusConnection *conn, long int *value); void ad_advertise_timeout(DBusConnection *conn, long int *value); void ad_advertise_data(DBusConnection *conn, int type, int argc, char *argv[]); +void ad_advertise_public_broadcast(DBusConnection *conn, int argc, + char *argv[]); void ad_disable_data(DBusConnection *conn, int type); void ad_advertise_discoverable(DBusConnection *conn, dbus_bool_t *value); void ad_advertise_discoverable_timeout(DBusConnection *conn, long int *value); diff --git a/client/main.c b/client/main.c index cb016a579..0e050b373 100644 --- a/client/main.c +++ b/client/main.c @@ -2892,6 +2892,17 @@ static char *ad_generator(const char *text, int state) return argument_generator(text, state, ad_arguments); } +static const char *public_broadcast_arguments[] = { + "sq", + "hq", + NULL +}; + +static char *public_broadcast_generator(const char *text, int state) +{ + return argument_generator(text, state, public_broadcast_arguments); +} + static void cmd_advertise_uuids(int argc, char *argv[]) { ad_advertise_uuids(dbus_conn, AD_TYPE_AD, argc, argv); @@ -2917,6 +2928,11 @@ static void cmd_advertise_data(int argc, char *argv[]) ad_advertise_data(dbus_conn, AD_TYPE_AD, argc, argv); } +static void cmd_advertise_public_broadcast(int argc, char *argv[]) +{ + ad_advertise_public_broadcast(dbus_conn, argc, argv); +} + static void cmd_advertise_sr_uuids(int argc, char *argv[]) { ad_advertise_uuids(dbus_conn, AD_TYPE_SRD, argc, argv); @@ -3600,6 +3616,9 @@ static const struct bt_shell_menu advertise_menu = { "Set/Get advertise manufacturer data" }, { "data", "[type] [data=xx xx ...]", cmd_advertise_data, "Set/Get advertise data" }, + { "public-broadcast", "[sq/hq]", cmd_advertise_public_broadcast, + "Set/Get BLE Audio Public Broadcast Announcement", + public_broadcast_generator }, { "sr-uuids", "[uuid1 uuid2 ...]", cmd_advertise_sr_uuids, "Set/Get scan response uuids" }, { "sr-solicit", "[uuid1 uuid2 ...]", cmd_advertise_sr_solicit, -- 2.53.0 ^ permalink raw reply related [flat|nested] 11+ messages in thread
* RE: client: add public-broadcast advertise helper 2026-03-17 14:26 ` [PATCH BlueZ 1/1] client: Add public-broadcast profile advertise command raghava447 @ 2026-03-17 16:21 ` bluez.test.bot 2026-03-17 18:00 ` [PATCH BlueZ 1/1] client: Add public-broadcast profile advertise command Luiz Augusto von Dentz 1 sibling, 0 replies; 11+ messages in thread From: bluez.test.bot @ 2026-03-17 16:21 UTC (permalink / raw) To: linux-bluetooth, raghavendra.rao [-- Attachment #1: Type: text/plain, Size: 1311 bytes --] This is automated email and please do not reply to this email! Dear submitter, Thank you for submitting the patches to the linux bluetooth mailing list. This is a CI test results with your patch series: PW Link:https://patchwork.kernel.org/project/bluetooth/list/?series=1068070 ---Test result--- Test Summary: CheckPatch PENDING 0.53 seconds GitLint PENDING 0.56 seconds BuildEll PASS 21.05 seconds BluezMake PASS 643.60 seconds MakeCheck PASS 19.19 seconds MakeDistcheck PASS 248.59 seconds CheckValgrind PASS 300.93 seconds CheckSmatch PASS 366.26 seconds bluezmakeextell PASS 191.21 seconds IncrementalBuild PENDING 0.48 seconds ScanBuild PASS 1029.10 seconds Details ############################## Test: CheckPatch - PENDING Desc: Run checkpatch.pl script Output: ############################## Test: GitLint - PENDING Desc: Run gitlint Output: ############################## Test: IncrementalBuild - PENDING Desc: Incremental build with the patches in the series Output: https://github.com/bluez/bluez/pull/1967/checks --- Regards, Linux Bluetooth ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH BlueZ 1/1] client: Add public-broadcast profile advertise command 2026-03-17 14:26 ` [PATCH BlueZ 1/1] client: Add public-broadcast profile advertise command raghava447 2026-03-17 16:21 ` client: add public-broadcast advertise helper bluez.test.bot @ 2026-03-17 18:00 ` Luiz Augusto von Dentz 2026-03-20 8:34 ` raghava447 1 sibling, 1 reply; 11+ messages in thread From: Luiz Augusto von Dentz @ 2026-03-17 18:00 UTC (permalink / raw) To: raghava447; +Cc: linux-bluetooth Hi, On Tue, Mar 17, 2026 at 10:42 AM raghava447 <raghavendra.rao@collabora.com> wrote: > > Add an advertise submenu command for BLE Audio public broadcast profile > advertising in bluetoothctl. > > The new "public-broadcast" command reuses the existing staged AD > ServiceData field for UUID 0x1856. > It supports: > - public-broadcast sq > - public-broadcast hq > - public-broadcast Perhaps we could detect when broadcast is in use and then set the name as the public broadcast name if one is set. > The sq and hq values map to the existing raw service-data payloads: > - sq -> 0x02 0x00 > - hq -> 0x04 0x00 > > When called without arguments, the command reports the currently staged > public-broadcast state if the staged AD service UUID is 0x1856. > Else it reports that public broadcast is not set. > --- > client/advertising.c | 70 ++++++++++++++++++++++++++++++++++++++++++++ > client/advertising.h | 2 ++ > client/main.c | 19 ++++++++++++ > 3 files changed, 91 insertions(+) > > diff --git a/client/advertising.c b/client/advertising.c > index f9df1b855..30b89a404 100644 > --- a/client/advertising.c > +++ b/client/advertising.c > @@ -20,6 +20,8 @@ > #include <string.h> > #include <errno.h> > > +#include "bluetooth/bluetooth.h" > +#include "bluetooth/uuid.h" > #include "gdbus/gdbus.h" > #include "src/shared/util.h" > #include "src/shared/shell.h" > @@ -27,6 +29,7 @@ > > #define AD_PATH "/org/bluez/advertising" > #define AD_IFACE "org.bluez.LEAdvertisement1" > +#define AD_PUBLIC_BROADCAST_UUID "0x1856" > > struct ad_data { > uint8_t data[245]; > @@ -1005,6 +1008,73 @@ static void ad_clear_data(int type) > memset(&ad.data[type], 0, sizeof(ad.data[type])); > } > > +static bool ad_is_public_broadcast_uuid(const char *uuid) > +{ > + return uuid && !bt_uuid_strcmp(uuid, AD_PUBLIC_BROADCAST_UUID); > +} > + > +static const char *ad_public_broadcast_state(void) > +{ > + if (!ad_is_public_broadcast_uuid(ad.service[AD_TYPE_AD].uuid)) > + return NULL; > + > + if (ad.service[AD_TYPE_AD].data.len != 2) > + return NULL; > + > + if (ad.service[AD_TYPE_AD].data.data[0] == 0x02 && > + ad.service[AD_TYPE_AD].data.data[1] == 0x00) > + return "sq"; > + > + if (ad.service[AD_TYPE_AD].data.data[0] == 0x04 && > + ad.service[AD_TYPE_AD].data.data[1] == 0x00) > + return "hq"; > + > + return NULL; > +} > + > +void ad_advertise_public_broadcast(DBusConnection *conn, int argc, char *argv[]) > +{ > + struct ad_data data = { > + .data = { 0x00, 0x00 }, > + .len = 2, > + }; > + const char *state; > + > + if (argc < 2) { > + state = ad_public_broadcast_state(); > + if (state) > + bt_shell_printf("Public Broadcast: %s\n", state); > + else > + bt_shell_printf("Public Broadcast not set\n"); > + > + return bt_shell_noninteractive_quit(EXIT_SUCCESS); > + } > + > + if (!strlen(argv[1])) { > + bt_shell_printf("Public broadcast value cannot be empty\n"); > + return bt_shell_noninteractive_quit(EXIT_FAILURE); > + } > + > + if (!strcasecmp(argv[1], "sq")) > + data.data[0] = 0x02; > + else if (!strcasecmp(argv[1], "hq")) > + data.data[0] = 0x04; > + else { > + bt_shell_printf("Invalid argument: accepted values are sq or hq\n"); > + return bt_shell_noninteractive_quit(EXIT_FAILURE); > + } > + > + ad_clear_service(AD_TYPE_AD); > + > + ad.service[AD_TYPE_AD].uuid = g_strdup(AD_PUBLIC_BROADCAST_UUID); > + ad.service[AD_TYPE_AD].data = data; > + > + g_dbus_emit_property_changed(conn, AD_PATH, AD_IFACE, > + prop_names.service[AD_TYPE_AD]); > + > + return bt_shell_noninteractive_quit(EXIT_SUCCESS); > +} > + > void ad_advertise_data(DBusConnection *conn, int type, int argc, char *argv[]) > { > char *endptr = NULL; > diff --git a/client/advertising.h b/client/advertising.h > index 9d124c7af..8c3fd041b 100644 > --- a/client/advertising.h > +++ b/client/advertising.h > @@ -34,6 +34,8 @@ void ad_advertise_local_appearance(DBusConnection *conn, long int *value); > void ad_advertise_duration(DBusConnection *conn, long int *value); > void ad_advertise_timeout(DBusConnection *conn, long int *value); > void ad_advertise_data(DBusConnection *conn, int type, int argc, char *argv[]); > +void ad_advertise_public_broadcast(DBusConnection *conn, int argc, > + char *argv[]); > void ad_disable_data(DBusConnection *conn, int type); > void ad_advertise_discoverable(DBusConnection *conn, dbus_bool_t *value); > void ad_advertise_discoverable_timeout(DBusConnection *conn, long int *value); > diff --git a/client/main.c b/client/main.c > index cb016a579..0e050b373 100644 > --- a/client/main.c > +++ b/client/main.c > @@ -2892,6 +2892,17 @@ static char *ad_generator(const char *text, int state) > return argument_generator(text, state, ad_arguments); > } > > +static const char *public_broadcast_arguments[] = { > + "sq", > + "hq", > + NULL > +}; > + > +static char *public_broadcast_generator(const char *text, int state) > +{ > + return argument_generator(text, state, public_broadcast_arguments); > +} > + > static void cmd_advertise_uuids(int argc, char *argv[]) > { > ad_advertise_uuids(dbus_conn, AD_TYPE_AD, argc, argv); > @@ -2917,6 +2928,11 @@ static void cmd_advertise_data(int argc, char *argv[]) > ad_advertise_data(dbus_conn, AD_TYPE_AD, argc, argv); > } > > +static void cmd_advertise_public_broadcast(int argc, char *argv[]) > +{ > + ad_advertise_public_broadcast(dbus_conn, argc, argv); > +} > + > static void cmd_advertise_sr_uuids(int argc, char *argv[]) > { > ad_advertise_uuids(dbus_conn, AD_TYPE_SRD, argc, argv); > @@ -3600,6 +3616,9 @@ static const struct bt_shell_menu advertise_menu = { > "Set/Get advertise manufacturer data" }, > { "data", "[type] [data=xx xx ...]", cmd_advertise_data, > "Set/Get advertise data" }, > + { "public-broadcast", "[sq/hq]", cmd_advertise_public_broadcast, > + "Set/Get BLE Audio Public Broadcast Announcement", > + public_broadcast_generator }, > { "sr-uuids", "[uuid1 uuid2 ...]", cmd_advertise_sr_uuids, > "Set/Get scan response uuids" }, > { "sr-solicit", "[uuid1 uuid2 ...]", cmd_advertise_sr_solicit, > -- > 2.53.0 > > -- Luiz Augusto von Dentz ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH BlueZ 1/1] client: Add public-broadcast profile advertise command 2026-03-17 18:00 ` [PATCH BlueZ 1/1] client: Add public-broadcast profile advertise command Luiz Augusto von Dentz @ 2026-03-20 8:34 ` raghava447 0 siblings, 0 replies; 11+ messages in thread From: raghava447 @ 2026-03-20 8:34 UTC (permalink / raw) To: linux-bluetooth; +Cc: Raghavendra Rao From: Raghavendra Rao <raghavendra.rao@collabora.com> Thanks for the suggestion. My intention with this patch was to keep it limited to the semantic helper for the Public Broadcast Announcement service. I can work on a follow-up patch to integrate the broadcast name handling so that when public broadcast is in use, a configured name can also be staged as the Public Broadcast Name AD field. ^ permalink raw reply [flat|nested] 11+ messages in thread
* [PATCH BlueZ v2 0/2] client: Add public broadcast advertising support 2026-03-17 14:26 [PATCH BlueZ 0/1] client: add public-broadcast advertise helper raghava447 2026-03-17 14:26 ` [PATCH BlueZ 1/1] client: Add public-broadcast profile advertise command raghava447 @ 2026-04-24 14:18 ` raghu447 2026-04-24 14:18 ` [PATCH v2 1/2] client: add public-broadcast advertising command raghu447 2026-04-24 14:18 ` [PATCH v2 2/2] client: make advertise.name use public broadcast name raghu447 1 sibling, 2 replies; 11+ messages in thread From: raghu447 @ 2026-04-24 14:18 UTC (permalink / raw) To: linux-bluetooth; +Cc: raghavendra From: raghavendra <raghavendra.rao@collabora.com> This series adds support for staging Public Broadcast Announcement advertising data used during LE Audio public broadcast qualification. Patch 1 adds a public-broadcast command to the advertise submenu. The command stages Public Broadcast Announcement ServiceData using UUID 0x1856 and supports SQ and HQ presets. Patch 2 makes advertise.name broadcast-aware. When Public Broadcast Announcement ServiceData is staged, advertise.name sets and gets the Public Broadcast Name using AD type 0x30. Outside public broadcast mode, the existing LocalName behavior is preserved. raghavendra (2): client: add public-broadcast advertising command client: make advertise.name use public broadcast name client/advertising.c | 145 +++++++++++++++++++++++++++++++++++++++++++ client/advertising.h | 2 + client/main.c | 19 ++++++ 3 files changed, 166 insertions(+) -- 2.53.0 ^ permalink raw reply [flat|nested] 11+ messages in thread
* [PATCH v2 1/2] client: add public-broadcast advertising command 2026-04-24 14:18 ` [PATCH BlueZ v2 0/2] client: Add public broadcast advertising support raghu447 @ 2026-04-24 14:18 ` raghu447 2026-04-24 14:41 ` Luiz Augusto von Dentz ` (2 more replies) 2026-04-24 14:18 ` [PATCH v2 2/2] client: make advertise.name use public broadcast name raghu447 1 sibling, 3 replies; 11+ messages in thread From: raghu447 @ 2026-04-24 14:18 UTC (permalink / raw) To: linux-bluetooth; +Cc: raghavendra From: raghavendra <raghavendra.rao@collabora.com> --- client/advertising.c | 71 ++++++++++++++++++++++++++++++++++++++++++++ client/advertising.h | 2 ++ client/main.c | 19 ++++++++++++ 3 files changed, 92 insertions(+) diff --git a/client/advertising.c b/client/advertising.c index f9df1b855..fc51f8193 100644 --- a/client/advertising.c +++ b/client/advertising.c @@ -20,6 +20,9 @@ #include <string.h> #include <errno.h> +#include "bluetooth/bluetooth.h" +#include "bluetooth/uuid.h" + #include "gdbus/gdbus.h" #include "src/shared/util.h" #include "src/shared/shell.h" @@ -27,6 +30,7 @@ #define AD_PATH "/org/bluez/advertising" #define AD_IFACE "org.bluez.LEAdvertisement1" +#define AD_PUBLIC_BROADCAST_UUID "0x1856" struct ad_data { uint8_t data[245]; @@ -1005,6 +1009,73 @@ static void ad_clear_data(int type) memset(&ad.data[type], 0, sizeof(ad.data[type])); } +static bool ad_is_public_broadcast_uuid(const char *uuid) +{ + return uuid && !bt_uuid_strcmp(uuid, AD_PUBLIC_BROADCAST_UUID); +} + +static const char *ad_public_broadcast_state(void) +{ + if (!ad_is_public_broadcast_uuid(ad.service[AD_TYPE_AD].uuid)) + return NULL; + + if (ad.service[AD_TYPE_AD].data.len != 2) + return NULL; + + if (ad.service[AD_TYPE_AD].data.data[0] == 0x02 && + ad.service[AD_TYPE_AD].data.data[1] == 0x00) + return "sq"; + + if (ad.service[AD_TYPE_AD].data.data[0] == 0x04 && + ad.service[AD_TYPE_AD].data.data[1] == 0x00) + return "hq"; + + return NULL; +} + +void ad_advertise_public_broadcast(DBusConnection *conn, int argc, char *argv[]) +{ + struct ad_data data = { + .data = { 0x00, 0x00 }, + .len = 2, + }; + const char *state; + + if (argc < 2) { + state = ad_public_broadcast_state(); + if (state) + bt_shell_printf("Public Broadcast: %s\n", state); + else + bt_shell_printf("Public Broadcast not set\n"); + + return bt_shell_noninteractive_quit(EXIT_SUCCESS); + } + + if (!strlen(argv[1])) { + bt_shell_printf("Public broadcast value cannot be empty\n"); + return bt_shell_noninteractive_quit(EXIT_FAILURE); + } + + if (!strcasecmp(argv[1], "sq")) + data.data[0] = 0x02; + else if (!strcasecmp(argv[1], "hq")) + data.data[0] = 0x04; + else { + bt_shell_printf("Invalid argument: accepted values are sq or hq\n"); + return bt_shell_noninteractive_quit(EXIT_FAILURE); + } + + ad_clear_service(AD_TYPE_AD); + + ad.service[AD_TYPE_AD].uuid = g_strdup(AD_PUBLIC_BROADCAST_UUID); + ad.service[AD_TYPE_AD].data = data; + + g_dbus_emit_property_changed(conn, AD_PATH, AD_IFACE, + prop_names.service[AD_TYPE_AD]); + + return bt_shell_noninteractive_quit(EXIT_SUCCESS); +} + void ad_advertise_data(DBusConnection *conn, int type, int argc, char *argv[]) { char *endptr = NULL; diff --git a/client/advertising.h b/client/advertising.h index 9d124c7af..98f1bc524 100644 --- a/client/advertising.h +++ b/client/advertising.h @@ -34,6 +34,8 @@ void ad_advertise_local_appearance(DBusConnection *conn, long int *value); void ad_advertise_duration(DBusConnection *conn, long int *value); void ad_advertise_timeout(DBusConnection *conn, long int *value); void ad_advertise_data(DBusConnection *conn, int type, int argc, char *argv[]); +void ad_advertise_public_broadcast(DBusConnection *conn, int argc, + char *argv[]); void ad_disable_data(DBusConnection *conn, int type); void ad_advertise_discoverable(DBusConnection *conn, dbus_bool_t *value); void ad_advertise_discoverable_timeout(DBusConnection *conn, long int *value); diff --git a/client/main.c b/client/main.c index 57fa13888..5fd21c80d 100644 --- a/client/main.c +++ b/client/main.c @@ -2920,6 +2920,17 @@ static char *scan_generator(const char *text, int state) return argument_generator(text, state, scan_arguments); } +static const char *public_broadcast_arguments[] = { + "sq", + "hq", + NULL +}; + +static char *public_broadcast_generator(const char *text, int state) +{ + return argument_generator(text, state, public_broadcast_arguments); +} + static void cmd_advertise(int argc, char *argv[]) { dbus_bool_t enable; @@ -2970,6 +2981,11 @@ static void cmd_advertise_data(int argc, char *argv[]) ad_advertise_data(dbus_conn, AD_TYPE_AD, argc, argv); } +static void cmd_advertise_public_broadcast(int argc, char *argv[]) +{ + ad_advertise_public_broadcast(dbus_conn, argc, argv); +} + static void cmd_advertise_sr_uuids(int argc, char *argv[]) { ad_advertise_uuids(dbus_conn, AD_TYPE_SRD, argc, argv); @@ -3653,6 +3669,9 @@ static const struct bt_shell_menu advertise_menu = { "Set/Get advertise manufacturer data" }, { "data", "[type] [data=xx xx ...]", cmd_advertise_data, "Set/Get advertise data" }, + { "public-broadcast", "[sq/hq]", cmd_advertise_public_broadcast, + "Set/Get BLE Audio Public Broadcast Announcement", + public_broadcast_generator }, { "sr-uuids", "[uuid1 uuid2 ...]", cmd_advertise_sr_uuids, "Set/Get scan response uuids" }, { "sr-solicit", "[uuid1 uuid2 ...]", cmd_advertise_sr_solicit, -- 2.53.0 ^ permalink raw reply related [flat|nested] 11+ messages in thread
* Re: [PATCH v2 1/2] client: add public-broadcast advertising command 2026-04-24 14:18 ` [PATCH v2 1/2] client: add public-broadcast advertising command raghu447 @ 2026-04-24 14:41 ` Luiz Augusto von Dentz 2026-04-24 14:44 ` Luiz Augusto von Dentz 2026-04-24 16:23 ` client: Add public broadcast advertising support bluez.test.bot 2 siblings, 0 replies; 11+ messages in thread From: Luiz Augusto von Dentz @ 2026-04-24 14:41 UTC (permalink / raw) To: raghu447; +Cc: linux-bluetooth Hi Raghu, On Fri, Apr 24, 2026 at 10:19 AM raghu447 <raghavendra.rao@collabora.com> wrote: > > From: raghavendra <raghavendra.rao@collabora.com> > > --- > client/advertising.c | 71 ++++++++++++++++++++++++++++++++++++++++++++ > client/advertising.h | 2 ++ > client/main.c | 19 ++++++++++++ > 3 files changed, 92 insertions(+) > > diff --git a/client/advertising.c b/client/advertising.c > index f9df1b855..fc51f8193 100644 > --- a/client/advertising.c > +++ b/client/advertising.c > @@ -20,6 +20,9 @@ > #include <string.h> > #include <errno.h> > > +#include "bluetooth/bluetooth.h" > +#include "bluetooth/uuid.h" > + > #include "gdbus/gdbus.h" > #include "src/shared/util.h" > #include "src/shared/shell.h" > @@ -27,6 +30,7 @@ > > #define AD_PATH "/org/bluez/advertising" > #define AD_IFACE "org.bluez.LEAdvertisement1" > +#define AD_PUBLIC_BROADCAST_UUID "0x1856" > > struct ad_data { > uint8_t data[245]; > @@ -1005,6 +1009,73 @@ static void ad_clear_data(int type) > memset(&ad.data[type], 0, sizeof(ad.data[type])); > } > > +static bool ad_is_public_broadcast_uuid(const char *uuid) > +{ > + return uuid && !bt_uuid_strcmp(uuid, AD_PUBLIC_BROADCAST_UUID); > +} > + > +static const char *ad_public_broadcast_state(void) > +{ > + if (!ad_is_public_broadcast_uuid(ad.service[AD_TYPE_AD].uuid)) > + return NULL; > + > + if (ad.service[AD_TYPE_AD].data.len != 2) > + return NULL; > + > + if (ad.service[AD_TYPE_AD].data.data[0] == 0x02 && > + ad.service[AD_TYPE_AD].data.data[1] == 0x00) > + return "sq"; > + > + if (ad.service[AD_TYPE_AD].data.data[0] == 0x04 && > + ad.service[AD_TYPE_AD].data.data[1] == 0x00) > + return "hq"; > + > + return NULL; > +} > + > +void ad_advertise_public_broadcast(DBusConnection *conn, int argc, char *argv[]) > +{ > + struct ad_data data = { > + .data = { 0x00, 0x00 }, > + .len = 2, > + }; > + const char *state; > + > + if (argc < 2) { > + state = ad_public_broadcast_state(); > + if (state) > + bt_shell_printf("Public Broadcast: %s\n", state); > + else > + bt_shell_printf("Public Broadcast not set\n"); > + > + return bt_shell_noninteractive_quit(EXIT_SUCCESS); > + } > + > + if (!strlen(argv[1])) { > + bt_shell_printf("Public broadcast value cannot be empty\n"); > + return bt_shell_noninteractive_quit(EXIT_FAILURE); > + } > + > + if (!strcasecmp(argv[1], "sq")) > + data.data[0] = 0x02; > + else if (!strcasecmp(argv[1], "hq")) > + data.data[0] = 0x04; Are these bits? If they are, let's use BIT(1) and BIT(2). > + else { > + bt_shell_printf("Invalid argument: accepted values are sq or hq\n"); > + return bt_shell_noninteractive_quit(EXIT_FAILURE); > + } > + > + ad_clear_service(AD_TYPE_AD); > + > + ad.service[AD_TYPE_AD].uuid = g_strdup(AD_PUBLIC_BROADCAST_UUID); > + ad.service[AD_TYPE_AD].data = data; > + > + g_dbus_emit_property_changed(conn, AD_PATH, AD_IFACE, > + prop_names.service[AD_TYPE_AD]); > + > + return bt_shell_noninteractive_quit(EXIT_SUCCESS); > +} > + > void ad_advertise_data(DBusConnection *conn, int type, int argc, char *argv[]) > { > char *endptr = NULL; > diff --git a/client/advertising.h b/client/advertising.h > index 9d124c7af..98f1bc524 100644 > --- a/client/advertising.h > +++ b/client/advertising.h > @@ -34,6 +34,8 @@ void ad_advertise_local_appearance(DBusConnection *conn, long int *value); > void ad_advertise_duration(DBusConnection *conn, long int *value); > void ad_advertise_timeout(DBusConnection *conn, long int *value); > void ad_advertise_data(DBusConnection *conn, int type, int argc, char *argv[]); > +void ad_advertise_public_broadcast(DBusConnection *conn, int argc, > + char *argv[]); > void ad_disable_data(DBusConnection *conn, int type); > void ad_advertise_discoverable(DBusConnection *conn, dbus_bool_t *value); > void ad_advertise_discoverable_timeout(DBusConnection *conn, long int *value); > diff --git a/client/main.c b/client/main.c > index 57fa13888..5fd21c80d 100644 > --- a/client/main.c > +++ b/client/main.c > @@ -2920,6 +2920,17 @@ static char *scan_generator(const char *text, int state) > return argument_generator(text, state, scan_arguments); > } > > +static const char *public_broadcast_arguments[] = { > + "sq", > + "hq", > + NULL > +}; I wonder if we should have better names for these, Android seems to have a setting for compatibility, not sure exactly what it means in terms of adv settings though. > + > +static char *public_broadcast_generator(const char *text, int state) > +{ > + return argument_generator(text, state, public_broadcast_arguments); > +} > + > static void cmd_advertise(int argc, char *argv[]) > { > dbus_bool_t enable; > @@ -2970,6 +2981,11 @@ static void cmd_advertise_data(int argc, char *argv[]) > ad_advertise_data(dbus_conn, AD_TYPE_AD, argc, argv); > } > > +static void cmd_advertise_public_broadcast(int argc, char *argv[]) > +{ > + ad_advertise_public_broadcast(dbus_conn, argc, argv); > +} > + > static void cmd_advertise_sr_uuids(int argc, char *argv[]) > { > ad_advertise_uuids(dbus_conn, AD_TYPE_SRD, argc, argv); > @@ -3653,6 +3669,9 @@ static const struct bt_shell_menu advertise_menu = { > "Set/Get advertise manufacturer data" }, > { "data", "[type] [data=xx xx ...]", cmd_advertise_data, > "Set/Get advertise data" }, > + { "public-broadcast", "[sq/hq]", cmd_advertise_public_broadcast, > + "Set/Get BLE Audio Public Broadcast Announcement", > + public_broadcast_generator }, > { "sr-uuids", "[uuid1 uuid2 ...]", cmd_advertise_sr_uuids, > "Set/Get scan response uuids" }, > { "sr-solicit", "[uuid1 uuid2 ...]", cmd_advertise_sr_solicit, > -- > 2.53.0 > > -- Luiz Augusto von Dentz ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v2 1/2] client: add public-broadcast advertising command 2026-04-24 14:18 ` [PATCH v2 1/2] client: add public-broadcast advertising command raghu447 2026-04-24 14:41 ` Luiz Augusto von Dentz @ 2026-04-24 14:44 ` Luiz Augusto von Dentz 2026-04-24 16:23 ` client: Add public broadcast advertising support bluez.test.bot 2 siblings, 0 replies; 11+ messages in thread From: Luiz Augusto von Dentz @ 2026-04-24 14:44 UTC (permalink / raw) To: raghu447; +Cc: linux-bluetooth Hi, On Fri, Apr 24, 2026 at 10:19 AM raghu447 <raghavendra.rao@collabora.com> wrote: > > From: raghavendra <raghavendra.rao@collabora.com> > > --- > client/advertising.c | 71 ++++++++++++++++++++++++++++++++++++++++++++ > client/advertising.h | 2 ++ > client/main.c | 19 ++++++++++++ > 3 files changed, 92 insertions(+) > > diff --git a/client/advertising.c b/client/advertising.c > index f9df1b855..fc51f8193 100644 > --- a/client/advertising.c > +++ b/client/advertising.c > @@ -20,6 +20,9 @@ > #include <string.h> > #include <errno.h> > > +#include "bluetooth/bluetooth.h" > +#include "bluetooth/uuid.h" > + > #include "gdbus/gdbus.h" > #include "src/shared/util.h" > #include "src/shared/shell.h" > @@ -27,6 +30,7 @@ > > #define AD_PATH "/org/bluez/advertising" > #define AD_IFACE "org.bluez.LEAdvertisement1" > +#define AD_PUBLIC_BROADCAST_UUID "0x1856" Forgot to mention, we don't seem to have a decoder for 0x1856, would you like to create one so we can actually see what is being programmed on btmon? That will probably require extending the monitor/packet.c:service_data_decoders > struct ad_data { > uint8_t data[245]; > @@ -1005,6 +1009,73 @@ static void ad_clear_data(int type) > memset(&ad.data[type], 0, sizeof(ad.data[type])); > } > > +static bool ad_is_public_broadcast_uuid(const char *uuid) > +{ > + return uuid && !bt_uuid_strcmp(uuid, AD_PUBLIC_BROADCAST_UUID); > +} > + > +static const char *ad_public_broadcast_state(void) > +{ > + if (!ad_is_public_broadcast_uuid(ad.service[AD_TYPE_AD].uuid)) > + return NULL; > + > + if (ad.service[AD_TYPE_AD].data.len != 2) > + return NULL; > + > + if (ad.service[AD_TYPE_AD].data.data[0] == 0x02 && > + ad.service[AD_TYPE_AD].data.data[1] == 0x00) > + return "sq"; > + > + if (ad.service[AD_TYPE_AD].data.data[0] == 0x04 && > + ad.service[AD_TYPE_AD].data.data[1] == 0x00) > + return "hq"; > + > + return NULL; > +} > + > +void ad_advertise_public_broadcast(DBusConnection *conn, int argc, char *argv[]) > +{ > + struct ad_data data = { > + .data = { 0x00, 0x00 }, > + .len = 2, > + }; > + const char *state; > + > + if (argc < 2) { > + state = ad_public_broadcast_state(); > + if (state) > + bt_shell_printf("Public Broadcast: %s\n", state); > + else > + bt_shell_printf("Public Broadcast not set\n"); > + > + return bt_shell_noninteractive_quit(EXIT_SUCCESS); > + } > + > + if (!strlen(argv[1])) { > + bt_shell_printf("Public broadcast value cannot be empty\n"); > + return bt_shell_noninteractive_quit(EXIT_FAILURE); > + } > + > + if (!strcasecmp(argv[1], "sq")) > + data.data[0] = 0x02; > + else if (!strcasecmp(argv[1], "hq")) > + data.data[0] = 0x04; > + else { > + bt_shell_printf("Invalid argument: accepted values are sq or hq\n"); > + return bt_shell_noninteractive_quit(EXIT_FAILURE); > + } > + > + ad_clear_service(AD_TYPE_AD); > + > + ad.service[AD_TYPE_AD].uuid = g_strdup(AD_PUBLIC_BROADCAST_UUID); > + ad.service[AD_TYPE_AD].data = data; > + > + g_dbus_emit_property_changed(conn, AD_PATH, AD_IFACE, > + prop_names.service[AD_TYPE_AD]); > + > + return bt_shell_noninteractive_quit(EXIT_SUCCESS); > +} > + > void ad_advertise_data(DBusConnection *conn, int type, int argc, char *argv[]) > { > char *endptr = NULL; > diff --git a/client/advertising.h b/client/advertising.h > index 9d124c7af..98f1bc524 100644 > --- a/client/advertising.h > +++ b/client/advertising.h > @@ -34,6 +34,8 @@ void ad_advertise_local_appearance(DBusConnection *conn, long int *value); > void ad_advertise_duration(DBusConnection *conn, long int *value); > void ad_advertise_timeout(DBusConnection *conn, long int *value); > void ad_advertise_data(DBusConnection *conn, int type, int argc, char *argv[]); > +void ad_advertise_public_broadcast(DBusConnection *conn, int argc, > + char *argv[]); > void ad_disable_data(DBusConnection *conn, int type); > void ad_advertise_discoverable(DBusConnection *conn, dbus_bool_t *value); > void ad_advertise_discoverable_timeout(DBusConnection *conn, long int *value); > diff --git a/client/main.c b/client/main.c > index 57fa13888..5fd21c80d 100644 > --- a/client/main.c > +++ b/client/main.c > @@ -2920,6 +2920,17 @@ static char *scan_generator(const char *text, int state) > return argument_generator(text, state, scan_arguments); > } > > +static const char *public_broadcast_arguments[] = { > + "sq", > + "hq", > + NULL > +}; > + > +static char *public_broadcast_generator(const char *text, int state) > +{ > + return argument_generator(text, state, public_broadcast_arguments); > +} > + > static void cmd_advertise(int argc, char *argv[]) > { > dbus_bool_t enable; > @@ -2970,6 +2981,11 @@ static void cmd_advertise_data(int argc, char *argv[]) > ad_advertise_data(dbus_conn, AD_TYPE_AD, argc, argv); > } > > +static void cmd_advertise_public_broadcast(int argc, char *argv[]) > +{ > + ad_advertise_public_broadcast(dbus_conn, argc, argv); > +} > + > static void cmd_advertise_sr_uuids(int argc, char *argv[]) > { > ad_advertise_uuids(dbus_conn, AD_TYPE_SRD, argc, argv); > @@ -3653,6 +3669,9 @@ static const struct bt_shell_menu advertise_menu = { > "Set/Get advertise manufacturer data" }, > { "data", "[type] [data=xx xx ...]", cmd_advertise_data, > "Set/Get advertise data" }, > + { "public-broadcast", "[sq/hq]", cmd_advertise_public_broadcast, > + "Set/Get BLE Audio Public Broadcast Announcement", > + public_broadcast_generator }, > { "sr-uuids", "[uuid1 uuid2 ...]", cmd_advertise_sr_uuids, > "Set/Get scan response uuids" }, > { "sr-solicit", "[uuid1 uuid2 ...]", cmd_advertise_sr_solicit, > -- > 2.53.0 > > -- Luiz Augusto von Dentz ^ permalink raw reply [flat|nested] 11+ messages in thread
* RE: client: Add public broadcast advertising support 2026-04-24 14:18 ` [PATCH v2 1/2] client: add public-broadcast advertising command raghu447 2026-04-24 14:41 ` Luiz Augusto von Dentz 2026-04-24 14:44 ` Luiz Augusto von Dentz @ 2026-04-24 16:23 ` bluez.test.bot 2 siblings, 0 replies; 11+ messages in thread From: bluez.test.bot @ 2026-04-24 16:23 UTC (permalink / raw) To: linux-bluetooth, raghavendra.rao [-- Attachment #1: Type: text/plain, Size: 12505 bytes --] This is automated email and please do not reply to this email! Dear submitter, Thank you for submitting the patches to the linux bluetooth mailing list. This is a CI test results with your patch series: PW Link:https://patchwork.kernel.org/project/bluetooth/list/?series=1085227 ---Test result--- Test Summary: CheckPatch FAIL 1.30 seconds GitLint PASS 0.67 seconds BuildEll PASS 19.93 seconds BluezMake PASS 604.72 seconds CheckSmatch PASS 318.76 seconds bluezmakeextell PASS 162.72 seconds IncrementalBuild PASS 597.91 seconds ScanBuild PASS 910.00 seconds Details ############################## Test: CheckPatch - FAIL Desc: Run checkpatch.pl script Output: [v2,1/2] client: add public-broadcast advertising command WARNING:LEADING_SPACE: please, no spaces at the start of a line #106: FILE: client/advertising.c:1014: + return uuid && !bt_uuid_strcmp(uuid, AD_PUBLIC_BROADCAST_UUID);$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #111: FILE: client/advertising.c:1019: + if (!ad_is_public_broadcast_uuid(ad.service[AD_TYPE_AD].uuid))$ ERROR:CODE_INDENT: code indent should use tabs where possible #112: FILE: client/advertising.c:1020: + return NULL;$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #112: FILE: client/advertising.c:1020: + return NULL;$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #114: FILE: client/advertising.c:1022: + if (ad.service[AD_TYPE_AD].data.len != 2)$ ERROR:CODE_INDENT: code indent should use tabs where possible #115: FILE: client/advertising.c:1023: + return NULL;$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #115: FILE: client/advertising.c:1023: + return NULL;$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #117: FILE: client/advertising.c:1025: + if (ad.service[AD_TYPE_AD].data.data[0] == 0x02 &&$ ERROR:CODE_INDENT: code indent should use tabs where possible #118: FILE: client/advertising.c:1026: + ad.service[AD_TYPE_AD].data.data[1] == 0x00)$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #118: FILE: client/advertising.c:1026: + ad.service[AD_TYPE_AD].data.data[1] == 0x00)$ ERROR:CODE_INDENT: code indent should use tabs where possible #119: FILE: client/advertising.c:1027: + return "sq";$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #119: FILE: client/advertising.c:1027: + return "sq";$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #121: FILE: client/advertising.c:1029: + if (ad.service[AD_TYPE_AD].data.data[0] == 0x04 &&$ ERROR:CODE_INDENT: code indent should use tabs where possible #122: FILE: client/advertising.c:1030: + ad.service[AD_TYPE_AD].data.data[1] == 0x00)$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #122: FILE: client/advertising.c:1030: + ad.service[AD_TYPE_AD].data.data[1] == 0x00)$ ERROR:CODE_INDENT: code indent should use tabs where possible #123: FILE: client/advertising.c:1031: + return "hq";$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #123: FILE: client/advertising.c:1031: + return "hq";$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #125: FILE: client/advertising.c:1033: + return NULL;$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #130: FILE: client/advertising.c:1038: + struct ad_data data = {$ ERROR:CODE_INDENT: code indent should use tabs where possible #131: FILE: client/advertising.c:1039: + .data = { 0x00, 0x00 },$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #131: FILE: client/advertising.c:1039: + .data = { 0x00, 0x00 },$ ERROR:CODE_INDENT: code indent should use tabs where possible #132: FILE: client/advertising.c:1040: + .len = 2,$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #132: FILE: client/advertising.c:1040: + .len = 2,$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #133: FILE: client/advertising.c:1041: + };$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #134: FILE: client/advertising.c:1042: + const char *state;$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #136: FILE: client/advertising.c:1044: + if (argc < 2) {$ ERROR:CODE_INDENT: code indent should use tabs where possible #137: FILE: client/advertising.c:1045: + state = ad_public_broadcast_state();$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #137: FILE: client/advertising.c:1045: + state = ad_public_broadcast_state();$ WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (16, 12) #138: FILE: client/advertising.c:1046: + if (state) + bt_shell_printf("Public Broadcast: %s\n", state); ERROR:CODE_INDENT: code indent should use tabs where possible #139: FILE: client/advertising.c:1047: + bt_shell_printf("Public Broadcast: %s\n", state);$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #139: FILE: client/advertising.c:1047: + bt_shell_printf("Public Broadcast: %s\n", state);$ ERROR:CODE_INDENT: code indent should use tabs where possible #140: FILE: client/advertising.c:1048: + else$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #140: FILE: client/advertising.c:1048: + else$ WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (8, 12) #140: FILE: client/advertising.c:1048: + else + bt_shell_printf("Public Broadcast not set\n"); ERROR:CODE_INDENT: code indent should use tabs where possible #141: FILE: client/advertising.c:1049: + bt_shell_printf("Public Broadcast not set\n");$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #141: FILE: client/advertising.c:1049: + bt_shell_printf("Public Broadcast not set\n");$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #144: FILE: client/advertising.c:1052: + }$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #146: FILE: client/advertising.c:1054: + if (!strlen(argv[1])) {$ ERROR:CODE_INDENT: code indent should use tabs where possible #147: FILE: client/advertising.c:1055: + bt_shell_printf("Public broadcast value cannot be empty\n");$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #147: FILE: client/advertising.c:1055: + bt_shell_printf("Public broadcast value cannot be empty\n");$ ERROR:CODE_INDENT: code indent should use tabs where possible #148: FILE: client/advertising.c:1056: + return bt_shell_noninteractive_quit(EXIT_FAILURE);$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #148: FILE: client/advertising.c:1056: + return bt_shell_noninteractive_quit(EXIT_FAILURE);$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #149: FILE: client/advertising.c:1057: + }$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #151: FILE: client/advertising.c:1059: + if (!strcasecmp(argv[1], "sq"))$ ERROR:CODE_INDENT: code indent should use tabs where possible #152: FILE: client/advertising.c:1060: + data.data[0] = 0x02;$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #152: FILE: client/advertising.c:1060: + data.data[0] = 0x02;$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #153: FILE: client/advertising.c:1061: + else if (!strcasecmp(argv[1], "hq"))$ ERROR:CODE_INDENT: code indent should use tabs where possible #154: FILE: client/advertising.c:1062: + data.data[0] = 0x04;$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #154: FILE: client/advertising.c:1062: + data.data[0] = 0x04;$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #155: FILE: client/advertising.c:1063: + else {$ ERROR:CODE_INDENT: code indent should use tabs where possible #156: FILE: client/advertising.c:1064: + bt_shell_printf("Invalid argument: accepted values are sq or hq\n");$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #156: FILE: client/advertising.c:1064: + bt_shell_printf("Invalid argument: accepted values are sq or hq\n");$ ERROR:CODE_INDENT: code indent should use tabs where possible #157: FILE: client/advertising.c:1065: + return bt_shell_noninteractive_quit(EXIT_FAILURE);$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #157: FILE: client/advertising.c:1065: + return bt_shell_noninteractive_quit(EXIT_FAILURE);$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #158: FILE: client/advertising.c:1066: + }$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #160: FILE: client/advertising.c:1068: + ad_clear_service(AD_TYPE_AD);$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #162: FILE: client/advertising.c:1070: + ad.service[AD_TYPE_AD].uuid = g_strdup(AD_PUBLIC_BROADCAST_UUID);$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #163: FILE: client/advertising.c:1071: + ad.service[AD_TYPE_AD].data = data;$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #165: FILE: client/advertising.c:1073: + g_dbus_emit_property_changed(conn, AD_PATH, AD_IFACE,$ ERROR:CODE_INDENT: code indent should use tabs where possible #166: FILE: client/advertising.c:1074: + prop_names.service[AD_TYPE_AD]);$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #166: FILE: client/advertising.c:1074: + prop_names.service[AD_TYPE_AD]);$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #168: FILE: client/advertising.c:1076: + return bt_shell_noninteractive_quit(EXIT_SUCCESS);$ ERROR:CODE_INDENT: code indent should use tabs where possible #183: FILE: client/advertising.h:38: + char *argv[]);$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #183: FILE: client/advertising.h:38: + char *argv[]);$ WARNING:STATIC_CONST_CHAR_ARRAY: static const char * array should probably be static const char * const #195: FILE: client/main.c:2923: +static const char *public_broadcast_arguments[] = { WARNING:LEADING_SPACE: please, no spaces at the start of a line #196: FILE: client/main.c:2924: + "sq",$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #197: FILE: client/main.c:2925: + "hq",$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #198: FILE: client/main.c:2926: + NULL$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #203: FILE: client/main.c:2931: + return argument_generator(text, state, public_broadcast_arguments);$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #215: FILE: client/main.c:2986: + ad_advertise_public_broadcast(dbus_conn, argc, argv);$ ERROR:CODE_INDENT: code indent should use tabs where possible #226: FILE: client/main.c:3673: + "Set/Get BLE Audio Public Broadcast Announcement",$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #226: FILE: client/main.c:3673: + "Set/Get BLE Audio Public Broadcast Announcement",$ ERROR:CODE_INDENT: code indent should use tabs where possible #227: FILE: client/main.c:3674: + public_broadcast_generator },$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #227: FILE: client/main.c:3674: + public_broadcast_generator },$ /github/workspace/src/patch/14537907.patch total: 22 errors, 52 warnings, 134 lines checked NOTE: For some of the reported defects, checkpatch may be able to mechanically convert to the typical style using --fix or --fix-inplace. NOTE: Whitespace errors detected. You may wish to use scripts/cleanpatch or scripts/cleanfile /github/workspace/src/patch/14537907.patch has style problems, please review. NOTE: Ignored message types: COMMIT_MESSAGE COMPLEX_MACRO CONST_STRUCT FILE_PATH_CHANGES MISSING_SIGN_OFF PREFER_PACKED SPDX_LICENSE_TAG SPLIT_STRING SSCANF_TO_KSTRTO NOTE: If any of the errors are false positives, please report them to the maintainer, see CHECKPATCH in MAINTAINERS. https://github.com/bluez/bluez/pull/2069 --- Regards, Linux Bluetooth ^ permalink raw reply [flat|nested] 11+ messages in thread
* [PATCH v2 2/2] client: make advertise.name use public broadcast name 2026-04-24 14:18 ` [PATCH BlueZ v2 0/2] client: Add public broadcast advertising support raghu447 2026-04-24 14:18 ` [PATCH v2 1/2] client: add public-broadcast advertising command raghu447 @ 2026-04-24 14:18 ` raghu447 1 sibling, 0 replies; 11+ messages in thread From: raghu447 @ 2026-04-24 14:18 UTC (permalink / raw) To: linux-bluetooth; +Cc: raghavendra From: raghavendra <raghavendra.rao@collabora.com> --- client/advertising.c | 74 ++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 74 insertions(+) diff --git a/client/advertising.c b/client/advertising.c index fc51f8193..6921bedec 100644 --- a/client/advertising.c +++ b/client/advertising.c @@ -31,6 +31,7 @@ #define AD_PATH "/org/bluez/advertising" #define AD_IFACE "org.bluez.LEAdvertisement1" #define AD_PUBLIC_BROADCAST_UUID "0x1856" +#define AD_TYPE_PUBLIC_BROADCAST_NAME 0x30 struct ad_data { uint8_t data[245]; @@ -1014,6 +1015,62 @@ static bool ad_is_public_broadcast_uuid(const char *uuid) return uuid && !bt_uuid_strcmp(uuid, AD_PUBLIC_BROADCAST_UUID); } +static bool ad_broadcast_in_use(void) +{ + return ad_is_public_broadcast_uuid(ad.service[AD_TYPE_AD].uuid); +} + +static bool ad_has_public_broadcast_name(void) +{ + return ad.data[AD_TYPE_AD].valid && + ad.data[AD_TYPE_AD].type == AD_TYPE_PUBLIC_BROADCAST_NAME && + ad.data[AD_TYPE_AD].data.len > 0; +} + +static void ad_print_public_broadcast_name(void) +{ + if (!ad_has_public_broadcast_name()) { + bt_shell_printf("Public Broadcast Name not set\n"); + return; + } + + bt_shell_printf("Public Broadcast Name: %.*s\n", + (int) ad.data[AD_TYPE_AD].data.len, + (char *) ad.data[AD_TYPE_AD].data.data); +} + +static bool ad_set_public_broadcast_name(DBusConnection *conn, + const char *name) +{ + struct ad_data data; + size_t len; + + if (!name || !strlen(name)) { + bt_shell_printf("Public Broadcast Name cannot be empty\n"); + return false; + } + + len = strlen(name); + if (len > sizeof(data.data)) { + bt_shell_printf("Public Broadcast Name too long\n"); + return false; + } + + memset(&data, 0, sizeof(data)); + memcpy(data.data, name, len); + data.len = len; + + ad_clear_data(AD_TYPE_AD); + ad.data[AD_TYPE_AD].valid = true; + ad.data[AD_TYPE_AD].type = AD_TYPE_PUBLIC_BROADCAST_NAME; + ad.data[AD_TYPE_AD].data = data; + + g_dbus_emit_property_changed(conn, AD_PATH, AD_IFACE, + prop_names.data[AD_TYPE_AD]); + + return true; +} + static const char *ad_public_broadcast_state(void) { if (!ad_is_public_broadcast_uuid(ad.service[AD_TYPE_AD].uuid)) @@ -1200,6 +1257,23 @@ void ad_advertise_name(DBusConnection *conn, bool value) void ad_advertise_local_name(DBusConnection *conn, const char *name) { + if (ad_broadcast_in_use()) { + if (!name) { + ad_print_public_broadcast_name(); + return bt_shell_noninteractive_quit(EXIT_SUCCESS); + } + + if (ad_has_public_broadcast_name() && + ad.data[AD_TYPE_AD].data.len == strlen(name) && + !memcmp(ad.data[AD_TYPE_AD].data.data, name, strlen(name))) + return bt_shell_noninteractive_quit(EXIT_SUCCESS); + + if (!ad_set_public_broadcast_name(conn, name)) + return bt_shell_noninteractive_quit(EXIT_FAILURE); + + return bt_shell_noninteractive_quit(EXIT_SUCCESS); + } + if (!name) { if (ad.local_name) bt_shell_printf("LocalName: %s\n", ad.local_name); -- 2.53.0 ^ permalink raw reply related [flat|nested] 11+ messages in thread
end of thread, other threads:[~2026-04-24 16:23 UTC | newest] Thread overview: 11+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2026-03-17 14:26 [PATCH BlueZ 0/1] client: add public-broadcast advertise helper raghava447 2026-03-17 14:26 ` [PATCH BlueZ 1/1] client: Add public-broadcast profile advertise command raghava447 2026-03-17 16:21 ` client: add public-broadcast advertise helper bluez.test.bot 2026-03-17 18:00 ` [PATCH BlueZ 1/1] client: Add public-broadcast profile advertise command Luiz Augusto von Dentz 2026-03-20 8:34 ` raghava447 2026-04-24 14:18 ` [PATCH BlueZ v2 0/2] client: Add public broadcast advertising support raghu447 2026-04-24 14:18 ` [PATCH v2 1/2] client: add public-broadcast advertising command raghu447 2026-04-24 14:41 ` Luiz Augusto von Dentz 2026-04-24 14:44 ` Luiz Augusto von Dentz 2026-04-24 16:23 ` client: Add public broadcast advertising support bluez.test.bot 2026-04-24 14:18 ` [PATCH v2 2/2] client: make advertise.name use public broadcast name raghu447
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox