Re: playback-only code in snd-dice?
Brought to you by:
aeb,
bencollins
From: Stefan R. <st...@s5...> - 2014-10-22 20:33:12
|
On Oct 22 Daniel Robbins wrote: > I now have firewire audio working with 3.17.1. But I still have to > hack the kernel sources to get this to work... the remaining problem > with dice.c is the fact that it disables itself if it sees that the > device has record capabilities, assuming you will use FFADO. With a little bit more patching than that, you could even modify the driver to support capture: https://github.com/takaswie/snd-firewire-improve . (I haven't tested this myself yet.) > Well, guess what? I would just like my firewire audio to just work > in-kernel without modifying source code. I request that you remove this > artificial limitation of the driver. I too would be in favor of this limitation being lifted (unless Sakamoto-san's work goes into the mainline sooner, of course), but with a qualification; see below. > My Mytek DSD DAC is used for playback only, so just because a device > advertises record capability doesn't actually mean it is a record > device in practice. The module should load and function, and if the > user doesn't want to use it, they can blacklist it. My opinion of > course, but what good reason is there to have snd-dice *not* function > without hacking source code when it could by default? For users of DICE devices for recording, it would currently be a regression if the kernel prevented FFADO from accessing the device. There would be two alternative solutions: Either recording users would need to add a module blacklist rule on their systems. Or the playback-only driver would be extended by a module parameter that tells it to bind to bidi devices too, and that parameter would be off by default. With the second approach, users who only require playback through their bidi device would have to add a module parameter config on their systems, while users who want to record do not need to reconfigure anything. -- Stefan Richter -=====-====- =-=- =-==- http://arcgraph.de/sr/ |