Received-SPF: pass (google.com: domain of linux-bluetooth-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   Nitin Jadhav <nitin.jadhav@nxp.com>
To:     linux-bluetooth@vger.kernel.org
Cc:     devyani.godbole@nxp.com, pav@iki.fi, luiz.dentz@gmail.com,
        sathish.narasimman@intel.com, nitin.jadhav@nxp.com
Subject: [PATCH BlueZ v4 3/3] shared/vcp: Reserve one more handle in VCS for VOCS
Date:   Tue, 25 Jul 2023 16:47:42 +0300
Message-Id: <20230725134742.51651-4-nitin.jadhav@nxp.com>
In-Reply-To: <20230725134742.51651-1-nitin.jadhav@nxp.com>
References: <20230725134742.51651-1-nitin.jadhav@nxp.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?lRvVmE+qFUkDjHZckEwRfGsoVBy0WEZNaC5LKoaDStDLXj4bJvqQmww2Wraj?=
 =?us-ascii?Q?Cnp6Ng0LP2apoggGO1lI1uBffdlTCNbbWSemPEzkmLTJe4I6YMo46/BumAGV?=
 =?us-ascii?Q?PRZKRL0JFIoPSPeq1b3JeRmg6WtxKwrmlwhOZORFkAMQNv+VdqC0QLfXhZNR?=
 =?us-ascii?Q?mSyjvX1mAaDu1KzCxZrgCkxkTkpeNXq49ivmVtZbvtDpJTTDcMW61+RGcJJW?=
 =?us-ascii?Q?sRl3tSjGKTsypr/oc+GFNcdiYqQEJBq28Um25o7i4bg2Y2dz2yYQrgC908kV?=
 =?us-ascii?Q?+E58ynzGjDPjBMK4V+rpCOvSuY040GRbbOD/Ul1jxIAxt/wYIwctn+sDBCXR?=
 =?us-ascii?Q?7V5TVjUgFgFfvIC13TctstswEeql8BOiUwGa7l538ZNu1oXvGUt65tcBjavB?=
 =?us-ascii?Q?JXbpP7yylZmYZbML2dwQFcjn9aEEE91CY/mZ4soX+CGpuru/aPI0EBDAlW1O?=
 =?us-ascii?Q?pzZWrIWlT3r2aFDxeKhS0bCf5v0HuCIveRu7Stw/AaDhVC0WbH9T5IJxxlYZ?=
 =?us-ascii?Q?TZyvtA0lwg8yfO9N4nqa/uDGGcPDebkAm58D6J16Dz9Azmnh55OpleQLQb/n?=
 =?us-ascii?Q?Dgi752IionqQJnarGU2oUecLHhlyGWW+VdAAtoXjM0MRMgY1rm24jfArzmaS?=
 =?us-ascii?Q?19QfQnSgBJjAworBk8fOjRfz74eopvx13KIN2dGuHIW0eExIJubs7g1O+Gjw?=
 =?us-ascii?Q?lXQnYUkcz5NGScn/P54DV70BsNChEcs+FEH52e+m+apDneFHKNrqlbm/XvgB?=
 =?us-ascii?Q?ldvJFsSeAETAXKUFSZtxomZAf89fWcrmRp3d91Lr+3CKkEqeP5f4upF+fSVf?=
 =?us-ascii?Q?J9fQKiR/BnjaZKnqD3ynsFVW2ZnEk59TnmuK67XxNjncgnC/14Rn7ZaaFB5N?=
 =?us-ascii?Q?o3P12avRnwZqCrDQZiZskAjevwP4W0799YaA0C5bf8EBg0cdpjzlohyZcGfM?=
 =?us-ascii?Q?EIZFHZwZ9JVqdcoQdOO15k/Mqx3wM4Q/KjojJmIwVXkFljNVyyPHXSmA02xJ?=
 =?us-ascii?Q?3TiCsgrH/lFStOdF6xkce6HlGYbTU6/6HG/JT/TyAKhQqTLIxTcLxdeje5ui?=
 =?us-ascii?Q?NGvknpOh7gRM/n2QOIfZAzFgSQUfw1N4J1qJhRMz3UnJMlo2U0kSMz5l9c5c?=
 =?us-ascii?Q?Dpt7DEz6Ny5QkIb0dD/aXN0YRkeJwDvB8V78qO5EtqZ3IAMyKE48DAjJYIor?=
 =?us-ascii?Q?sOfxgeuyH7OY7kTmFT/Yn2hddNS1AKdc50Wwsgo78CE0+jPiCkndN2LFwLU8?=
 =?us-ascii?Q?iCW+m3Zq3BZP3aqJSI31TET+nBdfSob6xemXPZqiyfDLki4w4J0HwbiXrA/+?=
 =?us-ascii?Q?lFdY3VS3UGru7nbD965PUrVIUgKyc5oUHwXBxKST5FbXmpou/RSpug2vy1RN?=
 =?us-ascii?Q?dk3LKrFol5zW2yvRB9ieURKUWKAr3Uvkgo1EHrY2oZHKQOTfghjQbMg8dTZu?=
 =?us-ascii?Q?GUVGtWTdb8+WeyadqyBkoLm2AX+zy2ZJVcFBiy0VEGKx6ACUpP9F2uVaWfEo?=
 =?us-ascii?Q?+UT0oJ+ryOyeJfj9UaESwtnjZHkynNoIqDxFBuMVWReTDO0aUAkLDomfHdK+?=
 =?us-ascii?Q?TDh4haGKik77KvoAGecVCQTpF7KQRgwKJ6uaLNt4?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 834ec679-f3c7-4de4-38ab-08db8d15c989
X-MS-Exchange-CrossTenant-AuthSource: AS8PR04MB9126.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 25 Jul 2023 13:48:11.9159
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: ZSD7gJh2eITAQUWwrREYIlwiHjKzLS2UKu/OpMIcb7a12jk1KoeYBBI0N+YnmkpqEv25o+lmYkxV47rwbzSIFA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PAXPR04MB9593
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE,
        RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS,T_SCC_BODY_TEXT_LINE,
        URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-bluetooth.vger.kernel.org>
X-Mailing-List: linux-bluetooth@vger.kernel.org

Fix: As per specs, VOCS is a secondary service;
hence it should be included under a primary service VCS.
The current number of handles is increased to accommodate the
included service
---
 src/shared/vcp.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/shared/vcp.c b/src/shared/vcp.c
index 0408531b2..263da4c30 100644
--- a/src/shared/vcp.c
+++ b/src/shared/vcp.c
@@ -998,7 +998,7 @@ static struct bt_vcs *vcs_new(struct gatt_db *db, struct bt_vcp_db *vdb)
 
 	/* Populate DB with VCS attributes */
 	bt_uuid16_create(&uuid, VCS_UUID);
-	vcs->service = gatt_db_add_service(db, &uuid, true, 9);
+	vcs->service = gatt_db_add_service(db, &uuid, true, 10);
 	gatt_db_service_add_included(vcs->service, vdb->vocs->service);
 	gatt_db_service_set_active(vdb->vocs->service, true);
 
-- 
2.34.1