Comment 14 for bug 348353

Revision history for this message
Mike Rans (rans) wrote :

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.