]> git.sesse.net Git - ffmpeg/blobdiff - libavformat/dvenc.c
mpegts: replace some magic numbers with the existing define
[ffmpeg] / libavformat / dvenc.c
index cd22e97684b38e3366b9e50fd69bc8e2a8f00c82..bffad44b9f04346b6a496873575525ec73518752 100644 (file)
@@ -6,44 +6,47 @@
  * of DV technical info.
  *
  * Raw DV format
- * Copyright (c) 2002 Fabrice Bellard.
+ * Copyright (c) 2002 Fabrice Bellard
  *
  * 50 Mbps (DVCPRO50) support
  * Copyright (c) 2006 Daniel Maas <dmaas@maasdigital.com>
  *
- * This file is part of FFmpeg.
+ * This file is part of Libav.
  *
- * FFmpeg is free software; you can redistribute it and/or
+ * Libav is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Lesser General Public
  * License as published by the Free Software Foundation; either
  * version 2.1 of the License, or (at your option) any later version.
  *
- * FFmpeg is distributed in the hope that it will be useful,
+ * Libav is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
  * Lesser General Public License for more details.
  *
  * You should have received a copy of the GNU Lesser General Public
- * License along with FFmpeg; if not, write to the Free Software
+ * License along with Libav; if not, write to the Free Software
  * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
  */
 #include <time.h>
 #include <stdarg.h>
+
 #include "avformat.h"
-#include "dvdata.h"
+#include "internal.h"
+#include "libavcodec/dvdata.h"
 #include "dv.h"
-#include "fifo.h"
+#include "libavutil/fifo.h"
+#include "libavutil/mathematics.h"
 
 struct DVMuxContext {
-    const DVprofile*  sys;    /* Current DV profile. E.g.: 525/60, 625/50 */
-    int         n_ast;        /* Number of stereo audio streams (up to 2) */
-    AVStream   *ast[2];       /* Stereo audio streams */
-    AVFifoBuffer  audio_data[2]; /* Fifo for storing excessive amounts of PCM */
-    int         frames;       /* Number of a current frame */
-    time_t      start_time;   /* Start time of recording */
-    int         has_audio;    /* frame under contruction has audio */
-    int         has_video;    /* frame under contruction has video */
-    uint8_t     frame_buf[DV_MAX_FRAME_SIZE]; /* frame under contruction */
+    const DVprofile*  sys;           /* current DV profile, e.g.: 525/60, 625/50 */
+    int               n_ast;         /* number of stereo audio streams (up to 2) */
+    AVStream         *ast[2];        /* stereo audio streams */
+    AVFifoBuffer     *audio_data[2]; /* FIFO for storing excessive amounts of PCM */
+    int               frames;        /* current frame number */
+    int64_t           start_time;    /* recording start time */
+    int               has_audio;     /* frame under contruction has audio */
+    int               has_video;     /* frame under contruction has video */
+    uint8_t           frame_buf[DV_MAX_FRAME_SIZE]; /* frame under contruction */
 };
 
 static const int dv_aaux_packs_dist[12][9] = {
@@ -63,7 +66,7 @@ static const int dv_aaux_packs_dist[12][9] = {
 
 static int dv_audio_frame_size(const DVprofile* sys, int frame)
 {
-    return sys->audio_samples_dist[frame % (sizeof(sys->audio_samples_dist)/
+    return sys->audio_samples_dist[frame % (sizeof(sys->audio_samples_dist) /
                                             sizeof(sys->audio_samples_dist[0]))];
 }
 
@@ -77,100 +80,100 @@ static int dv_write_pack(enum dv_pack_type pack_id, DVMuxContext *c, uint8_t* bu
     buf[0] = (uint8_t)pack_id;
     switch (pack_id) {
     case dv_timecode:
-          ct = (time_t)(c->frames / ((float)c->sys->frame_rate /
-                                     (float)c->sys->frame_rate_base));
-          brktimegm(ct, &tc);
-          /*
-           * LTC drop-frame frame counter drops two frames (0 and 1) every
-           * minute, unless it is exactly divisible by 10
-           */
-          ltc_frame = (c->frames + 2*ct/60 - 2*ct/600) % c->sys->ltc_divisor;
-          buf[1] = (0 << 7) | /* Color fame: 0 - unsync; 1 - sync mode */
-                   (1 << 6) | /* Drop frame timecode: 0 - nondrop; 1 - drop */
-                   ((ltc_frame / 10) << 4) | /* Tens of frames */
-                   (ltc_frame % 10);         /* Units of frames */
-          buf[2] = (1 << 7) | /* Biphase mark polarity correction: 0 - even; 1 - odd */
-                   ((tc.tm_sec / 10) << 4) | /* Tens of seconds */
-                   (tc.tm_sec % 10);         /* Units of seconds */
-          buf[3] = (1 << 7) | /* Binary group flag BGF0 */
-                   ((tc.tm_min / 10) << 4) | /* Tens of minutes */
-                   (tc.tm_min % 10);         /* Units of minutes */
-          buf[4] = (1 << 7) | /* Binary group flag BGF2 */
-                   (1 << 6) | /* Binary group flag BGF1 */
-                   ((tc.tm_hour / 10) << 4) | /* Tens of hours */
-                   (tc.tm_hour % 10);         /* Units of hours */
-          break;
+        ct = (time_t)av_rescale_rnd(c->frames, c->sys->time_base.num,
+                                    c->sys->time_base.den, AV_ROUND_DOWN);
+        brktimegm(ct, &tc);
+        /*
+         * LTC drop-frame frame counter drops two frames (0 and 1) every
+         * minute, unless it is exactly divisible by 10
+         */
+        ltc_frame = (c->frames + 2 * ct / 60 - 2 * ct / 600) % c->sys->ltc_divisor;
+        buf[1] = (0                 << 7) | /* color frame: 0 - unsync; 1 - sync mode */
+                 (1                 << 6) | /* drop frame timecode: 0 - nondrop; 1 - drop */
+                 ((ltc_frame / 10)  << 4) | /* tens of frames */
+                 (ltc_frame % 10);          /* units of frames */
+        buf[2] = (1                 << 7) | /* biphase mark polarity correction: 0 - even; 1 - odd */
+                 ((tc.tm_sec / 10)  << 4) | /* tens of seconds */
+                 (tc.tm_sec % 10);          /* units of seconds */
+        buf[3] = (1                 << 7) | /* binary group flag BGF0 */
+                 ((tc.tm_min / 10)  << 4) | /* tens of minutes */
+                 (tc.tm_min % 10);          /* units of minutes */
+        buf[4] = (1                 << 7) | /* binary group flag BGF2 */
+                 (1                 << 6) | /* binary group flag BGF1 */
+                 ((tc.tm_hour / 10) << 4) | /* tens of hours */
+                 (tc.tm_hour % 10);         /* units of hours */
+        break;
     case dv_audio_source:  /* AAUX source pack */
-          va_start(ap, buf);
-          buf[1] = (1 << 7) | /* locked mode -- SMPTE only supports locked mode */
-                   (1 << 6) | /* reserved -- always 1 */
-                   (dv_audio_frame_size(c->sys, c->frames) -
-                    c->sys->audio_min_samples[0]);
-                              /* # of samples      */
-          buf[2] = (0 << 7) | /* multi-stereo      */
-                   (0 << 5) | /* #of audio channels per block: 0 -- 1 channel */
-                   (0 << 4) | /* pair bit: 0 -- one pair of channels */
-                   !!va_arg(ap, int); /* audio mode        */
-          buf[3] = (1 << 7) | /* res               */
-                   (1 << 6) | /* multi-language flag */
-                   (c->sys->dsf << 5) | /*  system: 60fields/50fields */
-                   (c->sys->n_difchan & 2); /* definition: 0 -- 25Mbps, 2 -- 50Mbps */
-          buf[4] = (1 << 7) | /* emphasis: 1 -- off */
-                   (0 << 6) | /* emphasis time constant: 0 -- reserved */
-                   (0 << 3) | /* frequency: 0 -- 48Khz, 1 -- 44,1Khz, 2 -- 32Khz */
-                    0;        /* quantization: 0 -- 16bit linear, 1 -- 12bit nonlinear */
-          va_end(ap);
-          break;
+        va_start(ap, buf);
+        buf[1] = (1 << 7) | /* locked mode -- SMPTE only supports locked mode */
+                 (1 << 6) | /* reserved -- always 1 */
+                 (dv_audio_frame_size(c->sys, c->frames) -
+                  c->sys->audio_min_samples[0]);
+                            /* # of samples      */
+        buf[2] = (0 << 7) | /* multi-stereo      */
+                 (0 << 5) | /* #of audio channels per block: 0 -- 1 channel */
+                 (0 << 4) | /* pair bit: 0 -- one pair of channels */
+                 !!va_arg(ap, int); /* audio mode        */
+        buf[3] = (1 << 7) | /* res               */
+                 (1 << 6) | /* multi-language flag */
+                 (c->sys->dsf << 5) | /*  system: 60fields/50fields */
+                 (c->sys->n_difchan & 2); /* definition: 0 -- 25Mbps, 2 -- 50Mbps */
+        buf[4] = (1 << 7) | /* emphasis: 1 -- off */
+                 (0 << 6) | /* emphasis time constant: 0 -- reserved */
+                 (0 << 3) | /* frequency: 0 -- 48kHz, 1 -- 44,1kHz, 2 -- 32kHz */
+                  0;        /* quantization: 0 -- 16bit linear, 1 -- 12bit nonlinear */
+        va_end(ap);
+        break;
     case dv_audio_control:
-          buf[1] = (0 << 6) | /* copy protection: 0 -- unrestricted */
-                   (1 << 4) | /* input source: 1 -- digital input */
-                   (3 << 2) | /* compression: 3 -- no information */
-                    0;        /* misc. info/SMPTE emphasis off */
-          buf[2] = (1 << 7) | /* recording start point: 1 -- no */
-                   (1 << 6) | /* recording end point: 1 -- no */
-                   (1 << 3) | /* recording mode: 1 -- original */
-                    7;
-          buf[3] = (1 << 7) | /* direction: 1 -- forward */
-                   (c->sys->pix_fmt == PIX_FMT_YUV420P ? 0x20 : /* speed */
-                                                         c->sys->ltc_divisor*4);
-          buf[4] = (1 << 7) | /* reserved -- always 1 */
-                    0x7f;     /* genre category */
-          break;
+        buf[1] = (0 << 6) | /* copy protection: 0 -- unrestricted */
+                 (1 << 4) | /* input source: 1 -- digital input */
+                 (3 << 2) | /* compression: 3 -- no information */
+                  0;        /* misc. info/SMPTE emphasis off */
+        buf[2] = (1 << 7) | /* recording start point: 1 -- no */
+                 (1 << 6) | /* recording end point: 1 -- no */
+                 (1 << 3) | /* recording mode: 1 -- original */
+                  7;
+        buf[3] = (1 << 7) | /* direction: 1 -- forward */
+                 (c->sys->pix_fmt == PIX_FMT_YUV420P ? 0x20 : /* speed */
+                                                       c->sys->ltc_divisor * 4);
+        buf[4] = (1 << 7) | /* reserved -- always 1 */
+                  0x7f;     /* genre category */
+        break;
     case dv_audio_recdate:
     case dv_video_recdate:  /* VAUX recording date */
-          ct = c->start_time + (time_t)(c->frames /
-               ((float)c->sys->frame_rate / (float)c->sys->frame_rate_base));
-          brktimegm(ct, &tc);
-          buf[1] = 0xff; /* ds, tm, tens of time zone, units of time zone */
-                         /* 0xff is very likely to be "unknown" */
-          buf[2] = (3 << 6) | /* reserved -- always 1 */
-                   ((tc.tm_mday / 10) << 4) | /* Tens of day */
-                   (tc.tm_mday % 10);         /* Units of day */
-          buf[3] = /* we set high 4 bits to 0, shouldn't we set them to week? */
-                   ((tc.tm_mon / 10) << 4) |    /* Tens of month */
-                   (tc.tm_mon  % 10);           /* Units of month */
-          buf[4] = (((tc.tm_year % 100) / 10) << 4) | /* Tens of year */
-                   (tc.tm_year % 10);                 /* Units of year */
-          break;
+        ct = c->start_time + av_rescale_rnd(c->frames, c->sys->time_base.num,
+                                            c->sys->time_base.den, AV_ROUND_DOWN);
+        brktimegm(ct, &tc);
+        buf[1] = 0xff; /* ds, tm, tens of time zone, units of time zone */
+                       /* 0xff is very likely to be "unknown" */
+        buf[2] = (3 << 6) | /* reserved -- always 1 */
+                 ((tc.tm_mday / 10) << 4) | /* Tens of day */
+                 (tc.tm_mday % 10);         /* Units of day */
+        buf[3] = /* we set high 4 bits to 0, shouldn't we set them to week? */
+                 ((tc.tm_mon / 10) << 4) |    /* Tens of month */
+                 (tc.tm_mon  % 10);           /* Units of month */
+        buf[4] = (((tc.tm_year % 100) / 10) << 4) | /* Tens of year */
+                 (tc.tm_year % 10);                 /* Units of year */
+        break;
     case dv_audio_rectime:  /* AAUX recording time */
     case dv_video_rectime:  /* VAUX recording time */
-          ct = c->start_time + (time_t)(c->frames /
-               ((float)c->sys->frame_rate / (float)c->sys->frame_rate_base));
-          brktimegm(ct, &tc);
-          buf[1] = (3 << 6) | /* reserved -- always 1 */
-                   0x3f; /* tens of frame, units of frame: 0x3f - "unknown" ? */
-          buf[2] = (1 << 7) | /* reserved -- always 1 */
-                   ((tc.tm_sec / 10) << 4) | /* Tens of seconds */
-                   (tc.tm_sec % 10);         /* Units of seconds */
-          buf[3] = (1 << 7) | /* reserved -- always 1 */
-                   ((tc.tm_min / 10) << 4) | /* Tens of minutes */
-                   (tc.tm_min % 10);         /* Units of minutes */
-          buf[4] = (3 << 6) | /* reserved -- always 1 */
-                   ((tc.tm_hour / 10) << 4) | /* Tens of hours */
-                   (tc.tm_hour % 10);         /* Units of hours */
-          break;
+        ct = c->start_time + av_rescale_rnd(c->frames, c->sys->time_base.num,
+                                                       c->sys->time_base.den, AV_ROUND_DOWN);
+        brktimegm(ct, &tc);
+        buf[1] = (3 << 6) | /* reserved -- always 1 */
+                 0x3f; /* tens of frame, units of frame: 0x3f - "unknown" ? */
+        buf[2] = (1 << 7) | /* reserved -- always 1 */
+                 ((tc.tm_sec / 10) << 4) | /* Tens of seconds */
+                 (tc.tm_sec % 10);         /* Units of seconds */
+        buf[3] = (1 << 7) | /* reserved -- always 1 */
+                 ((tc.tm_min / 10) << 4) | /* Tens of minutes */
+                 (tc.tm_min % 10);         /* Units of minutes */
+        buf[4] = (3 << 6) | /* reserved -- always 1 */
+                 ((tc.tm_hour / 10) << 4) | /* Tens of hours */
+                 (tc.tm_hour % 10);         /* Units of hours */
+        break;
     default:
-          buf[1] = buf[2] = buf[3] = buf[4] = 0xff;
+        buf[1] = buf[2] = buf[3] = buf[4] = 0xff;
     }
     return 5;
 }
@@ -181,19 +184,19 @@ static void dv_inject_audio(DVMuxContext *c, int channel, uint8_t* frame_ptr)
     size = 4 * dv_audio_frame_size(c->sys, c->frames);
     frame_ptr += channel * c->sys->difseg_size * 150 * 80;
     for (i = 0; i < c->sys->difseg_size; i++) {
-       frame_ptr += 6 * 80; /* skip DIF segment header */
-       for (j = 0; j < 9; j++) {
-          dv_write_pack(dv_aaux_packs_dist[i][j], c, &frame_ptr[3], i >= c->sys->difseg_size/2);
-          for (d = 8; d < 80; d+=2) {
-             of = c->sys->audio_shuffle[i][j] + (d - 8)/2 * c->sys->audio_stride;
-             if (of*2 >= size)
-                 continue;
-
-             frame_ptr[d] = av_fifo_peek(&c->audio_data[channel], of*2+1); // FIXME: may be we have to admit
-             frame_ptr[d+1] = av_fifo_peek(&c->audio_data[channel], of*2); //        that DV is a big endian PCM
-          }
-          frame_ptr += 16 * 80; /* 15 Video DIFs + 1 Audio DIF */
-       }
+        frame_ptr += 6 * 80; /* skip DIF segment header */
+        for (j = 0; j < 9; j++) {
+            dv_write_pack(dv_aaux_packs_dist[i][j], c, &frame_ptr[3], i >= c->sys->difseg_size/2);
+            for (d = 8; d < 80; d+=2) {
+                of = c->sys->audio_shuffle[i][j] + (d - 8)/2 * c->sys->audio_stride;
+                if (of*2 >= size)
+                    continue;
+
+                frame_ptr[d]   = *av_fifo_peek2(c->audio_data[channel], of*2+1); // FIXME: maybe we have to admit
+                frame_ptr[d+1] = *av_fifo_peek2(c->audio_data[channel], of*2);   //        that DV is a big-endian PCM
+            }
+            frame_ptr += 16 * 80; /* 15 Video DIFs + 1 Audio DIF */
+        }
     }
 }
 
@@ -230,8 +233,8 @@ static void dv_inject_metadata(DVMuxContext *c, uint8_t* frame)
  * The following 3 functions constitute our interface to the world
  */
 
-int dv_assemble_frame(DVMuxContext *c, AVStream* st,
-                      const uint8_t* data, int data_size, uint8_t** frame)
+static int dv_assemble_frame(DVMuxContext *c, AVStream* st,
+                             uint8_t* data, int data_size, uint8_t** frame)
 {
     int i, reqasize;
 
@@ -239,38 +242,38 @@ int dv_assemble_frame(DVMuxContext *c, AVStream* st,
     reqasize = 4 * dv_audio_frame_size(c->sys, c->frames);
 
     switch (st->codec->codec_type) {
-    case CODEC_TYPE_VIDEO:
-          /* FIXME: we have to have more sensible approach than this one */
-          if (c->has_video)
-              av_log(st->codec, AV_LOG_ERROR, "Can't process DV frame #%d. Insufficient audio data or severe sync problem.\n", c->frames);
+    case AVMEDIA_TYPE_VIDEO:
+        /* FIXME: we have to have more sensible approach than this one */
+        if (c->has_video)
+            av_log(st->codec, AV_LOG_ERROR, "Can't process DV frame #%d. Insufficient audio data or severe sync problem.\n", c->frames);
 
-          memcpy(*frame, data, c->sys->frame_size);
-          c->has_video = 1;
-          break;
-    case CODEC_TYPE_AUDIO:
-          for (i = 0; i < c->n_ast && st != c->ast[i]; i++);
+        memcpy(*frame, data, c->sys->frame_size);
+        c->has_video = 1;
+        break;
+    case AVMEDIA_TYPE_AUDIO:
+        for (i = 0; i < c->n_ast && st != c->ast[i]; i++);
 
           /* FIXME: we have to have more sensible approach than this one */
-          if (av_fifo_size(&c->audio_data[i]) + data_size >= 100*AVCODEC_MAX_AUDIO_FRAME_SIZE)
-              av_log(st->codec, AV_LOG_ERROR, "Can't process DV frame #%d. Insufficient video data or severe sync problem.\n", c->frames);
-          av_fifo_write(&c->audio_data[i], data, data_size);
+        if (av_fifo_size(c->audio_data[i]) + data_size >= 100*AVCODEC_MAX_AUDIO_FRAME_SIZE)
+            av_log(st->codec, AV_LOG_ERROR, "Can't process DV frame #%d. Insufficient video data or severe sync problem.\n", c->frames);
+        av_fifo_generic_write(c->audio_data[i], data, data_size, NULL);
 
-          /* Lets see if we've got enough audio for one DV frame */
-          c->has_audio |= ((reqasize <= av_fifo_size(&c->audio_data[i])) << i);
+        /* Let us see if we've got enough audio for one DV frame. */
+        c->has_audio |= ((reqasize <= av_fifo_size(c->audio_data[i])) << i);
 
-          break;
+        break;
     default:
-          break;
+        break;
     }
 
-    /* Lets see if we have enough data to construct one DV frame */
-    if (c->has_video == 1 && c->has_audio + 1 == 1<<c->n_ast) {
+    /* Let us see if we have enough data to construct one DV frame. */
+    if (c->has_video == 1 && c->has_audio + 1 == 1 << c->n_ast) {
         dv_inject_metadata(c, *frame);
         c->has_audio = 0;
-        for (i=0; i<c->n_ast; i++) {
-             dv_inject_audio(c, i, *frame);
-             av_fifo_drain(&c->audio_data[i], reqasize);
-             c->has_audio |= ((reqasize <= av_fifo_size(&c->audio_data[i])) << i);
+        for (i=0; i < c->n_ast; i++) {
+            dv_inject_audio(c, i, *frame);
+            av_fifo_drain(c->audio_data[i], reqasize);
+            c->has_audio |= ((reqasize <= av_fifo_size(c->audio_data[i])) << i);
         }
 
         c->has_video = 0;
@@ -283,64 +286,71 @@ int dv_assemble_frame(DVMuxContext *c, AVStream* st,
     return 0;
 }
 
-DVMuxContext* dv_init_mux(AVFormatContext* s)
+static DVMuxContext* dv_init_mux(AVFormatContext* s)
 {
     DVMuxContext *c = s->priv_data;
     AVStream *vst = NULL;
+    AVDictionaryEntry *t;
     int i;
 
     /* we support at most 1 video and 2 audio streams */
     if (s->nb_streams > 3)
         return NULL;
 
-    c->n_ast = 0;
+    c->n_ast  = 0;
     c->ast[0] = c->ast[1] = NULL;
 
     /* We have to sort out where audio and where video stream is */
     for (i=0; i<s->nb_streams; i++) {
-         switch (s->streams[i]->codec->codec_type) {
-         case CODEC_TYPE_VIDEO:
-               if (vst) return NULL;
-               vst = s->streams[i];
-               break;
-         case CODEC_TYPE_AUDIO:
-             if (c->n_ast > 1) return NULL;
-             c->ast[c->n_ast++] = s->streams[i];
-             break;
-         default:
-               goto bail_out;
-         }
+        switch (s->streams[i]->codec->codec_type) {
+        case AVMEDIA_TYPE_VIDEO:
+            if (vst) return NULL;
+            vst = s->streams[i];
+            break;
+        case AVMEDIA_TYPE_AUDIO:
+            if (c->n_ast > 1) return NULL;
+            c->ast[c->n_ast++] = s->streams[i];
+            break;
+        default:
+            goto bail_out;
+        }
     }
 
     /* Some checks -- DV format is very picky about its incoming streams */
     if (!vst || vst->codec->codec_id != CODEC_ID_DVVIDEO)
         goto bail_out;
     for (i=0; i<c->n_ast; i++) {
-        if (c->ast[i] && (c->ast[i]->codec->codec_id != CODEC_ID_PCM_S16LE ||
+        if (c->ast[i] && (c->ast[i]->codec->codec_id    != CODEC_ID_PCM_S16LE ||
                           c->ast[i]->codec->sample_rate != 48000 ||
-                          c->ast[i]->codec->channels != 2))
+                          c->ast[i]->codec->channels    != 2))
             goto bail_out;
     }
-    c->sys = dv_codec_profile(vst->codec);
+    c->sys = avpriv_dv_codec_profile(vst->codec);
     if (!c->sys)
         goto bail_out;
 
-    if((c->n_ast > 1) && (c->sys->n_difchan < 2)) {
+    if ((c->n_ast > 1) && (c->sys->n_difchan < 2)) {
         /* only 1 stereo pair is allowed in 25Mbps mode */
         goto bail_out;
     }
 
     /* Ok, everything seems to be in working order */
-    c->frames = 0;
-    c->has_audio = 0;
-    c->has_video = 0;
-    c->start_time = (time_t)s->timestamp;
+    c->frames     = 0;
+    c->has_audio  = 0;
+    c->has_video  = 0;
+#if FF_API_TIMESTAMP
+    if (s->timestamp)
+        c->start_time = s->timestamp;
+    else
+#endif
+    if (t = av_dict_get(s->metadata, "creation_time", NULL, 0))
+        c->start_time = ff_iso8601_to_unix_time(t->value);
 
-    for (i=0; i<c->n_ast; i++) {
-        if (c->ast[i] && av_fifo_init(&c->audio_data[i], 100*AVCODEC_MAX_AUDIO_FRAME_SIZE) < 0) {
-            while (i>0) {
+    for (i=0; i < c->n_ast; i++) {
+        if (c->ast[i] && !(c->audio_data[i]=av_fifo_alloc(100*AVCODEC_MAX_AUDIO_FRAME_SIZE))) {
+            while (i > 0) {
                 i--;
-                av_fifo_free(&c->audio_data[i]);
+                av_fifo_free(c->audio_data[i]);
             }
             goto bail_out;
         }
@@ -352,20 +362,19 @@ bail_out:
     return NULL;
 }
 
-void dv_delete_mux(DVMuxContext *c)
+static void dv_delete_mux(DVMuxContext *c)
 {
     int i;
     for (i=0; i < c->n_ast; i++)
-        av_fifo_free(&c->audio_data[i]);
+        av_fifo_free(c->audio_data[i]);
 }
 
-#ifdef CONFIG_MUXERS
 static int dv_write_header(AVFormatContext *s)
 {
     if (!dv_init_mux(s)) {
         av_log(s, AV_LOG_ERROR, "Can't initialize DV format!\n"
                     "Make sure that you supply exactly two streams:\n"
-                    "     video: 25fps or 29.97fps, audio: 2ch/48Khz/PCM\n"
+                    "     video: 25fps or 29.97fps, audio: 2ch/48kHz/PCM\n"
                     "     (50Mbps allows an optional second audio stream)\n");
         return -1;
     }
@@ -380,8 +389,8 @@ static int dv_write_packet(struct AVFormatContext *s, AVPacket *pkt)
     fsize = dv_assemble_frame(s->priv_data, s->streams[pkt->stream_index],
                               pkt->data, pkt->size, &frame);
     if (fsize > 0) {
-        put_buffer(&s->pb, frame, fsize);
-        put_flush_packet(&s->pb);
+        avio_write(s->pb, frame, fsize);
+        avio_flush(s->pb);
     }
     return 0;
 }
@@ -397,19 +406,15 @@ static int dv_write_trailer(struct AVFormatContext *s)
     dv_delete_mux(s->priv_data);
     return 0;
 }
-#endif /* CONFIG_MUXERS */
-
-#ifdef CONFIG_DV_MUXER
-AVOutputFormat dv_muxer = {
-    "dv",
-    "DV video format",
-    NULL,
-    "dv",
-    sizeof(DVMuxContext),
-    CODEC_ID_PCM_S16LE,
-    CODEC_ID_DVVIDEO,
-    dv_write_header,
-    dv_write_packet,
-    dv_write_trailer,
+
+AVOutputFormat ff_dv_muxer = {
+    .name              = "dv",
+    .long_name         = NULL_IF_CONFIG_SMALL("DV video format"),
+    .extensions        = "dv",
+    .priv_data_size    = sizeof(DVMuxContext),
+    .audio_codec       = CODEC_ID_PCM_S16LE,
+    .video_codec       = CODEC_ID_DVVIDEO,
+    .write_header      = dv_write_header,
+    .write_packet      = dv_write_packet,
+    .write_trailer     = dv_write_trailer,
 };
-#endif