ti, 2025-02-25 kello 17:12 -0500, Luiz Augusto von Dentz kirjoitti:
> Hi Pauli,
>
> On Tue, Feb 25, 2025 at 5:10 PM Luiz Augusto von Dentz
> <luiz.dentz@xxxxxxxxx> wrote:
> >
> > Hi Pauli, Bastien,
> >
> > On Tue, Feb 25, 2025 at 5:01 PM Luiz Augusto von Dentz
> > <luiz.dentz@xxxxxxxxx> wrote:
> > >
> > > From: Luiz Augusto von Dentz <luiz.von.dentz@xxxxxxxxx>
> > >
> > > This adds support for PreferredBearer which is printed with the likes of
> > > info command:
> > >
> > > bluetoothctl> info <addr>
> > > ...
> > > PreferredBearer: last-seen
> > >
> > > It also introduces a new command to get/set the PreferredBearer:
> > >
> > > [bluetoothctl]> bearer --help
> > > Get/Set preferred bearer
> > > Usage:
> > > bearer <dev> [last-seen/bredr/le]
> > >
> > > [bluetoothctl]> bearer <addr>
> > > PreferredBearer: last-seen
> > > [bluetoothctl]> bearer <addr> le
> > > bluetoothd: @ MGMT Command: Add Device (0x0033) plen 8
> > > LE Address: <addr>
> > > Action: Auto-connect remote device (0x02)
> > > [CHG] Device <addr> PreferredBearer: le
> > > Changing le succeeded
> > > [bluetoothctl]> bearer <addr>
> > > PreferredBearer: le
> > > [bluetoothctl]> bearer <addr> bredr
> > > bluetoothd: @ MGMT Command: Remove Device (0x0034) plen 7
> > > LE Address: <addr>
> > > [CHG] Device <addr> PreferredBearer: bredr
> > > Changing bredr succeeded
> > > ---
> > > client/main.c | 27 +++++++++++++++++++++++++++
> > > 1 file changed, 27 insertions(+)
> > >
> > > diff --git a/client/main.c b/client/main.c
> > > index feb21a1163d2..76c9bc329c96 100644
> > > --- a/client/main.c
> > > +++ b/client/main.c
> > > @@ -1714,6 +1714,7 @@ static void cmd_info(int argc, char *argv[])
> > > print_property(proxy, "AdvertisingFlags");
> > > print_property(proxy, "AdvertisingData");
> > > print_property(proxy, "Sets");
> > > + print_property(proxy, "PreferredBearer");
> > >
> > > battery_proxy = find_proxies_by_path(battery_proxies,
> > > g_dbus_proxy_get_path(proxy));
> > > @@ -2086,6 +2087,30 @@ static void cmd_wake(int argc, char *argv[])
> > > return bt_shell_noninteractive_quit(EXIT_FAILURE);
> > > }
> > >
> > > +static void cmd_bearer(int argc, char *argv[])
> > > +{
> > > + GDBusProxy *proxy;
> > > + char *str;
> > > +
> > > + proxy = find_device(argc, argv);
> > > + if (!proxy)
> > > + return bt_shell_noninteractive_quit(EXIT_FAILURE);
> > > +
> > > + if (argc <= 2) {
> > > + print_property(proxy, "PreferredBearer");
> > > + return;
> > > + }
> > > +
> > > + str = strdup(argv[2]);
> > > +
> > > + if (g_dbus_proxy_set_property_basic(proxy, "PreferredBearer",
> > > + DBUS_TYPE_STRING, &str,
> > > + generic_callback, str, free))
> > > + return;
> > > +
> > > + return bt_shell_noninteractive_quit(EXIT_FAILURE);
> > > +}
> > > +
> > > static void cmd_list_attributes(int argc, char *argv[])
> > > {
> > > GDBusProxy *proxy;
> > > @@ -3247,6 +3272,8 @@ static const struct bt_shell_menu main_menu = {
> > > dev_generator },
> > > { "wake", "[dev] [on/off]", cmd_wake, "Get/Set wake support",
> > > dev_generator },
> > > + { "bearer", "<dev> [last-seen/bredr/le]", cmd_bearer,
> > > + "Get/Set preferred bearer", dev_generator },
> > > { } },
> > > };
> > >
> > > --
> > > 2.48.1
> >
> > So I went ahead and implemented the idea of having PreferredBearer,
> > this works great when setting bredr it really stops from connecting to
> > LE, the said the other way around when setting to le seems to confuse
> > some headsets like EarFun and it ends up connecting both bearers:
> >
> > [EarFun Air Pro 3]> transport.show
> > Transport /org/bluez/hci0/dev_70_5A_6F_63_B6_41/pac_source0/fd1
> > UUID: Sink PAC (00002bc9-0000-1000-8000-00805f9b34fb)
> > Codec: 0x06 (6)
> > Configuration.#0: len 0x02 type 0x01
> > Configuration.Sampling Frequency: 16 Khz (0x03)
> > Configuration.#1: len 0x02 type 0x02
> > Configuration.Frame Duration: 7.5 ms (0x00)
> > Configuration.#2: len 0x05 type 0x03
> > Configuration.Location: 0x00000001
> > Configuration.Location: Front Left (0x00000001)
> > Configuration.#3: len 0x03 type 0x04
> > Configuration.Frame Length: 30 (0x001e)
> > Configuration.#4: len 0x02 type 0x05
> > Configuration.Frame Blocks per SDU: 1 (0x01)
> > Device: /org/bluez/hci0/dev_70_5A_6F_63_B6_41
> > State: idle
> > Volume: 0x00c8 (200)
> > Endpoint: /org/bluez/hci0/dev_70_5A_6F_63_B6_41/pac_source0
> > QoS.CIG: 0x00 (0)
> > QoS.CIS: 0x00 (0)
> > QoS.Framing: 0x00 (0)
> > QoS.PresentationDelay: 0x00009c40 (40000)
> > QoS.Interval: 0x00001d4c (7500)
> > QoS.Latency: 0x0008 (8)
> > QoS.SDU: 0x001e (30)
> > QoS.PHY: 0x02 (2)
> > QoS.Retransmissions: 0x02 (2)
> > Location: 0x00000003 (3)
> > Links: /org/bluez/hci0/dev_70_5A_6F_63_B6_41/pac_sink0/fd3
> > Transport /org/bluez/hci0/dev_70_5A_6F_63_B6_41/pac_source0/fd2
> > UUID: Sink PAC (00002bc9-0000-1000-8000-00805f9b34fb)
> > Codec: 0x06 (6)
> > Configuration.#0: len 0x02 type 0x01
> > Configuration.Sampling Frequency: 16 Khz (0x03)
> > Configuration.#1: len 0x02 type 0x02
> > Configuration.Frame Duration: 7.5 ms (0x00)
> > Configuration.#2: len 0x05 type 0x03
> > Configuration.Location: 0x00000002
> > Configuration.Location: Front Right (0x00000002)
> > Configuration.#3: len 0x03 type 0x04
> > Configuration.Frame Length: 30 (0x001e)
> > Configuration.#4: len 0x02 type 0x05
> > Configuration.Frame Blocks per SDU: 1 (0x01)
> > Device: /org/bluez/hci0/dev_70_5A_6F_63_B6_41
> > State: idle
> > Volume: 0x00c8 (200)
> > Endpoint: /org/bluez/hci0/dev_70_5A_6F_63_B6_41/pac_source0
> > QoS.CIG: 0x00 (0)
> > QoS.CIS: 0x01 (1)
> > QoS.Framing: 0x00 (0)
> > QoS.PresentationDelay: 0x00009c40 (40000)
> > QoS.Interval: 0x00001d4c (7500)
> > QoS.Latency: 0x0008 (8)
> > QoS.SDU: 0x001e (30)
> > QoS.PHY: 0x02 (2)
> > QoS.Retransmissions: 0x02 (2)
> > Location: 0x00000003 (3)
> > Links: /org/bluez/hci0/dev_70_5A_6F_63_B6_41/pac_sink0/fd4
> > Transport /org/bluez/hci0/dev_70_5A_6F_63_B6_41/pac_sink0/fd3
> > UUID: Source PAC (00002bcb-0000-1000-8000-00805f9b34fb)
> > Codec: 0x06 (6)
> > Configuration.#0: len 0x02 type 0x01
> > Configuration.Sampling Frequency: 48 Khz (0x08)
> > Configuration.#1: len 0x02 type 0x02
> > Configuration.Frame Duration: 7.5 ms (0x00)
> > Configuration.#2: len 0x05 type 0x03
> > Configuration.Location: 0x00000001
> > Configuration.Location: Front Left (0x00000001)
> > Configuration.#3: len 0x03 type 0x04
> > Configuration.Frame Length: 90 (0x005a)
> > Configuration.#4: len 0x02 type 0x05
> > Configuration.Frame Blocks per SDU: 1 (0x01)
> > Device: /org/bluez/hci0/dev_70_5A_6F_63_B6_41
> > State: idle
> > Volume: 0x00c8 (200)
> > Endpoint: /org/bluez/hci0/dev_70_5A_6F_63_B6_41/pac_sink0
> > QoS.CIG: 0x00 (0)
> > QoS.CIS: 0x00 (0)
> > QoS.Framing: 0x00 (0)
> > QoS.PresentationDelay: 0x00009c40 (40000)
> > QoS.Interval: 0x00001d4c (7500)
> > QoS.Latency: 0x000f (15)
> > QoS.SDU: 0x005a (90)
> > QoS.PHY: 0x02 (2)
> > QoS.Retransmissions: 0x05 (5)
> > Location: 0x00000003 (3)
> > Links: /org/bluez/hci0/dev_70_5A_6F_63_B6_41/pac_source0/fd1
> > Transport /org/bluez/hci0/dev_70_5A_6F_63_B6_41/pac_sink0/fd4
> > UUID: Source PAC (00002bcb-0000-1000-8000-00805f9b34fb)
> > Codec: 0x06 (6)
> > Configuration.#0: len 0x02 type 0x01
> > Configuration.Sampling Frequency: 48 Khz (0x08)
> > Configuration.#1: len 0x02 type 0x02
> > Configuration.Frame Duration: 7.5 ms (0x00)
> > Configuration.#2: len 0x05 type 0x03
> > Configuration.Location: 0x00000002
> > Configuration.Location: Front Right (0x00000002)
> > Configuration.#3: len 0x03 type 0x04
> > Configuration.Frame Length: 90 (0x005a)
> > Configuration.#4: len 0x02 type 0x05
> > Configuration.Frame Blocks per SDU: 1 (0x01)
> > Device: /org/bluez/hci0/dev_70_5A_6F_63_B6_41
> > State: idle
> > Volume: 0x00c8 (200)
> > Endpoint: /org/bluez/hci0/dev_70_5A_6F_63_B6_41/pac_sink0
> > QoS.CIG: 0x00 (0)
> > QoS.CIS: 0x01 (1)
> > QoS.Framing: 0x00 (0)
> > QoS.PresentationDelay: 0x00009c40 (40000)
> > QoS.Interval: 0x00001d4c (7500)
> > QoS.Latency: 0x000f (15)
> > QoS.SDU: 0x005a (90)
> > QoS.PHY: 0x02 (2)
> > QoS.Retransmissions: 0x05 (5)
> > Location: 0x00000003 (3)
> > Links: /org/bluez/hci0/dev_70_5A_6F_63_B6_41/pac_source0/fd2
> > Transport /org/bluez/hci0/dev_70_5A_6F_63_B6_41/fd5
> > UUID: Audio Source (0000110a-0000-1000-8000-00805f9b34fb)
> > Codec: 0x02 (2)
> > Media Codec: MPEG24
> > Object Types: MPEG-4 AAC LC
> > Frequencies: 48kHz
> > Channels: 2
> > Bitrate: 320000
> > VBR: Yes
> > Device: /org/bluez/hci0/dev_70_5A_6F_63_B6_41
> > State: idle
> > Delay: 0x0960 (2400)
> > Volume: 0x0064 (100)
>
> Forgot to mention, but with the above transports it seems to confuse
> the gnome audio output selection, it doesn't seem to be able to mix
> A2DP and BAP transports for some reason, so when I select the device
> it enables BAP but A2DP is not shown as an option.
I'll have to see if I can reproduce that on current PW master branch.
The visibility of profiles in theory should only cares about whether
the UUIDs appear in both device properties and transport.
--
Pauli Virtanen
