Several changes to codec_dahdi to play nice with G723.
[asterisk/asterisk.git] / codecs / codec_g722.c
index ac0b179..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.
@@ -23,6 +24,9 @@
  *
  * \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
  *
@@ -42,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;
@@ -97,28 +101,15 @@ 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;
        int out_samples;
+       int in_samples;
 
-       out_samples = g722_decode(&tmp->g722, (int16_t *) &pvt->outbuf[pvt->samples * sizeof(int16_t)], 
-               src, f->samples);
+       /* g722_decode expects the samples to be in the invalid samples / 2 format */
+       in_samples = f->samples / 2;
 
-       pvt->samples += out_samples;
-
-       pvt->datalen += (out_samples * sizeof(int16_t));
-
-       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 g722tolin16_framein(struct ast_trans_pvt *pvt, struct ast_frame *f)
-{
-       struct g722_decoder_pvt *tmp = pvt->pvt;
-       int out_samples;
-
-       out_samples = g722_decode(&tmp->g722, (int16_t *) &pvt->outbuf[pvt->samples * sizeof(int16_t)], 
-               (uint8_t *) f->data, f->samples);
-
-       /* sample rate the same between formats, but don't assume that it won't output more ... */
        pvt->samples += out_samples;
 
        pvt->datalen += (out_samples * sizeof(int16_t));
@@ -131,94 +122,23 @@ static int lintog722_framein(struct ast_trans_pvt *pvt, struct ast_frame *f)
        struct g722_encoder_pvt *tmp = pvt->pvt;
        int outlen;
 
-       outlen = g722_encode(&tmp->g722, (uint8_t *) (&pvt->outbuf[pvt->datalen]), 
-               (int16_t *) f->data, f->samples);
-
-       pvt->samples += outlen;
-
-       pvt->datalen += outlen;
-
-       return 0;
-}
-
-static int lin16tog722_framein(struct ast_trans_pvt *pvt, struct ast_frame *f)
-{
-       struct g722_encoder_pvt *tmp = pvt->pvt;
-       int16_t *src = f->data;
-       int outlen;
-
-       outlen = g722_encode(&tmp->g722, (uint8_t*)(&pvt->outbuf[pvt->datalen]), src, f->samples);
+       outlen = g722_encode(&tmp->g722, (&pvt->outbuf.ui8[pvt->datalen]), 
+               (int16_t *) f->data.ptr, f->samples);
 
-       pvt->samples += outlen;
+       pvt->samples += outlen * 2;
 
        pvt->datalen += outlen;
 
        return 0;
 }
 
-static struct ast_frame *g722tolin_sample(void)
-{
-       static struct ast_frame f = {
-               .frametype = AST_FRAME_VOICE,
-               .subclass = AST_FORMAT_G722,
-               .datalen = sizeof(g722_slin_ex),
-               .samples = sizeof(g722_slin_ex),
-               .src = __PRETTY_FUNCTION__,
-               .data = g722_slin_ex,
-       };
-
-       return &f;
-}
-
-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),
-               .src = __PRETTY_FUNCTION__,
-               .data = slin_g722_ex,
-       };
-
-       return &f;
-}
-
-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;
-}
-
-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;
-}
-
 static struct ast_translator g722tolin = {
        .name = "g722tolin",
        .srcfmt = AST_FORMAT_G722,
        .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 / sizeof(int16_t),
        .buf_size = BUFFER_SAMPLES,
@@ -231,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,
 };
 
@@ -242,8 +162,8 @@ static struct ast_translator g722tolin16 = {
        .srcfmt = AST_FORMAT_G722,
        .dstfmt = AST_FORMAT_SLINEAR16,
        .newpvt = g722tolin16_new,      /* same for both directions */
-       .framein = g722tolin16_framein,
-       .sample = g722tolin16_sample,
+       .framein = g722tolin_framein,
+       .sample = g722_sample,
        .desc_size = sizeof(struct g722_decoder_pvt),
        .buffer_samples = BUFFER_SAMPLES / sizeof(int16_t),
        .buf_size = BUFFER_SAMPLES,
@@ -255,10 +175,10 @@ static struct ast_translator lin16tog722 = {
        .srcfmt = AST_FORMAT_SLINEAR16,
        .dstfmt = AST_FORMAT_G722,
        .newpvt = lin16tog722_new,      /* same for both directions */
-       .framein = lin16tog722_framein,
-       .sample = lin16tog722_sample,
+       .framein = lintog722_framein,
+       .sample = slin16_sample,
        .desc_size = sizeof(struct g722_encoder_pvt),
-       .buffer_samples = BUFFER_SAMPLES,
+       .buffer_samples = BUFFER_SAMPLES * 2,
        .buf_size = BUFFER_SAMPLES,
 };
 
@@ -268,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")) {