Skip to content

Commit

Permalink
Disable bdx protocol by default and enable in sdkconfig.defaults and
Browse files Browse the repository at this point in the history
address review comments
  • Loading branch information
shubhamdp committed Jan 29, 2024
1 parent 7770050 commit a839531
Show file tree
Hide file tree
Showing 6 changed files with 34 additions and 6 deletions.
2 changes: 1 addition & 1 deletion config/esp32/components/chip/Kconfig
Original file line number Diff line number Diff line change
Expand Up @@ -1040,7 +1040,7 @@ menu "CHIP Device Layer"

config CHIP_ENABLE_BDX_LOG_TRANSFER
bool "Enable BDX log transfer"
default y
default n
help
Enables the BDX protocol for diagnostics log transfer

Expand Down
4 changes: 2 additions & 2 deletions examples/temperature-measurement-app/esp32/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ Usage:

## Additional details

This example demonstrates the utilization of the diagnostics logs cluster to
This example demonstrates the utilization of the diagnostic logs cluster to
send diagnostic logs to the client.

In this scenario, the [main/diagnostic_logs](main/diagnostic_logs) directory
Expand All @@ -50,7 +50,7 @@ chip-tool pairing ble-wifi 1 SSID PASSPHRASE 20202021 3840
# Read end user support logs using response payload protocol
chip-tool diagnosticlogs retrieve-logs-request 0 0 1 0
# Read network diagnostic using bdx protocol
# Read network diagnostic using BDX protocol
chip-tool diagnosticlogs retrieve-logs-request 1 0 1 0 --TransferFileDesignator network-diag.log
```

Expand Down
Original file line number Diff line number Diff line change
@@ -1 +1,21 @@
// dummy crash log file
W (5047Guru Meditation Error: Core 0 panic'ed (LoadProhibited). Exception was unhandled.

Core 0 register dump:
PC : 0x4009579a PS : 0x00060c33 A0 : 0x800941e1 A1 : 0x3fff3630
0x4009579a: uxListRemove at /opt/espressif/esp-idf/components/freertos/FreeRTOS-Kernel/list.c:195

A2 : 0x00000006 A3 : 0x00060c20 A4 : 0x00000000 A5 : 0x00060c23
A6 : 0xb33fffff A7 : 0xb33fffff A8 : 0x800950f0 A9 : 0x3fff3600
A10 : 0x00000001 A11 : 0x000000fe A12 : 0x00000000 A13 : 0x00000000
A14 : 0x00000000 A15 : 0x00000000 SAR : 0x0000000a EXCCAUSE: 0x0000001c
EXCVADDR: 0x00000016 LBEG : 0x4000c2e0 LEND : 0x4000c2f6 LCOUNT : 0xffffffff
0x4000c2e0: memcpy in ROM
0x4000c2f6: memcpy in ROM

Backtrace: 0x40095797:0x3fff3630 0x400941de:0x3fff3650 0x40154b39:0x3fff3670 0x40154b53:0x3fff3690 0x4013e20d:0x3fff36b0 0x40094fa6:0x3fff36d0
0x40095797: uxListRemove at /opt/espressif/esp-idf/components/freertos/FreeRTOS-Kernel/list.c:202
0x400941de: vTaskDelete at /opt/espressif/esp-idf/components/freertos/FreeRTOS-Kernel/tasks.c:1434 (discriminator 4)
0x40154b39: esp_nimble_disable at /opt/espressif/esp-idf/components/bt/host/nimble/nimble/porting/npl/freertos/src/nimble_port_freertos.c:55
0x40154b53: nimble_port_freertos_deinit at /opt/espressif/esp-idf/components/bt/host/nimble/nimble/porting/npl/freertos/src/nimble_port_freertos.c:80
0x4013e20d: chip::DeviceLayer::Internal::BLEManagerImpl::bleprph_host_task(void*) at /home/smart/projects/smp_matter/build/esp-idf/chip/../../../../../../../opt/espressif/esp-matter/connectedhomeip/connectedhomeip/config/esp32/third_party/connectedhomeip/src/platform/ESP32/nimble/BLEManagerImpl.cpp:864
0x40094fa6: vPortTaskWrapper at /opt/espressif/esp-idf/components/freertos/FreeRTOS-Kernel/portable/xtensa/port.c:162
Original file line number Diff line number Diff line change
@@ -1 +1,3 @@
// dummy end user support log file
I (223374) chip[light]: Turning on the smart light.
I (233374) chip[light]: Setting smart light level to 78 %.
I (243374) chip[light]: Turning off the smart light.
Original file line number Diff line number Diff line change
@@ -1 +1,4 @@
// dummy network diagnostic log file
I (223374) chip[ndiag]: Wi-Fi connection status: 1
I (233374) chip[ndiag]: Wi-Fi RSSI: -67 dBm
I (243374) chip[ndiag]: Minimum ever Wi-Fi RSSI: -80 dBm
I (253374) chip[ndiag]: Wi-Fi disconnection count: 16
3 changes: 3 additions & 0 deletions examples/temperature-measurement-app/esp32/sdkconfig.defaults
Original file line number Diff line number Diff line change
Expand Up @@ -96,3 +96,6 @@ CONFIG_MBEDTLS_HKDF_C=y

# Increase LwIP IPv6 address number
CONFIG_LWIP_IPV6_NUM_ADDRESSES=6

# Enable the diagnostic logs transfer over BDX protocol
CONFIG_CHIP_ENABLE_BDX_LOG_TRANSFER=y

0 comments on commit a839531

Please sign in to comment.