From patchwork Fri Oct 20 23:42:11 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Luiz Augusto von Dentz X-Patchwork-Id: 13431292 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 55AD3C0032E for ; Fri, 20 Oct 2023 23:42:22 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229603AbjJTXmW (ORCPT ); Fri, 20 Oct 2023 19:42:22 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:51648 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229803AbjJTXmV (ORCPT ); Fri, 20 Oct 2023 19:42:21 -0400 Received: from mail-io1-xd33.google.com (mail-io1-xd33.google.com [IPv6:2607:f8b0:4864:20::d33]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 0AE33D79 for ; Fri, 20 Oct 2023 16:42:16 -0700 (PDT) Received: by mail-io1-xd33.google.com with SMTP id ca18e2360f4ac-7a68d1698adso33183039f.1 for ; Fri, 20 Oct 2023 16:42:16 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1697845335; x=1698450135; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:message-id:date:subject:to :from:from:to:cc:subject:date:message-id:reply-to; bh=XWqBNAeX6A/2t7F/B2+AP6gAdJuZDGSc/q1r/hSrihM=; b=ODzzqybhSLqUwLUeSf60vxSglL3tPYdZ3ClrQXnac13D0zG7Fckm4Pbps2Iqg7+v8p L+fH57aBsF1xYE8rm+EGhw/AAeJwFrljtT1uSSB6pMZh80CAe072jnn3ZY/k8em1UXDe s84CMBBHyxFnV5VzlNX1upDiegFyjaN3uXVU0VoZvLBxVN0kF0eOBGjixR+oRc63v+Cx 9j/SJyieaF/9r/RN/7AJ4FqEOgyizeS5SFHyNA5R70fh6QcgfuSEMq/LHbJ8gVsEQobz 7DmSX8m79ZncR6LjRzzLw9xPj9zwvEsvDsniJAXNv1sb/F9Shw/WM8utipp8WWTek9rJ ERDA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1697845335; x=1698450135; h=content-transfer-encoding:mime-version:message-id:date:subject:to :from:x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=XWqBNAeX6A/2t7F/B2+AP6gAdJuZDGSc/q1r/hSrihM=; b=GS9uEL3olCApLtsCEJ7VjOR+NNgb+abg7QR4OQozWqO7vKRvMRUwV/cmaRzTIDoc7B Angve3y8Cz3/1bFd5JeV4zJ/oe7g3n9uYCnsRH0XY95dOCt5N7WHiWUL5ZbjDWkusayx GX/kkEBQGcF5iUJ0q4wp2/IcLsLpQ/CuOBS0ggAVn1XyCUGRwgtyeHmfkhS4zj/hO0s7 xBgsJ82uqxcDZWKABp5xndtiOtNEagzPB1+yAqJkyZJgDZFgPTJ9rCo4e2JKg6bcLM/1 hyimxKzhV501zlDQAvAcLISeG+h1pTB534LpfhCQ+1b2YhT0L7il+WvlS4URHvXncFy+ qqcQ== X-Gm-Message-State: AOJu0Yx1aupdPBEE4fyi74n9ZDvXaMhSqfWMjfSPTIIbgLlygE2RkkcP tiaU03oDaK7RQmYeOfwnz2LQy0iLCPZDiZKzEwU= X-Google-Smtp-Source: AGHT+IHmNFO/kpHIxTFWFsleGdgKIY2ypifEmUJJwV7lK2M0SLW/5JBMhjH6wcXeUHdRcJ3+vXtP4Q== X-Received: by 2002:a6b:5904:0:b0:7a6:793d:2079 with SMTP id n4-20020a6b5904000000b007a6793d2079mr5780367iob.1.1697845334687; Fri, 20 Oct 2023 16:42:14 -0700 (PDT) Received: from lvondent-mobl4.. (c-98-232-221-87.hsd1.or.comcast.net. [98.232.221.87]) by smtp.gmail.com with ESMTPSA id br12-20020a05663846cc00b0043a1483abc9sm796376jab.74.2023.10.20.16.42.13 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 20 Oct 2023 16:42:13 -0700 (PDT) From: Luiz Augusto von Dentz To: linux-bluetooth@vger.kernel.org Subject: [PATCH BlueZ v2 1/2] shared/bap: Remove Locations, Support Context and Context defines Date: Fri, 20 Oct 2023 16:42:11 -0700 Message-ID: <20231020234212.3190403-1-luiz.dentz@gmail.com> X-Mailer: git-send-email 2.41.0 MIME-Version: 1.0 Precedence: bulk List-ID: X-Mailing-List: linux-bluetooth@vger.kernel.org From: Luiz Augusto von Dentz These values shall come from the upper stack. --- src/shared/bap.c | 15 --------------- 1 file changed, 15 deletions(-) diff --git a/src/shared/bap.c b/src/shared/bap.c index 14a62d9241eb..13bbcf7935bd 100644 --- a/src/shared/bap.c +++ b/src/shared/bap.c @@ -48,14 +48,6 @@ #define BAP_PROCESS_TIMEOUT 10 -#define PACS_SRC_LOCATION 0x00000001 -#define PACS_SNK_LOCATION 0x00000003 - -#define PACS_SRC_CTXT 0x000f -#define PACS_SUPPORTED_SRC_CTXT PACS_SRC_CTXT -#define PACS_SNK_CTXT 0x0fff -#define PACS_SUPPORTED_SNK_CTXT PACS_SNK_CTXT - struct bt_bap_pac_changed { unsigned int id; bt_bap_pac_func_t added; @@ -475,13 +467,6 @@ static struct bt_pacs *pacs_new(struct gatt_db *db) pacs = new0(struct bt_pacs, 1); - pacs->sink_loc_value = 0; - pacs->source_loc_value = 0; - pacs->sink_context_value = PACS_SNK_CTXT; - pacs->source_context_value = PACS_SRC_CTXT; - pacs->supported_sink_context_value = PACS_SUPPORTED_SNK_CTXT; - pacs->supported_source_context_value = PACS_SUPPORTED_SRC_CTXT; - /* Populate DB with PACS attributes */ bt_uuid16_create(&uuid, PACS_UUID); pacs->service = gatt_db_add_service(db, &uuid, true, 19); From patchwork Fri Oct 20 23:42:12 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Luiz Augusto von Dentz X-Patchwork-Id: 13431293 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 63B5DC001DF for ; Fri, 20 Oct 2023 23:42:26 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229766AbjJTXm0 (ORCPT ); Fri, 20 Oct 2023 19:42:26 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:51730 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229932AbjJTXmZ (ORCPT ); Fri, 20 Oct 2023 19:42:25 -0400 Received: from mail-io1-xd2c.google.com (mail-io1-xd2c.google.com [IPv6:2607:f8b0:4864:20::d2c]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id CBFB510C9 for ; Fri, 20 Oct 2023 16:42:17 -0700 (PDT) Received: by mail-io1-xd2c.google.com with SMTP id ca18e2360f4ac-7a689dc8131so46533839f.2 for ; Fri, 20 Oct 2023 16:42:17 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1697845336; x=1698450136; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:to:from:from:to:cc:subject:date:message-id :reply-to; bh=4IPBUiFJ/gaatV020+XX9xMxASAdovMoFX2DnHtPJjY=; b=MEYUkn9QLAAqSHbdjr1VntJ90uTMp5hHot9oIF8V2qJW1ANhIM6e4lABFPM/vMF1hy u31KhSU3VSo1d4lfamTh0/mM+UMVbff3E6Z+ga6GJkNFj2L+Iaq6jPY+GCt3fG8gMP9X gD8xBwvosdbkWQMyxlegJRP/nPRh/3fTspkT8nXsMG3Sl82Njeqjcnct6kz5YTpPCZZg 4RShtFNPe+tJjJMtvedWLZyD7fKBJ3xxt2ATqNgE4JKoI/vySjq6TDFrihb0nP89tBw7 C7X/CIogHETRS9c3aPpDLN2HrABlkZWbHP6JVeQZLK4497OlPSy7yXC6dURc3xCdwzhh +sAg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1697845336; x=1698450136; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=4IPBUiFJ/gaatV020+XX9xMxASAdovMoFX2DnHtPJjY=; b=F+mC+hLfeHWivfV2EcKxT1WDaeqYuMxDgKiUyrRaS7he3HYdgvgaTsPSgVG7W8T8fR HbeM0O26oCNQihtmho3ygWkeaHR0QpH9ajS86L6wkDzaOlpII+KZXkcbB3AQTutAbBwF gRpwI1hQXMhqq1fzi3Jr3C7MFkXL+nk73mBX4gOdATacEj25hYV64b8wUQ43mjR0XI5P q9eMH4l4C3J5kHYeiTiGepYcWP68LD6PKwUZ+50xDCqLa+lgrt4cRGFnp9urOOu2M7cZ BA4p474DWSE8tmc5Ot50dJowzhArxHJGay3UN/NGuYEL/Wn679xIMIizaygOrWEE9PY3 njEw== X-Gm-Message-State: AOJu0Yw9aIvhBKqiEFLlSjyL3k0JcwMYmjumaz503cl72RneZFwAyYE2 2XKfgeSm0JYvgb0x8PVAyx3N+/jXgpFzXjUXY+8= X-Google-Smtp-Source: AGHT+IHHHRpbW5to8hshBa8yGFwY2pFmYT3uT3f5JjGTdPvrdXotZPMwOW96zcO2t2uSvylMFe59dA== X-Received: by 2002:a05:6602:1490:b0:7a6:9f67:6085 with SMTP id a16-20020a056602149000b007a69f676085mr2999599iow.8.1697845336384; Fri, 20 Oct 2023 16:42:16 -0700 (PDT) Received: from lvondent-mobl4.. (c-98-232-221-87.hsd1.or.comcast.net. [98.232.221.87]) by smtp.gmail.com with ESMTPSA id br12-20020a05663846cc00b0043a1483abc9sm796376jab.74.2023.10.20.16.42.14 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 20 Oct 2023 16:42:15 -0700 (PDT) From: Luiz Augusto von Dentz To: linux-bluetooth@vger.kernel.org Subject: [PATCH BlueZ v2 2/2] client: Add support for setting Locations, SupportedContext and Context Date: Fri, 20 Oct 2023 16:42:12 -0700 Message-ID: <20231020234212.3190403-2-luiz.dentz@gmail.com> X-Mailer: git-send-email 2.41.0 In-Reply-To: <20231020234212.3190403-1-luiz.dentz@gmail.com> References: <20231020234212.3190403-1-luiz.dentz@gmail.com> MIME-Version: 1.0 Precedence: bulk List-ID: X-Mailing-List: linux-bluetooth@vger.kernel.org From: Luiz Augusto von Dentz This adds proper defaults for Locations, SupportedContext and Context properties since bluetoothd no longer automatically set proper defaults. --- client/player.c | 192 ++++++++++++++++++++++++++++++++++++++++++++---- 1 file changed, 178 insertions(+), 14 deletions(-) diff --git a/client/player.c b/client/player.c index 603447a9df46..715598aa9405 100644 --- a/client/player.c +++ b/client/player.c @@ -64,6 +64,14 @@ #define SEC_USEC(_t) (_t * 1000000L) #define TS_USEC(_ts) (SEC_USEC((_ts)->tv_sec) + NSEC_USEC((_ts)->tv_nsec)) +#define EP_SRC_LOCATIONS 0x00000001 +#define EP_SNK_LOCATIONS 0x00000003 + +#define EP_SRC_CTXT 0x000f +#define EP_SUPPORTED_SRC_CTXT EP_SRC_CTXT +#define EP_SNK_CTXT 0x0fff +#define EP_SUPPORTED_SNK_CTXT EP_SNK_CTXT + struct endpoint { char *path; char *uuid; @@ -72,6 +80,9 @@ struct endpoint { uint16_t vid; struct iovec *caps; struct iovec *meta; + uint32_t locations; + uint16_t supported_context; + uint16_t context; bool auto_accept; uint8_t max_transports; uint8_t iso_group; @@ -2363,6 +2374,63 @@ static gboolean endpoint_metadata_exists(const GDBusPropertyTable *property, return ep->meta ? TRUE : FALSE; } +static gboolean endpoint_get_locations(const GDBusPropertyTable *property, + DBusMessageIter *iter, void *data) +{ + struct endpoint *ep = data; + + dbus_message_iter_append_basic(iter, DBUS_TYPE_UINT32, &ep->locations); + + return TRUE; +} + +static gboolean endpoint_locations_exists(const GDBusPropertyTable *property, + void *data) +{ + struct endpoint *ep = data; + + return ep->supported_context ? TRUE : FALSE; +} + +static gboolean +endpoint_get_supported_context(const GDBusPropertyTable *property, + DBusMessageIter *iter, void *data) +{ + struct endpoint *ep = data; + + dbus_message_iter_append_basic(iter, DBUS_TYPE_UINT16, + &ep->supported_context); + + return TRUE; +} + +static gboolean +endpoint_supported_context_exists(const GDBusPropertyTable *property, + void *data) +{ + struct endpoint *ep = data; + + return ep->supported_context ? TRUE : FALSE; +} + +static gboolean endpoint_get_context(const GDBusPropertyTable *property, + DBusMessageIter *iter, void *data) +{ + struct endpoint *ep = data; + + dbus_message_iter_append_basic(iter, DBUS_TYPE_UINT16, &ep->context); + + return TRUE; +} + +static gboolean endpoint_context_exists(const GDBusPropertyTable *property, + void *data) +{ + struct endpoint *ep = data; + + return ep->context ? TRUE : FALSE; +} + static const GDBusPropertyTable endpoint_properties[] = { { "UUID", "s", endpoint_get_uuid, NULL, NULL }, { "Codec", "y", endpoint_get_codec, NULL, NULL }, @@ -2370,6 +2438,11 @@ static const GDBusPropertyTable endpoint_properties[] = { { "Metadata", "ay", endpoint_get_metadata, NULL, endpoint_metadata_exists }, { "Vendor", "u", endpoint_get_vendor, NULL, endpoint_vendor_exists }, + { "Locations", "u", endpoint_get_locations, NULL, + endpoint_locations_exists }, + { "SupportedContext", "q", endpoint_get_supported_context, NULL, + endpoint_supported_context_exists }, + { "Context", "q", endpoint_get_context, NULL, endpoint_context_exists }, { } }; @@ -2413,6 +2486,19 @@ static void register_endpoint_setup(DBusMessageIter *iter, void *user_data) bt_shell_hexdump(ep->meta->iov_base, ep->meta->iov_len); } + if (ep->locations) + g_dbus_dict_append_entry(&dict, "Locations", DBUS_TYPE_UINT32, + &ep->locations); + + if (ep->supported_context) + g_dbus_dict_append_entry(&dict, "SupportedContext", + DBUS_TYPE_UINT16, + &ep->supported_context); + + if (ep->context) + g_dbus_dict_append_entry(&dict, "Context", DBUS_TYPE_UINT16, + &ep->context); + dbus_message_iter_close_container(iter, &dict); } @@ -2552,6 +2638,67 @@ static void endpoint_iso_group(const char *input, void *user_data) endpoint_iso_stream, ep); } +static void endpoint_context(const char *input, void *user_data) +{ + struct endpoint *ep = user_data; + char *endptr = NULL; + int value; + + value = strtol(input, &endptr, 0); + + if (!endptr || *endptr != '\0' || value > UINT16_MAX) { + bt_shell_printf("Invalid argument: %s\n", input); + return bt_shell_noninteractive_quit(EXIT_FAILURE); + } + + ep->context = value; + + if (ep->broadcast) + bt_shell_prompt_input(ep->path, "BIG (auto/value):", + endpoint_iso_group, ep); + else + bt_shell_prompt_input(ep->path, "CIG (auto/value):", + endpoint_iso_group, ep); +} + +static void endpoint_supported_context(const char *input, void *user_data) +{ + struct endpoint *ep = user_data; + char *endptr = NULL; + int value; + + value = strtol(input, &endptr, 0); + + if (!endptr || *endptr != '\0' || value > UINT16_MAX) { + bt_shell_printf("Invalid argument: %s\n", input); + return bt_shell_noninteractive_quit(EXIT_FAILURE); + } + + ep->supported_context = value; + + bt_shell_prompt_input(ep->path, "Context (value):", endpoint_context, + ep); +} + +static void endpoint_locations(const char *input, void *user_data) +{ + struct endpoint *ep = user_data; + char *endptr = NULL; + int value; + + value = strtol(input, &endptr, 0); + + if (!endptr || *endptr != '\0') { + bt_shell_printf("Invalid argument: %s\n", input); + return bt_shell_noninteractive_quit(EXIT_FAILURE); + } + + ep->locations = value; + + bt_shell_prompt_input(ep->path, "Supported Context (value):", + endpoint_supported_context, ep); +} + static void endpoint_max_transports(const char *input, void *user_data) { struct endpoint *ep = user_data; @@ -2571,12 +2718,7 @@ static void endpoint_max_transports(const char *input, void *user_data) ep->max_transports = value; } - if (ep->broadcast) - bt_shell_prompt_input(ep->path, "BIG (auto/value):", - endpoint_iso_group, ep); - else - bt_shell_prompt_input(ep->path, "CIG (auto/value):", - endpoint_iso_group, ep); + bt_shell_prompt_input(ep->path, "Locations:", endpoint_locations, ep); } static void endpoint_auto_accept(const char *input, void *user_data) @@ -3338,14 +3480,41 @@ static const struct bt_shell_menu endpoint_menu = { {} }, }; +static void endpoint_init_defaults(struct endpoint *ep) +{ + ep->preset = find_presets(ep->uuid, ep->codec, ep->vid, ep->cid); + ep->max_transports = UINT8_MAX; + ep->auto_accept = true; + + if (!strcmp(ep->uuid, A2DP_SOURCE_UUID) || + !strcmp(ep->uuid, A2DP_SOURCE_UUID)) + return; + + ep->iso_group = BT_ISO_QOS_GROUP_UNSET; + ep->iso_stream = BT_ISO_QOS_STREAM_UNSET; + + ep->broadcast = (strcmp(ep->uuid, BCAA_SERVICE_UUID) && + strcmp(ep->uuid, BAA_SERVICE_UUID)) ? false : true; + if (ep->broadcast) + return; + + if (!strcmp(ep->uuid, PAC_SINK_UUID)) { + ep->locations = EP_SNK_LOCATIONS; + ep->supported_context = EP_SUPPORTED_SNK_CTXT; + ep->context = EP_SNK_CTXT; + } else if (!strcmp(ep->uuid, PAC_SOURCE_UUID)) { + ep->locations = EP_SRC_LOCATIONS; + ep->supported_context = EP_SUPPORTED_SRC_CTXT; + ep->context = EP_SRC_CTXT; + } +} + static struct endpoint *endpoint_new(const struct capabilities *cap) { struct endpoint *ep; ep = new0(struct endpoint, 1); ep->uuid = g_strdup(cap->uuid); - ep->broadcast = (strcmp(cap->uuid, BCAA_SERVICE_UUID) && - strcmp(cap->uuid, BAA_SERVICE_UUID)) ? false : true; ep->codec = cap->codec_id; ep->path = g_strdup_printf("%s/ep%u", BLUEZ_MEDIA_ENDPOINT_PATH, g_list_length(local_endpoints)); @@ -3368,12 +3537,7 @@ static void register_endpoints(GDBusProxy *proxy) continue; ep = endpoint_new(cap); - ep->preset = find_presets(ep->uuid, ep->codec, ep->vid, - ep->cid); - ep->max_transports = UINT8_MAX; - ep->auto_accept = true; - ep->iso_group = BT_ISO_QOS_GROUP_UNSET; - ep->iso_stream = BT_ISO_QOS_STREAM_UNSET; + endpoint_init_defaults(ep); endpoint_register(ep); } }