]> git.sesse.net Git - vlc/blobdiff - modules/access/decklink.cpp
macosx: add comment for application:openFiles: usage
[vlc] / modules / access / decklink.cpp
index 6dde9faa5bf64d587d00ed750837185c578b3d5e..25e7800ac85466af0bc5d134b30268bb0f2f773c 100644 (file)
@@ -2,23 +2,24 @@
  * decklink.cpp: BlackMagic DeckLink SDI input module
  *****************************************************************************
  * Copyright (C) 2010 Steinar H. Gunderson
+ * Copyright (C) 2012-2014 Rafaël Carré
  *
  * Authors: Steinar H. Gunderson <steinar+vlc@gunderson.no>
+            Rafaël Carré <funman@videolanorg>
  *
- * This library 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.
+ * This program 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.
  *
  * This library 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
+ * 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 this library; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301  USA
- *
+ * You should have received a copy of the GNU Lesser General Public License
+ * along with this program; if not, write to the Free Software Foundation,
+ * Inc., 51 Franklin Street, Fifth Floor, Boston MA 02110-1301, USA.
  *****************************************************************************/
 
 #define __STDC_CONSTANT_MACROS 1
 #include <DeckLinkAPI.h>
 #include <DeckLinkAPIDispatch.cpp>
 
-static int  Open ( vlc_object_t * );
-static void Close( vlc_object_t * );
+#include "sdi.h"
+
+static int  Open (vlc_object_t *);
+static void Close(vlc_object_t *);
 
 #define CARD_INDEX_TEXT N_("Input card to use")
 #define CARD_INDEX_LONGTEXT N_( \
     "DeckLink capture card to use, if multiple exist. " \
-    "The cards are numbered from 0." )
+    "The cards are numbered from 0.")
 
-#define MODE_TEXT N_("Desired input video mode")
+#define MODE_TEXT N_("Desired input video mode. Leave empty for autodetection.")
 #define MODE_LONGTEXT N_( \
     "Desired input video mode for DeckLink captures. " \
     "This value should be a FOURCC code in textual " \
-    "form, e.g. \"ntsc\"." )
+    "form, e.g. \"ntsc\".")
 
 #define AUDIO_CONNECTION_TEXT N_("Audio connection")
 #define AUDIO_CONNECTION_LONGTEXT N_( \
     "Audio connection to use for DeckLink captures. " \
     "Valid choices: embedded, aesebu, analog. " \
-    "Leave blank for card default." )
+    "Leave blank for card default.")
 
-#define RATE_TEXT N_("Audio sampling rate in Hz")
+#define RATE_TEXT N_("Audio samplerate (Hz)")
 #define RATE_LONGTEXT N_( \
     "Audio sampling rate (in hertz) for DeckLink captures. " \
-    "0 disables audio input." )
+    "0 disables audio input.")
 
 #define CHANNELS_TEXT N_("Number of audio channels")
 #define CHANNELS_LONGTEXT N_( \
     "Number of input audio channels for DeckLink captures. " \
-    "Must be 2, 8 or 16. 0 disables audio input." )
+    "Must be 2, 8 or 16. 0 disables audio input.")
 
 #define VIDEO_CONNECTION_TEXT N_("Video connection")
 #define VIDEO_CONNECTION_LONGTEXT N_( \
     "Video connection to use for DeckLink captures. " \
     "Valid choices: sdi, hdmi, opticalsdi, component, " \
     "composite, svideo. " \
-    "Leave blank for card default." )
+    "Leave blank for card default.")
 
 static const char *const ppsz_videoconns[] = {
     "sdi", "hdmi", "opticalsdi", "component", "composite", "svideo"
@@ -89,572 +92,633 @@ static const char *const ppsz_audioconns_text[] = {
 };
 
 #define ASPECT_RATIO_TEXT N_("Aspect ratio")
-#define ASPECT_RATIO_LONGTEXT N_( \
-    "Aspect ratio (4:3, 16:9). Default assumes square pixels." )
+#define ASPECT_RATIO_LONGTEXT N_(\
+    "Aspect ratio (4:3, 16:9). Default assumes square pixels.")
 
 vlc_module_begin ()
-    set_shortname( N_("DeckLink") )
-    set_description( N_("Blackmagic DeckLink SDI input") )
-    set_category( CAT_INPUT )
-    set_subcategory( SUBCAT_INPUT_ACCESS )
-
-    add_integer( "decklink-card-index", 0,
-                 CARD_INDEX_TEXT, CARD_INDEX_LONGTEXT, true )
-    add_string( "decklink-mode", "pal ",
-                 MODE_TEXT, MODE_LONGTEXT, true )
-    add_string( "decklink-audio-connection", 0,
-                 AUDIO_CONNECTION_TEXT, AUDIO_CONNECTION_LONGTEXT, true )
-        change_string_list( ppsz_audioconns, ppsz_audioconns_text, 0 )
-    add_integer( "decklink-audio-rate", 48000,
-                 RATE_TEXT, RATE_LONGTEXT, true )
-    add_integer( "decklink-audio-channels", 2,
-                 CHANNELS_TEXT, CHANNELS_LONGTEXT, true )
-    add_string( "decklink-video-connection", 0,
-                 VIDEO_CONNECTION_TEXT, VIDEO_CONNECTION_LONGTEXT, true )
-        change_string_list( ppsz_videoconns, ppsz_videoconns_text, 0 )
-    add_string( "decklink-aspect-ratio", NULL,
-                ASPECT_RATIO_TEXT, ASPECT_RATIO_LONGTEXT, true )
-
-    add_shortcut( "decklink" )
-    set_capability( "access_demux", 10 )
-    set_callbacks( Open, Close )
+    set_shortname(N_("DeckLink"))
+    set_description(N_("Blackmagic DeckLink SDI input"))
+    set_category(CAT_INPUT)
+    set_subcategory(SUBCAT_INPUT_ACCESS)
+
+    add_integer("decklink-card-index", 0,
+                 CARD_INDEX_TEXT, CARD_INDEX_LONGTEXT, true)
+    add_string("decklink-mode", NULL,
+                 MODE_TEXT, MODE_LONGTEXT, true)
+    add_string("decklink-audio-connection", 0,
+                 AUDIO_CONNECTION_TEXT, AUDIO_CONNECTION_LONGTEXT, true)
+        change_string_list(ppsz_audioconns, ppsz_audioconns_text)
+    add_integer("decklink-audio-rate", 48000,
+                 RATE_TEXT, RATE_LONGTEXT, true)
+    add_integer("decklink-audio-channels", 2,
+                 CHANNELS_TEXT, CHANNELS_LONGTEXT, true)
+    add_string("decklink-video-connection", 0,
+                 VIDEO_CONNECTION_TEXT, VIDEO_CONNECTION_LONGTEXT, true)
+        change_string_list(ppsz_videoconns, ppsz_videoconns_text)
+    add_string("decklink-aspect-ratio", NULL,
+                ASPECT_RATIO_TEXT, ASPECT_RATIO_LONGTEXT, true)
+    add_bool("decklink-tenbits", false, N_("10 bits"), N_("10 bits"), true)
+
+    add_shortcut("decklink")
+    set_capability("access_demux", 10)
+    set_callbacks(Open, Close)
 vlc_module_end ()
 
-static int Control( demux_t *, int, va_list );
+static int Control(demux_t *, int, va_list);
 
 class DeckLinkCaptureDelegate;
 
 struct demux_sys_t
 {
-    IDeckLink *p_card;
-    IDeckLinkInput *p_input;
-    DeckLinkCaptureDelegate *p_delegate;
+    IDeckLink *card;
+    IDeckLinkInput *input;
+    DeckLinkCaptureDelegate *delegate;
 
     /* We need to hold onto the IDeckLinkConfiguration object, or our settings will not apply.
        See section 2.4.15 of the Blackmagic Decklink SDK documentation. */
-    IDeckLinkConfiguration *p_config;
+    IDeckLinkConfiguration *config;
+    IDeckLinkAttributes *attributes;
+
+    bool autodetect;
 
-    es_out_id_t *p_video_es;
-    es_out_id_t *p_audio_es;
+    es_out_id_t *video_es;
+    es_out_id_t *audio_es;
+    es_out_id_t *cc_es;
 
     vlc_mutex_t pts_lock;
-    int i_last_pts;  /* protected by <pts_lock> */
+    int last_pts;  /* protected by <pts_lock> */
 
-    uint32_t i_dominance_flags;
-    int i_channels;
+    uint32_t dominance_flags;
+    int channels;
+
+    bool tenbits;
 };
 
+static const char *GetFieldDominance(BMDFieldDominance dom, uint32_t *flags)
+{
+    switch(dom)
+    {
+        case bmdProgressiveFrame:
+            return "";
+        case bmdProgressiveSegmentedFrame:
+            return ", segmented";
+        case bmdLowerFieldFirst:
+            *flags = BLOCK_FLAG_BOTTOM_FIELD_FIRST;
+            return ", interlaced [BFF]";
+        case bmdUpperFieldFirst:
+            *flags = BLOCK_FLAG_TOP_FIELD_FIRST;
+            return ", interlaced [TFF]";
+        case bmdUnknownFieldDominance:
+        default:
+            return ", unknown field dominance";
+    }
+}
+
+static es_format_t GetModeSettings(demux_t *demux, IDeckLinkDisplayMode *m)
+{
+    demux_sys_t *sys = demux->p_sys;
+    uint32_t flags = 0;
+    (void)GetFieldDominance(m->GetFieldDominance(), &flags);
+
+    BMDTimeValue frame_duration, time_scale;
+    if (m->GetFrameRate(&frame_duration, &time_scale) != S_OK) {
+        time_scale = 0;
+        frame_duration = 1;
+    }
+
+    es_format_t video_fmt;
+    vlc_fourcc_t chroma; chroma = sys->tenbits ? VLC_CODEC_I422_10L : VLC_CODEC_UYVY;
+    es_format_Init(&video_fmt, VIDEO_ES, chroma);
+
+    video_fmt.video.i_width = m->GetWidth();
+    video_fmt.video.i_height = m->GetHeight();
+    video_fmt.video.i_sar_num = 1;
+    video_fmt.video.i_sar_den = 1;
+    video_fmt.video.i_frame_rate = time_scale;
+    video_fmt.video.i_frame_rate_base = frame_duration;
+    video_fmt.i_bitrate = video_fmt.video.i_width * video_fmt.video.i_height * video_fmt.video.i_frame_rate * 2 * 8;
+
+    unsigned aspect_num, aspect_den;
+    if (!var_InheritURational(demux, &aspect_num, &aspect_den, "decklink-aspect-ratio") &&
+         aspect_num > 0 && aspect_den > 0) {
+        video_fmt.video.i_sar_num = aspect_num * video_fmt.video.i_height;
+        video_fmt.video.i_sar_den = aspect_den * video_fmt.video.i_width;
+    }
+
+    sys->dominance_flags = flags;
+
+    return video_fmt;
+}
+
 class DeckLinkCaptureDelegate : public IDeckLinkInputCallback
 {
 public:
-    DeckLinkCaptureDelegate( demux_t *p_demux ) : p_demux_(p_demux)
+    DeckLinkCaptureDelegate(demux_t *demux) : demux_(demux)
     {
-        vlc_atomic_set( &m_ref_, 1 );
+        atomic_store(&m_ref_, 1);
     }
 
-    virtual HRESULT STDMETHODCALLTYPE QueryInterface(REFIID iid, LPVOID *ppv) { return E_NOINTERFACE; }
+    virtual HRESULT STDMETHODCALLTYPE QueryInterface(REFIID, LPVOID *) { return E_NOINTERFACE; }
 
     virtual ULONG STDMETHODCALLTYPE AddRef(void)
     {
-        return vlc_atomic_inc( &m_ref_ );
+        return atomic_fetch_add(&m_ref_, 1);
     }
 
     virtual ULONG STDMETHODCALLTYPE Release(void)
     {
-        uintptr_t new_ref = vlc_atomic_dec( &m_ref_ );
-        if ( new_ref == 0 )
+        uintptr_t new_ref = atomic_fetch_sub(&m_ref_, 1);
+        if (new_ref == 0)
             delete this;
         return new_ref;
     }
 
-    virtual HRESULT STDMETHODCALLTYPE VideoInputFormatChanged(BMDVideoInputFormatChangedEvents, IDeckLinkDisplayMode*, BMDDetectedVideoInputFormatFlags)
+    virtual HRESULT STDMETHODCALLTYPE VideoInputFormatChanged(BMDVideoInputFormatChangedEvents events, IDeckLinkDisplayMode *mode, BMDDetectedVideoInputFormatFlags)
     {
-        msg_Dbg( p_demux_, "Video input format changed" );
+        demux_sys_t *sys = demux_->p_sys;
+
+        if( !(events & bmdVideoInputDisplayModeChanged ))
+            return S_OK;
+
+        const char *mode_name;
+        if (mode->GetName(&mode_name) != S_OK)
+            mode_name = "unknown";
+
+        msg_Dbg(demux_, "Video input format changed to %s", mode_name);
+        if (!sys->autodetect) {
+            msg_Err(demux_, "Video format detection disabled");
+            return S_OK;
+        }
+
+        es_out_Del(demux_->out, sys->video_es);
+        es_format_t video_fmt = GetModeSettings(demux_, mode);
+        sys->video_es = es_out_Add(demux_->out, &video_fmt);
+
+        BMDPixelFormat fmt = sys->tenbits ? bmdFormat10BitYUV : bmdFormat8BitYUV;
+        sys->input->PauseStreams();
+        sys->input->EnableVideoInput( mode->GetDisplayMode(), fmt, bmdVideoInputEnableFormatDetection );
+        sys->input->FlushStreams();
+        sys->input->StartStreams();
+
         return S_OK;
     }
 
     virtual HRESULT STDMETHODCALLTYPE VideoInputFrameArrived(IDeckLinkVideoInputFrame*, IDeckLinkAudioInputPacket*);
 
 private:
-    vlc_atomic_t m_ref_;
-    demux_t *p_demux_;
+    atomic_uint m_ref_;
+    demux_t *demux_;
 };
 
 HRESULT DeckLinkCaptureDelegate::VideoInputFrameArrived(IDeckLinkVideoInputFrame* videoFrame, IDeckLinkAudioInputPacket* audioFrame)
 {
-    demux_sys_t *p_sys = p_demux_->p_sys;
-    block_t *p_video_frame = NULL;
-    block_t *p_audio_frame = NULL;
+    demux_sys_t *sys = demux_->p_sys;
 
-    if( videoFrame )
-    {
-        if( videoFrame->GetFlags() & bmdFrameHasNoInputSource )
-        {
-            msg_Warn( p_demux_, "No input signal detected" );
+    if (videoFrame) {
+        if (videoFrame->GetFlags() & bmdFrameHasNoInputSource) {
+            msg_Warn(demux_, "No input signal detected");
             return S_OK;
         }
 
-        const int i_width = videoFrame->GetWidth();
-        const int i_height = videoFrame->GetHeight();
-        const int i_stride = videoFrame->GetRowBytes();
-        const int i_bpp = 2;
+        const int width = videoFrame->GetWidth();
+        const int height = videoFrame->GetHeight();
+        const int stride = videoFrame->GetRowBytes();
 
-        p_video_frame = block_New( p_demux_, i_width * i_height * i_bpp );
-        if( !p_video_frame )
-        {
-            msg_Err( p_demux_, "Could not allocate memory for video frame" );
+        int bpp = sys->tenbits ? 4 : 2;
+        block_t *video_frame = block_Alloc(width * height * bpp);
+        if (!video_frame)
             return S_OK;
-        }
 
-        void *frame_bytes;
-        videoFrame->GetBytes( &frame_bytes );
-        for( int y = 0; y < i_height; ++y )
-        {
-            const uint8_t *src = (const uint8_t *)frame_bytes + i_stride * y;
-            uint8_t *dst = p_video_frame->p_buffer + i_width * i_bpp * y;
-            memcpy( dst, src, i_width * i_bpp );
-        }
+        const uint32_t *frame_bytes;
+        videoFrame->GetBytes((void**)&frame_bytes);
 
         BMDTimeValue stream_time, frame_duration;
-        videoFrame->GetStreamTime( &stream_time, &frame_duration, CLOCK_FREQ );
-        p_video_frame->i_flags = BLOCK_FLAG_TYPE_I | p_sys->i_dominance_flags;
-        p_video_frame->i_pts = p_video_frame->i_dts = VLC_TS_0 + stream_time;
+        videoFrame->GetStreamTime(&stream_time, &frame_duration, CLOCK_FREQ);
+        video_frame->i_flags = BLOCK_FLAG_TYPE_I | sys->dominance_flags;
+        video_frame->i_pts = video_frame->i_dts = VLC_TS_0 + stream_time;
+
+        if (sys->tenbits) {
+            v210_convert((uint16_t*)video_frame->p_buffer, frame_bytes, width, height);
+            IDeckLinkVideoFrameAncillary *vanc;
+            if (videoFrame->GetAncillaryData(&vanc) == S_OK) {
+                for (int i = 1; i < 21; i++) {
+                    uint32_t *buf;
+                    if (vanc->GetBufferForVerticalBlankingLine(i, (void**)&buf) != S_OK)
+                        break;
+                    uint16_t dec[width * 2];
+                    v210_convert(&dec[0], buf, width, 1);
+                    block_t *cc = vanc_to_cc(demux_, dec, width * 2);
+                    if (!cc)
+                        continue;
+                    cc->i_pts = cc->i_dts = VLC_TS_0 + stream_time;
+
+                    if (!sys->cc_es) {
+                        es_format_t fmt;
+
+                        es_format_Init( &fmt, SPU_ES, VLC_FOURCC('c', 'c', '1' , ' ') );
+                        fmt.psz_description = strdup(N_("Closed captions 1"));
+                        if (fmt.psz_description) {
+                            sys->cc_es = es_out_Add(demux_->out, &fmt);
+                            msg_Dbg(demux_, "Adding Closed captions stream");
+                        }
+                    }
+                    if (sys->cc_es)
+                        es_out_Send(demux_->out, sys->cc_es, cc);
+                    else
+                        block_Release(cc);
+                    break; // we found the line with Closed Caption data
+                }
+                vanc->Release();
+            }
+        } else {
+            for (int y = 0; y < height; ++y) {
+                const uint8_t *src = (const uint8_t *)frame_bytes + stride * y;
+                uint8_t *dst = video_frame->p_buffer + width * 2 * y;
+                memcpy(dst, src, width * 2);
+            }
+        }
 
-        vlc_mutex_lock( &p_sys->pts_lock );
-        if( p_video_frame->i_pts > p_sys->i_last_pts )
-            p_sys->i_last_pts = p_video_frame->i_pts;
-        vlc_mutex_unlock( &p_sys->pts_lock );
+        vlc_mutex_lock(&sys->pts_lock);
+        if (video_frame->i_pts > sys->last_pts)
+            sys->last_pts = video_frame->i_pts;
+        vlc_mutex_unlock(&sys->pts_lock);
 
-        es_out_Control( p_demux_->out, ES_OUT_SET_PCR, p_video_frame->i_pts );
-        es_out_Send( p_demux_->out, p_sys->p_video_es, p_video_frame );
+        es_out_Control(demux_->out, ES_OUT_SET_PCR, video_frame->i_pts);
+        es_out_Send(demux_->out, sys->video_es, video_frame);
     }
 
-    if( audioFrame )
-    {
-        const int i_bytes = audioFrame->GetSampleFrameCount() * sizeof(int16_t) * p_sys->i_channels;
-
-        p_audio_frame = block_New( p_demux_, i_bytes );
-        if( !p_audio_frame )
-        {
-            msg_Err( p_demux_, "Could not allocate memory for audio frame" );
-            if( p_video_frame )
-                block_Release( p_video_frame );
+    if (audioFrame) {
+        const int bytes = audioFrame->GetSampleFrameCount() * sizeof(int16_t) * sys->channels;
+
+        block_t *audio_frame = block_Alloc(bytes);
+        if (!audio_frame)
             return S_OK;
-        }
 
         void *frame_bytes;
-        audioFrame->GetBytes( &frame_bytes );
-        memcpy( p_audio_frame->p_buffer, frame_bytes, i_bytes );
+        audioFrame->GetBytes(&frame_bytes);
+        memcpy(audio_frame->p_buffer, frame_bytes, bytes);
 
         BMDTimeValue packet_time;
-        audioFrame->GetPacketTime( &packet_time, CLOCK_FREQ );
-        p_audio_frame->i_pts = p_audio_frame->i_dts = VLC_TS_0 + packet_time;
+        audioFrame->GetPacketTime(&packet_time, CLOCK_FREQ);
+        audio_frame->i_pts = audio_frame->i_dts = VLC_TS_0 + packet_time;
 
-        vlc_mutex_lock( &p_sys->pts_lock );
-        if( p_audio_frame->i_pts > p_sys->i_last_pts )
-            p_sys->i_last_pts = p_audio_frame->i_pts;
-        vlc_mutex_unlock( &p_sys->pts_lock );
-        if( p_audio_frame->i_pts > p_sys->i_last_pts )
+        vlc_mutex_lock(&sys->pts_lock);
+        if (audio_frame->i_pts > sys->last_pts)
+            sys->last_pts = audio_frame->i_pts;
+        vlc_mutex_unlock(&sys->pts_lock);
 
-        es_out_Control( p_demux_->out, ES_OUT_SET_PCR, p_audio_frame->i_pts );
-        es_out_Send( p_demux_->out, p_sys->p_audio_es, p_audio_frame );
+        es_out_Control(demux_->out, ES_OUT_SET_PCR, audio_frame->i_pts);
+        es_out_Send(demux_->out, sys->audio_es, audio_frame);
     }
 
     return S_OK;
 }
 
-static int Open( vlc_object_t *p_this )
+
+static int GetAudioConn(demux_t *demux)
 {
-    demux_t     *p_demux = (demux_t*)p_this;
-    demux_sys_t *p_sys;
+    demux_sys_t *sys = demux->p_sys;
+
+    char *opt = var_CreateGetNonEmptyString(demux, "decklink-audio-connection");
+    if (!opt)
+        return VLC_SUCCESS;
+
+    BMDAudioConnection c;
+    if (!strcmp(opt, "embedded"))
+        c = bmdAudioConnectionEmbedded;
+    else if (!strcmp(opt, "aesebu"))
+        c = bmdAudioConnectionAESEBU;
+    else if (!strcmp(opt, "analog"))
+        c = bmdAudioConnectionAnalog;
+    else {
+        msg_Err(demux, "Invalid audio-connection: `%s\' specified", opt);
+        free(opt);
+        return VLC_EGENERIC;
+    }
+
+    if (sys->config->SetInt(bmdDeckLinkConfigAudioInputConnection, c) != S_OK) {
+        msg_Err(demux, "Failed to set audio input connection");
+        return VLC_EGENERIC;
+    }
+
+    return VLC_SUCCESS;
+}
+
+static int GetVideoConn(demux_t *demux)
+{
+    demux_sys_t *sys = demux->p_sys;
+
+    char *opt = var_InheritString(demux, "decklink-video-connection");
+    if (!opt)
+        return VLC_SUCCESS;
+
+    BMDVideoConnection c;
+    if (!strcmp(opt, "sdi"))
+        c = bmdVideoConnectionSDI;
+    else if (!strcmp(opt, "hdmi"))
+        c = bmdVideoConnectionHDMI;
+    else if (!strcmp(opt, "opticalsdi"))
+        c = bmdVideoConnectionOpticalSDI;
+    else if (!strcmp(opt, "component"))
+        c = bmdVideoConnectionComponent;
+    else if (!strcmp(opt, "composite"))
+        c = bmdVideoConnectionComposite;
+    else if (!strcmp(opt, "svideo"))
+        c = bmdVideoConnectionSVideo;
+    else {
+        msg_Err(demux, "Invalid video-connection: `%s\' specified", opt);
+        free(opt);
+        return VLC_EGENERIC;
+    }
+
+    free(opt);
+    if (sys->config->SetInt(bmdDeckLinkConfigVideoInputConnection, c) != S_OK) {
+        msg_Err(demux, "Failed to set video input connection");
+        return VLC_EGENERIC;
+    }
+
+    return VLC_SUCCESS;
+}
+
+static int Open(vlc_object_t *p_this)
+{
+    demux_t     *demux = (demux_t*)p_this;
+    demux_sys_t *sys;
     int         ret = VLC_EGENERIC;
-    char        *psz_aspect;
-    char        *psz_display_mode = NULL;
-    char        *psz_video_connection = NULL;
-    char        *psz_audio_connection = NULL;
-    bool        b_found_mode;
-    int         i_card_index;
-    int         i_width, i_height, i_fps_num, i_fps_den;
-    int         i_rate;
-    unsigned    u_aspect_num, u_aspect_den;
+    int         card_index;
+    int         physical_channels = 0;
+    int         rate;
+    BMDVideoInputFlags flags = bmdVideoInputFlagDefault;
 
     /* Only when selected */
-    if( *p_demux->psz_access == '\0' )
+    if (*demux->psz_access == '\0')
         return VLC_EGENERIC;
 
-    /* Set up p_demux */
-    p_demux->pf_demux = NULL;
-    p_demux->pf_control = Control;
-    p_demux->info.i_update = 0;
-    p_demux->info.i_title = 0;
-    p_demux->info.i_seekpoint = 0;
-    p_demux->p_sys = p_sys = (demux_sys_t*)calloc( 1, sizeof( demux_sys_t ) );
-    if( !p_sys )
+    /* Set up demux */
+    demux->pf_demux = NULL;
+    demux->pf_control = Control;
+    demux->info.i_update = 0;
+    demux->info.i_title = 0;
+    demux->info.i_seekpoint = 0;
+    demux->p_sys = sys = (demux_sys_t*)calloc(1, sizeof(demux_sys_t));
+    if (!sys)
         return VLC_ENOMEM;
 
-    vlc_mutex_init( &p_sys->pts_lock );
+    vlc_mutex_init(&sys->pts_lock);
 
-    IDeckLinkDisplayModeIterator *p_display_iterator = NULL;
+    sys->tenbits = var_InheritBool(p_this, "decklink-tenbits");
 
     IDeckLinkIterator *decklink_iterator = CreateDeckLinkIteratorInstance();
-    if( !decklink_iterator )
-    {
-        msg_Err( p_demux, "DeckLink drivers not found." );
+    if (!decklink_iterator) {
+        msg_Err(demux, "DeckLink drivers not found.");
         goto finish;
     }
 
-    HRESULT result;
-
-    i_card_index = var_InheritInteger( p_demux, "decklink-card-index" );
-    if( i_card_index < 0 )
-    {
-        msg_Err( p_demux, "Invalid card index %d", i_card_index );
+    card_index = var_InheritInteger(demux, "decklink-card-index");
+    if (card_index < 0) {
+        msg_Err(demux, "Invalid card index %d", card_index);
         goto finish;
     }
 
-    for( int i = 0; i <= i_card_index; ++i )
-    {
-        if( p_sys->p_card )
-            p_sys->p_card->Release();
-        result = decklink_iterator->Next( &p_sys->p_card );
-        if( result != S_OK )
-            break;
-    }
-
-    if( result != S_OK )
-    {
-        msg_Err( p_demux, "DeckLink PCI card %d not found", i_card_index );
-        goto finish;
+    for (int i = 0; i <= card_index; i++) {
+        if (sys->card)
+            sys->card->Release();
+        if (decklink_iterator->Next(&sys->card) != S_OK) {
+            msg_Err(demux, "DeckLink PCI card %d not found", card_index);
+            goto finish;
+        }
     }
 
-    const char *psz_model_name;
-    result = p_sys->p_card->GetModelName( &psz_model_name );
-
-    if( result != S_OK )
-    {
-        msg_Err( p_demux, "Could not get model name" );
-        goto finish;
-    }
+    const char *model_name;
+    if (sys->card->GetModelName(&model_name) != S_OK)
+        model_name = "unknown";
 
-    msg_Dbg( p_demux, "Opened DeckLink PCI card %d (%s)", i_card_index, psz_model_name );
+    msg_Dbg(demux, "Opened DeckLink PCI card %d (%s)", card_index, model_name);
 
-    if( p_sys->p_card->QueryInterface( IID_IDeckLinkInput, (void**)&p_sys->p_input) != S_OK )
-    {
-        msg_Err( p_demux, "Card has no inputs" );
+    if (sys->card->QueryInterface(IID_IDeckLinkInput, (void**)&sys->input) != S_OK) {
+        msg_Err(demux, "Card has no inputs");
         goto finish;
     }
 
     /* Set up the video and audio sources. */
-    if( p_sys->p_card->QueryInterface( IID_IDeckLinkConfiguration, (void**)&p_sys->p_config) != S_OK )
-    {
-        msg_Err( p_demux, "Failed to get configuration interface" );
+    if (sys->card->QueryInterface(IID_IDeckLinkConfiguration, (void**)&sys->config) != S_OK) {
+        msg_Err(demux, "Failed to get configuration interface");
         goto finish;
     }
 
-    psz_video_connection = var_InheritString( p_demux, "decklink-video-connection" );
-    if( psz_video_connection )
-    {
-        BMDVideoConnection conn;
-        if ( !strcmp( psz_video_connection, "sdi" ) )
-            conn = bmdVideoConnectionSDI;
-        else if ( !strcmp( psz_video_connection, "hdmi" ) )
-            conn = bmdVideoConnectionHDMI;
-        else if ( !strcmp( psz_video_connection, "opticalsdi" ) )
-            conn = bmdVideoConnectionOpticalSDI;
-        else if ( !strcmp( psz_video_connection, "component" ) )
-            conn = bmdVideoConnectionComponent;
-        else if ( !strcmp( psz_video_connection, "composite" ) )
-            conn = bmdVideoConnectionComposite;
-        else if ( !strcmp( psz_video_connection, "svideo" ) )
-            conn = bmdVideoConnectionSVideo;
-        else
-        {
-            msg_Err( p_demux, "Invalid --decklink-video-connection specified; choose one of " \
-                              "sdi, hdmi, opticalsdi, component, composite, or svideo." );
-            goto finish;
-        }
-
-        msg_Dbg( p_demux, "Setting video input connection to 0x%x", conn);
-        result = p_sys->p_config->SetInt( bmdDeckLinkConfigVideoInputConnection, conn );
-        if( result != S_OK )
-        {
-            msg_Err( p_demux, "Failed to set video input connection" );
-            goto finish;
-        }
+    if (sys->card->QueryInterface(IID_IDeckLinkAttributes, (void**)&sys->attributes) != S_OK) {
+        msg_Err(demux, "Failed to get attributes interface");
+        goto finish;
     }
 
-    psz_audio_connection = var_CreateGetNonEmptyString( p_demux, "decklink-audio-connection" );
-    if( psz_audio_connection )
-    {
-        BMDAudioConnection conn;
-        if ( !strcmp( psz_audio_connection, "embedded" ) )
-            conn = bmdAudioConnectionEmbedded;
-        else if ( !strcmp( psz_audio_connection, "aesebu" ) )
-            conn = bmdAudioConnectionAESEBU;
-        else if ( !strcmp( psz_audio_connection, "analog" ) )
-            conn = bmdAudioConnectionAnalog;
-        else
-        {
-            msg_Err( p_demux, "Invalid --decklink-audio-connection specified; choose one of " \
-                              "embedded, aesebu, or analog." );
-            goto finish;
-        }
-
-        msg_Dbg( p_demux, "Setting audio input format to 0x%x", conn);
-        result = p_sys->p_config->SetInt( bmdDeckLinkConfigAudioInputConnection, conn );
-        if( result != S_OK )
-        {
-            msg_Err( p_demux, "Failed to set audio input connection" );
-            goto finish;
-        }
-    }
+    if (GetVideoConn(demux) || GetAudioConn(demux))
+        goto finish;
 
-    /* Get the list of display modes. */
-    result = p_sys->p_input->GetDisplayModeIterator( &p_display_iterator );
-    if( result != S_OK )
-    {
-        msg_Err( p_demux, "Failed to enumerate display modes" );
+    BMDPixelFormat fmt;
+    fmt = sys->tenbits ? bmdFormat10BitYUV : bmdFormat8BitYUV;
+    if (sys->attributes->GetFlag(BMDDeckLinkSupportsInputFormatDetection, &sys->autodetect) != S_OK) {
+        msg_Err(demux, "Failed to query card attribute");
         goto finish;
     }
 
-    psz_display_mode = var_CreateGetNonEmptyString( p_demux, "decklink-mode" );
-    if( !psz_display_mode || strlen( psz_display_mode ) > 4 ) {
-        msg_Err( p_demux, "Missing or invalid --decklink-mode string" );
+    /* Get the list of display modes. */
+    IDeckLinkDisplayModeIterator *mode_it;
+    if (sys->input->GetDisplayModeIterator(&mode_it) != S_OK) {
+        msg_Err(demux, "Failed to enumerate display modes");
         goto finish;
     }
 
-    /*
-     * Pad the --decklink-mode string to four characters, so the user can specify e.g. "pal"
-     * without having to add the trailing space.
-     */
-    char sz_display_mode_padded[5];
-    strcpy(sz_display_mode_padded, "    ");
-    for( int i = 0; i < strlen( psz_display_mode ); ++i )
-        sz_display_mode_padded[i] = psz_display_mode[i];
+    union {
+        BMDDisplayMode id;
+        char str[4];
+    } u;
+
+    u.id = 0;
+
+    char *mode;
+    mode = var_CreateGetNonEmptyString(demux, "decklink-mode");
+    if (mode)
+        sys->autodetect = false; // disable autodetection if mode was set
+
+    if (sys->autodetect) {
+        msg_Dbg(demux, "Card supports input format detection");
+        flags |= bmdVideoInputEnableFormatDetection;
+        /* Enable a random format, we will reconfigure on format detection */
+        u.id = htonl(bmdModeHD1080p2997);
+    } else {
+        if (!mode || strlen(mode) < 3 || strlen(mode) > 4) {
+            msg_Err(demux, "Invalid mode: \'%s\'", mode ? mode : "");
+            free(mode);
+            goto finish;
+        }
 
-    BMDDisplayMode wanted_mode_id;
-    memcpy( &wanted_mode_id, &sz_display_mode_padded, sizeof(wanted_mode_id) );
+        msg_Dbg(demux, "Looking for mode \'%s\'", mode);
+        memcpy(u.str, mode, 4);
+        if (u.str[3] == '\0')
+            u.str[3] = ' '; /* 'pal'\0 -> 'pal ' */
+        free(mode);
+    }
 
-    b_found_mode = false;
+    es_format_t video_fmt;
+    video_fmt.video.i_width = 0;
 
-    for (;;)
-    {
-        IDeckLinkDisplayMode *p_display_mode;
-        result = p_display_iterator->Next( &p_display_mode );
-        if( result != S_OK || !p_display_mode )
+    for (IDeckLinkDisplayMode *m;; m->Release()) {
+        if ((mode_it->Next(&m) != S_OK) || !m)
             break;
 
-        char sz_mode_id_text[5] = {0};
-        BMDDisplayMode mode_id = ntohl( p_display_mode->GetDisplayMode() );
-        memcpy( sz_mode_id_text, &mode_id, sizeof(mode_id) );
-
-        const char *psz_mode_name;
-        result = p_display_mode->GetName( &psz_mode_name );
-        if( result != S_OK )
-        {
-            msg_Err( p_demux, "Failed to get display mode name" );
-            p_display_mode->Release();
-            goto finish;
-        }
-
+        const char *mode_name;
         BMDTimeValue frame_duration, time_scale;
-        result = p_display_mode->GetFrameRate( &frame_duration, &time_scale );
-        if( result != S_OK )
-        {
-            msg_Err( p_demux, "Failed to get frame rate" );
-            p_display_mode->Release();
-            goto finish;
+        uint32_t field_flags;
+        const char *field = GetFieldDominance(m->GetFieldDominance(), &field_flags);
+        BMDDisplayMode id = ntohl(m->GetDisplayMode());
+
+        if (m->GetName(&mode_name) != S_OK)
+            mode_name = "unknown";
+        if (m->GetFrameRate(&frame_duration, &time_scale) != S_OK) {
+            time_scale = 0;
+            frame_duration = 1;
         }
 
-        const char *psz_field_dominance;
-        uint32_t i_dominance_flags = 0;
-        switch( p_display_mode->GetFieldDominance() )
-        {
-        case bmdProgressiveFrame:
-            psz_field_dominance = "";
-            break;
-        case bmdProgressiveSegmentedFrame:
-            psz_field_dominance = ", segmented";
-            break;
-        case bmdLowerFieldFirst:
-            psz_field_dominance = ", interlaced [BFF]";
-            i_dominance_flags = BLOCK_FLAG_BOTTOM_FIELD_FIRST;
-            break;
-        case bmdUpperFieldFirst:
-            psz_field_dominance = ", interlaced [TFF]";
-            i_dominance_flags = BLOCK_FLAG_TOP_FIELD_FIRST;
-            break;
-        case bmdUnknownFieldDominance:
-        default:
-            psz_field_dominance = ", unknown field dominance";
-            break;
-        }
+        msg_Dbg(demux, "Found mode '%4.4s': %s (%dx%d, %.3f fps%s)",
+                 (char*)&id, mode_name,
+                 (int)m->GetWidth(), (int)m->GetHeight(),
+                 double(time_scale) / frame_duration, field);
 
-        msg_Dbg( p_demux, "Found mode '%s': %s (%dx%d, %.3f fps%s)",
-                 sz_mode_id_text, psz_mode_name,
-                 p_display_mode->GetWidth(), p_display_mode->GetHeight(),
-                 double(time_scale) / frame_duration, psz_field_dominance );
-
-        if( wanted_mode_id == mode_id )
-        {
-            b_found_mode = true;
-            i_width = p_display_mode->GetWidth();
-            i_height = p_display_mode->GetHeight();
-            i_fps_num = time_scale;
-            i_fps_den = frame_duration;
-            p_sys->i_dominance_flags = i_dominance_flags;
+        if (u.id == id) {
+            video_fmt = GetModeSettings(demux, m);
+            msg_Dbg(demux, "Using that mode");
         }
-
-        p_display_mode->Release();
     }
 
-    if( !b_found_mode )
-    {
-        msg_Err( p_demux, "Unknown video mode specified. " \
-                          "Run VLC with -v --verbose-objects=-all,+decklink " \
-                          "to get a list of supported modes." );
+    mode_it->Release();
+
+    if (video_fmt.video.i_width == 0) {
+        msg_Err(demux, "Unknown video mode `%4.4s\' specified.", (char*)&u.id);
         goto finish;
     }
 
-    result = p_sys->p_input->EnableVideoInput( htonl( wanted_mode_id ), bmdFormat8BitYUV, 0 );
-    if( result != S_OK )
-    {
-        msg_Err( p_demux, "Failed to enable video input" );
+    if (sys->input->EnableVideoInput(htonl(u.id), fmt, flags) != S_OK) {
+        msg_Err(demux, "Failed to enable video input");
         goto finish;
     }
 
     /* Set up audio. */
-    p_sys->i_channels = var_InheritInteger( p_demux, "decklink-audio-channels" );
-    i_rate = var_InheritInteger( p_demux, "decklink-audio-rate" );
-    if( i_rate > 0 && p_sys->i_channels > 0 )
-    {
-        result = p_sys->p_input->EnableAudioInput( i_rate, bmdAudioSampleType16bitInteger, p_sys->i_channels );
-        if( result != S_OK )
-        {
-            msg_Err( p_demux, "Failed to enable audio input" );
+    sys->channels = var_InheritInteger(demux, "decklink-audio-channels");
+    switch (sys->channels) {
+    case 0:
+        break;
+    case 2:
+        physical_channels = AOUT_CHANS_STEREO;
+        break;
+    case 8:
+        physical_channels = AOUT_CHANS_7_1;
+        break;
+    //case 16:
+    default:
+        msg_Err(demux, "Invalid number of channels (%d), disabling audio", sys->channels);
+        sys->channels = 0;
+    }
+    rate = var_InheritInteger(demux, "decklink-audio-rate");
+    if (rate > 0 && sys->channels > 0) {
+        if (sys->input->EnableAudioInput(rate, bmdAudioSampleType16bitInteger, sys->channels) != S_OK) {
+            msg_Err(demux, "Failed to enable audio input");
             goto finish;
         }
     }
 
-    p_sys->p_delegate = new DeckLinkCaptureDelegate( p_demux );
-    p_sys->p_input->SetCallback( p_sys->p_delegate );
+    sys->delegate = new DeckLinkCaptureDelegate(demux);
+    sys->input->SetCallback(sys->delegate);
 
-    result = p_sys->p_input->StartStreams();
-    if( result != S_OK )
-    {
-        msg_Err( p_demux, "Could not start streaming from SDI card. This could be caused "
-                          "by invalid video mode or flags, access denied, or card already in use." );
+    if (sys->input->StartStreams() != S_OK) {
+        msg_Err(demux, "Could not start streaming from SDI card. This could be caused "
+                          "by invalid video mode or flags, access denied, or card already in use.");
         goto finish;
     }
 
-    /* Declare elementary streams */
-    es_format_t video_fmt;
-    es_format_Init( &video_fmt, VIDEO_ES, VLC_CODEC_UYVY );
-    video_fmt.video.i_width = i_width;
-    video_fmt.video.i_height = i_height;
-    video_fmt.video.i_sar_num = 1;
-    video_fmt.video.i_sar_den = 1;
-    video_fmt.video.i_frame_rate = i_fps_num;
-    video_fmt.video.i_frame_rate_base = i_fps_den;
-    video_fmt.i_bitrate = video_fmt.video.i_width * video_fmt.video.i_height * video_fmt.video.i_frame_rate * 2 * 8;
-
-    if ( !var_InheritURational( p_demux, &u_aspect_num, &u_aspect_den, "decklink-aspect-ratio" ) &&
-         u_aspect_num > 0 && u_aspect_den > 0 ) {
-        video_fmt.video.i_sar_num = u_aspect_num * video_fmt.video.i_height;
-        video_fmt.video.i_sar_den = u_aspect_den * video_fmt.video.i_width;
-    }
-
-    msg_Dbg( p_demux, "added new video es %4.4s %dx%d",
-             (char*)&video_fmt.i_codec, video_fmt.video.i_width, video_fmt.video.i_height );
-    p_sys->p_video_es = es_out_Add( p_demux->out, &video_fmt );
+    msg_Dbg(demux, "added new video es %4.4s %dx%d",
+             (char*)&video_fmt.i_codec, video_fmt.video.i_width, video_fmt.video.i_height);
+    sys->video_es = es_out_Add(demux->out, &video_fmt);
 
     es_format_t audio_fmt;
-    es_format_Init( &audio_fmt, AUDIO_ES, VLC_CODEC_S16N );
-    audio_fmt.audio.i_channels = p_sys->i_channels;
-    audio_fmt.audio.i_rate = i_rate;
+    es_format_Init(&audio_fmt, AUDIO_ES, VLC_CODEC_S16N);
+    audio_fmt.audio.i_channels = sys->channels;
+    audio_fmt.audio.i_physical_channels = physical_channels;
+    audio_fmt.audio.i_rate = rate;
     audio_fmt.audio.i_bitspersample = 16;
     audio_fmt.audio.i_blockalign = audio_fmt.audio.i_channels * audio_fmt.audio.i_bitspersample / 8;
     audio_fmt.i_bitrate = audio_fmt.audio.i_channels * audio_fmt.audio.i_rate * audio_fmt.audio.i_bitspersample;
 
-    msg_Dbg( p_demux, "added new audio es %4.4s %dHz %dbpp %dch",
+    msg_Dbg(demux, "added new audio es %4.4s %dHz %dbpp %dch",
              (char*)&audio_fmt.i_codec, audio_fmt.audio.i_rate, audio_fmt.audio.i_bitspersample, audio_fmt.audio.i_channels);
-    p_sys->p_audio_es = es_out_Add( p_demux->out, &audio_fmt );
+    sys->audio_es = es_out_Add(demux->out, &audio_fmt);
 
     ret = VLC_SUCCESS;
 
 finish:
-    if( decklink_iterator )
+    if (decklink_iterator)
         decklink_iterator->Release();
 
-    free( psz_video_connection );
-    free( psz_audio_connection );
-    free( psz_display_mode );
-
-    if( p_display_iterator )
-        p_display_iterator->Release();
-
-    if( ret != VLC_SUCCESS )
-        Close( p_this );
+    if (ret != VLC_SUCCESS)
+        Close(p_this);
 
     return ret;
 }
 
-static void Close( vlc_object_t *p_this )
+static void Close(vlc_object_t *p_this)
 {
-    demux_t     *p_demux = (demux_t *)p_this;
-    demux_sys_t *p_sys   = p_demux->p_sys;
+    demux_t     *demux = (demux_t *)p_this;
+    demux_sys_t *sys   = demux->p_sys;
 
-    if( p_sys->p_config )
-        p_sys->p_config->Release();
+    if (sys->attributes)
+        sys->attributes->Release();
 
-    if( p_sys->p_input )
-    {
-        p_sys->p_input->StopStreams();
-        p_sys->p_input->Release();
+    if (sys->config)
+        sys->config->Release();
+
+    if (sys->input) {
+        sys->input->StopStreams();
+        sys->input->Release();
     }
 
-    if( p_sys->p_card )
-        p_sys->p_card->Release();
+    if (sys->card)
+        sys->card->Release();
 
-    if( p_sys->p_delegate )
-        p_sys->p_delegate->Release();
+    if (sys->delegate)
+        sys->delegate->Release();
 
-    vlc_mutex_destroy( &p_sys->pts_lock );
-    free( p_sys );
+    vlc_mutex_destroy(&sys->pts_lock);
+    free(sys);
 }
 
-static int Control( demux_t *p_demux, int i_query, va_list args )
+static int Control(demux_t *demux, int query, va_list args)
 {
-    demux_sys_t *p_sys = p_demux->p_sys;
+    demux_sys_t *sys = demux->p_sys;
     bool *pb;
-    int64_t    *pi64;
+    int64_t *pi64;
 
-    switch( i_query )
+    switch(query)
     {
         /* Special for access_demux */
         case DEMUX_CAN_PAUSE:
         case DEMUX_CAN_SEEK:
         case DEMUX_CAN_CONTROL_PACE:
-            pb = (bool*)va_arg( args, bool * );
+            pb = (bool*)va_arg(args, bool *);
             *pb = false;
             return VLC_SUCCESS;
 
         case DEMUX_GET_PTS_DELAY:
-            pi64 = (int64_t*)va_arg( args, int64_t * );
-            *pi64 =
-                INT64_C(1000) * var_InheritInteger( p_demux, "live-caching" );
+            pi64 = (int64_t*)va_arg(args, int64_t *);
+            *pi64 = INT64_C(1000) * var_InheritInteger(demux, "live-caching");
             return VLC_SUCCESS;
 
         case DEMUX_GET_TIME:
-            pi64 = (int64_t*)va_arg( args, int64_t * );
-            vlc_mutex_lock( &p_sys->pts_lock );
-            *pi64 = p_sys->i_last_pts;
-            vlc_mutex_unlock( &p_sys->pts_lock );
+            pi64 = (int64_t*)va_arg(args, int64_t *);
+            vlc_mutex_lock(&sys->pts_lock);
+            *pi64 = sys->last_pts;
+            vlc_mutex_unlock(&sys->pts_lock);
             return VLC_SUCCESS;
 
         default:
             return VLC_EGENERIC;
     }
-
-    return VLC_EGENERIC;
 }