Opened 20 years ago

Closed 19 years ago

Last modified 19 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@…> 20 years ago.
Patch against r9546
audiopassthru.patch (24.9 KB ) - added by Martin Ebourne <martin@…> 19 years ago.
Generic solution - allows user to specify separate passthru device

Download all attachments as: .zip

Change History (9)

by Martin Ebourne <martin@…>, 20 years ago

Attachment: alsa-params.patch added

Patch against r9546

comment:1 by Isaac Richards, 19 years ago

Owner: changed from Isaac Richards to danielk

comment:2 by martin@…, 19 years ago

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 by danielk, 19 years ago

Milestone: 0.20
Resolution: invalid
Status: newclosed

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

by Martin Ebourne <martin@…>, 19 years ago

Attachment: audiopassthru.patch added

Generic solution - allows user to specify separate passthru device

comment:4 by Martin Ebourne <martin@…>, 19 years ago

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 by danielk, 19 years ago

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 by danielk, 19 years ago

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

comment:7 by danielk, 19 years ago

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

Note: See TracTickets for help on using tickets.