Several changes to codec_dahdi to play nice with G723.
[asterisk/asterisk.git] / codecs / codec_g722.c
index a81b9b6..bdd175e 100644 (file)
@@ -1,9 +1,10 @@
 /*
  * Asterisk -- An open source telephony toolkit.
  *
- * Copyright (C) 1999 - 2006, Digium, Inc.
+ * Copyright (C) 1999 - 2008, Digium, Inc.
  *
  * Matthew Fredrickson <creslin@digium.com>
+ * Russell Bryant <russell@digium.com>
  *
  * Special thanks to Steve Underwood for the implementation
  * and for doing the 8khz<->g.722 direct translation code.
  *
  * \brief codec_g722.c - translate between signed linear and ITU G.722-64kbps
  *
+ * \author Matthew Fredrickson <creslin@digium.com>
+ * \author Russell Bryant <russell@digium.com>
+ *
+ * \arg http://soft-switch.org/downloads/non-gpl-bits.tgz
+ * \arg http://lists.digium.com/pipermail/asterisk-dev/2006-September/022866.html
+ *
  * \ingroup codecs
  */
 
@@ -39,11 +46,11 @@ ASTERISK_FILE_VERSION(__FILE__, "$Revision$")
 #define BUFFER_SAMPLES   8096  /* size for the translation buffers */
 #define BUF_SHIFT      5
 
-/* Sample frame data */
-
 #include "g722/g722.h"
-#include "slin_g722_ex.h"
-#include "g722_slin_ex.h"
+
+/* Sample frame data */
+#include "asterisk/slin.h"
+#include "ex_g722.h"
 
 struct g722_encoder_pvt {
        g722_encode_state_t g722;
@@ -94,84 +101,35 @@ static int g722tolin16_new(struct ast_trans_pvt *pvt)
 static int g722tolin_framein(struct ast_trans_pvt *pvt, struct ast_frame *f)
 {
        struct g722_decoder_pvt *tmp = pvt->pvt;
-       unsigned char *src = f->data;
-       int16_t *dst = (int16_t *) pvt->outbuf + pvt->samples;
+       int out_samples;
+       int in_samples;
 
-       g722_decode(&tmp->g722, dst, src, f->samples);
-       pvt->samples += f->samples;
-       pvt->datalen += 2 * f->samples;
+       /* g722_decode expects the samples to be in the invalid samples / 2 format */
+       in_samples = f->samples / 2;
 
-       return 0;
-}
+       out_samples = g722_decode(&tmp->g722, &pvt->outbuf.i16[pvt->samples * sizeof(int16_t)], 
+               (uint8_t *) f->data.ptr, in_samples);
 
-static int lintog722_framein(struct ast_trans_pvt *pvt, struct ast_frame *f)
-{
-       struct g722_encoder_pvt *tmp = pvt->pvt;
-       int16_t *src = f->data;
+       pvt->samples += out_samples;
 
-       g722_encode(&tmp->g722, (uint8_t*)(&pvt->outbuf[pvt->datalen]), src, f->samples);
-       /* Since G.722 64kbps per second is one bye per sample, all of these
-          calculations are easy */
-       pvt->samples += f->samples;
-       pvt->datalen += f->samples;
+       pvt->datalen += (out_samples * sizeof(int16_t));
 
        return 0;
 }
 
-static struct ast_frame *g722tolin_sample(void)
+static int lintog722_framein(struct ast_trans_pvt *pvt, struct ast_frame *f)
 {
-       static struct ast_frame f = {
-               .frametype = AST_FRAME_VOICE,
-               .subclass = AST_FORMAT_G722,
-               .datalen = sizeof(g722_slin_ex),
-               .samples = sizeof(g722_slin_ex) / sizeof(g722_slin_ex[0]),
-               .src = __PRETTY_FUNCTION__,
-               .data = g722_slin_ex,
-       };
-
-       return &f;
-}
+       struct g722_encoder_pvt *tmp = pvt->pvt;
+       int outlen;
 
-static struct ast_frame *g722tolin16_sample(void)
-{
-       static struct ast_frame f = {
-               .frametype = AST_FRAME_VOICE,
-               .subclass = AST_FORMAT_G722,
-               .datalen = sizeof(slin_g722_ex),
-               .samples = sizeof(slin_g722_ex) / sizeof(slin_g722_ex[0]),
-               .src = __PRETTY_FUNCTION__,
-               .data = slin_g722_ex,
-       };
-
-       return &f;
-}
+       outlen = g722_encode(&tmp->g722, (&pvt->outbuf.ui8[pvt->datalen]), 
+               (int16_t *) f->data.ptr, f->samples);
 
-static struct ast_frame *lintog722_sample (void)
-{
-       static struct ast_frame f = {
-               .frametype = AST_FRAME_VOICE,
-               .subclass = AST_FORMAT_SLINEAR,
-               .datalen = sizeof(slin_g722_ex),
-               .samples = sizeof(slin_g722_ex) / sizeof(slin_g722_ex[0]),
-               .src = __PRETTY_FUNCTION__,
-               .data = slin_g722_ex,
-       };
-
-       return &f;
-}
+       pvt->samples += outlen * 2;
 
-static struct ast_frame *lin16tog722_sample (void)
-{
-       static struct ast_frame f = {
-               .frametype = AST_FRAME_VOICE,
-               .subclass = AST_FORMAT_SLINEAR16,
-               .datalen = sizeof(slin_g722_ex),
-               .samples = sizeof(slin_g722_ex) / sizeof(slin_g722_ex[0]),
-               .src = __PRETTY_FUNCTION__,
-               .data = slin_g722_ex,
-       };
-
-       return &f;
+       pvt->datalen += outlen;
+
+       return 0;
 }
 
 static struct ast_translator g722tolin = {
@@ -180,9 +138,9 @@ static struct ast_translator g722tolin = {
        .dstfmt = AST_FORMAT_SLINEAR,
        .newpvt = g722tolin_new,        /* same for both directions */
        .framein = g722tolin_framein,
-       .sample = g722tolin_sample,
+       .sample = g722_sample,
        .desc_size = sizeof(struct g722_decoder_pvt),
-       .buffer_samples = BUFFER_SAMPLES,
+       .buffer_samples = BUFFER_SAMPLES / sizeof(int16_t),
        .buf_size = BUFFER_SAMPLES,
        .plc_samples = 160,
 };
@@ -193,9 +151,9 @@ static struct ast_translator lintog722 = {
        .dstfmt = AST_FORMAT_G722,
        .newpvt = lintog722_new,        /* same for both directions */
        .framein = lintog722_framein,
-       .sample = lintog722_sample,
+       .sample = slin8_sample,
        .desc_size = sizeof(struct g722_encoder_pvt),
-       .buffer_samples = BUFFER_SAMPLES,
+       .buffer_samples = BUFFER_SAMPLES * 2,
        .buf_size = BUFFER_SAMPLES,
 };
 
@@ -205,9 +163,9 @@ static struct ast_translator g722tolin16 = {
        .dstfmt = AST_FORMAT_SLINEAR16,
        .newpvt = g722tolin16_new,      /* same for both directions */
        .framein = g722tolin_framein,
-       .sample = g722tolin16_sample,
+       .sample = g722_sample,
        .desc_size = sizeof(struct g722_decoder_pvt),
-       .buffer_samples = BUFFER_SAMPLES,
+       .buffer_samples = BUFFER_SAMPLES / sizeof(int16_t),
        .buf_size = BUFFER_SAMPLES,
        .plc_samples = 160,
 };
@@ -218,9 +176,9 @@ static struct ast_translator lin16tog722 = {
        .dstfmt = AST_FORMAT_G722,
        .newpvt = lin16tog722_new,      /* same for both directions */
        .framein = lintog722_framein,
-       .sample = lin16tog722_sample,
+       .sample = slin16_sample,
        .desc_size = sizeof(struct g722_encoder_pvt),
-       .buffer_samples = BUFFER_SAMPLES,
+       .buffer_samples = BUFFER_SAMPLES * 2,
        .buf_size = BUFFER_SAMPLES,
 };
 
@@ -230,9 +188,7 @@ static int parse_config(int reload)
        struct ast_flags config_flags = { reload ? CONFIG_FLAG_FILEUNCHANGED : 0 };
        struct ast_config *cfg = ast_config_load("codecs.conf", config_flags);
 
-       if (cfg == NULL)
-               return 0;
-       if (cfg == CONFIG_STATUS_FILEUNCHANGED)
+       if (cfg == CONFIG_STATUS_FILEMISSING || cfg == CONFIG_STATUS_FILEUNCHANGED || cfg == CONFIG_STATUS_FILEINVALID)
                return 0;
        for (var = ast_variable_browse(cfg, "plc"); var; var = var->next) {
                if (!strcasecmp(var->name, "genericplc")) {