Hello Maintainers
This patch series handles code implementation for mandatory features of
Volume Offset Control Service.
Implementation of following features have been handled in this patch
series
- Volume Offset Control Service
- Volume Offset State Characteristic (Read, Notify)
- Audio Location Characteristic (Read)
- Volume Offset Control Point Characteristic (Write)
- Audio Output Description Characteristic (Read)
- Set Volume Offset
All the code implementation has been tested using PTS testing the
verdict is PASS for all mandatory test cases.
Thank you in advance for your review.
Warm Regards
Nitin Jadhav
NJNXP (2):
Added initial code for handling VOCS
Fixed the following issue observed during PTS testing for mandatory
test cases - Specified Upper and Lower Limit for Volume offset -
Corrected the number of handles for VOCS - VOCS is made as included
service of VCS because VOCS is secondary service and VSC is primary
service
lib/uuid.h | 5 +
src/shared/vcp.c | 547 ++++++++++++++++++++++++++++++++++++++++++++++-
2 files changed, 550 insertions(+), 2 deletions(-)
--
2.34.1
---
src/shared/vcp.c | 26 ++++++++++++++++++--------
1 file changed, 18 insertions(+), 8 deletions(-)
diff --git a/src/shared/vcp.c b/src/shared/vcp.c
index 05e341108..b6e8ffdaf 100644
--- a/src/shared/vcp.c
+++ b/src/shared/vcp.c
@@ -32,9 +32,13 @@
#define VCP_STEP_SIZE 1
+#define VOCS_VOL_OFFSET_UPPER_LIMIT 255
+#define VOCS_VOL_OFFSET_LOWER_LIMIT -255
+
/* Apllication Error Code */
#define BT_ATT_ERROR_INVALID_CHANGE_COUNTER 0x80
#define BT_ATT_ERROR_OPCODE_NOT_SUPPORTED 0x81
+#define BT_ATT_ERROR_VALUE_OUT_OF_RANGE 0x82
#define GEN_AUDIO_AUDLOC_NA 0x00000000
#define GEN_AUDIO_AUDLOC_FL 0x00000001
@@ -100,7 +104,7 @@ struct bt_vcs_ab_vol {
struct bt_vocs_set_vol_off {
uint8_t change_counter;
- uint8_t set_vol_offset;
+ int16_t set_vol_offset;
} __packed;
struct bt_vcp_cb {
@@ -167,8 +171,8 @@ struct bt_vcs {
/* Contains local bt_vcp_db */
struct vol_offset_state {
- uint16_t vol_offset;
- uint8_t counter;
+ int16_t vol_offset;
+ uint8_t counter;
} __packed;
struct bt_vocs {
@@ -705,6 +709,10 @@ static uint8_t vocs_set_vol_offset(struct bt_vocs *vocs, struct bt_vcp *vcp,
return BT_ATT_ERROR_INVALID_CHANGE_COUNTER;
}
+ if(req->set_vol_offset > VOCS_VOL_OFFSET_UPPER_LIMIT || req->set_vol_offset < VOCS_VOL_OFFSET_LOWER_LIMIT) {
+ DBG(vcp, "error: Value Out of Range");
+ return BT_ATT_ERROR_VALUE_OUT_OF_RANGE;
+ }
vstate->vol_offset = req->set_vol_offset;
vstate->counter = -~vstate->counter; /*Increment Change Counter*/
@@ -971,7 +979,7 @@ static void vocs_voaodec_read(struct gatt_db_attribute *attrib,
iov.iov_len);
}
-static struct bt_vcs *vcs_new(struct gatt_db *db)
+static struct bt_vcs *vcs_new(struct gatt_db *db, struct bt_vcp_db *vdb)
{
struct bt_vcs *vcs;
struct vol_state *vstate;
@@ -990,6 +998,8 @@ static struct bt_vcs *vcs_new(struct gatt_db *db)
/* Populate DB with VCS attributes */
bt_uuid16_create(&uuid, VCS_UUID);
vcs->service = gatt_db_add_service(db, &uuid, true, 9);
+ gatt_db_service_add_included(vcs->service, vdb->vocs->service);
+ gatt_db_service_set_active(vdb->vocs->service, true);
bt_uuid16_create(&uuid, VOL_STATE_CHRC_UUID);
vcs->vs = gatt_db_service_add_characteristic(vcs->service,
@@ -1048,7 +1058,8 @@ static struct bt_vocs *vocs_new(struct gatt_db *db)
/* Populate DB with VOCS attributes */
bt_uuid16_create(&uuid, VOL_OFFSET_CS_UUID);
- vocs->service = gatt_db_add_service(db, &uuid, true, 9);
+
+ vocs->service = gatt_db_add_service(db, &uuid, false, 12);
bt_uuid16_create(&uuid, VOL_OFFSET_STATE_CHAR_UUID);
vocs->vos = gatt_db_service_add_characteristic(vocs->service,
@@ -1110,11 +1121,10 @@ static struct bt_vcp_db *vcp_db_new(struct gatt_db *db)
if (!vcp_db)
vcp_db = queue_new();
- vdb->vcs = vcs_new(db);
- vdb->vcs->vdb = vdb;
-
vdb->vocs = vocs_new(db);
vdb->vocs->vdb = vdb;
+ vdb->vcs = vcs_new(db, vdb);
+ vdb->vcs->vdb = vdb;
queue_push_tail(vcp_db, vdb);
--
2.34.1
This commit handles the code implementation of VOCS service and characteristics
---
lib/uuid.h | 5 +
src/shared/vcp.c | 533 +++++++++++++++++++++++++++++++++++++++++++++++
2 files changed, 538 insertions(+)
diff --git a/lib/uuid.h b/lib/uuid.h
index 5cdfedb4b..e8c7f8e37 100644
--- a/lib/uuid.h
+++ b/lib/uuid.h
@@ -179,6 +179,11 @@ extern "C" {
#define VOL_CP_CHRC_UUID 0x2B7E
#define VOL_FLAG_CHRC_UUID 0x2B7F
+#define VOL_OFFSET_STATE_CHAR_UUID 0x2B80
+#define AUDIO_LOCATION_CHRC_UUID 0x2B81
+#define VOL_OFFSET_CP_CHRC_UUID 0x2B82
+#define AUDIO_OUTPUT_DESC_CHAR_UUID 0x2B83
+
#define GMCS_UUID 0x1849
#define MEDIA_PLAYER_NAME_CHRC_UUID 0x2b93
#define MEDIA_TRACK_CHNGD_CHRC_UUID 0x2b96
diff --git a/src/shared/vcp.c b/src/shared/vcp.c
index 5459cf892..05e341108 100644
--- a/src/shared/vcp.c
+++ b/src/shared/vcp.c
@@ -36,9 +36,40 @@
#define BT_ATT_ERROR_INVALID_CHANGE_COUNTER 0x80
#define BT_ATT_ERROR_OPCODE_NOT_SUPPORTED 0x81
+#define GEN_AUDIO_AUDLOC_NA 0x00000000
+#define GEN_AUDIO_AUDLOC_FL 0x00000001
+#define GEN_AUDIO_AUDLOC_FR 0x00000002
+#define GEN_AUDIO_AUDLOC_FC 0x00000004
+#define GEN_AUDIO_AUDLOC_LOW_FRQ_EFF_1 0x00000008
+#define GEN_AUDIO_AUDLOC_BL 0x00000010
+#define GEN_AUDIO_AUDLOC_BR 0x00000020
+#define GEN_AUDIO_AUDLOC_FLC 0x00000040
+#define GEN_AUDIO_AUDLOC_FRC 0x00000080
+#define GEN_AUDIO_AUDLOC_BC 0x00000100
+#define GEN_AUDIO_AUDLOC_LOW_FRQ_EFF_2 0x00000200
+#define GEN_AUDIO_AUDLOC_SL 0x00000400
+#define GEN_AUDIO_AUDLOC_SR 0x00000800
+#define GEN_AUDIO_AUDLOC_TFL 0x00001000
+#define GEN_AUDIO_AUDLOC_TFR 0x00002000
+#define GEN_AUDIO_AUDLOC_TFC 0x00004000
+#define GEN_AUDIO_AUDLOC_TC 0x00008000
+#define GEN_AUDIO_AUDLOC_TBL 0x00010000
+#define GEN_AUDIO_AUDLOC_TBR 0x00020000
+#define GEN_AUDIO_AUDLOC_TSL 0x00040000
+#define GEN_AUDIO_AUDLOC_TSR 0x00080000
+#define GEN_AUDIO_AUDLOC_TBC 0x00100000
+#define GEN_AUDIO_AUDLOC_BFC 0x00200000
+#define GEN_AUDIO_AUDLOC_BFL 0x00400000
+#define GEN_AUDIO_AUDLOC_BFR 0x00800000
+#define GEN_AUDIO_AUDLOC_FLW 0x01000000
+#define GEN_AUDIO_AUDLOC_FRW 0x02000000
+#define GEN_AUDIO_AUDLOC_LS 0x04000000
+#define GEN_AUDIO_AUDLOC_RS 0x08000000
+
struct bt_vcp_db {
struct gatt_db *db;
struct bt_vcs *vcs;
+ struct bt_vocs *vocs;
};
typedef void (*vcp_func_t)(struct bt_vcp *vcp, bool success, uint8_t att_ecode,
@@ -57,11 +88,21 @@ struct bt_vcs_param {
uint8_t change_counter;
} __packed;
+struct bt_vocs_param {
+ uint8_t op;
+ uint8_t change_counter;
+} __packed;
+
struct bt_vcs_ab_vol {
uint8_t change_counter;
uint8_t vol_set;
} __packed;
+struct bt_vocs_set_vol_off {
+ uint8_t change_counter;
+ uint8_t set_vol_offset;
+} __packed;
+
struct bt_vcp_cb {
unsigned int id;
bt_vcp_func_t attached;
@@ -89,6 +130,10 @@ struct bt_vcp {
unsigned int vstate_id;
unsigned int vflag_id;
+ unsigned int vostate_id;
+ unsigned int vocs_audio_loc_id;
+ unsigned int vocs_ao_dec_id;
+
struct queue *notify;
struct queue *pending;
@@ -120,6 +165,27 @@ struct bt_vcs {
struct gatt_db_attribute *vf_ccc;
};
+/* Contains local bt_vcp_db */
+struct vol_offset_state {
+ uint16_t vol_offset;
+ uint8_t counter;
+} __packed;
+
+struct bt_vocs {
+ struct bt_vcp_db *vdb;
+ struct vol_offset_state *vostate;
+ uint32_t vocs_audio_loc;
+ char *vocs_ao_dec;
+ struct gatt_db_attribute *service;
+ struct gatt_db_attribute *vos;
+ struct gatt_db_attribute *vos_ccc;
+ struct gatt_db_attribute *voal;
+ struct gatt_db_attribute *voal_ccc;
+ struct gatt_db_attribute *vo_cp;
+ struct gatt_db_attribute *voaodec;
+ struct gatt_db_attribute *voaodec_ccc;
+};
+
static struct queue *vcp_db;
static struct queue *vcp_cbs;
static struct queue *sessions;
@@ -159,6 +225,17 @@ static struct vol_state *vdb_get_vstate(struct bt_vcp_db *vdb)
return NULL;
}
+static struct vol_offset_state *vdb_get_vostate(struct bt_vcp_db *vdb)
+{
+ if (!vdb->vocs)
+ return NULL;
+
+ if (vdb->vocs->vostate)
+ return vdb->vocs->vostate;
+
+ return NULL;
+}
+
static struct bt_vcs *vcp_get_vcs(struct bt_vcp *vcp)
{
if (!vcp)
@@ -173,6 +250,20 @@ static struct bt_vcs *vcp_get_vcs(struct bt_vcp *vcp)
return vcp->rdb->vcs;
}
+static struct bt_vocs *vcp_get_vocs(struct bt_vcp *vcp)
+{
+ if (!vcp)
+ return NULL;
+
+ if (vcp->rdb->vocs)
+ return vcp->rdb->vocs;
+
+ vcp->rdb->vocs = new0(struct bt_vocs, 1);
+ vcp->rdb->vocs->vdb = vcp->rdb;
+
+ return vcp->rdb->vocs;
+}
+
static void vcp_detached(void *data, void *user_data)
{
struct bt_vcp_cb *cb = data;
@@ -202,6 +293,7 @@ static void vcp_db_free(void *data)
gatt_db_unref(vdb->db);
free(vdb->vcs);
+ free(vdb->vocs);
free(vdb);
}
@@ -583,6 +675,45 @@ static uint8_t vcs_mute(struct bt_vcs *vcs, struct bt_vcp *vcp,
return 0;
}
+static uint8_t vocs_set_vol_offset(struct bt_vocs *vocs, struct bt_vcp *vcp,
+ struct iovec *iov)
+{
+ struct bt_vcp_db *vdb;
+ struct vol_offset_state *vstate;
+ struct bt_vocs_set_vol_off *req;
+
+ DBG(vcp, "Set Volume Offset");
+
+ vdb = vcp_get_vdb(vcp);
+ if (!vdb) {
+ DBG(vcp, "error: VDB not available");
+ return 0;
+ }
+
+ vstate = vdb_get_vostate(vdb);
+ if (!vstate) {
+ DBG(vcp, "error: VSTATE not available");
+ return 0;
+ }
+
+ req = iov_pull_mem(iov, sizeof(*req));
+ if (!req)
+ return 0;
+
+ if (req->change_counter != vstate->counter) {
+ DBG(vcp, "Change Counter Mismatch Volume not decremented!");
+ return BT_ATT_ERROR_INVALID_CHANGE_COUNTER;
+ }
+
+ vstate->vol_offset = req->set_vol_offset;
+ vstate->counter = -~vstate->counter; /*Increment Change Counter*/
+
+ gatt_db_attribute_notify(vdb->vocs->vos, (void *)vstate,
+ sizeof(struct vol_offset_state),
+ bt_vcp_get_att(vcp));
+ return 0;
+}
+
#define BT_VCS_REL_VOL_DOWN 0x00
#define BT_VCS_REL_VOL_UP 0x01
#define BT_VCS_UNMUTE_REL_VOL_DOWN 0x02
@@ -591,6 +722,8 @@ static uint8_t vcs_mute(struct bt_vcs *vcs, struct bt_vcp *vcp,
#define BT_VCS_UNMUTE 0x05
#define BT_VCS_MUTE 0x06
+#define BT_VOCS_SET_VOL_OFFSET 0x01
+
#define VCS_OP(_str, _op, _size, _func) \
{ \
.str = _str, \
@@ -623,6 +756,26 @@ struct vcs_op_handler {
{}
};
+#define VOCS_OP(_str, _op, _size, _func) \
+ { \
+ .str = _str, \
+ .op = _op, \
+ .size = _size, \
+ .func = _func, \
+ }
+
+struct vocs_op_handler {
+ const char *str;
+ uint8_t op;
+ size_t size;
+ uint8_t (*func)(struct bt_vocs *vocs, struct bt_vcp *vcp,
+ struct iovec *iov);
+} vocp_handlers[] = {
+ VOCS_OP("Set Volume Offset", BT_VOCS_SET_VOL_OFFSET,
+ sizeof(uint8_t), vocs_set_vol_offset),
+ {}
+};
+
static void vcs_cp_write(struct gatt_db_attribute *attrib,
unsigned int id, uint16_t offset,
const uint8_t *value, size_t len,
@@ -683,6 +836,66 @@ respond:
gatt_db_attribute_write_result(attrib, id, ret);
}
+static void vocs_cp_write(struct gatt_db_attribute *attrib,
+ unsigned int id, uint16_t offset,
+ const uint8_t *value, size_t len,
+ uint8_t opcode, struct bt_att *att,
+ void *user_data)
+{
+ struct bt_vocs *vocs = user_data;
+ struct bt_vcp *vcp = vcp_get_session(att, vocs->vdb->db);
+ struct iovec iov = {
+ .iov_base = (void *) value,
+ .iov_len = len,
+ };
+ uint8_t *vcp_op;
+ struct vocs_op_handler *handler;
+ uint8_t ret = BT_ATT_ERROR_REQUEST_NOT_SUPPORTED;
+
+ DBG(vcp, "VOCP Control Point Write");
+
+ if (offset) {
+ DBG(vcp, "invalid offset %d", offset);
+ ret = BT_ATT_ERROR_INVALID_OFFSET;
+ goto respond;
+ }
+
+ if (len < sizeof(*vcp_op)) {
+ DBG(vcp, "invalid len %ld < %ld sizeof(*param)", len,
+ sizeof(*vcp_op));
+ ret = BT_ATT_ERROR_INVALID_ATTRIBUTE_VALUE_LEN;
+ goto respond;
+ }
+
+ vcp_op = iov_pull_mem(&iov, sizeof(*vcp_op));
+
+ for (handler = vocp_handlers; handler && handler->str; handler++) {
+ if (handler->op != *vcp_op)
+ continue;
+
+ if (iov.iov_len < handler->size) {
+ DBG(vcp, "invalid len %ld < %ld handler->size", len,
+ handler->size);
+ ret = BT_ATT_ERROR_OPCODE_NOT_SUPPORTED;
+ goto respond;
+ }
+
+ break;
+ }
+
+ if (handler && handler->str) {
+ DBG(vcp, "%s", handler->str);
+
+ ret = handler->func(vocs, vcp, &iov);
+ } else {
+ DBG(vcp, "Unknown opcode 0x%02x", *vcp_op);
+ ret = BT_ATT_ERROR_OPCODE_NOT_SUPPORTED;
+ }
+
+respond:
+ gatt_db_attribute_write_result(attrib, id, ret);
+}
+
static void vcs_state_read(struct gatt_db_attribute *attrib,
unsigned int id, uint16_t offset,
uint8_t opcode, struct bt_att *att,
@@ -698,6 +911,21 @@ static void vcs_state_read(struct gatt_db_attribute *attrib,
iov.iov_len);
}
+static void vocs_state_read(struct gatt_db_attribute *attrib,
+ unsigned int id, uint16_t offset,
+ uint8_t opcode, struct bt_att *att,
+ void *user_data)
+{
+ struct bt_vocs *vocs = user_data;
+ struct iovec iov;
+
+ iov.iov_base = vocs->vostate;
+ iov.iov_len = sizeof(*vocs->vostate);
+
+ gatt_db_attribute_read_result(attrib, id, 0, iov.iov_base,
+ iov.iov_len);
+}
+
static void vcs_flag_read(struct gatt_db_attribute *attrib,
unsigned int id, uint16_t offset,
uint8_t opcode, struct bt_att *att,
@@ -713,6 +941,36 @@ static void vcs_flag_read(struct gatt_db_attribute *attrib,
iov.iov_len);
}
+static void vocs_voal_read(struct gatt_db_attribute *attrib,
+ unsigned int id, uint16_t offset,
+ uint8_t opcode, struct bt_att *att,
+ void *user_data)
+{
+ struct bt_vocs *vocs = user_data;
+ struct iovec iov;
+
+ iov.iov_base = &vocs->vocs_audio_loc;
+ iov.iov_len = sizeof(vocs->vocs_audio_loc);
+
+ gatt_db_attribute_read_result(attrib, id, 0, iov.iov_base,
+ iov.iov_len);
+}
+
+static void vocs_voaodec_read(struct gatt_db_attribute *attrib,
+ unsigned int id, uint16_t offset,
+ uint8_t opcode, struct bt_att *att,
+ void *user_data)
+{
+ struct bt_vocs *vocs = user_data;
+ struct iovec iov;
+
+ iov.iov_base = &vocs->vocs_ao_dec;
+ iov.iov_len = strlen(vocs->vocs_ao_dec);
+
+ gatt_db_attribute_read_result(attrib, id, 0, iov.iov_base,
+ iov.iov_len);
+}
+
static struct bt_vcs *vcs_new(struct gatt_db *db)
{
struct bt_vcs *vcs;
@@ -771,6 +1029,74 @@ static struct bt_vcs *vcs_new(struct gatt_db *db)
return vcs;
}
+static struct bt_vocs *vocs_new(struct gatt_db *db)
+{
+ struct bt_vocs *vocs;
+ struct vol_offset_state *vostate;
+ bt_uuid_t uuid;
+
+ if (!db)
+ return NULL;
+
+ vocs = new0(struct bt_vocs, 1);
+
+ vostate = new0(struct vol_offset_state, 1);
+
+ vocs->vostate = vostate;
+ vocs->vocs_audio_loc = GEN_AUDIO_AUDLOC_FL;
+ vocs->vocs_ao_dec = "Left Speaker";
+
+ /* Populate DB with VOCS attributes */
+ bt_uuid16_create(&uuid, VOL_OFFSET_CS_UUID);
+ vocs->service = gatt_db_add_service(db, &uuid, true, 9);
+
+ bt_uuid16_create(&uuid, VOL_OFFSET_STATE_CHAR_UUID);
+ vocs->vos = gatt_db_service_add_characteristic(vocs->service,
+ &uuid,
+ BT_ATT_PERM_READ,
+ BT_GATT_CHRC_PROP_READ |
+ BT_GATT_CHRC_PROP_NOTIFY,
+ vocs_state_read, NULL,
+ vocs);
+
+ vocs->vos_ccc = gatt_db_service_add_ccc(vocs->service,
+ BT_ATT_PERM_READ | BT_ATT_PERM_WRITE);
+
+ bt_uuid16_create(&uuid, AUDIO_LOCATION_CHRC_UUID);
+ vocs->voal = gatt_db_service_add_characteristic(vocs->service,
+ &uuid,
+ BT_ATT_PERM_READ,
+ BT_GATT_CHRC_PROP_READ |
+ BT_GATT_CHRC_PROP_NOTIFY,
+ vocs_voal_read, NULL,
+ vocs);
+
+ vocs->voal_ccc = gatt_db_service_add_ccc(vocs->service,
+ BT_ATT_PERM_READ | BT_ATT_PERM_WRITE);
+
+ bt_uuid16_create(&uuid, VOL_OFFSET_CP_CHRC_UUID);
+ vocs->vo_cp = gatt_db_service_add_characteristic(vocs->service,
+ &uuid,
+ BT_ATT_PERM_WRITE,
+ BT_GATT_CHRC_PROP_WRITE,
+ NULL, vocs_cp_write,
+ vocs);
+
+ bt_uuid16_create(&uuid, AUDIO_OUTPUT_DESC_CHAR_UUID);
+ vocs->voaodec = gatt_db_service_add_characteristic(vocs->service,
+ &uuid,
+ BT_ATT_PERM_READ,
+ BT_GATT_CHRC_PROP_READ |
+ BT_GATT_CHRC_PROP_NOTIFY,
+ vocs_voaodec_read, NULL,
+ vocs);
+
+ vocs->voaodec_ccc = gatt_db_service_add_ccc(vocs->service,
+ BT_ATT_PERM_READ | BT_ATT_PERM_WRITE);
+
+ return vocs;
+}
+
static struct bt_vcp_db *vcp_db_new(struct gatt_db *db)
{
struct bt_vcp_db *vdb;
@@ -787,6 +1113,9 @@ static struct bt_vcp_db *vcp_db_new(struct gatt_db *db)
vdb->vcs = vcs_new(db);
vdb->vcs->vdb = vdb;
+ vdb->vocs = vocs_new(db);
+ vdb->vocs->vdb = vdb;
+
queue_push_tail(vcp_db, vdb);
return vdb;
@@ -911,6 +1240,41 @@ static void vcp_vstate_notify(struct bt_vcp *vcp, uint16_t value_handle,
DBG(vcp, "Vol Counter 0x%x", vstate.counter);
}
+static void vcp_voffset_state_notify(struct bt_vcp *vcp, uint16_t value_handle,
+ const uint8_t *value, uint16_t length,
+ void *user_data)
+{
+ struct vol_offset_state vostate;
+
+ memcpy(&vostate, value, sizeof(struct vol_offset_state));
+
+ DBG(vcp, "Vol Offset 0x%x", vostate.vol_offset);
+ DBG(vcp, "Vol Offset Counter 0x%x", vostate.counter);
+}
+
+static void vcp_audio_loc_notify(struct bt_vcp *vcp, uint16_t value_handle,
+ const uint8_t *value, uint16_t length,
+ void *user_data)
+{
+ uint32_t *vocs_audio_loc_n = 0;
+
+ memcpy(vocs_audio_loc_n, value, sizeof(uint32_t));
+
+ DBG(vcp, "VOCS Audio Location 0x%x", *vocs_audio_loc_n);
+}
+
+
+static void vcp_audio_descriptor_notify(struct bt_vcp *vcp, uint16_t value_handle,
+ const uint8_t *value, uint16_t length,
+ void *user_data)
+{
+ char vocs_audio_dec_n[256] = {'\0'};
+
+ memcpy(vocs_audio_dec_n, value, length);
+
+ DBG(vcp, "VOCS Audio Descriptor 0x%s", *vocs_audio_dec_n);
+}
+
static void vcp_vflag_notify(struct bt_vcp *vcp, uint16_t value_handle,
const uint8_t *value, uint16_t length,
void *user_data)
@@ -972,6 +1336,80 @@ static void read_vol_state(struct bt_vcp *vcp, bool success, uint8_t att_ecode,
DBG(vcp, "Vol Counter:%x", vs->counter);
}
+static void read_vol_offset_state(struct bt_vcp *vcp, bool success, uint8_t att_ecode,
+ const uint8_t *value, uint16_t length,
+ void *user_data)
+{
+ struct vol_offset_state *vos;
+ struct iovec iov = {
+ .iov_base = (void *) value,
+ .iov_len = length,
+ };
+
+ if (!success) {
+ DBG(vcp, "Unable to read Vol Offset State: error 0x%02x", att_ecode);
+ return;
+ }
+
+ vos = iov_pull_mem(&iov, sizeof(*vos));
+ if (!vos) {
+ DBG(vcp, "Unable to get Vol Offset State");
+ return;
+ }
+
+ DBG(vcp, "Vol Set:%x", vos->vol_offset);
+ DBG(vcp, "Vol Counter:%x", vos->counter);
+}
+
+static void read_vocs_audio_location(struct bt_vcp *vcp, bool success, uint8_t att_ecode,
+ const uint8_t *value, uint16_t length,
+ void *user_data)
+{
+ uint32_t *vocs_audio_loc;
+ struct iovec iov = {
+ .iov_base = (void *) value,
+ .iov_len = length,
+ };
+
+ if (!success) {
+ DBG(vcp, "Unable to read VOCS Audio Location: error 0x%02x", att_ecode);
+ return;
+ }
+
+ vocs_audio_loc = iov_pull_mem(&iov, sizeof(uint32_t));
+ if (!*vocs_audio_loc) {
+ DBG(vcp, "Unable to get VOCS Audio Location");
+ return;
+ }
+
+ DBG(vcp, "VOCS Audio Loc:%x", *vocs_audio_loc);
+}
+
+
+static void read_vocs_audio_descriptor(struct bt_vcp *vcp, bool success, uint8_t att_ecode,
+ const uint8_t *value, uint16_t length,
+ void *user_data)
+{
+ char *vocs_ao_dec_r;
+ struct iovec iov = {
+ .iov_base = (void *) value,
+ .iov_len = length,
+ };
+
+ if (!success) {
+ DBG(vcp, "Unable to read VOCS Audio Descriptor: error 0x%02x", att_ecode);
+ return;
+ }
+
+ vocs_ao_dec_r = iov_pull_mem(&iov, length);
+ if (!*vocs_ao_dec_r) {
+ DBG(vcp, "Unable to get VOCS Audio Descriptor");
+ return;
+ }
+
+ DBG(vcp, "VOCS Audio Descriptor:%s", *vocs_ao_dec_r);
+}
+
static void vcp_pending_destroy(void *data)
{
struct bt_vcp_pending *pending = data;
@@ -1128,6 +1566,85 @@ static void foreach_vcs_char(struct gatt_db_attribute *attr, void *user_data)
}
}
+static void foreach_vocs_char(struct gatt_db_attribute *attr, void *user_data)
+{
+ struct bt_vcp *vcp = user_data;
+ uint16_t value_handle;
+ bt_uuid_t uuid, uuid_vostate, uuid_audio_loc, uuid_vo_cp, uuid_audio_op_decs;
+ struct bt_vocs *vocs;
+
+ if (!gatt_db_attribute_get_char_data(attr, NULL, &value_handle,
+ NULL, NULL, &uuid))
+ return;
+
+ bt_uuid16_create(&uuid_vostate, VOL_OFFSET_STATE_CHAR_UUID);
+ bt_uuid16_create(&uuid_audio_loc, AUDIO_LOCATION_CHRC_UUID);
+ bt_uuid16_create(&uuid_vo_cp, VOL_OFFSET_CP_CHRC_UUID);
+ bt_uuid16_create(&uuid_audio_op_decs, AUDIO_OUTPUT_DESC_CHAR_UUID);
+
+ if (!bt_uuid_cmp(&uuid, &uuid_vostate)) {
+ DBG(vcp, "VOCS Vol state found: handle 0x%04x", value_handle);
+
+ vocs = vcp_get_vocs(vcp);
+ if (!vocs || vocs->vos)
+ return;
+
+ vocs->vos = attr;
+
+ vcp_read_value(vcp, value_handle, read_vol_offset_state, vcp);
+
+ vcp->vostate_id = vcp_register_notify(vcp, value_handle,
+ vcp_voffset_state_notify, NULL);
+
+ return;
+ }
+
+ if (!bt_uuid_cmp(&uuid, &uuid_audio_loc)) {
+ DBG(vcp, "VOCS Volume Audio Location found: handle 0x%04x", value_handle);
+
+ vocs = vcp_get_vocs(vcp);
+ if (!vocs || vocs->voal)
+ return;
+
+ vocs->voal = attr;
+
+ vcp_read_value(vcp, value_handle, read_vocs_audio_location, vcp);
+
+ vcp->vocs_audio_loc_id = vcp_register_notify(vcp, value_handle,
+ vcp_audio_loc_notify, NULL);
+
+ return;
+ }
+
+ if (!bt_uuid_cmp(&uuid, &uuid_vo_cp)) {
+ DBG(vcp, "VOCS Volume CP found: handle 0x%04x", value_handle);
+
+ vocs = vcp_get_vocs(vcp);
+ if (!vocs || vocs->vo_cp)
+ return;
+
+ vocs->vo_cp = attr;
+
+ return;
+ }
+
+ if (!bt_uuid_cmp(&uuid, &uuid_audio_op_decs)) {
+ DBG(vcp, "VOCS Vol Audio Descriptor found: handle 0x%04x", value_handle);
+
+ vocs = vcp_get_vocs(vcp);
+ if (!vocs || vocs->voaodec)
+ return;
+
+ vocs->voaodec = attr;
+
+ vcp_read_value(vcp, value_handle, read_vocs_audio_descriptor, vcp);
+ vcp->vocs_ao_dec_id = vcp_register_notify(vcp, value_handle,
+ vcp_audio_descriptor_notify, NULL);
+
+ }
+
+}
+
static void foreach_vcs_service(struct gatt_db_attribute *attr,
void *user_data)
{
@@ -1141,6 +1658,19 @@ static void foreach_vcs_service(struct gatt_db_attribute *attr,
gatt_db_service_foreach_char(attr, foreach_vcs_char, vcp);
}
+static void foreach_vocs_service(struct gatt_db_attribute *attr,
+ void *user_data)
+{
+ struct bt_vcp *vcp = user_data;
+ struct bt_vocs *vocs = vcp_get_vocs(vcp);
+
+ vocs->service = attr;
+
+ gatt_db_service_set_claimed(attr, true);
+
+ gatt_db_service_foreach_char(attr, foreach_vocs_char, vcp);
+}
+
bool bt_vcp_attach(struct bt_vcp *vcp, struct bt_gatt_client *client)
{
bt_uuid_t uuid;
@@ -1163,6 +1693,9 @@ bool bt_vcp_attach(struct bt_vcp *vcp, struct bt_gatt_client *client)
bt_uuid16_create(&uuid, VCS_UUID);
gatt_db_foreach_service(vcp->ldb->db, &uuid, foreach_vcs_service, vcp);
+ bt_uuid16_create(&uuid, VOL_OFFSET_CS_UUID);
+ gatt_db_foreach_service(vcp->ldb->db, &uuid, foreach_vocs_service, vcp);
+
return true;
}
--
2.34.1
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=754428
---Test result---
Test Summary:
CheckPatch FAIL 1.73 seconds
GitLint FAIL 0.91 seconds
BuildEll PASS 26.19 seconds
BluezMake PASS 755.55 seconds
MakeCheck PASS 11.55 seconds
MakeDistcheck PASS 152.47 seconds
CheckValgrind PASS 246.78 seconds
CheckSmatch PASS 330.82 seconds
bluezmakeextell PASS 99.84 seconds
IncrementalBuild PASS 1273.46 seconds
ScanBuild WARNING 986.70 seconds
Details
##############################
Test: CheckPatch - FAIL
Desc: Run checkpatch.pl script
Output:
[BlueZ,v1,1/2] Added initial code for handling VOCS
WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line)
#97:
This commit handles the code implementation of VOCS service and characteristics
ERROR:TRAILING_WHITESPACE: trailing whitespace
#528: FILE: src/shared/vcp.c:1076:
+^I$
WARNING:LONG_LINE: line length of 82 exceeds 80 columns
#593: FILE: src/shared/vcp.c:1267:
+static void vcp_audio_descriptor_notify(struct bt_vcp *vcp, uint16_t value_handle,
WARNING:LONG_LINE: line length of 86 exceeds 80 columns
#611: FILE: src/shared/vcp.c:1339:
+static void read_vol_offset_state(struct bt_vcp *vcp, bool success, uint8_t att_ecode,
WARNING:LONG_LINE: line length of 85 exceeds 80 columns
#622: FILE: src/shared/vcp.c:1350:
+ DBG(vcp, "Unable to read Vol Offset State: error 0x%02x", att_ecode);
WARNING:LONG_LINE: line length of 89 exceeds 80 columns
#636: FILE: src/shared/vcp.c:1364:
+static void read_vocs_audio_location(struct bt_vcp *vcp, bool success, uint8_t att_ecode,
WARNING:LONG_LINE: line length of 88 exceeds 80 columns
#647: FILE: src/shared/vcp.c:1375:
+ DBG(vcp, "Unable to read VOCS Audio Location: error 0x%02x", att_ecode);
WARNING:LONG_LINE: line length of 91 exceeds 80 columns
#661: FILE: src/shared/vcp.c:1389:
+static void read_vocs_audio_descriptor(struct bt_vcp *vcp, bool success, uint8_t att_ecode,
WARNING:LONG_LINE: line length of 90 exceeds 80 columns
#672: FILE: src/shared/vcp.c:1400:
+ DBG(vcp, "Unable to read VOCS Audio Descriptor: error 0x%02x", att_ecode);
WARNING:LONG_LINE: line length of 85 exceeds 80 columns
#696: FILE: src/shared/vcp.c:1573:
+ bt_uuid_t uuid, uuid_vostate, uuid_audio_loc, uuid_vo_cp, uuid_audio_op_decs;
WARNING:LONG_LINE: line length of 85 exceeds 80 columns
#720: FILE: src/shared/vcp.c:1597:
+ vcp_voffset_state_notify, NULL);
WARNING:LONG_LINE: line length of 90 exceeds 80 columns
#726: FILE: src/shared/vcp.c:1603:
+ DBG(vcp, "VOCS Volume Audio Location found: handle 0x%04x", value_handle);
ERROR:TRAILING_WHITESPACE: trailing whitespace
#729: FILE: src/shared/vcp.c:1606:
+^I^Iif (!vocs || vocs->voal) $
WARNING:LONG_LINE: line length of 81 exceeds 80 columns
#734: FILE: src/shared/vcp.c:1611:
+ vcp_read_value(vcp, value_handle, read_vocs_audio_location, vcp);
WARNING:LONG_LINE: line length of 81 exceeds 80 columns
#737: FILE: src/shared/vcp.c:1614:
+ vcp_audio_loc_notify, NULL);
WARNING:LONG_LINE: line length of 89 exceeds 80 columns
#755: FILE: src/shared/vcp.c:1632:
+ DBG(vcp, "VOCS Vol Audio Descriptor found: handle 0x%04x", value_handle);
WARNING:LONG_LINE: line length of 83 exceeds 80 columns
#763: FILE: src/shared/vcp.c:1640:
+ vcp_read_value(vcp, value_handle, read_vocs_audio_descriptor, vcp);
WARNING:LONG_LINE: line length of 87 exceeds 80 columns
#765: FILE: src/shared/vcp.c:1642:
+ vcp_audio_descriptor_notify, NULL);
/github/workspace/src/src/13269044.patch total: 2 errors, 16 warnings, 672 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/src/13269044.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.
[BlueZ,v1,2/2] Fixed the following issue observed during PTS testing for mandatory test cases - Specified Upper and Lower Limit for Volume offset - Corrected the number of handles for VOCS - VOCS is made as included service of VCS because VOCS is secondar
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#137: FILE: src/shared/vcp.c:174:
+ int16_t vol_offset;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#138: FILE: src/shared/vcp.c:175:
+ uint8_t counter;$
WARNING:LONG_LINE: line length of 116 exceeds 80 columns
#146: FILE: src/shared/vcp.c:712:
+ if(req->set_vol_offset > VOCS_VOL_OFFSET_UPPER_LIMIT || req->set_vol_offset < VOCS_VOL_OFFSET_LOWER_LIMIT) {
ERROR:SPACING: space required before the open parenthesis '('
#146: FILE: src/shared/vcp.c:712:
+ if(req->set_vol_offset > VOCS_VOL_OFFSET_UPPER_LIMIT || req->set_vol_offset < VOCS_VOL_OFFSET_LOWER_LIMIT) {
/github/workspace/src/src/13269045.patch total: 1 errors, 3 warnings, 79 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.
/github/workspace/src/src/13269045.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.
##############################
Test: GitLint - FAIL
Desc: Run gitlint
Output:
[BlueZ,v1,2/2] Fixed the following issue observed during PTS testing for mandatory test cases - Specified Upper and Lower Limit for Volume offset - Corrected the number of handles for VOCS - VOCS is made as included service of VCS because VOCS is secondar
WARNING: I3 - ignore-body-lines: gitlint will be switching from using Python regex 'match' (match beginning) to 'search' (match anywhere) semantics. Please review your ignore-body-lines.regex option accordingly. To remove this warning, set general.regex-style-search=True. More details: https://jorisroovers.github.io/gitlint/configuration/#regex-style-search
1: T1 Title exceeds max length (255>80): "[BlueZ,v1,2/2] Fixed the following issue observed during PTS testing for mandatory test cases - Specified Upper and Lower Limit for Volume offset - Corrected the number of handles for VOCS - VOCS is made as included service of VCS because VOCS is secondar"
##############################
Test: ScanBuild - WARNING
Desc: Run Scan Build
Output:
src/shared/vcp.c:1271:2: warning: Null pointer passed to 1st parameter expecting 'nonnull'
memcpy(vocs_audio_loc_n, value, sizeof(uint32_t));
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1 warning generated.
---
Regards,
Linux Bluetooth
Hi Nitin,
On Tue, Jun 6, 2023 at 5:09 AM NJNXP <[email protected]> wrote:
>
> This commit handles the code implementation of VOCS service and characteristics
> ---
> lib/uuid.h | 5 +
> src/shared/vcp.c | 533 +++++++++++++++++++++++++++++++++++++++++++++++
> 2 files changed, 538 insertions(+)
>
> diff --git a/lib/uuid.h b/lib/uuid.h
> index 5cdfedb4b..e8c7f8e37 100644
> --- a/lib/uuid.h
> +++ b/lib/uuid.h
> @@ -179,6 +179,11 @@ extern "C" {
> #define VOL_CP_CHRC_UUID 0x2B7E
> #define VOL_FLAG_CHRC_UUID 0x2B7F
>
> +#define VOL_OFFSET_STATE_CHAR_UUID 0x2B80
> +#define AUDIO_LOCATION_CHRC_UUID 0x2B81
> +#define VOL_OFFSET_CP_CHRC_UUID 0x2B82
> +#define AUDIO_OUTPUT_DESC_CHAR_UUID 0x2B83
It might be a good idea to prefix these with VCS just to make sure,
something like:
<SERVICE>_<ATTRIBUTE NAME>_UUID
> #define GMCS_UUID 0x1849
> #define MEDIA_PLAYER_NAME_CHRC_UUID 0x2b93
> #define MEDIA_TRACK_CHNGD_CHRC_UUID 0x2b96
> diff --git a/src/shared/vcp.c b/src/shared/vcp.c
> index 5459cf892..05e341108 100644
> --- a/src/shared/vcp.c
> +++ b/src/shared/vcp.c
> @@ -36,9 +36,40 @@
> #define BT_ATT_ERROR_INVALID_CHANGE_COUNTER 0x80
> #define BT_ATT_ERROR_OPCODE_NOT_SUPPORTED 0x81
>
> +#define GEN_AUDIO_AUDLOC_NA 0x00000000
> +#define GEN_AUDIO_AUDLOC_FL 0x00000001
> +#define GEN_AUDIO_AUDLOC_FR 0x00000002
> +#define GEN_AUDIO_AUDLOC_FC 0x00000004
> +#define GEN_AUDIO_AUDLOC_LOW_FRQ_EFF_1 0x00000008
> +#define GEN_AUDIO_AUDLOC_BL 0x00000010
> +#define GEN_AUDIO_AUDLOC_BR 0x00000020
> +#define GEN_AUDIO_AUDLOC_FLC 0x00000040
> +#define GEN_AUDIO_AUDLOC_FRC 0x00000080
> +#define GEN_AUDIO_AUDLOC_BC 0x00000100
> +#define GEN_AUDIO_AUDLOC_LOW_FRQ_EFF_2 0x00000200
> +#define GEN_AUDIO_AUDLOC_SL 0x00000400
> +#define GEN_AUDIO_AUDLOC_SR 0x00000800
> +#define GEN_AUDIO_AUDLOC_TFL 0x00001000
> +#define GEN_AUDIO_AUDLOC_TFR 0x00002000
> +#define GEN_AUDIO_AUDLOC_TFC 0x00004000
> +#define GEN_AUDIO_AUDLOC_TC 0x00008000
> +#define GEN_AUDIO_AUDLOC_TBL 0x00010000
> +#define GEN_AUDIO_AUDLOC_TBR 0x00020000
> +#define GEN_AUDIO_AUDLOC_TSL 0x00040000
> +#define GEN_AUDIO_AUDLOC_TSR 0x00080000
> +#define GEN_AUDIO_AUDLOC_TBC 0x00100000
> +#define GEN_AUDIO_AUDLOC_BFC 0x00200000
> +#define GEN_AUDIO_AUDLOC_BFL 0x00400000
> +#define GEN_AUDIO_AUDLOC_BFR 0x00800000
> +#define GEN_AUDIO_AUDLOC_FLW 0x01000000
> +#define GEN_AUDIO_AUDLOC_FRW 0x02000000
> +#define GEN_AUDIO_AUDLOC_LS 0x04000000
> +#define GEN_AUDIO_AUDLOC_RS 0x08000000
Id prefer we do something like: BT_VCP_FRONT_LEFT, BT_VCP_FRONT_RIGHT,
or we have it in a common header since it appears this is shared with
the likes of PACS.
> struct bt_vcp_db {
> struct gatt_db *db;
> struct bt_vcs *vcs;
> + struct bt_vocs *vocs;
> };
>
> typedef void (*vcp_func_t)(struct bt_vcp *vcp, bool success, uint8_t att_ecode,
> @@ -57,11 +88,21 @@ struct bt_vcs_param {
> uint8_t change_counter;
> } __packed;
>
> +struct bt_vocs_param {
> + uint8_t op;
> + uint8_t change_counter;
> +} __packed;
> +
> struct bt_vcs_ab_vol {
> uint8_t change_counter;
> uint8_t vol_set;
> } __packed;
>
> +struct bt_vocs_set_vol_off {
> + uint8_t change_counter;
> + uint8_t set_vol_offset;
> +} __packed;
> +
> struct bt_vcp_cb {
> unsigned int id;
> bt_vcp_func_t attached;
> @@ -89,6 +130,10 @@ struct bt_vcp {
> unsigned int vstate_id;
> unsigned int vflag_id;
>
> + unsigned int vostate_id;
> + unsigned int vocs_audio_loc_id;
> + unsigned int vocs_ao_dec_id;
No need to prefix these if they can be inferred by the struct name, so
you can use state_id, location_id, etc.
> +
> struct queue *notify;
> struct queue *pending;
>
> @@ -120,6 +165,27 @@ struct bt_vcs {
> struct gatt_db_attribute *vf_ccc;
> };
>
> +/* Contains local bt_vcp_db */
> +struct vol_offset_state {
> + uint16_t vol_offset;
> + uint8_t counter;
> +} __packed;
> +
> +struct bt_vocs {
> + struct bt_vcp_db *vdb;
> + struct vol_offset_state *vostate;
> + uint32_t vocs_audio_loc;
> + char *vocs_ao_dec;
> + struct gatt_db_attribute *service;
> + struct gatt_db_attribute *vos;
> + struct gatt_db_attribute *vos_ccc;
> + struct gatt_db_attribute *voal;
> + struct gatt_db_attribute *voal_ccc;
> + struct gatt_db_attribute *vo_cp;
> + struct gatt_db_attribute *voaodec;
> + struct gatt_db_attribute *voaodec_ccc;
> +};
> +
> static struct queue *vcp_db;
> static struct queue *vcp_cbs;
> static struct queue *sessions;
> @@ -159,6 +225,17 @@ static struct vol_state *vdb_get_vstate(struct bt_vcp_db *vdb)
> return NULL;
> }
>
> +static struct vol_offset_state *vdb_get_vostate(struct bt_vcp_db *vdb)
> +{
> + if (!vdb->vocs)
> + return NULL;
> +
> + if (vdb->vocs->vostate)
> + return vdb->vocs->vostate;
> +
> + return NULL;
> +}
> +
> static struct bt_vcs *vcp_get_vcs(struct bt_vcp *vcp)
> {
> if (!vcp)
> @@ -173,6 +250,20 @@ static struct bt_vcs *vcp_get_vcs(struct bt_vcp *vcp)
> return vcp->rdb->vcs;
> }
>
> +static struct bt_vocs *vcp_get_vocs(struct bt_vcp *vcp)
> +{
> + if (!vcp)
> + return NULL;
> +
> + if (vcp->rdb->vocs)
> + return vcp->rdb->vocs;
> +
> + vcp->rdb->vocs = new0(struct bt_vocs, 1);
> + vcp->rdb->vocs->vdb = vcp->rdb;
> +
> + return vcp->rdb->vocs;
> +}
> +
> static void vcp_detached(void *data, void *user_data)
> {
> struct bt_vcp_cb *cb = data;
> @@ -202,6 +293,7 @@ static void vcp_db_free(void *data)
> gatt_db_unref(vdb->db);
>
> free(vdb->vcs);
> + free(vdb->vocs);
> free(vdb);
> }
>
> @@ -583,6 +675,45 @@ static uint8_t vcs_mute(struct bt_vcs *vcs, struct bt_vcp *vcp,
> return 0;
> }
>
> +static uint8_t vocs_set_vol_offset(struct bt_vocs *vocs, struct bt_vcp *vcp,
> + struct iovec *iov)
> +{
> + struct bt_vcp_db *vdb;
> + struct vol_offset_state *vstate;
> + struct bt_vocs_set_vol_off *req;
> +
> + DBG(vcp, "Set Volume Offset");
> +
> + vdb = vcp_get_vdb(vcp);
> + if (!vdb) {
> + DBG(vcp, "error: VDB not available");
> + return 0;
> + }
> +
> + vstate = vdb_get_vostate(vdb);
> + if (!vstate) {
> + DBG(vcp, "error: VSTATE not available");
> + return 0;
> + }
> +
> + req = iov_pull_mem(iov, sizeof(*req));
> + if (!req)
> + return 0;
> +
> + if (req->change_counter != vstate->counter) {
> + DBG(vcp, "Change Counter Mismatch Volume not decremented!");
> + return BT_ATT_ERROR_INVALID_CHANGE_COUNTER;
> + }
> +
> + vstate->vol_offset = req->set_vol_offset;
> + vstate->counter = -~vstate->counter; /*Increment Change Counter*/
> +
> + gatt_db_attribute_notify(vdb->vocs->vos, (void *)vstate,
> + sizeof(struct vol_offset_state),
> + bt_vcp_get_att(vcp));
> + return 0;
> +}
> +
> #define BT_VCS_REL_VOL_DOWN 0x00
> #define BT_VCS_REL_VOL_UP 0x01
> #define BT_VCS_UNMUTE_REL_VOL_DOWN 0x02
> @@ -591,6 +722,8 @@ static uint8_t vcs_mute(struct bt_vcs *vcs, struct bt_vcp *vcp,
> #define BT_VCS_UNMUTE 0x05
> #define BT_VCS_MUTE 0x06
>
> +#define BT_VOCS_SET_VOL_OFFSET 0x01
> +
> #define VCS_OP(_str, _op, _size, _func) \
> { \
> .str = _str, \
> @@ -623,6 +756,26 @@ struct vcs_op_handler {
> {}
> };
>
> +#define VOCS_OP(_str, _op, _size, _func) \
> + { \
> + .str = _str, \
> + .op = _op, \
> + .size = _size, \
> + .func = _func, \
> + }
> +
> +struct vocs_op_handler {
> + const char *str;
> + uint8_t op;
> + size_t size;
> + uint8_t (*func)(struct bt_vocs *vocs, struct bt_vcp *vcp,
> + struct iovec *iov);
> +} vocp_handlers[] = {
> + VOCS_OP("Set Volume Offset", BT_VOCS_SET_VOL_OFFSET,
> + sizeof(uint8_t), vocs_set_vol_offset),
> + {}
> +};
> +
> static void vcs_cp_write(struct gatt_db_attribute *attrib,
> unsigned int id, uint16_t offset,
> const uint8_t *value, size_t len,
> @@ -683,6 +836,66 @@ respond:
> gatt_db_attribute_write_result(attrib, id, ret);
> }
>
> +static void vocs_cp_write(struct gatt_db_attribute *attrib,
> + unsigned int id, uint16_t offset,
> + const uint8_t *value, size_t len,
> + uint8_t opcode, struct bt_att *att,
> + void *user_data)
> +{
> + struct bt_vocs *vocs = user_data;
> + struct bt_vcp *vcp = vcp_get_session(att, vocs->vdb->db);
> + struct iovec iov = {
> + .iov_base = (void *) value,
> + .iov_len = len,
> + };
> + uint8_t *vcp_op;
> + struct vocs_op_handler *handler;
> + uint8_t ret = BT_ATT_ERROR_REQUEST_NOT_SUPPORTED;
> +
> + DBG(vcp, "VOCP Control Point Write");
> +
> + if (offset) {
> + DBG(vcp, "invalid offset %d", offset);
> + ret = BT_ATT_ERROR_INVALID_OFFSET;
> + goto respond;
> + }
> +
> + if (len < sizeof(*vcp_op)) {
> + DBG(vcp, "invalid len %ld < %ld sizeof(*param)", len,
> + sizeof(*vcp_op));
> + ret = BT_ATT_ERROR_INVALID_ATTRIBUTE_VALUE_LEN;
> + goto respond;
> + }
> +
> + vcp_op = iov_pull_mem(&iov, sizeof(*vcp_op));
> +
> + for (handler = vocp_handlers; handler && handler->str; handler++) {
> + if (handler->op != *vcp_op)
> + continue;
> +
> + if (iov.iov_len < handler->size) {
> + DBG(vcp, "invalid len %ld < %ld handler->size", len,
> + handler->size);
> + ret = BT_ATT_ERROR_OPCODE_NOT_SUPPORTED;
> + goto respond;
> + }
> +
> + break;
> + }
> +
> + if (handler && handler->str) {
> + DBG(vcp, "%s", handler->str);
> +
> + ret = handler->func(vocs, vcp, &iov);
> + } else {
> + DBG(vcp, "Unknown opcode 0x%02x", *vcp_op);
> + ret = BT_ATT_ERROR_OPCODE_NOT_SUPPORTED;
> + }
> +
> +respond:
> + gatt_db_attribute_write_result(attrib, id, ret);
> +}
> +
> static void vcs_state_read(struct gatt_db_attribute *attrib,
> unsigned int id, uint16_t offset,
> uint8_t opcode, struct bt_att *att,
> @@ -698,6 +911,21 @@ static void vcs_state_read(struct gatt_db_attribute *attrib,
> iov.iov_len);
> }
>
> +static void vocs_state_read(struct gatt_db_attribute *attrib,
> + unsigned int id, uint16_t offset,
> + uint8_t opcode, struct bt_att *att,
> + void *user_data)
> +{
> + struct bt_vocs *vocs = user_data;
> + struct iovec iov;
> +
> + iov.iov_base = vocs->vostate;
> + iov.iov_len = sizeof(*vocs->vostate);
> +
> + gatt_db_attribute_read_result(attrib, id, 0, iov.iov_base,
> + iov.iov_len);
> +}
> +
> static void vcs_flag_read(struct gatt_db_attribute *attrib,
> unsigned int id, uint16_t offset,
> uint8_t opcode, struct bt_att *att,
> @@ -713,6 +941,36 @@ static void vcs_flag_read(struct gatt_db_attribute *attrib,
> iov.iov_len);
> }
>
> +static void vocs_voal_read(struct gatt_db_attribute *attrib,
> + unsigned int id, uint16_t offset,
> + uint8_t opcode, struct bt_att *att,
> + void *user_data)
> +{
> + struct bt_vocs *vocs = user_data;
> + struct iovec iov;
> +
> + iov.iov_base = &vocs->vocs_audio_loc;
> + iov.iov_len = sizeof(vocs->vocs_audio_loc);
> +
> + gatt_db_attribute_read_result(attrib, id, 0, iov.iov_base,
> + iov.iov_len);
> +}
> +
> +static void vocs_voaodec_read(struct gatt_db_attribute *attrib,
> + unsigned int id, uint16_t offset,
> + uint8_t opcode, struct bt_att *att,
> + void *user_data)
> +{
> + struct bt_vocs *vocs = user_data;
> + struct iovec iov;
> +
> + iov.iov_base = &vocs->vocs_ao_dec;
> + iov.iov_len = strlen(vocs->vocs_ao_dec);
> +
> + gatt_db_attribute_read_result(attrib, id, 0, iov.iov_base,
> + iov.iov_len);
> +}
> +
> static struct bt_vcs *vcs_new(struct gatt_db *db)
> {
> struct bt_vcs *vcs;
> @@ -771,6 +1029,74 @@ static struct bt_vcs *vcs_new(struct gatt_db *db)
> return vcs;
> }
>
> +static struct bt_vocs *vocs_new(struct gatt_db *db)
> +{
> + struct bt_vocs *vocs;
> + struct vol_offset_state *vostate;
> + bt_uuid_t uuid;
> +
> + if (!db)
> + return NULL;
> +
> + vocs = new0(struct bt_vocs, 1);
> +
> + vostate = new0(struct vol_offset_state, 1);
> +
> + vocs->vostate = vostate;
> + vocs->vocs_audio_loc = GEN_AUDIO_AUDLOC_FL;
> + vocs->vocs_ao_dec = "Left Speaker";
> +
> + /* Populate DB with VOCS attributes */
> + bt_uuid16_create(&uuid, VOL_OFFSET_CS_UUID);
> + vocs->service = gatt_db_add_service(db, &uuid, true, 9);
> +
> + bt_uuid16_create(&uuid, VOL_OFFSET_STATE_CHAR_UUID);
> + vocs->vos = gatt_db_service_add_characteristic(vocs->service,
> + &uuid,
> + BT_ATT_PERM_READ,
> + BT_GATT_CHRC_PROP_READ |
> + BT_GATT_CHRC_PROP_NOTIFY,
> + vocs_state_read, NULL,
> + vocs);
> +
> + vocs->vos_ccc = gatt_db_service_add_ccc(vocs->service,
> + BT_ATT_PERM_READ | BT_ATT_PERM_WRITE);
> +
> + bt_uuid16_create(&uuid, AUDIO_LOCATION_CHRC_UUID);
> + vocs->voal = gatt_db_service_add_characteristic(vocs->service,
> + &uuid,
> + BT_ATT_PERM_READ,
> + BT_GATT_CHRC_PROP_READ |
> + BT_GATT_CHRC_PROP_NOTIFY,
> + vocs_voal_read, NULL,
> + vocs);
> +
> + vocs->voal_ccc = gatt_db_service_add_ccc(vocs->service,
> + BT_ATT_PERM_READ | BT_ATT_PERM_WRITE);
> +
> + bt_uuid16_create(&uuid, VOL_OFFSET_CP_CHRC_UUID);
> + vocs->vo_cp = gatt_db_service_add_characteristic(vocs->service,
> + &uuid,
> + BT_ATT_PERM_WRITE,
> + BT_GATT_CHRC_PROP_WRITE,
> + NULL, vocs_cp_write,
> + vocs);
> +
> + bt_uuid16_create(&uuid, AUDIO_OUTPUT_DESC_CHAR_UUID);
> + vocs->voaodec = gatt_db_service_add_characteristic(vocs->service,
> + &uuid,
> + BT_ATT_PERM_READ,
> + BT_GATT_CHRC_PROP_READ |
> + BT_GATT_CHRC_PROP_NOTIFY,
> + vocs_voaodec_read, NULL,
> + vocs);
> +
> + vocs->voaodec_ccc = gatt_db_service_add_ccc(vocs->service,
> + BT_ATT_PERM_READ | BT_ATT_PERM_WRITE);
> +
> + return vocs;
> +}
> +
> static struct bt_vcp_db *vcp_db_new(struct gatt_db *db)
> {
> struct bt_vcp_db *vdb;
> @@ -787,6 +1113,9 @@ static struct bt_vcp_db *vcp_db_new(struct gatt_db *db)
> vdb->vcs = vcs_new(db);
> vdb->vcs->vdb = vdb;
>
> + vdb->vocs = vocs_new(db);
> + vdb->vocs->vdb = vdb;
> +
> queue_push_tail(vcp_db, vdb);
>
> return vdb;
> @@ -911,6 +1240,41 @@ static void vcp_vstate_notify(struct bt_vcp *vcp, uint16_t value_handle,
> DBG(vcp, "Vol Counter 0x%x", vstate.counter);
> }
>
> +static void vcp_voffset_state_notify(struct bt_vcp *vcp, uint16_t value_handle,
> + const uint8_t *value, uint16_t length,
> + void *user_data)
> +{
> + struct vol_offset_state vostate;
> +
> + memcpy(&vostate, value, sizeof(struct vol_offset_state));
> +
> + DBG(vcp, "Vol Offset 0x%x", vostate.vol_offset);
> + DBG(vcp, "Vol Offset Counter 0x%x", vostate.counter);
> +}
> +
> +static void vcp_audio_loc_notify(struct bt_vcp *vcp, uint16_t value_handle,
> + const uint8_t *value, uint16_t length,
> + void *user_data)
> +{
> + uint32_t *vocs_audio_loc_n = 0;
> +
> + memcpy(vocs_audio_loc_n, value, sizeof(uint32_t));
> +
> + DBG(vcp, "VOCS Audio Location 0x%x", *vocs_audio_loc_n);
> +}
> +
> +
> +static void vcp_audio_descriptor_notify(struct bt_vcp *vcp, uint16_t value_handle,
> + const uint8_t *value, uint16_t length,
> + void *user_data)
> +{
> + char vocs_audio_dec_n[256] = {'\0'};
> +
> + memcpy(vocs_audio_dec_n, value, length);
> +
> + DBG(vcp, "VOCS Audio Descriptor 0x%s", *vocs_audio_dec_n);
> +}
> +
> static void vcp_vflag_notify(struct bt_vcp *vcp, uint16_t value_handle,
> const uint8_t *value, uint16_t length,
> void *user_data)
> @@ -972,6 +1336,80 @@ static void read_vol_state(struct bt_vcp *vcp, bool success, uint8_t att_ecode,
> DBG(vcp, "Vol Counter:%x", vs->counter);
> }
>
> +static void read_vol_offset_state(struct bt_vcp *vcp, bool success, uint8_t att_ecode,
> + const uint8_t *value, uint16_t length,
> + void *user_data)
> +{
> + struct vol_offset_state *vos;
> + struct iovec iov = {
> + .iov_base = (void *) value,
> + .iov_len = length,
> + };
> +
> + if (!success) {
> + DBG(vcp, "Unable to read Vol Offset State: error 0x%02x", att_ecode);
> + return;
> + }
> +
> + vos = iov_pull_mem(&iov, sizeof(*vos));
> + if (!vos) {
> + DBG(vcp, "Unable to get Vol Offset State");
> + return;
> + }
> +
> + DBG(vcp, "Vol Set:%x", vos->vol_offset);
> + DBG(vcp, "Vol Counter:%x", vos->counter);
> +}
> +
> +static void read_vocs_audio_location(struct bt_vcp *vcp, bool success, uint8_t att_ecode,
> + const uint8_t *value, uint16_t length,
> + void *user_data)
> +{
> + uint32_t *vocs_audio_loc;
> + struct iovec iov = {
> + .iov_base = (void *) value,
> + .iov_len = length,
> + };
> +
> + if (!success) {
> + DBG(vcp, "Unable to read VOCS Audio Location: error 0x%02x", att_ecode);
> + return;
> + }
> +
> + vocs_audio_loc = iov_pull_mem(&iov, sizeof(uint32_t));
> + if (!*vocs_audio_loc) {
> + DBG(vcp, "Unable to get VOCS Audio Location");
> + return;
> + }
> +
> + DBG(vcp, "VOCS Audio Loc:%x", *vocs_audio_loc);
> +}
> +
> +
> +static void read_vocs_audio_descriptor(struct bt_vcp *vcp, bool success, uint8_t att_ecode,
> + const uint8_t *value, uint16_t length,
> + void *user_data)
> +{
> + char *vocs_ao_dec_r;
> + struct iovec iov = {
> + .iov_base = (void *) value,
> + .iov_len = length,
> + };
> +
> + if (!success) {
> + DBG(vcp, "Unable to read VOCS Audio Descriptor: error 0x%02x", att_ecode);
> + return;
> + }
> +
> + vocs_ao_dec_r = iov_pull_mem(&iov, length);
> + if (!*vocs_ao_dec_r) {
> + DBG(vcp, "Unable to get VOCS Audio Descriptor");
> + return;
> + }
> +
> + DBG(vcp, "VOCS Audio Descriptor:%s", *vocs_ao_dec_r);
> +}
> +
> static void vcp_pending_destroy(void *data)
> {
> struct bt_vcp_pending *pending = data;
> @@ -1128,6 +1566,85 @@ static void foreach_vcs_char(struct gatt_db_attribute *attr, void *user_data)
> }
> }
>
> +static void foreach_vocs_char(struct gatt_db_attribute *attr, void *user_data)
> +{
> + struct bt_vcp *vcp = user_data;
> + uint16_t value_handle;
> + bt_uuid_t uuid, uuid_vostate, uuid_audio_loc, uuid_vo_cp, uuid_audio_op_decs;
> + struct bt_vocs *vocs;
> +
> + if (!gatt_db_attribute_get_char_data(attr, NULL, &value_handle,
> + NULL, NULL, &uuid))
> + return;
> +
> + bt_uuid16_create(&uuid_vostate, VOL_OFFSET_STATE_CHAR_UUID);
> + bt_uuid16_create(&uuid_audio_loc, AUDIO_LOCATION_CHRC_UUID);
> + bt_uuid16_create(&uuid_vo_cp, VOL_OFFSET_CP_CHRC_UUID);
> + bt_uuid16_create(&uuid_audio_op_decs, AUDIO_OUTPUT_DESC_CHAR_UUID);
> +
> + if (!bt_uuid_cmp(&uuid, &uuid_vostate)) {
> + DBG(vcp, "VOCS Vol state found: handle 0x%04x", value_handle);
> +
> + vocs = vcp_get_vocs(vcp);
> + if (!vocs || vocs->vos)
> + return;
> +
> + vocs->vos = attr;
> +
> + vcp_read_value(vcp, value_handle, read_vol_offset_state, vcp);
> +
> + vcp->vostate_id = vcp_register_notify(vcp, value_handle,
> + vcp_voffset_state_notify, NULL);
> +
> + return;
> + }
> +
> + if (!bt_uuid_cmp(&uuid, &uuid_audio_loc)) {
> + DBG(vcp, "VOCS Volume Audio Location found: handle 0x%04x", value_handle);
> +
> + vocs = vcp_get_vocs(vcp);
> + if (!vocs || vocs->voal)
> + return;
> +
> + vocs->voal = attr;
> +
> + vcp_read_value(vcp, value_handle, read_vocs_audio_location, vcp);
> +
> + vcp->vocs_audio_loc_id = vcp_register_notify(vcp, value_handle,
> + vcp_audio_loc_notify, NULL);
> +
> + return;
> + }
> +
> + if (!bt_uuid_cmp(&uuid, &uuid_vo_cp)) {
> + DBG(vcp, "VOCS Volume CP found: handle 0x%04x", value_handle);
> +
> + vocs = vcp_get_vocs(vcp);
> + if (!vocs || vocs->vo_cp)
> + return;
> +
> + vocs->vo_cp = attr;
> +
> + return;
> + }
> +
> + if (!bt_uuid_cmp(&uuid, &uuid_audio_op_decs)) {
> + DBG(vcp, "VOCS Vol Audio Descriptor found: handle 0x%04x", value_handle);
> +
> + vocs = vcp_get_vocs(vcp);
> + if (!vocs || vocs->voaodec)
> + return;
> +
> + vocs->voaodec = attr;
> +
> + vcp_read_value(vcp, value_handle, read_vocs_audio_descriptor, vcp);
> + vcp->vocs_ao_dec_id = vcp_register_notify(vcp, value_handle,
> + vcp_audio_descriptor_notify, NULL);
> +
> + }
> +
> +}
> +
> static void foreach_vcs_service(struct gatt_db_attribute *attr,
> void *user_data)
> {
> @@ -1141,6 +1658,19 @@ static void foreach_vcs_service(struct gatt_db_attribute *attr,
> gatt_db_service_foreach_char(attr, foreach_vcs_char, vcp);
> }
>
> +static void foreach_vocs_service(struct gatt_db_attribute *attr,
> + void *user_data)
> +{
> + struct bt_vcp *vcp = user_data;
> + struct bt_vocs *vocs = vcp_get_vocs(vcp);
> +
> + vocs->service = attr;
> +
> + gatt_db_service_set_claimed(attr, true);
> +
> + gatt_db_service_foreach_char(attr, foreach_vocs_char, vcp);
> +}
> +
> bool bt_vcp_attach(struct bt_vcp *vcp, struct bt_gatt_client *client)
> {
> bt_uuid_t uuid;
> @@ -1163,6 +1693,9 @@ bool bt_vcp_attach(struct bt_vcp *vcp, struct bt_gatt_client *client)
> bt_uuid16_create(&uuid, VCS_UUID);
> gatt_db_foreach_service(vcp->ldb->db, &uuid, foreach_vcs_service, vcp);
>
> + bt_uuid16_create(&uuid, VOL_OFFSET_CS_UUID);
> + gatt_db_foreach_service(vcp->ldb->db, &uuid, foreach_vocs_service, vcp);
> +
> return true;
> }
>
> --
> 2.34.1
>
--
Luiz Augusto von Dentz
Hi Nitin,
On Tue, Jun 6, 2023 at 5:09 AM NJNXP <[email protected]> wrote:
>
> Hello Maintainers
>
> This patch series handles code implementation for mandatory features of
> Volume Offset Control Service.
>
> Implementation of following features have been handled in this patch
> series
> - Volume Offset Control Service
> - Volume Offset State Characteristic (Read, Notify)
> - Audio Location Characteristic (Read)
> - Volume Offset Control Point Characteristic (Write)
> - Audio Output Description Characteristic (Read)
> - Set Volume Offset
>
> All the code implementation has been tested using PTS testing the
> verdict is PASS for all mandatory test cases.
Great, though at this stage I think it would be even more important to
have a unit tester for the bt_vcp instance, just as we do have for
bt_bap and bt_bass in unit/test-bap.c and unit/test-bass.c
respectively, this would help us making sure there are no memory
issues and avoid introducing regressions going forward.
> Thank you in advance for your review.
>
> Warm Regards
> Nitin Jadhav
>
> NJNXP (2):
> Added initial code for handling VOCS
> Fixed the following issue observed during PTS testing for mandatory
> test cases - Specified Upper and Lower Limit for Volume offset -
> Corrected the number of handles for VOCS - VOCS is made as included
> service of VCS because VOCS is secondary service and VSC is primary
> service
>
> lib/uuid.h | 5 +
> src/shared/vcp.c | 547 ++++++++++++++++++++++++++++++++++++++++++++++-
> 2 files changed, 550 insertions(+), 2 deletions(-)
>
> --
> 2.34.1
>
--
Luiz Augusto von Dentz
Hello Luiz,
Thanks for the message. As per your suggestion we will check on implementing of unit tester for the bt_vcp and raise a separate patch for it.
Right now we are able to fix the other review comments for VOCS mentioned in v1*.patch. Please help to review process ahead.
Thanks,
Warm Regards,
Nitin Jadhav
> -----Original Message-----
> From: Luiz Augusto von Dentz <[email protected]>
> Sent: Wednesday, June 7, 2023 4:44 AM
> To: Nitin Jadhav <[email protected]>
> Cc: [email protected]; Devyani Godbole
> <[email protected]>; Mihai-Octavian Urzica <mihai-
> [email protected]>; Silviu Florian Barbulescu
> <[email protected]>
> Subject: [EXT] Re: [PATCH BlueZ v1 0/2] Code handling for VOCS service
>
> Caution: This is an external email. Please take care when clicking links or
> opening attachments. When in doubt, report the message using the 'Report
> this email' button
>
>
> Hi Nitin,
>
> On Tue, Jun 6, 2023 at 5:09 AM NJNXP <[email protected]> wrote:
> >
> > Hello Maintainers
> >
> > This patch series handles code implementation for mandatory features
> > of Volume Offset Control Service.
> >
> > Implementation of following features have been handled in this patch
> > series
> > - Volume Offset Control Service
> > - Volume Offset State Characteristic (Read, Notify)
> > - Audio Location Characteristic (Read)
> > - Volume Offset Control Point Characteristic (Write)
> > - Audio Output Description Characteristic (Read)
> > - Set Volume Offset
> >
> > All the code implementation has been tested using PTS testing the
> > verdict is PASS for all mandatory test cases.
>
> Great, though at this stage I think it would be even more important to have a
> unit tester for the bt_vcp instance, just as we do have for bt_bap and bt_bass
> in unit/test-bap.c and unit/test-bass.c respectively, this would help us
> making sure there are no memory issues and avoid introducing regressions
> going forward.
>
> > Thank you in advance for your review.
> >
> > Warm Regards
> > Nitin Jadhav
> >
> > NJNXP (2):
> > Added initial code for handling VOCS
> > Fixed the following issue observed during PTS testing for mandatory
> > test cases - Specified Upper and Lower Limit for Volume offset -
> > Corrected the number of handles for VOCS - VOCS is made as included
> > service of VCS because VOCS is secondary service and VSC is primary
> > service
> >
> > lib/uuid.h | 5 +
> > src/shared/vcp.c | 547
> > ++++++++++++++++++++++++++++++++++++++++++++++-
> > 2 files changed, 550 insertions(+), 2 deletions(-)
> >
> > --
> > 2.34.1
> >
>
>
> --
> Luiz Augusto von Dentz