Commit: 5a234ca06c185f336a6398efe09f78c4d0dacf0b - system-bt (git)

system/bt

Commit MetaInfo

• Revisão: 5a234ca06c185f336a6398efe09f78c4d0dacf0b (tree)
• Hora: 2016-09-13 16:29:34
• Autor: Luke Zhang <lukez@qca....>
• Commiter: Ivan Grinko

Mensagem de Log

Fix the command timeout issue with TX idle timer

Start the idle timer after sending out each command to prevent power
collapse.

CRs-fixed:926763
Change-Id: I98e649fa40a4622e3c6bce4ea5c53d51e25413b3

Mudança Sumário

• modified: hci/include/low_power_manager.h
• modified: hci/src/hci_layer.c
• modified: hci/src/low_power_manager.c

Diff

--- a/hci/include/low_power_manager.h
+++ b/hci/include/low_power_manager.h
@@ -41,6 +41,9 @@ typedef struct low_power_manager_t {
   // Tell the low power manager that you're done transmitting data. Must be
   // called on the thread provided at initialization time.
   void (*transmit_done)(void);
+
+  void (*start_idle_timer)(bool check_LPM);
+  void (*stop_idle_timer)();
 } low_power_manager_t;
 
 const low_power_manager_t *low_power_manager_get_interface();

--- a/hci/src/hci_layer.c
+++ b/hci/src/hci_layer.c
@@ -534,9 +534,9 @@ static void event_command_ready(fixed_queue_t *queue, UNUSED_ATTR void *context)
     pthread_mutex_unlock(&commands_pending_response_lock);
 
     // Send it off
-    low_power_manager->wake_assert();
+    low_power_manager->stop_idle_timer();
     packet_fragmenter->fragment_and_dispatch(wait_entry->command);
-    low_power_manager->transmit_done();
+    low_power_manager->start_idle_timer(false);
 
     update_command_response_timer();
   }

--- a/hci/src/low_power_manager.c
+++ b/hci/src/low_power_manager.c
@@ -61,8 +61,8 @@ static void event_allow_device_sleep(void *context);
 static void event_idle_timeout(void *context);
 
 static void reset_state();
-static void start_idle_timer();
-static void stop_idle_timer();
+void start_idle_timer(bool check_LPM);
+void stop_idle_timer();
 
 static thread_fn event_functions[] = {
   event_disable,
@@ -129,7 +129,7 @@ static void transmit_done() {
   transmit_is_done = true;
   if (wake_state == LPM_WAKE_W4_TX_DONE) {
     wake_state = LPM_WAKE_W4_TIMEOUT;
-    start_idle_timer();
+    start_idle_timer(true);
   }
 }
 
@@ -163,7 +163,7 @@ static void allow_device_sleep() {
   if (state == LPM_ENABLED && wake_state == LPM_WAKE_ASSERTED) {
     if (transmit_is_done) {
       wake_state = LPM_WAKE_W4_TIMEOUT;
-      start_idle_timer();
+      start_idle_timer(true);
     } else {
       wake_state = LPM_WAKE_W4_TX_DONE;
     }
@@ -190,8 +190,8 @@ static void idle_timer_expired(UNUSED_ATTR void *context) {
     thread_post(thread, event_idle_timeout, NULL);
 }
 
-static void start_idle_timer() {
-  if (state == LPM_ENABLED) {
+void start_idle_timer(bool check_LPM) {
+  if (state == LPM_ENABLED || !check_LPM) {
     if (idle_timeout_ms == 0) {
        wake_deassert();
     } else {
@@ -200,8 +200,9 @@ static void start_idle_timer() {
   }
 }
 
-static void stop_idle_timer() {
+void stop_idle_timer() {
   alarm_cancel(idle_alarm);
+  LOG_DEBUG("%s", __func__);
 }
 
 static void event_disable(UNUSED_ATTR void *context) {
@@ -240,7 +241,9 @@ static const low_power_manager_t interface = {
   cleanup,
   post_command,
   wake_assert,
-  transmit_done
+  transmit_done,
+  start_idle_timer,
+  stop_idle_timer
 };
 
 const low_power_manager_t *low_power_manager_get_interface() {