diff mbox series

[BlueZ,6/9] btgatt-client: Implement read by type

Message ID 20230323103835.571037-7-simon.mikuda@streamunlimited.com (mailing list archive)
State New, archived
Headers show
Series gatt-db fix + btgatt-client features | expand

Checks

Context Check Description
tedd_an/pre-ci_am success Success
tedd_an/CheckPatch warning WARNING:LONG_LINE: line length of 81 exceeds 80 columns #115: FILE: tools/btgatt-client.c:531: + while (bt_gatt_iter_next_read_by_type(&iter, &handle, &length, &value)) { /github/workspace/src/src/13185479.patch total: 0 errors, 1 warnings, 90 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/13185479.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.
tedd_an/GitLint success Gitlint PASS

Commit Message

Simon Mikuda March 23, 2023, 10:38 a.m. UTC
---
 tools/btgatt-client.c | 72 +++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 72 insertions(+)
diff mbox series

Patch

diff --git a/tools/btgatt-client.c b/tools/btgatt-client.c
index 152e6ee70..0681074f9 100644
--- a/tools/btgatt-client.c
+++ b/tools/btgatt-client.c
@@ -34,6 +34,7 @@ 
 #include "src/shared/queue.h"
 #include "src/shared/gatt-db.h"
 #include "src/shared/gatt-client.h"
+#include "src/shared/gatt-helpers.h"
 
 #define ATT_CID 4
 #define ATT_PSM 31
@@ -510,6 +511,75 @@  static void cmd_read_multiple(int argc, char **argv)
 	free(value);
 }
 
+void read_by_type_cb(bool success, uint8_t att_ecode,
+						struct bt_gatt_result *result,
+						void *user_data)
+{
+	const uint8_t *value;
+	uint16_t length, handle;
+	struct bt_gatt_iter iter;
+	char line[MAX_LEN_LINE];
+	int i;
+
+	if (!success) {
+		error("Read by type request failed: %s (0x%02x)",
+				ecode_to_string(att_ecode), att_ecode);
+		return;
+	}
+
+	bt_gatt_iter_init(&iter, result);
+	while (bt_gatt_iter_next_read_by_type(&iter, &handle, &length, &value)) {
+		line[0] = '\0';
+		append(line, "\tValue handle 0x%04x", handle);
+
+		if (length == 0) {
+			print("%s: 0 bytes", line);
+			return;
+		}
+
+		append(line, " (%u bytes): ", length);
+
+		for (i = 0; i < length; i++)
+			append(line, "%02x ", value[i]);
+
+		print("%s", line);
+	}
+}
+
+static void cmd_read_by_type(int argc, char **argv)
+{
+	bt_uuid_t uuid;
+	uint16_t start_handle = 0x0001, end_handle = 0xFFFF;
+	char *endptr = NULL;
+
+	if (bt_string_to_uuid(&uuid, argv[1]) < 0) {
+		error("Invalid UUID: %s", optarg);
+		return;
+	}
+	if (argc > 2) {
+		start_handle = strtol(argv[2], &endptr, 0);
+		if (!endptr || *endptr != '\0' || !start_handle) {
+			error("Invalid start_handle : %s", argv[1]);
+			return;
+		}
+	}
+	if (argc > 3) {
+		end_handle = strtol(argv[3], &endptr, 0);
+		if (!endptr || *endptr != '\0' || !end_handle) {
+			error("Invalid end_handle : %s", argv[1]);
+			return;
+		}
+	}
+	if (start_handle > end_handle) {
+		error("start_handle cannot by larger than end_handle");
+		return;
+	}
+
+	if (!bt_gatt_read_by_type(cli->att, start_handle, end_handle,
+				&uuid, read_by_type_cb, NULL, NULL))
+		error("Failed to initiate read value procedure");
+}
+
 static void read_cb(bool success, uint8_t att_ecode, const uint8_t *value,
 					uint16_t length, void *user_data)
 {
@@ -1163,6 +1233,8 @@  static const struct bt_shell_menu main_menu = {
 		cmd_read_long_value, "Read a long characteristic or desctriptor value" },
 	{ "read-multiple", "<handles...>",
 		cmd_read_multiple, "Read Multiple" },
+	{ "read-by-type", "<uuid> [start_handle] [end_handle]",
+		cmd_read_by_type, "Read a value by UUID" },
 	{ "write-value", " [-w|-s] <value_handle> <value...>",
 		cmd_write_value, "Write a characteristic or descriptor value\n"
 		"Options:\n"