Opened 14 years ago

Closed 13 years ago

Last modified 13 years ago

#1598 closed patch (fixed)

AC3 passthru broken with some alsa drivers (eg. snd_usb_audio)

Reported by: Martin Ebourne <martin@…> Owned by: danielk
Priority: minor Milestone: 0.20
Component: mythtv Version: head
Severity: medium Keywords:
Cc: Ticket locked: no

Description

Passthru has been broken on some drivers since r8941. This automatically adds ':{ AES0 0x02 }' on to the end of the device name, but not all drivers support that. eg. snd_usb_audio returns an error code in this case.

Patch enclosed to try it both ways, been working for me for a few weeks.

Attachments (2)

alsa-params.patch (1.3 KB) - added by Martin Ebourne <martin@…> 14 years ago.
Patch against r9546
audiopassthru.patch (24.9 KB) - added by Martin Ebourne <martin@…> 14 years ago.
Generic solution - allows user to specify separate passthru device

Download all attachments as: .zip

Change History (9)

Changed 14 years ago by Martin Ebourne <martin@…>

Attachment: alsa-params.patch added

Patch against r9546

comment:1 Changed 14 years ago by Isaac Richards

Owner: changed from Isaac Richards to danielk

comment:2 Changed 14 years ago by martin@…

Daniel, if you're looking at this I guess you're also aware of #1608 which is a dup with a different but also not fully successful patch. See also this thread:

http://www.gossamer-threads.com/lists/mythtv/dev/194225

I've started working on the generic solution: the input field patch suggested by Markus in the second email, but it touches a lot of places and I haven't had chance to finish & test it yet.

comment:3 Changed 14 years ago by danielk

Milestone: 0.20
Resolution: invalid
Status: newclosed

I'm closing both AC3 tickets (#1598,#1608) until there is a workable solution.

Changed 14 years ago by Martin Ebourne <martin@…>

Attachment: audiopassthru.patch added

Generic solution - allows user to specify separate passthru device

comment:4 Changed 14 years ago by Martin Ebourne <martin@…>

Resolution: invalid
Status: closedreopened

Added patch for the generic solution. Provides edit field for separate passthru device, defaults to the same as the audio device. Only AudioOutputALSA pays any attention to the separate device at the moment, though others will be easy to add if required. Not been able to test other platforms (eg. mac) but should be safe.

comment:5 Changed 13 years ago by danielk

Resolution: fixed
Status: reopenedclosed

(In [10201]) Closes #1598. Adds AC3/DTS passthough device setting.

This is needed because some hardware doesn't detect the output type or uses different audio connectors for different types of audio. By default the old behaviour of sending all output to the same device is used.

This also makes the configure autodetection of ARTS support find the arts headers in Gentoo's new file location regime.

This changes the myth binary revision so a distclean and plugin recompile is required.

comment:6 Changed 13 years ago by danielk

(In [10202]) Refs #1598. Ran commit in wrong place for [10201], this commit included plugin and mfd changes.

comment:7 Changed 13 years ago by danielk

(In [10219]) Refs #1598. Fixes compile error on OSX.

Note: See TracTickets for help on using tickets.