shared/gatt-client: Omit valid errors for primary discovery 38/204938/1
authorLuiz Augusto von Dentz <luiz.von.dentz@intel.com>
Wed, 14 Mar 2018 12:56:30 +0000 (14:56 +0200)
committerAmit Purwar <amit.purwar@samsung.com>
Fri, 26 Apr 2019 10:44:25 +0000 (16:14 +0530)
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.

Change-Id: Ideebfd3b7646f5082f07edb0ae03c74272f23ec5
Signed-off-by: Amit Purwar <amit.purwar@samsung.com>
src/shared/gatt-client.c

index fe86689..353aaa1 100644 (file)
@@ -1161,9 +1161,6 @@ static void discover_primary_cb(bool success, uint8_t att_ecode,
        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:
@@ -1171,6 +1168,9 @@ 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."
+                                       " ATT ECODE: 0x%02x", att_ecode);
                        goto done;
                }
        }