After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 346326 - alsasrc doesn't support many high end sound cards due to channel and caps restrictions
alsasrc doesn't support many high end sound cards due to channel and caps res...
Status: RESOLVED DUPLICATE of bug 345188
Product: GStreamer
Classification: Platform
Component: gst-plugins-base
git master
Other Linux
: Normal normal
: NONE
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2006-06-30 23:58 UTC by Michael Sheldon
Modified: 2006-07-02 11:13 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Patch to gstalsa.c and gstalsasrc.c (1.02 KB, patch)
2006-07-01 00:01 UTC, Michael Sheldon
committed Details | Review

Description Michael Sheldon 2006-06-30 23:58:52 UTC
As far as I can tell the problems arise in the following functions (testing with a Delta 44 sound card):

gst_alsa_detect_formats:
 Returns EMPTY Caps. This is due to the template caps only supporting widths of 8 or 16. The Delta 44 requires 32 bit.

gst_alsa_detect_channels:
 GST_ALSA_MAX_CHANNELS is limited to 8. The minimum (and maximum) required by the Delta 44 is 12 channels. There's also no check to see if the minimum is greater than GST_ALSA_MAX_CHANNELS when setting the maximum to GST_ALSA_MAX_CHANNELS.
Comment 1 Michael Sheldon 2006-07-01 00:01:22 UTC
Created attachment 68230 [details] [review]
Patch to gstalsa.c and gstalsasrc.c

This is a rough patch which fixes the problem for me. In the long run I guess it'd be best to extend the range of channel_positions (and so raise the GST_ALSA_MAX_CHANNELS limit).
Comment 2 Michael Sheldon 2006-07-01 01:08:00 UTC
Although, I'm not sure limiting input by the number of channel positions necessarily makes sense, since they're often irrelevant. E.g. In the case of the Delta 44 they are simply 12 mono inputs (4 physical inputs handled in different ways).
Comment 3 Tim-Philipp Müller 2006-07-02 11:13:46 UTC
Thanks for the patch! I've committed the alsasrc template caps bit with some modifications:

 - 32 bps should be the first set of caps, as the caps should be sorted
   by preference
 - if you bump the channels from [1,2] to [1,MAX] you should do that for
   16bps and 8bps as well

The rest is the same as bug #345188. I think that should be done a bit differently so that there is channel layout info in the caps (and if it is only 10x MONO) ..


*** This bug has been marked as a duplicate of 345188 ***