]> git.sesse.net Git - vlc/blobdiff - modules/audio_output/auhal.c
skins2: reuse graphics from generic bitmap cache (radialslider)
[vlc] / modules / audio_output / auhal.c
index a0419a0fa5960ee83b433051161a598248c45f47..b520dc61b273a7cad70e7a5e8aaf811607b607a3 100644 (file)
@@ -6,6 +6,7 @@
  *
  * Authors: Derk-Jan Hartman <hartman at videolan dot org>
  *          Felix Paul Kühne <fkuehne at videolan dot org>
+ *          David Fuhrmann <david dot fuhrmann at googlemail dot com>
  *
  * 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
@@ -22,9 +23,7 @@
  * Inc., 51 Franklin Street, Fifth Floor, Boston MA 02110-1301, USA.
  *****************************************************************************/
 
-/*****************************************************************************
- * Preamble
- *****************************************************************************/
+#pragma mark includes
 
 #ifdef HAVE_CONFIG_H
 # import "config.h"
@@ -42,6 +41,9 @@
 
 #import "TPCircularBuffer.h"
 
+#pragma mark -
+#pragma mark private declarations
+
 #ifndef verify_noerr
 # define verify_noerr(a) assert((a) == noErr)
 #endif
 
 #define AOUT_VAR_SPDIF_FLAG 0xf00000
 
-#define kBufferLength 2048 * 8 * 8 * 4
+#define AUDIO_BUFFER_SIZE_IN_SECONDS (AOUT_MAX_ADVANCE_TIME / CLOCK_FREQ)
+
 
 #define AOUT_VOLUME_DEFAULT             256
 #define AOUT_VOLUME_MAX                 512
 
+#define VOLUME_TEXT N_("Audio volume")
+#define VOLUME_LONGTEXT VOLUME_TEXT
+
+#define DEVICE_TEXT N_("Last audio device")
+#define DEVICE_LONGTEXT DEVICE_TEXT
 
 /*****************************************************************************
  * aout_sys_t: private audio output method descriptor
@@ -97,12 +105,12 @@ struct aout_sys_t
     bool                        b_got_first_sample; /* did the aout core provide something to render? */
 
     int                         i_rate;             /* media sample rate */
-    mtime_t                     i_played_length;    /* how much did we play already */
-    mtime_t                     i_last_sample_time; /* last sample time played by the AudioUnit */
+    int                         i_bytes_per_sample;
 
     struct audio_device_t       *devices;
 
     vlc_mutex_t                 lock;
+    vlc_cond_t                  cond;
 };
 
 struct audio_device_t
@@ -112,44 +120,41 @@ struct audio_device_t
     char *name;
 };
 
-/*****************************************************************************
- * Local prototypes.
- *****************************************************************************/
+
+#pragma mark -
+#pragma mark local prototypes & module descriptor
+
 static int      Open                    (vlc_object_t *);
-static int      OpenAnalog              (audio_output_t *, audio_sample_format_t *);
-static int      OpenSPDIF               (audio_output_t *, audio_sample_format_t *);
 static void     Close                   (vlc_object_t *);
-
-static void     Play                    (audio_output_t *, block_t *);
-static void     Flush                   (audio_output_t *, bool);
-static void     Pause                   (audio_output_t *, bool, mtime_t);
-static int      TimeGet                 (audio_output_t *, mtime_t *);
+static int      Start                   (audio_output_t *, audio_sample_format_t *);
+static int      StartAnalog             (audio_output_t *, audio_sample_format_t *);
+static int      StartSPDIF              (audio_output_t *, audio_sample_format_t *);
+static void     Stop                    (audio_output_t *);
 
 static int      DeviceList              (audio_output_t *p_aout, char ***namesp, char ***descsp);
 static void     RebuildDeviceList       (audio_output_t *);
 static int      SwitchAudioDevice       (audio_output_t *p_aout, const char *name);
+static int      VolumeSet               (audio_output_t *, float);
+static int      MuteSet                 (audio_output_t *, bool);
 
-static int      AudioDeviceHasOutput    (AudioDeviceID);
-static int      AudioDeviceSupportsDigital(audio_output_t *, AudioDeviceID);
-static int      AudioStreamSupportsDigital(audio_output_t *, AudioStreamID);
-static int      AudioStreamChangeFormat (audio_output_t *, AudioStreamID, AudioStreamBasicDescription);
-
-static OSStatus RenderCallbackAnalog    (vlc_object_t *, AudioUnitRenderActionFlags *, const AudioTimeStamp *, UInt32 , UInt32, AudioBufferList *);
+static void     Play                    (audio_output_t *, block_t *);
+static void     Pause                   (audio_output_t *, bool, mtime_t);
+static void     Flush                   (audio_output_t *, bool);
+static int      TimeGet                 (audio_output_t *, mtime_t *);
+static OSStatus RenderCallbackAnalog    (vlc_object_t *, AudioUnitRenderActionFlags *, const AudioTimeStamp *,
+                                         UInt32 , UInt32, AudioBufferList *);
 
 static OSStatus RenderCallbackSPDIF     (AudioDeviceID, const AudioTimeStamp *, const void *, const AudioTimeStamp *,
-                                          AudioBufferList *, const AudioTimeStamp *, void *);
+                                         AudioBufferList *, const AudioTimeStamp *, void *);
+
 static OSStatus HardwareListener        (AudioObjectID, UInt32, const AudioObjectPropertyAddress *, void *);
 static OSStatus StreamListener          (AudioObjectID, UInt32, const AudioObjectPropertyAddress *, void *);
 
-static int      VolumeSet               (audio_output_t *, float);
-static int      MuteSet                 (audio_output_t *, bool);
-
+static int      AudioDeviceHasOutput    (AudioDeviceID);
+static int      AudioDeviceSupportsDigital(audio_output_t *, AudioDeviceID);
+static int      AudioStreamSupportsDigital(audio_output_t *, AudioStreamID);
+static int      AudioStreamChangeFormat (audio_output_t *, AudioStreamID, AudioStreamBasicDescription);
 
-/*****************************************************************************
- * Module descriptor
- *****************************************************************************/
-#define VOLUME_TEXT N_("Audio volume")
-#define VOLUME_LONGTEXT VOLUME_TEXT
 
 vlc_module_begin ()
     set_shortname("auhal")
@@ -161,12 +166,63 @@ vlc_module_begin ()
     add_integer("auhal-volume", AOUT_VOLUME_DEFAULT,
                 VOLUME_TEXT, VOLUME_LONGTEXT, true)
     change_integer_range(0, AOUT_VOLUME_MAX)
+    add_string("auhal-audio-device", "", DEVICE_TEXT, DEVICE_LONGTEXT, true)
     add_obsolete_integer("macosx-audio-device") /* since 2.1.0 */
 vlc_module_end ()
 
-/*****************************************************************************
- * Open: open macosx audio output
- *****************************************************************************/
+#pragma mark -
+#pragma mark initialization
+
+static int Open(vlc_object_t *obj)
+{
+    audio_output_t *aout = (audio_output_t *)obj;
+    aout_sys_t *sys = malloc(sizeof (*sys));
+
+    if (unlikely(sys == NULL))
+        return VLC_ENOMEM;
+
+    vlc_mutex_init(&sys->lock);
+    vlc_cond_init(&sys->cond);
+
+    aout->sys = sys;
+    aout->start = Start;
+    aout->stop = Stop;
+    aout->volume_set = VolumeSet;
+    aout->mute_set = MuteSet;
+    aout->device_enum = DeviceList;
+    aout->sys->devices = NULL;
+    aout->device_select = SwitchAudioDevice;
+
+    RebuildDeviceList(aout);
+
+    /* remember the volume */
+    aout_VolumeReport(aout, var_InheritInteger(aout, "auhal-volume") / (float)AOUT_VOLUME_DEFAULT);
+    MuteSet(aout, var_InheritBool(aout, "mute"));
+
+    SwitchAudioDevice(aout, config_GetPsz(aout, "auhal-audio-device"));
+
+    return VLC_SUCCESS;
+}
+
+static void Close(vlc_object_t *obj)
+{
+    audio_output_t *aout = (audio_output_t *)obj;
+    aout_sys_t *sys = aout->sys;
+
+    config_PutPsz(aout, "auhal-audio-device", aout_DeviceGet(aout));
+
+    for (struct audio_device_t * device = sys->devices, *next; device != NULL; device = next) {
+        next = device->next;
+        free(device->name);
+        free(device);
+    }
+
+    vlc_mutex_destroy(&sys->lock);
+    vlc_cond_destroy(&sys->cond);
+
+    free(sys);
+}
+
 static int Start(audio_output_t *p_aout, audio_sample_format_t *restrict fmt)
 {
     OSStatus                err = noErr;
@@ -177,6 +233,8 @@ static int Start(audio_output_t *p_aout, audio_sample_format_t *restrict fmt)
      * property size */
     int                     b_alive = false;
 
+    bool                    b_start_digital = false;
+
     p_sys = p_aout->sys;
     p_sys->b_digital = false;
     p_sys->au_component = NULL;
@@ -187,12 +245,10 @@ static int Start(audio_output_t *p_aout, audio_sample_format_t *restrict fmt)
     p_sys->i_stream_index = -1;
     p_sys->b_revert = false;
     p_sys->b_changed_mixing = false;
+    p_sys->i_bytes_per_sample = 0;
 
     aout_FormatPrint(p_aout, "VLC is looking for:", fmt);
 
-    if (p_sys->b_selected_dev_is_digital)
-        msg_Dbg(p_aout, "audio device supports digital output");
-
     msg_Dbg(p_aout, "attempting to use device %i", p_sys->i_selected_dev);
 
     /* Check if the desired device is alive and usable */
@@ -210,10 +266,19 @@ static int Start(audio_output_t *p_aout, audio_sample_format_t *restrict fmt)
     }
 
     if (!b_alive) {
-        msg_Warn(p_aout, "selected audio device is not alive, switching to default device");
+        msg_Warn(p_aout, "selected audio device is not alive, switching to default device with id %i", p_sys->i_default_dev);
         p_sys->i_selected_dev = p_sys->i_default_dev;
+        p_sys->b_selected_dev_is_digital = false;
     }
 
+    // recheck if device still supports digital
+    b_start_digital = p_sys->b_selected_dev_is_digital;
+    if(!AudioDeviceSupportsDigital(p_aout, p_sys->i_selected_dev))
+        b_start_digital = false;
+
+    if (b_start_digital)
+        msg_Dbg(p_aout, "Use audio device for digital output");
+
     /* add a callback to see if the device dies later on */
     err = AudioObjectAddPropertyListener(p_sys->i_selected_dev, &audioDeviceAliveAddress, HardwareListener, (void *)p_aout);
     if (err != noErr) {
@@ -245,13 +310,13 @@ static int Start(audio_output_t *p_aout, audio_sample_format_t *restrict fmt)
     bool b_success = false;
 
     /* Check for Digital mode or Analog output mode */
-    if (AOUT_FMT_SPDIF (fmt) && p_sys->b_selected_dev_is_digital) {
-        if (OpenSPDIF (p_aout, fmt)) {
+    if (AOUT_FMT_SPDIF (fmt) && b_start_digital) {
+        if (StartSPDIF (p_aout, fmt)) {
             msg_Dbg(p_aout, "digital output successfully opened");
             b_success = true;
         }
     } else {
-        if (OpenAnalog(p_aout, fmt)) {
+        if (StartAnalog(p_aout, fmt)) {
             msg_Dbg(p_aout, "analog output successfully opened");
             b_success = true;
         }
@@ -267,14 +332,14 @@ static int Start(audio_output_t *p_aout, audio_sample_format_t *restrict fmt)
 
 error:
     /* If we reach this, this aout has failed */
-    msg_Err(p_aout, "opening the auhal output failed");
+    msg_Err(p_aout, "opening auhal output failed");
     return VLC_EGENERIC;
 }
 
-/*****************************************************************************
- * Open: open and setup a HAL AudioUnit to do analog (multichannel) audio output
- *****************************************************************************/
-static int OpenAnalog(audio_output_t *p_aout, audio_sample_format_t *fmt)
+/*
+ * StartAnalog: open and setup a HAL AudioUnit to do PCM audio output
+ */
+static int StartAnalog(audio_output_t *p_aout, audio_sample_format_t *fmt)
 {
     struct aout_sys_t           *p_sys = p_aout->sys;
     OSStatus                    err = noErr;
@@ -581,11 +646,10 @@ static int OpenAnalog(audio_output_t *p_aout, audio_sample_format_t *fmt)
     p_sys->clock_diff += mdate();
 
     /* setup circular buffer */
-    TPCircularBufferInit(&p_sys->circular_buffer, kBufferLength);
+    TPCircularBufferInit(&p_sys->circular_buffer, AUDIO_BUFFER_SIZE_IN_SECONDS *
+                         fmt->i_rate * fmt->i_bytes_per_frame);
 
     p_sys->b_got_first_sample = false;
-    p_sys->i_played_length = 0;
-    p_sys->i_last_sample_time = 0;
 
     /* Set volume for output unit */
     float volume = var_InheritInteger(p_aout, "auhal-volume") / (float)AOUT_VOLUME_DEFAULT;
@@ -600,10 +664,10 @@ static int OpenAnalog(audio_output_t *p_aout, audio_sample_format_t *fmt)
     return true;
 }
 
-/*****************************************************************************
- * Setup a encoded digital stream (SPDIF)
- *****************************************************************************/
-static int OpenSPDIF (audio_output_t * p_aout, audio_sample_format_t *fmt)
+/*
+ * StartSPDIF: Setup an encoded digital stream (SPDIF) output
+ */
+static int StartSPDIF (audio_output_t * p_aout, audio_sample_format_t *fmt)
 {
     struct aout_sys_t       *p_sys = p_aout->sys;
     OSStatus                err = noErr;
@@ -678,7 +742,7 @@ static int OpenSPDIF (audio_output_t * p_aout, audio_sample_format_t *fmt)
         /* Retrieve all the stream formats supported by each output stream */
         err = AudioObjectGetPropertyDataSize(p_streams[i], &physicalFormatsAddress, 0, NULL, &i_param_size);
         if (err != noErr) {
-            msg_Err(p_aout, "OpenSPDIF: could not get number of streamformats: [%s] (%i)", (char *)&err, (int32_t)err);
+            msg_Err(p_aout, "could not get number of streamformats: [%s] (%i)", (char *)&err, (int32_t)err);
             continue;
         }
 
@@ -805,13 +869,12 @@ static int OpenSPDIF (audio_output_t * p_aout, audio_sample_format_t *fmt)
         return false;
     }
 
+    /* setup circular buffer */
+    TPCircularBufferInit(&p_sys->circular_buffer, 200 * AOUT_SPDIF_SIZE);
+
     return true;
 }
 
-
-/*****************************************************************************
- * Close: Close HAL AudioUnit
- *****************************************************************************/
 static void Stop(audio_output_t *p_aout)
 {
     struct aout_sys_t   *p_sys = p_aout->sys;
@@ -888,13 +951,15 @@ static void Stop(audio_output_t *p_aout)
             msg_Err(p_aout, "Could not release hogmode: [%4.4s]", (char *)&err);
     }
 
-    p_sys->i_played_length = 0;
-    p_sys->i_last_sample_time = 0;
+    p_sys->i_bytes_per_sample = 0;
 
     /* clean-up circular buffer */
     TPCircularBufferCleanup(&p_sys->circular_buffer);
 }
 
+#pragma mark -
+#pragma mark core interaction
+
 static int DeviceList(audio_output_t *p_aout, char ***namesp, char ***descsp)
 {
     struct aout_sys_t   *p_sys = p_aout->sys;
@@ -949,12 +1014,13 @@ static void RebuildDeviceList(audio_output_t * p_aout)
             free(device);
         }
     }
+    p_sys->devices = NULL;
 
     /* Get number of devices */
     AudioObjectPropertyAddress audioDevicesAddress = { kAudioHardwarePropertyDevices, kAudioObjectPropertyScopeGlobal, kAudioObjectPropertyElementMaster };
     err = AudioObjectGetPropertyDataSize(kAudioObjectSystemObject, &audioDevicesAddress, 0, NULL, &propertySize);
     if (err != noErr) {
-        msg_Err(p_aout, "Could not get number of devices: [%s]", (char *)&err);
+        msg_Err(p_aout, "Could not get number of devices: [%4.4s]", (char *)&err);
         return;
     }
 
@@ -974,7 +1040,7 @@ static void RebuildDeviceList(audio_output_t * p_aout)
     /* Populate DeviceID array */
     err = AudioObjectGetPropertyData(kAudioObjectSystemObject, &audioDevicesAddress, 0, NULL, &propertySize, deviceIDs);
     if (err != noErr) {
-        msg_Err(p_aout, "could not get the device IDs: [%s]", (char *)&err);
+        msg_Err(p_aout, "could not get the device IDs: [%4.4s]", (char *)&err);
         return;
     }
 
@@ -983,7 +1049,7 @@ static void RebuildDeviceList(audio_output_t * p_aout)
     propertySize = sizeof(AudioObjectID);
     err= AudioObjectGetPropertyData(kAudioObjectSystemObject, &defaultDeviceAddress, 0, NULL, &propertySize, &defaultDeviceID);
     if (err != noErr) {
-        msg_Err(p_aout, "could not get default audio device: [%s]", (char *)&err);
+        msg_Err(p_aout, "could not get default audio device: [%4.4s]", (char *)&err);
         return;
     }
     p_sys->i_default_dev = defaultDeviceID;
@@ -1072,271 +1138,137 @@ static int SwitchAudioDevice(audio_output_t *p_aout, const char *name)
     return 0;
 }
 
-/*****************************************************************************
- * AudioDeviceHasOutput: Checks if the Device actually provides any outputs at all
- *****************************************************************************/
-static int AudioDeviceHasOutput(AudioDeviceID i_dev_id)
+static int VolumeSet(audio_output_t * p_aout, float volume)
 {
-    UInt32 dataSize = 0;
-    OSStatus status;
+    struct aout_sys_t *p_sys = p_aout->sys;
+    OSStatus ostatus;
 
-    AudioObjectPropertyAddress streamsAddress = { kAudioDevicePropertyStreams, kAudioDevicePropertyScopeOutput, kAudioObjectPropertyElementMaster };
-    status = AudioObjectGetPropertyDataSize(i_dev_id, &streamsAddress, 0, NULL, &dataSize);
+    aout_VolumeReport(p_aout, volume);
 
-    if (dataSize == 0 || status != noErr)
-        return FALSE;
+    /* Set volume for output unit */
+    ostatus = AudioUnitSetParameter(p_sys->au_unit,
+                                    kHALOutputParam_Volume,
+                                    kAudioUnitScope_Global,
+                                    0,
+                                    volume * volume * volume,
+                                    0);
 
-    return TRUE;
+    if (var_InheritBool(p_aout, "volume-save"))
+        config_PutInt(p_aout, "auhal-volume", lroundf(volume * AOUT_VOLUME_DEFAULT));
+
+    return ostatus;
 }
 
-/*****************************************************************************
- * AudioDeviceSupportsDigital: Check i_dev_id for digital stream support.
- *****************************************************************************/
-static int AudioDeviceSupportsDigital(audio_output_t *p_aout, AudioDeviceID i_dev_id)
+static int MuteSet(audio_output_t * p_aout, bool mute)
 {
-    OSStatus                    err = noErr;
-    UInt32                      i_param_size = 0;
-    AudioStreamID               *p_streams = NULL;
-    int                         i_streams = 0;
-    bool                        b_return = false;
+    struct   aout_sys_t *p_sys = p_aout->sys;
+    OSStatus ostatus;
 
-    /* Retrieve all the output streams */
-    AudioObjectPropertyAddress streamsAddress = { kAudioDevicePropertyStreams, kAudioDevicePropertyScopeOutput, kAudioObjectPropertyElementMaster };
-    err = AudioObjectGetPropertyDataSize(i_dev_id, &streamsAddress, 0, NULL, &i_param_size);
-    if (err != noErr) {
-        msg_Err(p_aout, "could not get number of streams: [%s] (%i)", (char *)&err, (int32_t)err);
-        return false;
-    }
+    aout_MuteReport(p_aout, mute);
 
-    i_streams = i_param_size / sizeof(AudioStreamID);
-    p_streams = (AudioStreamID *)malloc(i_param_size);
-    if (p_streams == NULL)
-        return VLC_ENOMEM;
+    float volume = .0;
 
-    err = AudioObjectGetPropertyData(i_dev_id, &streamsAddress, 0, NULL, &i_param_size, p_streams);
-    if (err != noErr) {
-        msg_Err(p_aout, "could not get list of streams: [%s]", (char *)&err);
-        return false;
-    }
+    if (!mute)
+        volume = var_InheritInteger(p_aout, "auhal-volume") / (float)AOUT_VOLUME_DEFAULT;
 
-    for (int i = 0; i < i_streams; i++) {
-        if (AudioStreamSupportsDigital(p_aout, p_streams[i]))
-            b_return = true;
-    }
+    ostatus = AudioUnitSetParameter(p_sys->au_unit,
+                                    kHALOutputParam_Volume,
+                                    kAudioUnitScope_Global,
+                                    0,
+                                    volume * volume * volume,
+                                    0);
 
-    free(p_streams);
-    return b_return;
+    return ostatus;
 }
 
-/*****************************************************************************
- * AudioStreamSupportsDigital: Check i_stream_id for digital stream support.
- *****************************************************************************/
-static int AudioStreamSupportsDigital(audio_output_t *p_aout, AudioStreamID i_stream_id)
+#pragma mark -
+#pragma mark actual playback
+
+static void Play (audio_output_t * p_aout, block_t * p_block)
 {
-    OSStatus                    err = noErr;
-    UInt32                      i_param_size = 0;
-    AudioStreamRangedDescription *p_format_list = NULL;
-    int                         i_formats = 0;
-    bool                        b_return = false;
+    struct aout_sys_t *p_sys = p_aout->sys;
 
-    /* Retrieve all the stream formats supported by each output stream */
-    AudioObjectPropertyAddress physicalFormatsAddress = { kAudioStreamPropertyAvailablePhysicalFormats, kAudioObjectPropertyScopeGlobal, 0 };
-    err = AudioObjectGetPropertyDataSize(i_stream_id, &physicalFormatsAddress, 0, NULL, &i_param_size);
-    if (err != noErr) {
-        msg_Err(p_aout, "could not get number of streamformats: [%s] (%i)", (char *)&err, (int32_t)err);
-        return false;
-    }
+    if (p_block->i_nb_samples > 0) {
+        if (!p_sys->b_got_first_sample) {
+            /* Start the AU */
+            verify_noerr(AudioOutputUnitStart(p_sys->au_unit));
+            p_sys->b_got_first_sample = true;
+        }
 
-    i_formats = i_param_size / sizeof(AudioStreamRangedDescription);
-    msg_Dbg(p_aout, "found %i stream formats", i_formats);
+        /* Do the channel reordering */
+        if (p_sys->chans_to_reorder && !p_sys->b_digital) {
+           aout_ChannelReorder(p_block->p_buffer,
+                               p_block->i_buffer,
+                               p_sys->chans_to_reorder,
+                               p_sys->chan_table,
+                               VLC_CODEC_FL32);
+        }
 
-    p_format_list = (AudioStreamRangedDescription *)malloc(i_param_size);
-    if (p_format_list == NULL)
-        return false;
+        /* move data to buffer */
+        if (unlikely(!TPCircularBufferProduceBytes(&p_sys->circular_buffer, p_block->p_buffer, p_block->i_buffer)))
+            msg_Warn(p_aout, "dropped buffer");
 
-    err = AudioObjectGetPropertyData(i_stream_id, &physicalFormatsAddress, 0, NULL, &i_param_size, p_format_list);
-    if (err != noErr) {
-        msg_Err(p_aout, "could not get the list of streamformats: [%4.4s]", (char *)&err);
-        free(p_format_list);
-        p_format_list = NULL;
-        return false;
+        if (!p_sys->i_bytes_per_sample)
+            p_sys->i_bytes_per_sample = p_block->i_buffer / p_block->i_nb_samples;
     }
 
-    for (int i = 0; i < i_formats; i++) {
-#ifndef NDEBUG
-        msg_Dbg(p_aout, STREAM_FORMAT_MSG("supported format: ", p_format_list[i].mFormat));
-#endif
+    block_Release(p_block);
+}
 
-        if (p_format_list[i].mFormat.mFormatID == 'IAC3' ||
-            p_format_list[i].mFormat.mFormatID == 'iac3' ||
-            p_format_list[i].mFormat.mFormatID == kAudioFormat60958AC3 ||
-            p_format_list[i].mFormat.mFormatID == kAudioFormatAC3)
-            b_return = true;
-    }
+static void Pause (audio_output_t *p_aout, bool pause, mtime_t date)
+{
+    struct aout_sys_t * p_sys = p_aout->sys;
+    VLC_UNUSED(date);
 
-    free(p_format_list);
-    return b_return;
+    if (p_aout->sys->b_digital) {
+        if (pause)
+            AudioDeviceStop(p_sys->i_selected_dev, p_sys->i_procID);
+        else
+            AudioDeviceStart(p_sys->i_selected_dev, p_sys->i_procID);
+    } else {
+        if (pause)
+            AudioOutputUnitStop(p_sys->au_unit);
+        else
+            AudioOutputUnitStart(p_sys->au_unit);
+    }
 }
 
-/*****************************************************************************
- * AudioStreamChangeFormat: Change i_stream_id to change_format
- *****************************************************************************/
-static int AudioStreamChangeFormat(audio_output_t *p_aout, AudioStreamID i_stream_id, AudioStreamBasicDescription change_format)
+static void Flush(audio_output_t *p_aout, bool wait)
 {
-    OSStatus            err = noErr;
-    UInt32              i_param_size = 0;
+    struct aout_sys_t *p_sys = p_aout->sys;
 
-    AudioObjectPropertyAddress physicalFormatAddress = { kAudioStreamPropertyPhysicalFormat, kAudioObjectPropertyScopeGlobal, kAudioObjectPropertyElementMaster };
+    if (wait) {
+        int32_t availableBytes;
+        vlc_mutex_lock(&p_sys->lock);
+        TPCircularBufferTail(&p_sys->circular_buffer, &availableBytes);
+        while (availableBytes > 0) {
+            vlc_cond_wait(&p_sys->cond, &p_sys->lock);
+            TPCircularBufferTail(&p_sys->circular_buffer, &availableBytes);
+        }
+        vlc_mutex_unlock(&p_sys->lock);
 
-    struct { vlc_mutex_t lock; vlc_cond_t cond; } w;
-
-    msg_Dbg(p_aout, STREAM_FORMAT_MSG("setting stream format: ", change_format));
-
-    /* Condition because SetProperty is asynchronious */
-    vlc_cond_init(&w.cond);
-    vlc_mutex_init(&w.lock);
-    vlc_mutex_lock(&w.lock);
-
-    /* Install the callback */
-    err = AudioObjectAddPropertyListener(i_stream_id, &physicalFormatAddress, StreamListener, (void *)&w);
-    if (err != noErr) {
-        msg_Err(p_aout, "AudioObjectAddPropertyListener for kAudioStreamPropertyPhysicalFormat failed: [%4.4s]", (char *)&err);
-        return false;
-    }
-
-    /* change the format */
-    err = AudioObjectSetPropertyData(i_stream_id, &physicalFormatAddress, 0, NULL, sizeof(AudioStreamBasicDescription),
-                                     &change_format);
-    if (err != noErr) {
-        msg_Err(p_aout, "could not set the stream format: [%4.4s]", (char *)&err);
-        return false;
-    }
-
-    /* The AudioStreamSetProperty is not only asynchronious (requiring the locks)
-     * it is also not atomic in its behaviour.
-     * Therefore we check 5 times before we really give up.
-     * FIXME: failing isn't actually implemented yet. */
-    for (int i = 0; i < 5; i++) {
-        AudioStreamBasicDescription actual_format;
-        mtime_t timeout = mdate() + 500000;
-
-        if (vlc_cond_timedwait(&w.cond, &w.lock, timeout))
-            msg_Dbg(p_aout, "reached timeout");
-
-        i_param_size = sizeof(AudioStreamBasicDescription);
-        err = AudioObjectGetPropertyData(i_stream_id, &physicalFormatAddress, 0, NULL, &i_param_size, &actual_format);
-
-        msg_Dbg(p_aout, STREAM_FORMAT_MSG("actual format in use: ", actual_format));
-        if (actual_format.mSampleRate == change_format.mSampleRate &&
-            actual_format.mFormatID == change_format.mFormatID &&
-            actual_format.mFramesPerPacket == change_format.mFramesPerPacket) {
-            /* The right format is now active */
-            break;
-        }
-        /* We need to check again */
-    }
-
-    /* Removing the property listener */
-    err = AudioObjectRemovePropertyListener(i_stream_id, &physicalFormatAddress, StreamListener, (void *)&w);
-    if (err != noErr) {
-        msg_Err(p_aout, "AudioStreamRemovePropertyListener failed: [%4.4s]", (char *)&err);
-        return false;
-    }
-
-    /* Destroy the lock and condition */
-    vlc_mutex_unlock(&w.lock);
-    vlc_mutex_destroy(&w.lock);
-    vlc_cond_destroy(&w.cond);
-
-    return true;
-}
-
-static void Play (audio_output_t * p_aout, block_t * p_block)
-{
-    struct aout_sys_t *p_sys = p_aout->sys;
-
-    if (p_block->i_nb_samples > 0) {
-        if (!p_sys->b_got_first_sample) {
-            /* Start the AU */
-            verify_noerr(AudioOutputUnitStart(p_sys->au_unit));
-            p_sys->b_got_first_sample = true;
-        }
-
-        /* Do the channel reordering */
-        if (p_sys->chans_to_reorder && !p_sys->b_digital) {
-           aout_ChannelReorder(p_block->p_buffer,
-                               p_block->i_buffer,
-                               p_sys->chans_to_reorder,
-                               p_sys->chan_table,
-                               VLC_CODEC_FL32);
-        }
-
-        /* Render audio into buffer */
-        AudioBufferList bufferList;
-        bufferList.mNumberBuffers = 1;
-        bufferList.mBuffers[0].mNumberChannels = p_sys->i_numberOfChannels;
-        bufferList.mBuffers[0].mData = malloc(p_block->i_nb_samples * sizeof(Float32) * p_sys->i_numberOfChannels);
-        bufferList.mBuffers[0].mDataByteSize = p_block->i_nb_samples * sizeof(Float32) * p_sys->i_numberOfChannels;
-
-        memcpy(bufferList.mBuffers[0].mData, p_block->p_buffer, p_block->i_buffer);
-
-        /* keep track of the played data */
-        p_aout->sys->i_played_length += p_block->i_length;
-
-        /* move data to buffer */
-        TPCircularBufferProduceBytes(&p_sys->circular_buffer, bufferList.mBuffers[0].mData, bufferList.mBuffers[0].mDataByteSize);
-    }
-
-    block_Release(p_block);
-}
-
-static void Pause (audio_output_t *p_aout, bool pause, mtime_t date)
-{
-    struct aout_sys_t * p_sys = p_aout->sys;
-    VLC_UNUSED(date);
-
-    if (p_aout->sys->b_digital) {
-        if (pause)
-            AudioDeviceStop(p_sys->i_selected_dev, p_sys->i_procID);
-        else
-            AudioDeviceStart(p_sys->i_selected_dev, p_sys->i_procID);
     } else {
-        if (pause)
-            AudioOutputUnitStop(p_sys->au_unit);
-        else
-            AudioOutputUnitStart(p_sys->au_unit);
+        p_sys->b_got_first_sample = false;
+
+        /* flush circular buffer */
+        AudioOutputUnitStop(p_aout->sys->au_unit);
+        TPCircularBufferClear(&p_aout->sys->circular_buffer);
     }
 }
 
-static void Flush(audio_output_t *p_aout, bool wait)
+static int TimeGet(audio_output_t *p_aout, mtime_t *delay)
 {
     struct aout_sys_t * p_sys = p_aout->sys;
-    VLC_UNUSED(wait);
-
-    p_sys->b_got_first_sample = false;
 
-    /* flush circular buffer */
-    AudioOutputUnitStop(p_aout->sys->au_unit);
-    TPCircularBufferClear(&p_aout->sys->circular_buffer);
+    if (!p_sys->i_bytes_per_sample)
+        return -1;
 
-    p_sys->i_played_length = 0;
-    p_sys->i_last_sample_time = 0;
-}
+    int32_t availableBytes;
+    TPCircularBufferTail(&p_sys->circular_buffer, &availableBytes);
 
-static int TimeGet(audio_output_t *p_aout, mtime_t *delay)
-{
-    struct aout_sys_t * p_sys = p_aout->sys;
+    *delay = (availableBytes / p_sys->i_bytes_per_sample) * CLOCK_FREQ / p_sys->i_rate;
 
-    vlc_mutex_lock(&p_sys->lock);
-    mtime_t i_pos = p_sys->i_last_sample_time * CLOCK_FREQ / p_sys->i_rate;
-    vlc_mutex_unlock(&p_sys->lock);
-
-    if (i_pos > 0) {
-        *delay = p_aout->sys->i_played_length - i_pos;
-        return 0;
-    }
-    else
-        return -1;
+    return 0;
 }
 
 /*****************************************************************************
@@ -1354,6 +1286,7 @@ static OSStatus RenderCallbackAnalog(vlc_object_t *p_obj,
     VLC_UNUSED(ioActionFlags);
     VLC_UNUSED(inTimeStamp);
     VLC_UNUSED(inBusNumber);
+    VLC_UNUSED(inNumberFrames);
 
     audio_output_t * p_aout = (audio_output_t *)p_obj;
     struct aout_sys_t * p_sys = p_aout->sys;
@@ -1361,6 +1294,7 @@ static OSStatus RenderCallbackAnalog(vlc_object_t *p_obj,
     int bytesToCopy = ioData->mBuffers[0].mDataByteSize;
     Float32 *targetBuffer = (Float32*)ioData->mBuffers[0].mData;
 
+    vlc_mutex_lock(&p_sys->lock);
     /* Pull audio from buffer */
     int32_t availableBytes;
     Float32 *buffer = TPCircularBufferTail(&p_sys->circular_buffer, &availableBytes);
@@ -1373,18 +1307,17 @@ static OSStatus RenderCallbackAnalog(vlc_object_t *p_obj,
     } else {
         memcpy(targetBuffer, buffer, __MIN(bytesToCopy, availableBytes));
         TPCircularBufferConsume(&p_sys->circular_buffer, __MIN(bytesToCopy, availableBytes));
-        VLC_UNUSED(inNumberFrames);
-        vlc_mutex_lock(&p_sys->lock);
-        p_sys->i_last_sample_time = inTimeStamp->mSampleTime;
-        vlc_mutex_unlock(&p_sys->lock);
     }
 
+    vlc_cond_signal(&p_sys->cond);
+    vlc_mutex_unlock(&p_sys->lock);
+
     return noErr;
 }
 
-/*****************************************************************************
+/*
  * RenderCallbackSPDIF: callback for SPDIF audio output
- *****************************************************************************/
+ */
 static OSStatus RenderCallbackSPDIF (AudioDeviceID inDevice,
                                     const AudioTimeStamp * inNow,
                                     const void * inInputData,
@@ -1397,16 +1330,18 @@ static OSStatus RenderCallbackSPDIF (AudioDeviceID inDevice,
     VLC_UNUSED(inDevice);
     VLC_UNUSED(inInputData);
     VLC_UNUSED(inInputTime);
+    VLC_UNUSED(inOutputTime);
 
     audio_output_t * p_aout = (audio_output_t *)threadGlobals;
     struct aout_sys_t * p_sys = p_aout->sys;
 
     int bytesToCopy = outOutputData->mBuffers[p_sys->i_stream_index].mDataByteSize;
-    Float32 *targetBuffer = (Float32*)outOutputData->mBuffers[p_sys->i_stream_index].mData;
+    char *targetBuffer = outOutputData->mBuffers[p_sys->i_stream_index].mData;
 
+    vlc_mutex_lock(&p_sys->lock);
     /* Pull audio from buffer */
     int32_t availableBytes;
-    Float32 *buffer = TPCircularBufferTail(&p_sys->circular_buffer, &availableBytes);
+    char *buffer = TPCircularBufferTail(&p_sys->circular_buffer, &availableBytes);
 
     /* check if we have enough data */
     if (!availableBytes) {
@@ -1415,17 +1350,20 @@ static OSStatus RenderCallbackSPDIF (AudioDeviceID inDevice,
     } else {
         memcpy(targetBuffer, buffer, __MIN(bytesToCopy, availableBytes));
         TPCircularBufferConsume(&p_sys->circular_buffer, __MIN(bytesToCopy, availableBytes));
-        vlc_mutex_lock(&p_sys->lock);
-        p_sys->i_last_sample_time = inOutputTime->mSampleTime;
-        vlc_mutex_unlock(&p_sys->lock);
     }
 
+    vlc_cond_signal(&p_sys->cond);
+    vlc_mutex_unlock(&p_sys->lock);
+
     return noErr;
 }
 
-/*****************************************************************************
+#pragma mark -
+#pragma mark Stream / Hardware Listeners
+
+/*
  * HardwareListener: Warns us of changes in the list of registered devices
- *****************************************************************************/
+ */
 static OSStatus HardwareListener(AudioObjectID inObjectID,  UInt32 inNumberAddresses, const AudioObjectPropertyAddress inAddresses[], void*inClientData)
 {
     OSStatus err = noErr;
@@ -1460,13 +1398,14 @@ static OSStatus HardwareListener(AudioObjectID inObjectID,  UInt32 inNumberAddre
 #endif
 
     RebuildDeviceList(p_aout);
+    aout_RestartRequest(p_aout, AOUT_RESTART_OUTPUT);
 
     return err;
 }
 
-/*****************************************************************************
- * StreamListener
- *****************************************************************************/
+/*
+ * StreamListener: check whether the device's physical format changes on-the-fly (unlikely)
+ */
 static OSStatus StreamListener(AudioObjectID inObjectID,  UInt32 inNumberAddresses, const AudioObjectPropertyAddress inAddresses[], void*inClientData)
 {
     OSStatus err = noErr;
@@ -1485,92 +1424,184 @@ static OSStatus StreamListener(AudioObjectID inObjectID,  UInt32 inNumberAddress
     return err;
 }
 
-/*****************************************************************************
- * VolumeSet: Implements volume_set(). Update the CoreAudio AU volume immediately.
- *****************************************************************************/
-static int VolumeSet(audio_output_t * p_aout, float volume)
-{
-    struct aout_sys_t *p_sys = p_aout->sys;
-    OSStatus ostatus;
+#pragma mark -
+#pragma mark helpers
 
-    aout_VolumeReport(p_aout, volume);
+/*
+ * AudioDeviceHasOutput: Checks if the device is actually an output device
+ */
+static int AudioDeviceHasOutput(AudioDeviceID i_dev_id)
+{
+    UInt32 dataSize = 0;
+    OSStatus status;
 
-    /* Set volume for output unit */
-    ostatus = AudioUnitSetParameter(p_sys->au_unit,
-                                    kHALOutputParam_Volume,
-                                    kAudioUnitScope_Global,
-                                    0,
-                                    volume * volume * volume,
-                                    0);
+    AudioObjectPropertyAddress streamsAddress = { kAudioDevicePropertyStreams, kAudioDevicePropertyScopeOutput, kAudioObjectPropertyElementMaster };
+    status = AudioObjectGetPropertyDataSize(i_dev_id, &streamsAddress, 0, NULL, &dataSize);
 
-    if (var_InheritBool(p_aout, "volume-save"))
-        config_PutInt(p_aout, "auhal-volume", lroundf(volume * AOUT_VOLUME_DEFAULT));
+    if (dataSize == 0 || status != noErr)
+        return FALSE;
 
-    return ostatus;
+    return TRUE;
 }
 
-static int MuteSet(audio_output_t * p_aout, bool mute)
+/*
+ * AudioDeviceSupportsDigital: Checks if device supports raw bitstreams
+ */
+static int AudioDeviceSupportsDigital(audio_output_t *p_aout, AudioDeviceID i_dev_id)
 {
-    struct   aout_sys_t *p_sys = p_aout->sys;
-    OSStatus ostatus;
+    OSStatus                    err = noErr;
+    UInt32                      i_param_size = 0;
+    AudioStreamID               *p_streams = NULL;
+    int                         i_streams = 0;
+    bool                        b_return = false;
 
-    aout_MuteReport(p_aout, mute);
+    /* Retrieve all the output streams */
+    AudioObjectPropertyAddress streamsAddress = { kAudioDevicePropertyStreams, kAudioDevicePropertyScopeOutput, kAudioObjectPropertyElementMaster };
+    err = AudioObjectGetPropertyDataSize(i_dev_id, &streamsAddress, 0, NULL, &i_param_size);
+    if (err != noErr) {
+        msg_Err(p_aout, "could not get number of streams: [%s] (%i)", (char *)&err, (int32_t)err);
+        return false;
+    }
 
-    float volume = .0;
+    i_streams = i_param_size / sizeof(AudioStreamID);
+    p_streams = (AudioStreamID *)malloc(i_param_size);
+    if (p_streams == NULL)
+        return VLC_ENOMEM;
 
-    if (!mute)
-        volume = var_InheritInteger(p_aout, "auhal-volume") / (float)AOUT_VOLUME_DEFAULT;
+    err = AudioObjectGetPropertyData(i_dev_id, &streamsAddress, 0, NULL, &i_param_size, p_streams);
+    if (err != noErr) {
+        msg_Err(p_aout, "could not get list of streams: [%s]", (char *)&err);
+        return false;
+    }
 
-    ostatus = AudioUnitSetParameter(p_sys->au_unit,
-                                    kHALOutputParam_Volume,
-                                    kAudioUnitScope_Global,
-                                    0,
-                                    volume * volume * volume,
-                                    0);
+    for (int i = 0; i < i_streams; i++) {
+        if (AudioStreamSupportsDigital(p_aout, p_streams[i]))
+            b_return = true;
+    }
 
-    return ostatus;
+    free(p_streams);
+    return b_return;
 }
 
-static int Open(vlc_object_t *obj)
+/*
+ * AudioStreamSupportsDigital: Checks if audio stream is compatible with raw bitstreams
+ */
+static int AudioStreamSupportsDigital(audio_output_t *p_aout, AudioStreamID i_stream_id)
 {
-    audio_output_t *aout = (audio_output_t *)obj;
-    aout_sys_t *sys = malloc(sizeof (*sys));
+    OSStatus                    err = noErr;
+    UInt32                      i_param_size = 0;
+    AudioStreamRangedDescription *p_format_list = NULL;
+    int                         i_formats = 0;
+    bool                        b_return = false;
 
-    if (unlikely(sys == NULL))
-        return VLC_ENOMEM;
+    /* Retrieve all the stream formats supported by each output stream */
+    AudioObjectPropertyAddress physicalFormatsAddress = { kAudioStreamPropertyAvailablePhysicalFormats, kAudioObjectPropertyScopeGlobal, 0 };
+    err = AudioObjectGetPropertyDataSize(i_stream_id, &physicalFormatsAddress, 0, NULL, &i_param_size);
+    if (err != noErr) {
+        msg_Err(p_aout, "could not get number of streamformats: [%s] (%i)", (char *)&err, (int32_t)err);
+        return false;
+    }
 
-    vlc_mutex_init(&sys->lock);
+    i_formats = i_param_size / sizeof(AudioStreamRangedDescription);
+    msg_Dbg(p_aout, "found %i stream formats", i_formats);
 
-    aout->sys = sys;
-    aout->start = Start;
-    aout->stop = Stop;
-    aout->volume_set = VolumeSet;
-    aout->mute_set = MuteSet;
-    aout->device_enum = DeviceList;
-    aout->sys->devices = NULL;
-    aout->device_select = SwitchAudioDevice;
+    p_format_list = (AudioStreamRangedDescription *)malloc(i_param_size);
+    if (p_format_list == NULL)
+        return false;
 
-    RebuildDeviceList(aout);
+    err = AudioObjectGetPropertyData(i_stream_id, &physicalFormatsAddress, 0, NULL, &i_param_size, p_format_list);
+    if (err != noErr) {
+        msg_Err(p_aout, "could not get the list of streamformats: [%4.4s]", (char *)&err);
+        free(p_format_list);
+        p_format_list = NULL;
+        return false;
+    }
 
-    /* remember the volume */
-    aout_VolumeReport(aout, var_InheritInteger(aout, "auhal-volume") / (float)AOUT_VOLUME_DEFAULT);
-    MuteSet(aout, var_InheritBool(aout, "mute"));
+    for (int i = 0; i < i_formats; i++) {
+#ifndef NDEBUG
+        msg_Dbg(p_aout, STREAM_FORMAT_MSG("supported format: ", p_format_list[i].mFormat));
+#endif
 
-    return VLC_SUCCESS;
+        if (p_format_list[i].mFormat.mFormatID == 'IAC3' ||
+            p_format_list[i].mFormat.mFormatID == 'iac3' ||
+            p_format_list[i].mFormat.mFormatID == kAudioFormat60958AC3 ||
+            p_format_list[i].mFormat.mFormatID == kAudioFormatAC3)
+            b_return = true;
+    }
+
+    free(p_format_list);
+    return b_return;
 }
 
-static void Close(vlc_object_t *obj)
+/*
+ * AudioStreamChangeFormat: switch stream format based on the provided description
+ */
+static int AudioStreamChangeFormat(audio_output_t *p_aout, AudioStreamID i_stream_id, AudioStreamBasicDescription change_format)
 {
-    audio_output_t *aout = (audio_output_t *)obj;
-    aout_sys_t *sys = aout->sys;
+    OSStatus            err = noErr;
+    UInt32              i_param_size = 0;
 
-    for (struct audio_device_t * device = sys->devices, *next; device != NULL; device = next) {
-        next = device->next;
-        free(device->name);
-        free(device);
+    AudioObjectPropertyAddress physicalFormatAddress = { kAudioStreamPropertyPhysicalFormat, kAudioObjectPropertyScopeGlobal, kAudioObjectPropertyElementMaster };
+
+    struct { vlc_mutex_t lock; vlc_cond_t cond; } w;
+
+    msg_Dbg(p_aout, STREAM_FORMAT_MSG("setting stream format: ", change_format));
+
+    /* Condition because SetProperty is asynchronious */
+    vlc_cond_init(&w.cond);
+    vlc_mutex_init(&w.lock);
+    vlc_mutex_lock(&w.lock);
+
+    /* Install the callback */
+    err = AudioObjectAddPropertyListener(i_stream_id, &physicalFormatAddress, StreamListener, (void *)&w);
+    if (err != noErr) {
+        msg_Err(p_aout, "AudioObjectAddPropertyListener for kAudioStreamPropertyPhysicalFormat failed: [%4.4s]", (char *)&err);
+        return false;
     }
 
-    vlc_mutex_destroy(&sys->lock);
+    /* change the format */
+    err = AudioObjectSetPropertyData(i_stream_id, &physicalFormatAddress, 0, NULL, sizeof(AudioStreamBasicDescription),
+                                     &change_format);
+    if (err != noErr) {
+        msg_Err(p_aout, "could not set the stream format: [%4.4s]", (char *)&err);
+        return false;
+    }
 
-    free(sys);
+    /* The AudioStreamSetProperty is not only asynchronious (requiring the locks)
+     * it is also not atomic in its behaviour.
+     * Therefore we check 5 times before we really give up.
+     * FIXME: failing isn't actually implemented yet. */
+    for (int i = 0; i < 5; i++) {
+        AudioStreamBasicDescription actual_format;
+        mtime_t timeout = mdate() + 500000;
+
+        if (vlc_cond_timedwait(&w.cond, &w.lock, timeout))
+            msg_Dbg(p_aout, "reached timeout");
+
+        i_param_size = sizeof(AudioStreamBasicDescription);
+        err = AudioObjectGetPropertyData(i_stream_id, &physicalFormatAddress, 0, NULL, &i_param_size, &actual_format);
+
+        msg_Dbg(p_aout, STREAM_FORMAT_MSG("actual format in use: ", actual_format));
+        if (actual_format.mSampleRate == change_format.mSampleRate &&
+            actual_format.mFormatID == change_format.mFormatID &&
+            actual_format.mFramesPerPacket == change_format.mFramesPerPacket) {
+            /* The right format is now active */
+            break;
+        }
+        /* We need to check again */
+    }
+
+    /* Removing the property listener */
+    err = AudioObjectRemovePropertyListener(i_stream_id, &physicalFormatAddress, StreamListener, (void *)&w);
+    if (err != noErr) {
+        msg_Err(p_aout, "AudioStreamRemovePropertyListener failed: [%4.4s]", (char *)&err);
+        return false;
+    }
+
+    /* Destroy the lock and condition */
+    vlc_mutex_unlock(&w.lock);
+    vlc_mutex_destroy(&w.lock);
+    vlc_cond_destroy(&w.cond);
+
+    return true;
 }