Enable auto reply for queue messages sent with queue_send. It's only nescessary to use queue_reply to return a value other than zero or to return a result before waiting on the queue again.

git-svn-id: svn://svn.rockbox.org/rockbox/trunk@14923 a1c6a512-1295-4272-9138-f99709370657

2 files changed, 18 insertions, 9 deletions

diff --git a/firmware/kernel.c b/firmware/kernel.c
index 77b0c6ffef..03be6db624 100644
--- a/firmware/kernel.c
+++ b/firmware/kernel.c
@@ -38,8 +38,6 @@ void (*tick_funcs[MAX_NUM_TICK_TASKS])(void);
 static struct event_queue *all_queues[32] NOCACHEBSS_ATTR;
 static int num_queues NOCACHEBSS_ATTR;
 
-void queue_wait(struct event_queue *q, struct event *ev) ICODE_ATTR;
-
 /****************************************************************************
  * Standard kernel stuff
  ****************************************************************************/
@@ -234,7 +232,15 @@ void queue_wait(struct event_queue *q, struct event *ev)
     int oldlevel;
     unsigned int rd;
 
-    oldlevel = set_irq_level(HIGHEST_IRQ_LEVEL);    
+    oldlevel = set_irq_level(HIGHEST_IRQ_LEVEL);
+
+#ifdef HAVE_EXTENDED_MESSAGING_AND_NAME
+    if(q->send && q->send->curr_sender)
+    {
+        /* auto-reply */
+        queue_release_sender(&q->send->curr_sender, 0);
+    }
+#endif
     
     if (q->read == q->write)
     {
@@ -259,6 +265,14 @@ void queue_wait(struct event_queue *q, struct event *ev)
 void queue_wait_w_tmo(struct event_queue *q, struct event *ev, int ticks)
 {
     int oldlevel = set_irq_level(HIGHEST_IRQ_LEVEL);
+
+#ifdef HAVE_EXTENDED_MESSAGING_AND_NAME
+    if (q->send && q->send->curr_sender)
+    {
+        /* auto-reply */
+        queue_release_sender(&q->send->curr_sender, 0);
+    }
+#endif
     
     if (q->read == q->write && ticks > 0)
     {
diff --git a/firmware/pcm_record.c b/firmware/pcm_record.c
index f3cb21e491..a5d0e51c30 100644
--- a/firmware/pcm_record.c
+++ b/firmware/pcm_record.c
@@ -1358,7 +1358,6 @@ static void pcmrec_stop(void)
     if (is_recording)
     {
         dma_lock = true;    /* lock dma write position */
-        queue_reply(&pcmrec_queue, 0);
 
         /* flush all available data first to avoid overflow while waiting
            for encoding to finish */
@@ -1519,15 +1518,11 @@ static void pcmrec_thread(void)
                 if (is_recording)
                     break;
                 pcmrec_close();
-                /* Be sure other threads are released if waiting */
-                queue_clear(&pcmrec_queue);
-                flush_interrupts = 0;
                 usb_acknowledge(SYS_USB_CONNECTED_ACK);
                 usb_wait_for_disconnect(&pcmrec_queue);
+                flush_interrupts = 0;
                 break;
         } /* end switch */
-
-        queue_reply(&pcmrec_queue, 0);
     } /* end while */
 } /* pcmrec_thread */