All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2] USB-Audio: Blacklist iec958 for some USB devices
@ 2012-03-09 10:46 David Henningsson
  2012-03-09 11:37 ` Takashi Iwai
  0 siblings, 1 reply; 5+ messages in thread
From: David Henningsson @ 2012-03-09 10:46 UTC (permalink / raw)
  To: alsa-devel
  Cc: Andy Whitcroft, tiwai, clemens, Raymond Yau, 940145,
	David Henningsson

Some common USB devices are analog only, so prevent the digital
devices from being opened. This simplifies probing the device
in PulseAudio.

I have a "Logitech USB Headset" myself, the "Logitech Web Camera"
was reported by Raymond Yau, and the "Blue Snowball" was reported by
Andy Whitcroft.

Cc: Andy Whitcroft <andy.whitcroft@canonical.com>
Cc: Raymond Yau <superquad.vortex2@gmail.com>
BugLink: https://bugs.launchpad.net/bugs/940145
Signed-off-by: David Henningsson <david.henningsson@canonical.com>
---
 src/conf/cards/USB-Audio.conf |    5 +++++
 1 files changed, 5 insertions(+), 0 deletions(-)

diff --git a/src/conf/cards/USB-Audio.conf b/src/conf/cards/USB-Audio.conf
index 0726c34..d7c121e 100644
--- a/src/conf/cards/USB-Audio.conf
+++ b/src/conf/cards/USB-Audio.conf
@@ -36,6 +36,11 @@ USB-Audio.pcm.surround40_type {
 # number for the iec958 device can be changed here.
 USB-Audio.pcm.iec958_device {
 	# "NoiseBlaster 3000" 42
+
+	# The below don't have digital in/out, so prevent them from being opened.
+	"Logitech USB Headset" 999
+	"Logitech Web Camera" 999
+	"Blue Snowball" 999
 }
 
 
-- 
1.7.9

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [PATCH v2] USB-Audio: Blacklist iec958 for some USB devices
  2012-03-09 10:46 [PATCH v2] USB-Audio: Blacklist iec958 for some USB devices David Henningsson
@ 2012-03-09 11:37 ` Takashi Iwai
  2012-03-09 11:52   ` Raymond Yau
  0 siblings, 1 reply; 5+ messages in thread
From: Takashi Iwai @ 2012-03-09 11:37 UTC (permalink / raw)
  To: David Henningsson
  Cc: Raymond Yau, 940145, alsa-devel, clemens, Andy Whitcroft

At Fri,  9 Mar 2012 11:46:06 +0100,
David Henningsson wrote:
> 
> Some common USB devices are analog only, so prevent the digital
> devices from being opened. This simplifies probing the device
> in PulseAudio.
> 
> I have a "Logitech USB Headset" myself, the "Logitech Web Camera"
> was reported by Raymond Yau, and the "Blue Snowball" was reported by
> Andy Whitcroft.
> 
> Cc: Andy Whitcroft <andy.whitcroft@canonical.com>
> Cc: Raymond Yau <superquad.vortex2@gmail.com>
> BugLink: https://bugs.launchpad.net/bugs/940145
> Signed-off-by: David Henningsson <david.henningsson@canonical.com>

Thanks, applied now.


Takashi

> ---
>  src/conf/cards/USB-Audio.conf |    5 +++++
>  1 files changed, 5 insertions(+), 0 deletions(-)
> 
> diff --git a/src/conf/cards/USB-Audio.conf b/src/conf/cards/USB-Audio.conf
> index 0726c34..d7c121e 100644
> --- a/src/conf/cards/USB-Audio.conf
> +++ b/src/conf/cards/USB-Audio.conf
> @@ -36,6 +36,11 @@ USB-Audio.pcm.surround40_type {
>  # number for the iec958 device can be changed here.
>  USB-Audio.pcm.iec958_device {
>  	# "NoiseBlaster 3000" 42
> +
> +	# The below don't have digital in/out, so prevent them from being opened.
> +	"Logitech USB Headset" 999
> +	"Logitech Web Camera" 999
> +	"Blue Snowball" 999
>  }
>  
>  
> -- 
> 1.7.9
> 

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH v2] USB-Audio: Blacklist iec958 for some USB devices
  2012-03-09 11:37 ` Takashi Iwai
@ 2012-03-09 11:52   ` Raymond Yau
  2012-03-09 12:48     ` David Henningsson
  0 siblings, 1 reply; 5+ messages in thread
From: Raymond Yau @ 2012-03-09 11:52 UTC (permalink / raw)
  To: Takashi Iwai; +Cc: ALSA Development Mailing List

2012/3/9, Takashi Iwai <tiwai@suse.de>:
> At Fri,  9 Mar 2012 11:46:06 +0100,
> David Henningsson wrote:
>>
>> Some common USB devices are analog only, so prevent the digital
>> devices from being opened. This simplifies probing the device
>> in PulseAudio.
>>
>> I have a "Logitech USB Headset" myself, the "Logitech Web Camera"
>> was reported by Raymond Yau, and the "Blue Snowball" was reported by
>> Andy Whitcroft.
>>
>> Cc: Andy Whitcroft <andy.whitcroft@canonical.com>
>> Cc: Raymond Yau <superquad.vortex2@gmail.com>
>> BugLink: https://bugs.launchpad.net/bugs/940145
>> Signed-off-by: David Henningsson <david.henningsson@canonical.com>
>
> Thanks, applied now.
>
>
> Takashi
>
>> ---
>>  src/conf/cards/USB-Audio.conf |    5 +++++
>>  1 files changed, 5 insertions(+), 0 deletions(-)
>>
>> diff --git a/src/conf/cards/USB-Audio.conf b/src/conf/cards/USB-Audio.conf
>> index 0726c34..d7c121e 100644
>> --- a/src/conf/cards/USB-Audio.conf
>> +++ b/src/conf/cards/USB-Audio.conf
>> @@ -36,6 +36,11 @@ USB-Audio.pcm.surround40_type {
>>  # number for the iec958 device can be changed here.
>>  USB-Audio.pcm.iec958_device {
>>  	# "NoiseBlaster 3000" 42
>> +
>> +	# The below don't have digital in/out, so prevent them from being
>> opened.
>> +	"Logitech USB Headset" 999
>> +	"Logitech Web Camera" 999
>> +	"Blue Snowball" 999
>>  }
>>

How do the blacklist work ? since there is no "Logitect Web Camera" in
lsusb info

My  QuickCam Communicate Deluxe only support mono, so it is unlikely
for pulseaudio to open for digital capture

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH v2] USB-Audio: Blacklist iec958 for some USB devices
  2012-03-09 11:52   ` Raymond Yau
@ 2012-03-09 12:48     ` David Henningsson
  2012-03-09 12:58       ` Raymond Yau
  0 siblings, 1 reply; 5+ messages in thread
From: David Henningsson @ 2012-03-09 12:48 UTC (permalink / raw)
  To: Raymond Yau; +Cc: ALSA Development Mailing List

On 03/09/2012 12:52 PM, Raymond Yau wrote:
> 2012/3/9, Takashi Iwai<tiwai@suse.de>:
>> At Fri,  9 Mar 2012 11:46:06 +0100,
>> David Henningsson wrote:
>>>
>>> Some common USB devices are analog only, so prevent the digital
>>> devices from being opened. This simplifies probing the device
>>> in PulseAudio.
>>>
>>> I have a "Logitech USB Headset" myself, the "Logitech Web Camera"
>>> was reported by Raymond Yau, and the "Blue Snowball" was reported by
>>> Andy Whitcroft.
>>>
>>> Cc: Andy Whitcroft<andy.whitcroft@canonical.com>
>>> Cc: Raymond Yau<superquad.vortex2@gmail.com>
>>> BugLink: https://bugs.launchpad.net/bugs/940145
>>> Signed-off-by: David Henningsson<david.henningsson@canonical.com>
>>
>> Thanks, applied now.
>>
>>
>> Takashi
>>
>>> ---
>>>   src/conf/cards/USB-Audio.conf |    5 +++++
>>>   1 files changed, 5 insertions(+), 0 deletions(-)
>>>
>>> diff --git a/src/conf/cards/USB-Audio.conf b/src/conf/cards/USB-Audio.conf
>>> index 0726c34..d7c121e 100644
>>> --- a/src/conf/cards/USB-Audio.conf
>>> +++ b/src/conf/cards/USB-Audio.conf
>>> @@ -36,6 +36,11 @@ USB-Audio.pcm.surround40_type {
>>>   # number for the iec958 device can be changed here.
>>>   USB-Audio.pcm.iec958_device {
>>>   	# "NoiseBlaster 3000" 42
>>> +
>>> +	# The below don't have digital in/out, so prevent them from being
>>> opened.
>>> +	"Logitech USB Headset" 999
>>> +	"Logitech Web Camera" 999
>>> +	"Blue Snowball" 999
>>>   }
>>>
>
> How do the blacklist work ? since there is no "Logitect Web Camera" in
> lsusb info
>
> My  QuickCam Communicate Deluxe only support mono, so it is unlikely
> for pulseaudio to open for digital capture

Sorry, maybe I misunderstood you then - could you give me the contents 
of "/proc/asound/cards" please?



-- 
David Henningsson, Canonical Ltd.
http://launchpad.net/~diwic

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH v2] USB-Audio: Blacklist iec958 for some USB devices
  2012-03-09 12:48     ` David Henningsson
@ 2012-03-09 12:58       ` Raymond Yau
  0 siblings, 0 replies; 5+ messages in thread
From: Raymond Yau @ 2012-03-09 12:58 UTC (permalink / raw)
  To: David Henningsson; +Cc: ALSA Development Mailing List

2012/3/9, David Henningsson <david.henningsson@canonical.com>:
> On 03/09/2012 12:52 PM, Raymond Yau wrote:
>> 2012/3/9, Takashi Iwai<tiwai@suse.de>:
>>> At Fri,  9 Mar 2012 11:46:06 +0100,
>>> David Henningsson wrote:
>>>>
>>>> Some common USB devices are analog only, so prevent the digital
>>>> devices from being opened. This simplifies probing the device
>>>> in PulseAudio.
>>>>
>>>> I have a "Logitech USB Headset" myself, the "Logitech Web Camera"
>>>> was reported by Raymond Yau, and the "Blue Snowball" was reported by
>>>> Andy Whitcroft.
>>>>
>>>> Cc: Andy Whitcroft<andy.whitcroft@canonical.com>
>>>> Cc: Raymond Yau<superquad.vortex2@gmail.com>
>>>> BugLink: https://bugs.launchpad.net/bugs/940145
>>>> Signed-off-by: David Henningsson<david.henningsson@canonical.com>
>>>
>>> Thanks, applied now.
>>>
>>>
>>> Takashi
>>>
>>>> ---
>>>>   src/conf/cards/USB-Audio.conf |    5 +++++
>>>>   1 files changed, 5 insertions(+), 0 deletions(-)
>>>>
>>>> diff --git a/src/conf/cards/USB-Audio.conf
>>>> b/src/conf/cards/USB-Audio.conf
>>>> index 0726c34..d7c121e 100644
>>>> --- a/src/conf/cards/USB-Audio.conf
>>>> +++ b/src/conf/cards/USB-Audio.conf
>>>> @@ -36,6 +36,11 @@ USB-Audio.pcm.surround40_type {
>>>>   # number for the iec958 device can be changed here.
>>>>   USB-Audio.pcm.iec958_device {
>>>>   	# "NoiseBlaster 3000" 42
>>>> +
>>>> +	# The below don't have digital in/out, so prevent them from being
>>>> opened.
>>>> +	"Logitech USB Headset" 999
>>>> +	"Logitech Web Camera" 999
>>>> +	"Blue Snowball" 999
>>>>   }
>>>>
>>
>> How do the blacklist work ? since there is no "Logitect Web Camera" in
>> lsusb info
>>
>> My  QuickCam Communicate Deluxe only support mono, so it is unlikely
>> for pulseaudio to open for digital capture
>
> Sorry, maybe I misunderstood you then - could you give me the contents
> of "/proc/asound/cards" please?
>
>
>

It  only support 16000Hz mono and S16_LE analog capture

 2 [U0x46d0x992    ]: USB-Audio - USB Device 0x46d:0x992
                      USB Device 0x46d:0x992 at usb-0000:00:1d.7-3, high speed

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2012-03-09 12:58 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-03-09 10:46 [PATCH v2] USB-Audio: Blacklist iec958 for some USB devices David Henningsson
2012-03-09 11:37 ` Takashi Iwai
2012-03-09 11:52   ` Raymond Yau
2012-03-09 12:48     ` David Henningsson
2012-03-09 12:58       ` Raymond Yau

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.