michaelh/zephyr
1
0
Fork 0
Code Releases Activity

Bluetooth: tester: Add logging in core handlers

Browse source 
Add warnings when something goes unhandled.

Signed-off-by: Carles Cufi <carles.cufi@nordicsemi.no>
This commit is contained in:
Carles Cufi 2020-02-21 10:23:54 +01:00 • committed by Johan Hedberg
commit 7f5ae21a6d
1 changed files with 5 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

5
tests/bluetooth/tester/src/bttester.c
View file

@ -104,6 +104,7 @@ static void register_service(u8_t *data, u16_t len)
break; break;
#endif /* CONFIG_BT_MESH */ #endif /* CONFIG_BT_MESH */
default: default:
LOG_WRN("unknown id: 0x%02x", cmd->id);
status = BTP_STATUS_FAILED; status = BTP_STATUS_FAILED;
break; break;
} }
@ -136,6 +137,7 @@ static void unregister_service(u8_t *data, u16_t len)
break; break;
#endif /* CONFIG_BT_MESH */ #endif /* CONFIG_BT_MESH */
default: default:
LOG_WRN("unknown id: 0x%x", cmd->id);
status = BTP_STATUS_FAILED; status = BTP_STATUS_FAILED;
break; break;
} }
@ -148,6 +150,7 @@ static void handle_core(u8_t opcode, u8_t index, u8_t *data,
u16_t len) u16_t len)
{ {
if (index != BTP_INDEX_NONE) { if (index != BTP_INDEX_NONE) {
LOG_WRN("index != BTP_INDEX_NONE: 0x%x", index);
tester_rsp(BTP_SERVICE_ID_CORE, opcode, index, BTP_STATUS_FAILED); tester_rsp(BTP_SERVICE_ID_CORE, opcode, index, BTP_STATUS_FAILED);
return; return;
} }
@ -166,6 +169,7 @@ static void handle_core(u8_t opcode, u8_t index, u8_t *data,
unregister_service(data, len); unregister_service(data, len);
return; return;
default: default:
LOG_WRN("unknown opcode: 0x%x", opcode);
tester_rsp(BTP_SERVICE_ID_CORE, opcode, BTP_INDEX_NONE, tester_rsp(BTP_SERVICE_ID_CORE, opcode, BTP_INDEX_NONE,
BTP_STATUS_UNKNOWN_CMD); BTP_STATUS_UNKNOWN_CMD);
return; return;
@ -212,6 +216,7 @@ static void cmd_handler(void *p1, void *p2, void *p3)
break; break;
#endif /* CONFIG_BT_MESH */ #endif /* CONFIG_BT_MESH */
default: default:
LOG_WRN("unknown service: 0x%x", cmd->hdr.service);
tester_rsp(cmd->hdr.service, cmd->hdr.opcode, tester_rsp(cmd->hdr.service, cmd->hdr.opcode,
cmd->hdr.index, BTP_STATUS_FAILED); cmd->hdr.index, BTP_STATUS_FAILED);
break; break;