Thanks for making the PPA. It should make it a lot easier for users.

I just had a quick look at your patch and I think there's a couple of problems.
     pa_sink_set_max_request(u->sink, u->hwbuf_size);
     pa_sink_set_max_rewind(u->sink, u->hwbuf_size); <--- this one should be 
deleted
 
+    if (strcmp(u->device_name, "a52") == 0) {
+        pa_sink_set_max_rewind(u->sink, 0);
+    } else {
+        pa_sink_set_max_rewind(u->sink, u->hwbuf_size);
+    }

 [Mapping iec958-ac3-surround-40]
There's second profile which from memory I think is:  iec958-ac3-surround-51. 
This also needs a52 instead of a52:%f

Lastly, I suppose it would be possible to patch /etc/pulse/daemon.conf
in this PPA so that it has default-sample-rate = 48000. Then the user
doesn't need their own ~/.pulse/daemon.conf

This would mean they only need to edit ~/.asoundrc unless you can come
up with a better way.

Now that the Pulseaudio guys have a patch for SPDIF passthrough,
hopefully someone will look at putting ac3 encoding in Pulseaudio.

-- 
Pulseaudio fails with Alsa a52 (ac3) plugin (Ubuntu 10.04 Lucid)
https://bugs.launchpad.net/bugs/348353
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to