commit e88fe5f9bd4b63faa2aaf2cce30ceb1de32c4e9a
author Luiz Augusto von Dentz <luiz.von.dentz@intel.com> Wed Mar 14 14:56:30 2018 +0200
committer Luiz Augusto von Dentz <luiz.von.dentz@intel.com> Thu Mar 15 10:38:20 2018 +0200
tree c6496de06441e177857398ae01ca3bff72b961d7
parent 9e3fa053432a9e84564853dc0ca2609078b92c5a

shared/gatt-client: Omit valid errors for primary discovery

BT_ATT_ERROR_ATTRIBUTE_NOT_FOUND shall not cause anything to be printed
since it is a valid response and just mark the end of primary discovery.

src/shared/gatt-client.c

diff --git a/src/shared/gatt-client.c b/src/shared/gatt-client.c
index 1285c37..d61668d 100644
--- a/src/shared/gatt-client.c
+++ b/src/shared/gatt-client.c
@@ -1023,9 +1023,6 @@
 	discovery_req_clear(client);
 
 	if (!success) {
-		util_debug(client->debug_callback, client->debug_data,
-					"Primary service discovery failed."
-					" ATT ECODE: 0x%02x", att_ecode);
 		/* Reset error in case of not found */
 		switch (att_ecode) {
 		case BT_ATT_ERROR_ATTRIBUTE_NOT_FOUND:
@@ -1033,6 +1030,9 @@
 			att_ecode = 0;
 			goto secondary;
 		default:
+			util_debug(client->debug_callback, client->debug_data,
+					"Primary service discovery failed."
+					" ATT ECODE: 0x%02x", att_ecode);
 			goto done;
 		}
 	}