]> granicus.if.org Git - esp-idf/commitdiff
ESP BLE Mesh: Minor fix - Print the address and the on/off value appropriately
authorHrishikesh Dhayagude <hrishi@espressif.com>
Mon, 14 Oct 2019 13:42:35 +0000 (21:42 +0800)
committerJiang Jiang Jian <jack@espressif.com>
Mon, 14 Oct 2019 13:42:35 +0000 (21:42 +0800)
The new value should be printed in the set function and not the previous
one.

examples/bluetooth/esp_ble_mesh/ble_mesh_client_model/main/ble_mesh_demo_main.c
examples/bluetooth/esp_ble_mesh/ble_mesh_node/main/ble_mesh_demo_main.c

index 3dde33207d8b6adda8f7996e5b2d97d9714992d8..f7466992defd0e1daa9e985677c2a76b5fd5a3d7 100644 (file)
@@ -150,12 +150,12 @@ static void gen_onoff_set_unack_handler(esp_ble_mesh_model_t *model,
     uint8_t prev_onoff;
     esp_err_t err;
 
-    ESP_LOGI(TAG, "%s, addr 0x%02x onoff 0x%02x", __func__, model->element->element_addr, led->current);
-
     prev_onoff = led->previous;
     led->current = data[0];
     remote_onoff = led->current;
 
+    ESP_LOGI(TAG, "%s, addr 0x%04x onoff 0x%02x", __func__, model->element->element_addr, led->current);
+
     board_led_operation(led->pin, led->current);
 
     /* If Generic OnOff state is changed, and the publish address of Generic OnOff Server
@@ -181,12 +181,12 @@ static void gen_onoff_set_handler(esp_ble_mesh_model_t *model,
     uint8_t prev_onoff, send_data;
     esp_err_t err;
 
-    ESP_LOGI(TAG, "%s, addr 0x%02x onoff 0x%02x", __func__, model->element->element_addr, led->current);
-
     prev_onoff = led->previous;
     led->current = data[0];
     remote_onoff = led->current;
 
+    ESP_LOGI(TAG, "%s, addr 0x%04x onoff 0x%02x", __func__, model->element->element_addr, led->current);
+
     board_led_operation(led->pin, led->current);
 
     send_data = led->current;
index 0e0aeaf8457d003093fc46dfefdfb91ef5ac08a8..1d8fb2cd56372a99a4419a485e2df4ca105c31c4 100644 (file)
@@ -146,11 +146,11 @@ static void gen_onoff_set_unack_handler(esp_ble_mesh_model_t *model,
     uint8_t prev_onoff;
     esp_err_t err;
 
-    ESP_LOGI(TAG, "%s, addr 0x%02x onoff 0x%02x", __func__, model->element->element_addr, led->current);
-
     prev_onoff = led->previous;
     led->current = data[0];
 
+    ESP_LOGI(TAG, "%s, addr 0x%04x onoff 0x%02x", __func__, model->element->element_addr, led->current);
+
     board_led_operation(led->pin, led->current);
 
     /* If Generic OnOff state is changed, and the publish address of Generic OnOff Server
@@ -175,11 +175,11 @@ static void gen_onoff_set_handler(esp_ble_mesh_model_t *model,
     uint8_t prev_onoff, send_data;
     esp_err_t err;
 
-    ESP_LOGI(TAG, "%s, addr 0x%02x onoff 0x%02x", __func__, model->element->element_addr, led->current);
-
     prev_onoff = led->previous;
     led->current = data[0];
 
+    ESP_LOGI(TAG, "%s, addr 0x%04x onoff 0x%02x", __func__, model->element->element_addr, led->current);
+
     board_led_operation(led->pin, led->current);
 
     send_data = led->current;