From: Luiz Augusto von Dentz <[email protected]>
This adds logging of file and function names.
---
src/shared/att.c | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/src/shared/att.c b/src/shared/att.c
index 169f726e4..7344b0c46 100644
--- a/src/shared/att.c
+++ b/src/shared/att.c
@@ -308,10 +308,12 @@ static void att_log(struct bt_att *att, uint8_t level, const char *format,
}
#define att_debug(_att, _format, _arg...) \
- att_log(_att, BT_ATT_DEBUG, _format, ## _arg)
+ att_log(_att, BT_ATT_DEBUG, "%s:%s() " _format, __FILE__, __func__,\
+ ## _arg)
#define att_verbose(_att, _format, _arg...) \
- att_log(_att, BT_ATT_DEBUG_VERBOSE, _format, ## _arg)
+ att_log(_att, BT_ATT_DEBUG_VERBOSE, "%s:%s() " _format, __FILE__, \
+ __func__, ## _arg)
static void att_hexdump(struct bt_att *att, char dir, const void *data,
size_t len)
--
2.35.1
From: Luiz Augusto von Dentz <[email protected]>
This adds gatt_log wrapper for util_debug and DBG macro so file and
function names are printed which is more consistent with other parts of
the daemon code.
---
src/shared/gatt-client.c | 160 +++++++++++++++++----------------------
1 file changed, 70 insertions(+), 90 deletions(-)
diff --git a/src/shared/gatt-client.c b/src/shared/gatt-client.c
index e24c9603c..ba9228ddf 100644
--- a/src/shared/gatt-client.c
+++ b/src/shared/gatt-client.c
@@ -37,6 +37,8 @@
#define GATT_SVC_UUID 0x1801
#define SVC_CHNGD_UUID 0x2a05
+#define DBG(_client, _format, arg...) \
+ gatt_log(_client, "%s:%s() " _format, __FILE__, __func__, ## arg)
struct ready_cb {
bt_gatt_client_callback_t callback;
@@ -376,6 +378,18 @@ static void discovery_op_free(struct discovery_op *op)
static bool read_db_hash(struct discovery_op *op);
+static void gatt_log(struct bt_gatt_client *client, const char *format, ...)
+{
+ va_list ap;
+
+ if (!client || !format || !client->debug_callback)
+ return;
+
+ va_start(ap, format);
+ util_debug_va(client->debug_callback, client->debug_data, format, ap);
+ va_end(ap);
+}
+
static void discovery_op_complete(struct discovery_op *op, bool success,
uint8_t err)
{
@@ -406,9 +420,8 @@ static void discovery_op_complete(struct discovery_op *op, bool success,
gatt_db_attribute_get_service_data(attr, &start, &end,
NULL, NULL);
- util_debug(op->client->debug_callback, op->client->debug_data,
- "service disappeared: start 0x%04x end 0x%04x",
- start, end);
+ DBG(op->client, "service disappeared: start 0x%04x end 0x%04x",
+ start, end);
gatt_db_remove_service(op->client->db, attr);
}
@@ -542,9 +555,7 @@ static void discover_incl_cb(bool success, uint8_t att_ecode,
if (includes_count == 0)
goto failed;
- util_debug(client->debug_callback, client->debug_data,
- "Included services found: %u",
- includes_count);
+ DBG(client, "Included services found: %u", includes_count);
for (i = 0; i < includes_count; i++) {
if (!bt_gatt_iter_next_included_service(&iter, &handle, &start,
@@ -555,13 +566,12 @@ static void discover_incl_cb(bool success, uint8_t att_ecode,
/* Log debug message */
bt_uuid_to_string(&uuid, uuid_str, sizeof(uuid_str));
- util_debug(client->debug_callback, client->debug_data,
- "handle: 0x%04x, start: 0x%04x, end: 0x%04x,"
+ DBG(client, "handle: 0x%04x, start: 0x%04x, end: 0x%04x,"
"uuid: %s", handle, start, end, uuid_str);
attr = gatt_db_get_attribute(client->db, start);
if (!attr) {
- util_debug(client->debug_callback, client->debug_data,
+ DBG(client,
"Unable to find attribute at 0x%04x: skipping",
start);
continue;
@@ -569,7 +579,7 @@ static void discover_incl_cb(bool success, uint8_t att_ecode,
attr = gatt_db_insert_included(client->db, handle, attr);
if (!attr) {
- util_debug(client->debug_callback, client->debug_data,
+ DBG(client,
"Unable to add include attribute at 0x%04x",
handle);
goto failed;
@@ -582,7 +592,7 @@ static void discover_incl_cb(bool success, uint8_t att_ecode,
* attribute.
*/
if (gatt_db_attribute_get_handle(attr) != handle) {
- util_debug(client->debug_callback, client->debug_data,
+ DBG(client,
"Invalid attribute 0x%04x expect it at 0x%04x",
gatt_db_attribute_get_handle(attr), handle);
goto failed;
@@ -604,8 +614,8 @@ next:
if (client->discovery_req)
return;
- util_debug(client->debug_callback, client->debug_data,
- "Failed to start characteristic discovery");
+ DBG(client, "Failed to start characteristic discovery");
+
discovery_op_unref(op);
failed:
discovery_op_complete(op, false, att_ecode);
@@ -656,7 +666,7 @@ static bool discover_descs(struct discovery_op *op, bool *discovering)
NULL, NULL, NULL);
if (!attr) {
- util_debug(client->debug_callback, client->debug_data,
+ DBG(client,
"Failed to insert characteristic at 0x%04x",
chrc_data->value_handle);
@@ -729,8 +739,8 @@ static bool discover_descs(struct discovery_op *op, bool *discovering)
goto done;
}
- util_debug(client->debug_callback, client->debug_data,
- "Failed to start descriptor discovery");
+ DBG(client, "Failed to start descriptor discovery");
+
discovery_op_unref(op);
goto failed;
@@ -750,8 +760,7 @@ static void ext_prop_write_cb(struct gatt_db_attribute *attrib,
{
struct bt_gatt_client *client = user_data;
- util_debug(client->debug_callback, client->debug_data,
- "Value set status: %d", err);
+ DBG(client, "Value set status: %d", err);
}
static void ext_prop_read_cb(bool success, uint8_t att_ecode,
@@ -790,8 +799,7 @@ static void ext_prop_read_cb(bool success, uint8_t att_ecode,
if (!success)
goto done;
- util_debug(client->debug_callback, client->debug_data,
- "Ext. prop value: 0x%04x", (uint16_t)value[0]);
+ DBG(client, "Ext. prop value: 0x%04x", (uint16_t)value[0]);
desc_attr = queue_pop_head(op->ext_prop_desc);
if (!desc_attr)
@@ -857,8 +865,7 @@ static void discover_descs_cb(bool success, uint8_t att_ecode,
if (desc_count == 0)
goto failed;
- util_debug(client->debug_callback, client->debug_data,
- "Descriptors found: %u", desc_count);
+ DBG(client, "Descriptors found: %u", desc_count);
bt_uuid16_create(&ext_prop_uuid, GATT_CHARAC_EXT_PROPER_UUID);
@@ -867,9 +874,8 @@ static void discover_descs_cb(bool success, uint8_t att_ecode,
/* Log debug message */
bt_uuid_to_string(&uuid, uuid_str, sizeof(uuid_str));
- util_debug(client->debug_callback, client->debug_data,
- "handle: 0x%04x, uuid: %s",
- handle, uuid_str);
+
+ DBG(client, "handle: 0x%04x, uuid: %s", handle, uuid_str);
attr = gatt_db_insert_descriptor(client->db, handle,
&uuid, 0, NULL, NULL,
@@ -880,8 +886,7 @@ static void discover_descs_cb(bool success, uint8_t att_ecode,
gatt_db_attribute_get_type(attr)))
continue;
- util_debug(client->debug_callback, client->debug_data,
- "Failed to insert descriptor at 0x%04x",
+ DBG(client, "Failed to insert descriptor at 0x%04x",
handle);
goto failed;
}
@@ -947,8 +952,8 @@ static void discover_chrcs_cb(bool success, uint8_t att_ecode,
goto failed;
chrc_count = bt_gatt_result_characteristic_count(result);
- util_debug(client->debug_callback, client->debug_data,
- "Characteristics found: %u", chrc_count);
+
+ DBG(client, "Characteristics found: %u", chrc_count);
if (chrc_count == 0)
goto failed;
@@ -959,8 +964,7 @@ static void discover_chrcs_cb(bool success, uint8_t att_ecode,
/* Log debug message */
bt_uuid_to_string(&uuid, uuid_str, sizeof(uuid_str));
- util_debug(client->debug_callback, client->debug_data,
- "start: 0x%04x, end: 0x%04x, value: 0x%04x, "
+ DBG(client, "start: 0x%04x, end: 0x%04x, value: 0x%04x, "
"props: 0x%02x, uuid: %s",
start, end, value, properties, uuid_str);
@@ -997,8 +1001,7 @@ next:
if (client->discovery_req)
return;
- util_debug(client->debug_callback, client->debug_data,
- "Failed to start included services discovery");
+ DBG(client, "Failed to start included services discovery");
discovery_op_unref(op);
@@ -1110,8 +1113,7 @@ static bool discovery_parse_services(struct discovery_op *op, bool primary,
/* Log debug message */
bt_uuid_to_string(&uuid, uuid_str, sizeof(uuid_str));
- util_debug(client->debug_callback, client->debug_data,
- "start: 0x%04x, end: 0x%04x, uuid: %s",
+ DBG(client, "start: 0x%04x, end: 0x%04x, uuid: %s",
start, end, uuid_str);
/* Store the service */
@@ -1122,9 +1124,7 @@ static bool discovery_parse_services(struct discovery_op *op, bool primary,
attr = gatt_db_insert_service(client->db, start, &uuid,
false, end - start + 1);
if (!attr) {
- util_debug(client->debug_callback,
- client->debug_data,
- "Failed to store service");
+ DBG(client, "Failed to store service");
return false;
}
/* Database has changed adjust last handle */
@@ -1157,8 +1157,7 @@ static void discover_secondary_cb(bool success, uint8_t att_ecode,
att_ecode = 0;
goto next;
default:
- util_debug(client->debug_callback, client->debug_data,
- "Secondary service discovery failed."
+ DBG(client, "Secondary service discovery failed."
" ATT ECODE: 0x%02x", att_ecode);
goto done;
}
@@ -1169,8 +1168,7 @@ static void discover_secondary_cb(bool success, uint8_t att_ecode,
goto done;
}
- util_debug(client->debug_callback, client->debug_data,
- "Secondary services found: %u",
+ DBG(client, "Secondary services found: %u",
bt_gatt_result_service_count(result));
if (!discovery_parse_services(op, false, &iter)) {
@@ -1199,8 +1197,8 @@ next:
if (client->discovery_req)
return;
- util_debug(client->debug_callback, client->debug_data,
- "Failed to start included services discovery");
+ DBG(client, "Failed to start included services discovery");
+
discovery_op_unref(op);
success = false;
@@ -1226,8 +1224,7 @@ static void discover_primary_cb(bool success, uint8_t att_ecode,
att_ecode = 0;
goto secondary;
default:
- util_debug(client->debug_callback, client->debug_data,
- "Primary service discovery failed."
+ DBG(client, "Primary service discovery failed."
" ATT ECODE: 0x%02x", att_ecode);
goto done;
}
@@ -1238,8 +1235,7 @@ static void discover_primary_cb(bool success, uint8_t att_ecode,
goto done;
}
- util_debug(client->debug_callback, client->debug_data,
- "Primary services found: %u",
+ DBG(client, "Primary services found: %u",
bt_gatt_result_service_count(result));
if (!discovery_parse_services(op, true, &iter)) {
@@ -1266,8 +1262,8 @@ secondary:
if (client->discovery_req)
return;
- util_debug(client->debug_callback, client->debug_data,
- "Failed to start secondary service discovery");
+ DBG(client, "Failed to start secondary service discovery");
+
discovery_op_unref(op);
success = false;
@@ -1331,8 +1327,7 @@ static void discover_all(struct discovery_op *op)
if (client->discovery_req)
return;
- util_debug(client->debug_callback, client->debug_data,
- "Failed to initiate primary service discovery");
+ DBG(client, "Failed to initiate primary service discovery");
client->in_init = false;
notify_client_ready(client, false, BT_ATT_ERROR_UNLIKELY);
@@ -1345,8 +1340,7 @@ static void db_hash_write_value_cb(struct gatt_db_attribute *attrib,
{
struct bt_gatt_client *client = user_data;
- util_debug(client->debug_callback, client->debug_data,
- "Value set status: %d", err);
+ DBG(client, "Value set status: %d", err);
}
static void db_hash_read_value_cb(struct gatt_db_attribute *attrib,
@@ -1377,9 +1371,8 @@ static void db_hash_read_cb(bool success, uint8_t att_ecode,
bt_gatt_iter_init(&iter, result);
bt_gatt_iter_next_read_by_type(&iter, &handle, &len, &value);
- util_debug(client->debug_callback, client->debug_data,
- "DB Hash found: handle 0x%04x length 0x%04x",
- handle, len);
+ DBG(client, "DB Hash found: handle 0x%04x length 0x%04x",
+ handle, len);
if (len != 16)
goto discover;
@@ -1390,15 +1383,14 @@ static void db_hash_read_cb(bool success, uint8_t att_ecode,
/* Check if the has has changed since last time */
if (hash && !memcmp(hash, value, len)) {
- util_debug(client->debug_callback, client->debug_data,
- "DB Hash match: skipping discovery");
+ DBG(client, "DB Hash match: skipping discovery");
queue_remove_all(op->pending_svcs, NULL, NULL, NULL);
discovery_op_complete(op, true, 0);
return;
}
- util_debug(client->debug_callback, client->debug_data,
- "DB Hash value:");
+ DBG(client, "DB Hash value:");
+
util_hexdump(' ', value, len, client->debug_callback,
client->debug_data);
@@ -1469,8 +1461,7 @@ static void db_server_feat_read(bool success, uint8_t att_ecode,
bt_gatt_iter_init(&iter, result);
bt_gatt_iter_next_read_by_type(&iter, &handle, &len, &value);
- util_debug(client->debug_callback, client->debug_data,
- "Server Features found: handle 0x%04x "
+ DBG(client, "Server Features found: handle 0x%04x "
"length 0x%04x value 0x%02x", handle, len,
value[0]);
@@ -1524,8 +1515,7 @@ static void exchange_mtu_cb(bool success, uint8_t att_ecode, void *user_data)
client->mtu_req_id = 0;
if (!success) {
- util_debug(client->debug_callback, client->debug_data,
- "MTU Exchange failed. ATT ECODE: 0x%02x",
+ DBG(client, "MTU Exchange failed. ATT ECODE: 0x%02x",
att_ecode);
/*
@@ -1543,8 +1533,7 @@ static void exchange_mtu_cb(bool success, uint8_t att_ecode, void *user_data)
return;
}
- util_debug(client->debug_callback, client->debug_data,
- "MTU exchange complete, with MTU: %u",
+ DBG(client, "MTU exchange complete, with MTU: %u",
bt_att_get_mtu(client->att));
discover:
@@ -1749,7 +1738,7 @@ static void service_changed_register_cb(uint16_t att_ecode, void *user_data)
struct bt_gatt_client *client = user_data;
if (att_ecode) {
- util_debug(client->debug_callback, client->debug_data,
+ DBG(client,
"Failed to register handler for \"Service Changed\"");
success = false;
client->svc_chngd_ind_id = 0;
@@ -1758,8 +1747,7 @@ static void service_changed_register_cb(uint16_t att_ecode, void *user_data)
client->svc_chngd_registered = true;
success = true;
- util_debug(client->debug_callback, client->debug_data,
- "Registered handler for \"Service Changed\": %u",
+ DBG(client, "Registered handler for \"Service Changed\": %u",
client->svc_chngd_ind_id);
done:
@@ -1807,7 +1795,7 @@ static void service_changed_complete(struct discovery_op *op, bool success,
client->in_svc_chngd = false;
if (!success && att_ecode != BT_ATT_ERROR_ATTRIBUTE_NOT_FOUND) {
- util_debug(client->debug_callback, client->debug_data,
+ DBG(client,
"Failed to discover services within changed range - "
"error: 0x%02x", att_ecode);
@@ -1841,7 +1829,7 @@ static void service_changed_complete(struct discovery_op *op, bool success,
if (register_service_changed(client))
return;
- util_debug(client->debug_callback, client->debug_data,
+ DBG(client,
"Failed to re-register handler for \"Service Changed\"");
}
@@ -1877,9 +1865,8 @@ static void process_service_changed(struct bt_gatt_client *client,
discovery_op_free(op);
fail:
- util_debug(client->debug_callback, client->debug_data,
- "Failed to initiate service discovery"
- " after Service Changed");
+ DBG(client,
+ "Failed to initiate service discovery after Service Changed");
}
static void service_changed_cb(uint16_t value_handle, const uint8_t *value,
@@ -1896,13 +1883,12 @@ static void service_changed_cb(uint16_t value_handle, const uint8_t *value,
end = get_le16(value + 2);
if (start > end) {
- util_debug(client->debug_callback, client->debug_data,
+ DBG(client,
"Service Changed received with invalid handles");
return;
}
- util_debug(client->debug_callback, client->debug_data,
- "Service Changed received - start: 0x%04x end: 0x%04x",
+ DBG(client, "Service Changed received - start: 0x%04x end: 0x%04x",
start, end);
if (!client->in_svc_chngd) {
@@ -1923,8 +1909,7 @@ static void server_feat_write_value(struct gatt_db_attribute *attrib,
{
struct bt_gatt_client *client = user_data;
- util_debug(client->debug_callback, client->debug_data,
- "Server Features Value set status: %d", err);
+ DBG(client, "Server Features Value set status: %d", err);
}
static void write_server_features(struct bt_gatt_client *client, uint8_t feat)
@@ -1943,8 +1928,7 @@ static void write_server_features(struct bt_gatt_client *client, uint8_t feat)
if (!gatt_db_attribute_write(attr, 0, &feat, sizeof(feat),
0, NULL, server_feat_write_value,
client))
- util_debug(client->debug_callback, client->debug_data,
- "Unable to store Server Features");
+ DBG(client, "Unable to store Server Features");
}
static void write_client_features(struct bt_gatt_client *client)
@@ -1981,8 +1965,7 @@ static void write_client_features(struct bt_gatt_client *client)
client->features |= BT_GATT_CHRC_CLI_FEAT_NFY_MULTI;
- util_debug(client->debug_callback, client->debug_data,
- "Writing Client Features 0x%02x", client->features);
+ DBG(client, "Writing Client Features 0x%02x", client->features);
bt_gatt_client_write_value(client, handle, &client->features,
sizeof(client->features), NULL, NULL, NULL);
@@ -2006,13 +1989,11 @@ static void init_complete(struct discovery_op *op, bool success,
if (register_service_changed(client))
goto done;
- util_debug(client->debug_callback, client->debug_data,
- "Failed to register handler for \"Service Changed\"");
+ DBG(client, "Failed to register handler for \"Service Changed\"");
success = false;
fail:
- util_debug(client->debug_callback, client->debug_data,
- "Failed to initialize gatt-client");
+ DBG(client, "Failed to initialize gatt-client");
op->success = false;
@@ -3501,8 +3482,7 @@ unsigned int bt_gatt_client_prepare_write(struct bt_gatt_client *client,
* prepare writes or this is brand new reliable session (id == 0)
*/
if (id != client->reliable_write_session_id) {
- util_debug(client->debug_callback, client->debug_data,
- "There is other reliable write session ongoing %u",
+ DBG(client, "There is other reliable write session ongoing %u",
client->reliable_write_session_id);
return 0;
--
2.35.1
From: Luiz Augusto von Dentz <[email protected]>
att_debug and att_verbose are macros which are more common to be
used as uppercase, this also change them to use DBG like other parts of
the code.
---
src/shared/att.c | 38 +++++++++++++++++++-------------------
1 file changed, 19 insertions(+), 19 deletions(-)
diff --git a/src/shared/att.c b/src/shared/att.c
index 7344b0c46..f7bef08bc 100644
--- a/src/shared/att.c
+++ b/src/shared/att.c
@@ -307,11 +307,11 @@ static void att_log(struct bt_att *att, uint8_t level, const char *format,
va_end(va);
}
-#define att_debug(_att, _format, _arg...) \
+#define DBG(_att, _format, _arg...) \
att_log(_att, BT_ATT_DEBUG, "%s:%s() " _format, __FILE__, __func__,\
## _arg)
-#define att_verbose(_att, _format, _arg...) \
+#define VERBOSE(_att, _format, _arg...) \
att_log(_att, BT_ATT_DEBUG_VERBOSE, "%s:%s() " _format, __FILE__, \
__func__, ## _arg)
@@ -359,7 +359,7 @@ static bool encode_pdu(struct bt_att *att, struct att_send_op *op,
sign_cnt, &((uint8_t *) op->pdu)[1 + length])))
return true;
- att_debug(att, "ATT unable to generate signature");
+ DBG(att, "ATT unable to generate signature");
fail:
free(op->pdu);
@@ -488,7 +488,7 @@ static bool timeout_cb(void *user_data)
if (!op)
return false;
- att_debug(att, "(chan %p) Operation timed out: 0x%02x", chan,
+ DBG(att, "(chan %p) Operation timed out: 0x%02x", chan,
op->opcode);
if (att->timeout_callback)
@@ -524,11 +524,11 @@ static ssize_t bt_att_chan_write(struct bt_att_chan *chan, uint8_t opcode,
iov.iov_base = (void *) pdu;
iov.iov_len = len;
- att_verbose(att, "(chan %p) ATT op 0x%02x", chan, opcode);
+ VERBOSE(att, "(chan %p) ATT op 0x%02x", chan, opcode);
ret = io_send(chan->io, &iov, 1);
if (ret < 0) {
- att_debug(att, "(chan %p) write failed: %s", chan,
+ DBG(att, "(chan %p) write failed: %s", chan,
strerror(-ret));
return ret;
}
@@ -661,12 +661,12 @@ static bool disconnect_cb(struct io *io, void *user_data)
len = sizeof(err);
if (getsockopt(chan->fd, SOL_SOCKET, SO_ERROR, &err, &len) < 0) {
- att_debug(att, "(chan %p) Failed to obtain disconnect "
+ DBG(att, "(chan %p) Failed to obtain disconnect "
"error: %s", chan, strerror(errno));
err = 0;
}
- att_debug(att, "Channel %p disconnected: %s", chan, strerror(err));
+ DBG(att, "Channel %p disconnected: %s", chan, strerror(err));
/* Dettach channel */
queue_remove(att->chans, chan);
@@ -795,7 +795,7 @@ static bool handle_error_rsp(struct bt_att_chan *chan, uint8_t *pdu,
op->timeout_id = 0;
}
- att_debug(att, "(chan %p) Retrying operation %p", chan, op);
+ DBG(att, "(chan %p) Retrying operation %p", chan, op);
chan->pending_req = NULL;
@@ -818,7 +818,7 @@ static void handle_rsp(struct bt_att_chan *chan, uint8_t opcode, uint8_t *pdu,
* the bearer.
*/
if (!op) {
- att_debug(att, "(chan %p) Received unexpected ATT response",
+ DBG(att, "(chan %p) Received unexpected ATT response",
chan);
io_shutdown(chan->io);
return;
@@ -850,7 +850,7 @@ static void handle_rsp(struct bt_att_chan *chan, uint8_t opcode, uint8_t *pdu,
goto done;
fail:
- att_debug(att, "(chan %p) Failed to handle response PDU; opcode: "
+ DBG(att, "(chan %p) Failed to handle response PDU; opcode: "
"0x%02x", chan, opcode);
rsp_opcode = BT_ATT_OP_ERROR_RSP;
@@ -875,7 +875,7 @@ static void handle_conf(struct bt_att_chan *chan, uint8_t *pdu, ssize_t pdu_len)
* invalid.
*/
if (!op || pdu_len) {
- att_debug(att, "(chan %p) Received unexpected/invalid ATT "
+ DBG(att, "(chan %p) Received unexpected/invalid ATT "
"confirmation", chan);
io_shutdown(chan->io);
return;
@@ -949,7 +949,7 @@ static bool handle_signed(struct bt_att *att, uint8_t *pdu, ssize_t pdu_len)
return true;
fail:
- att_debug(att, "ATT failed to verify signature: 0x%02x", opcode);
+ DBG(att, "ATT failed to verify signature: 0x%02x", opcode);
return false;
}
@@ -1032,7 +1032,7 @@ static bool can_read_data(struct io *io, void *user_data)
if (bytes_read < 0)
return false;
- att_verbose(att, "(chan %p) ATT received: %zd", chan, bytes_read);
+ VERBOSE(att, "(chan %p) ATT received: %zd", chan, bytes_read);
att_hexdump(att, '>', chan->buf, bytes_read);
@@ -1047,12 +1047,12 @@ static bool can_read_data(struct io *io, void *user_data)
/* Act on the received PDU based on the opcode type */
switch (get_op_type(opcode)) {
case ATT_OP_TYPE_RSP:
- att_verbose(att, "(chan %p) ATT response received: 0x%02x",
+ VERBOSE(att, "(chan %p) ATT response received: 0x%02x",
chan, opcode);
handle_rsp(chan, opcode, pdu + 1, bytes_read - 1);
break;
case ATT_OP_TYPE_CONF:
- att_verbose(att, "(chan %p) ATT confirmation received: 0x%02x",
+ VERBOSE(att, "(chan %p) ATT confirmation received: 0x%02x",
chan, opcode);
handle_conf(chan, pdu + 1, bytes_read - 1);
break;
@@ -1063,7 +1063,7 @@ static bool can_read_data(struct io *io, void *user_data)
* promptly notify the upper layer via disconnect handlers.
*/
if (chan->in_req) {
- att_debug(att, "(chan %p) Received request while "
+ DBG(att, "(chan %p) Received request while "
"another is pending: 0x%02x",
chan, opcode);
io_shutdown(chan->io);
@@ -1083,7 +1083,7 @@ static bool can_read_data(struct io *io, void *user_data)
/* For all other opcodes notify the upper layer of the PDU and
* let them act on it.
*/
- att_debug(att, "(chan %p) ATT PDU received: 0x%02x", chan,
+ DBG(att, "(chan %p) ATT PDU received: 0x%02x", chan,
opcode);
handle_notify(chan, pdu, bytes_read);
break;
@@ -1237,7 +1237,7 @@ static void bt_att_attach_chan(struct bt_att *att, struct bt_att_chan *chan)
io_set_close_on_destroy(chan->io, att->close_on_unref);
- att_debug(att, "Channel %p attached", chan);
+ DBG(att, "Channel %p attached", chan);
wakeup_chan_writer(chan, NULL);
}
--
2.35.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=625273
---Test result---
Test Summary:
CheckPatch PASS 7.48 seconds
GitLint PASS 5.00 seconds
Prep - Setup ELL PASS 44.76 seconds
Build - Prep PASS 0.75 seconds
Build - Configure PASS 9.00 seconds
Build - Make PASS 1526.60 seconds
Make Check PASS 11.54 seconds
Make Check w/Valgrind PASS 461.85 seconds
Make Distcheck PASS 264.50 seconds
Build w/ext ELL - Configure PASS 9.28 seconds
Build w/ext ELL - Make PASS 1521.31 seconds
Incremental Build with patchesPASS 7600.04 seconds
---
Regards,
Linux Bluetooth
Hello:
This series was applied to bluetooth/bluez.git (master)
by Luiz Augusto von Dentz <[email protected]>:
On Mon, 21 Mar 2022 13:55:25 -0700 you wrote:
> From: Luiz Augusto von Dentz <[email protected]>
>
> This adds logging of file and function names.
> ---
> src/shared/att.c | 6 ++++--
> 1 file changed, 4 insertions(+), 2 deletions(-)
Here is the summary with links:
- [BlueZ,1/5] att: Log file and function names
https://git.kernel.org/pub/scm/bluetooth/bluez.git/?id=8039d42687fd
- [BlueZ,2/5] device: Don't use DBG in gatt_debug
(no matching commit)
- [BlueZ,3/5] gatt-client: Add DBG macro
https://git.kernel.org/pub/scm/bluetooth/bluez.git/?id=e0870ce5e1fe
- [BlueZ,4/5] gatt-server: Add DBG macro
https://git.kernel.org/pub/scm/bluetooth/bluez.git/?id=55c25d91e4d6
- [BlueZ,5/5] att: Rename att_debug and att_verbose to DBG and VERBOSE
https://git.kernel.org/pub/scm/bluetooth/bluez.git/?id=e1b808c128fa
You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html