On Wed, 09 Mar 2016 14:15:03 +0100, Takashi Sakamoto wrote: > > Hi, > > On mar 7 2016 22:35, Takashi Sakamoto wrote: > > Hi, > > > > This patchset updates my previous RFT, and goes for merging. > > [RFT][PATCH 0/4] ALSA: dice: enabled to handle several streams > > http://mailman.alsa-project.org/pipermail/alsa-devel/2016-March/105387.html > > > > For the previous one, Stefan Richter worked as a tester with his devices > > (Focusrite Saffire Pro 24 and 40), while current patchset includes changes > > related to userspace interface. Therefore, I don't add Tested-by. > > > > > > In previous one, ALSA dice driver uses subdevice for userspace to access > > any PCM substreams. On the other hand, the design of PCM subdevice leaves > > a bit cumbersome for userspace to handle several subdevices simultaneously > > in one process. These PCM substreams on a batch of AMDTP streams and it's > > better for ALSA dice driver to enable one process to handle them with > > simpler way. > > > > Based on this concept, this patchset adds PCM character devices, instead of > > PCM subdevice. Thus, userspace applications can utilize PCM substreams via > > individual PCM character devices. In configuration space of alsa-lib, via > > 'hw:0,0' and 'hw:0,1' if available, like: > > > > $ arecord -l > > ... > > card 1: Pro26000f5a [Pro26-000f5a], device 0: DICE [Pro26-000f5a] > > Subdevices: 1/1 > > Subdevice #0: subdevice #0 > > card 1: Pro26000f5a [Pro26-000f5a], device 1: DICE [Pro26-000f5a] > > Subdevices: 1/1 > > Subdevice #0: subdevice #0 > > > > > > For the other parts, there's no code changes. Therefore, Stefan's result > > still has meanings. > > > > Changes: > > * Add additional PCM character device instead of PCM subdevice > > * Correct string entries to force-pcm > > * Improve comments > > > > Takashi Sakamoto (4): > > ALSA: dice: have two sets of isochronous resources/streams > > ALSA: dice: handle whole available isochronous streams > > ALSA: dice: handle several PCM substreams when any isochronous streams > > are available > > ALSA: dice: force to add two pcm devices for listed models > > > > sound/firewire/dice/dice-midi.c | 8 +- > > sound/firewire/dice/dice-pcm.c | 147 +++++++++----- > > sound/firewire/dice/dice-stream.c | 398 ++++++++++++++++++++++++-------------- > > sound/firewire/dice/dice.c | 41 ++++ > > sound/firewire/dice/dice.h | 33 +++- > > 5 files changed, 424 insertions(+), 203 deletions(-) > > I strongly request to merge this patchset to linux-next for 4.6 kernel. > For already supported models, this patchset basically adds no changes > except for actual codes. Therefore few possibilities of regressions. I expected some tested-by tags or such, but OK, it's fine to merge without that. There are minor nitpicks, but they can be improved later. Now applied all patches. thanks, Takashi _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
