Changeset 389ef25 in mainline for uspace/srv/audio/hound/audio_sink.c

Timestamp:

2012-07-17T06:53:55Z (12 years ago)

Author:

Jan Vesely <jano.vesely@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

950110ee

Parents:

bb67def

Message:

hound: Check format before starting playback.

File:

• uspace/srv/audio/hound/audio_sink.c (modified) (5 diffs)

uspace/srv/audio/hound/audio_sink.c

@@ -46 +46 @@
 int audio_sink_init(audio_sink_t *sink, const char *name,
     void *private_data, int (*connection_change)(audio_sink_t *, bool),
-    const audio_format_t *f)
+    int (*check_format)(audio_sink_t *sink), const audio_format_t *f)
 {
         assert(sink);
@@ -59 +59 @@
         sink->format = *f;
         sink->connection_change = connection_change;
+        sink->check_format = check_format;
         log_verbose("Initialized sink (%p) '%s'", sink, sink->name);
         return EOK;
@@ -70 +71 @@
         sink->name = NULL;
 }
-
 
 int audio_sink_add_source(audio_sink_t *sink, audio_source_t *source)
@@ -85 +85 @@
                 /* Set audio format according to the first source */
                 if (audio_format_is_any(&sink->format)) {
-                        /* Source does not care */
-                        if (audio_format_is_any(&source->format)) {
-                                log_verbose("Set default format for sink %s.",
-                                    sink->name);
-                                sink->format = AUDIO_FORMAT_DEFAULT;
-                        } else {
-                                log_verbose("Set format based on the first "
-                                    "source(%s): %u channel(s), %uHz, %s for "
-                                    "sink %s.", source->name,
-                                    source->format.channels,
-                                    source->format.sampling_rate,
-                                    pcm_sample_format_str(
-                                        source->format.sample_format),
-                                    sink->name);
-                                sink->format = source->format;
-                        }
+                        int ret = audio_sink_set_format(sink, &source->format);
+                        if (ret != EOK)
+                                return ret;
                 }
         }
@@ -120 +107 @@
             source->name, sink->name);
 
+        return EOK;
+}
+
+int audio_sink_set_format(audio_sink_t *sink, const audio_format_t *format)
+{
+        assert(sink);
+        assert(format);
+        if (!audio_format_is_any(&sink->format)) {
+                log_debug("Sink %s already has a format", sink->name);
+                return EEXISTS;
+        }
+        const audio_format_t old_format;
+
+        if (audio_format_is_any(format)) {
+                log_verbose("Setting DEFAULT format for sink %s", sink->name);
+                sink->format = AUDIO_FORMAT_DEFAULT;
+        } else {
+                sink->format = *format;
+        }
+        if (sink->check_format) {
+                const int ret = sink->check_format(sink);
+                if (ret != EOK && ret != ELIMIT) {
+                        log_debug("Format check failed on sink %s", sink->name);
+                        sink->format = old_format;
+                        return ret;
+                }
+        }
+        log_verbose("Set format for sink %s: %u channel(s), %uHz, %s",
+            sink->name, format->channels, format->sampling_rate,
+            pcm_sample_format_str(format->sample_format));
         return EOK;
 }