From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1KNt16-0004vF-5e for qemu-devel@nongnu.org; Tue, 29 Jul 2008 13:29:44 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1KNt15-0004ue-Bh for qemu-devel@nongnu.org; Tue, 29 Jul 2008 13:29:43 -0400 Received: from [199.232.76.173] (port=49995 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1KNt15-0004uY-5y for qemu-devel@nongnu.org; Tue, 29 Jul 2008 13:29:43 -0400 Received: from savannah.gnu.org ([199.232.41.3]:42317 helo=sv.gnu.org) by monty-python.gnu.org with esmtps (TLS-1.0:RSA_AES_256_CBC_SHA1:32) (Exim 4.60) (envelope-from ) id 1KNt14-0007KG-RZ for qemu-devel@nongnu.org; Tue, 29 Jul 2008 13:29:43 -0400 Received: from cvs.savannah.gnu.org ([199.232.41.69]) by sv.gnu.org with esmtp (Exim 4.63) (envelope-from ) id 1KNt14-0000Vw-Hk for qemu-devel@nongnu.org; Tue, 29 Jul 2008 17:29:42 +0000 Received: from balrog by cvs.savannah.gnu.org with local (Exim 4.63) (envelope-from ) id 1KNt14-0000Vr-Ad for qemu-devel@nongnu.org; Tue, 29 Jul 2008 17:29:42 +0000 MIME-Version: 1.0 Errors-To: balrog Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit From: Andrzej Zaborowski Message-Id: Date: Tue, 29 Jul 2008 17:29:42 +0000 Subject: [Qemu-devel] [4969] Correct audio api usage in OMAP EAC (spotted by malc). Reply-To: qemu-devel@nongnu.org List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Revision: 4969 http://svn.sv.gnu.org/viewvc/?view=rev&root=qemu&revision=4969 Author: balrog Date: 2008-07-29 17:29:41 +0000 (Tue, 29 Jul 2008) Log Message: ----------- Correct audio api usage in OMAP EAC (spotted by malc). This is to improve the usage of audio API thanks to explanation from malc. Functionally may not be better. Modified Paths: -------------- trunk/hw/omap2.c Modified: trunk/hw/omap2.c =================================================================== --- trunk/hw/omap2.c 2008-07-29 14:19:16 UTC (rev 4968) +++ trunk/hw/omap2.c 2008-07-29 17:29:41 UTC (rev 4969) @@ -1447,6 +1447,7 @@ #define EAC_BUF_LEN 1024 uint32_t rxbuf[EAC_BUF_LEN]; + int rxoff; int rxlen; int rxavail; uint32_t txbuf[EAC_BUF_LEN]; @@ -1478,7 +1479,7 @@ static inline void omap_eac_in_dmarequest_update(struct omap_eac_s *s) { - qemu_set_irq(s->codec.rxdrq, s->codec.rxavail + s->codec.rxlen && + qemu_set_irq(s->codec.rxdrq, (s->codec.rxavail || s->codec.rxlen) && ((s->codec.config[1] >> 12) & 1)); /* DMAREN */ } @@ -1490,26 +1491,61 @@ static inline void omap_eac_in_refill(struct omap_eac_s *s) { - int left, start = 0; + int left = MIN(EAC_BUF_LEN - s->codec.rxlen, s->codec.rxavail) << 2; + int start = ((s->codec.rxoff + s->codec.rxlen) & (EAC_BUF_LEN - 1)) << 2; + int leftwrap = MIN(left, (EAC_BUF_LEN << 2) - start); + int recv = 1; + uint8_t *buf = (uint8_t *) s->codec.rxbuf + start; - s->codec.rxlen = MIN(s->codec.rxavail, EAC_BUF_LEN); - s->codec.rxavail -= s->codec.rxlen; + left -= leftwrap; + start = 0; + while (leftwrap && (recv = AUD_read(s->codec.in_voice, buf + start, + leftwrap)) > 0) { /* Be defensive */ + start += recv; + leftwrap -= recv; + } + if (recv <= 0) + s->codec.rxavail = 0; + else + s->codec.rxavail -= start >> 2; + s->codec.rxlen += start >> 2; - for (left = s->codec.rxlen << 2; left; start = (EAC_BUF_LEN << 2) - left) - left -= AUD_read(s->codec.in_voice, - (uint8_t *) s->codec.rxbuf + start, left); + if (recv > 0 && left > 0) { + start = 0; + while (left && (recv = AUD_read(s->codec.in_voice, + (uint8_t *) s->codec.rxbuf + start, + left)) > 0) { /* Be defensive */ + start += recv; + left -= recv; + } + if (recv <= 0) + s->codec.rxavail = 0; + else + s->codec.rxavail -= start >> 2; + s->codec.rxlen += start >> 2; + } } static inline void omap_eac_out_empty(struct omap_eac_s *s) { - int left, start = 0; + int left = s->codec.txlen << 2; + int start = 0; + int sent = 1; - for (left = s->codec.txlen << 2; left; start = (s->codec.txlen << 2) - left) - left -= AUD_write(s->codec.out_voice, - (uint8_t *) s->codec.txbuf + start, left); + while (left && (sent = AUD_write(s->codec.out_voice, + (uint8_t *) s->codec.txbuf + start, + left)) > 0) { /* Be defensive */ + start += sent; + left -= sent; + } - s->codec.txavail -= s->codec.txlen; - s->codec.txlen = 0; + if (!sent) { + s->codec.txavail = 0; + omap_eac_out_dmarequest_update(s); + } + + if (start) + s->codec.txlen = 0; } static void omap_eac_in_cb(void *opaque, int avail_b) @@ -1517,8 +1553,9 @@ struct omap_eac_s *s = (struct omap_eac_s *) opaque; s->codec.rxavail = avail_b >> 2; + omap_eac_in_refill(s); + /* TODO: possibly discard current buffer if overrun */ omap_eac_in_dmarequest_update(s); - /* TODO: possibly discard current buffer if overrun */ } static void omap_eac_out_cb(void *opaque, int free_b) @@ -1526,10 +1563,10 @@ struct omap_eac_s *s = (struct omap_eac_s *) opaque; s->codec.txavail = free_b >> 2; - if (s->codec.txlen > s->codec.txavail) - s->codec.txlen = s->codec.txavail; - omap_eac_out_empty(s); - omap_eac_out_dmarequest_update(s); + if (s->codec.txlen) + omap_eac_out_empty(s); + else + omap_eac_out_dmarequest_update(s); } static void omap_eac_enable_update(struct omap_eac_s *s) @@ -1591,7 +1628,9 @@ { audsettings_t fmt; - omap_eac_out_empty(s); + /* The hardware buffers at most one sample */ + if (s->codec.rxlen) + s->codec.rxlen = 1; if (s->codec.in_voice) { AUD_set_active_in(s->codec.in_voice, 0); @@ -1599,10 +1638,14 @@ s->codec.in_voice = 0; } if (s->codec.out_voice) { + omap_eac_out_empty(s); AUD_set_active_out(s->codec.out_voice, 0); AUD_close_out(&s->codec.card, s->codec.out_voice); s->codec.out_voice = 0; + s->codec.txavail = 0; } + /* Discard what couldn't be written */ + s->codec.txlen = 0; omap_eac_enable_update(s); if (!s->codec.enable) @@ -1663,6 +1706,7 @@ s->codec.config[1] = 0x0000; s->codec.config[2] = 0x0007; s->codec.config[3] = 0x1ffc; + s->codec.rxoff = 0; s->codec.rxlen = 0; s->codec.txlen = 0; s->codec.rxavail = 0; @@ -1676,6 +1720,7 @@ { struct omap_eac_s *s = (struct omap_eac_s *) opaque; int offset = addr - s->base; + uint32_t ret; switch (offset) { case 0x000: /* CPCFR1 */ @@ -1739,16 +1784,19 @@ /* This should be write-only? Docs list it as read-only. */ return 0x0000; case 0x0b8: /* ADRDR */ - if (likely(s->codec.rxlen > 1)) - return s->codec.rxbuf[EAC_BUF_LEN - s->codec.rxlen --]; - else if (s->codec.rxlen) { + if (likely(s->codec.rxlen > 1)) { + ret = s->codec.rxbuf[s->codec.rxoff ++]; + s->codec.rxlen --; + s->codec.rxoff &= EAC_BUF_LEN - 1; + return ret; + } else if (s->codec.rxlen) { + ret = s->codec.rxbuf[s->codec.rxoff ++]; + s->codec.rxlen --; + s->codec.rxoff &= EAC_BUF_LEN - 1; if (s->codec.rxavail) omap_eac_in_refill(s); - else { - s->codec.rxlen = 0; - omap_eac_in_dmarequest_update(s); - } - return s->codec.rxbuf[EAC_BUF_LEN - 1]; + omap_eac_in_dmarequest_update(s); + return ret; } return 0x0000; case 0x0bc: /* AGCFR */ @@ -1881,8 +1929,8 @@ s->codec.txlen == s->codec.txavail)) { if (s->codec.txavail) omap_eac_out_empty(s); - else - s->codec.txlen = 0; + /* Discard what couldn't be written */ + s->codec.txlen = 0; } break;