Alsa-Devel Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] alsaucm: Report failure to set a device due to missing verb setup
@ 2010-10-08 21:11 Mark Brown
  2010-10-10 10:45 ` Liam Girdwood
  0 siblings, 1 reply; 2+ messages in thread
From: Mark Brown @ 2010-10-08 21:11 UTC (permalink / raw)
  To: Liam Girdwood; +Cc: alsa-devel, Mark Brown

Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
---
 alsaucm/usecase.c |    4 +++-
 1 files changed, 3 insertions(+), 1 deletions(-)

diff --git a/alsaucm/usecase.c b/alsaucm/usecase.c
index 2db26f1..d1420c0 100644
--- a/alsaucm/usecase.c
+++ b/alsaucm/usecase.c
@@ -94,8 +94,10 @@ static int list_verb_device_status(snd_use_case_mgr_t *uc_mgr)
 	int i, enabled, num;
 
 	verb = snd_use_case_get_verb(uc_mgr);
-	if (verb == NULL)
+	if (verb == NULL) {
+		printf(" no verb currently enabled.\n");
 		return -ENODEV;
+	}
 
 	num = snd_use_case_get_device_list(uc_mgr, verb, &device_list);
 	if (num <= 0) {
-- 
1.7.1

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

* Re: [PATCH] alsaucm: Report failure to set a device due to missing verb setup
  2010-10-08 21:11 [PATCH] alsaucm: Report failure to set a device due to missing verb setup Mark Brown
@ 2010-10-10 10:45 ` Liam Girdwood
  0 siblings, 0 replies; 2+ messages in thread
From: Liam Girdwood @ 2010-10-10 10:45 UTC (permalink / raw)
  To: Mark Brown, Jaroslav Kysela; +Cc: alsa-devel

On Fri, 2010-10-08 at 14:11 -0700, Mark Brown wrote:
> Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
> ---
>  alsaucm/usecase.c |    4 +++-
>  1 files changed, 3 insertions(+), 1 deletions(-)
> 
> diff --git a/alsaucm/usecase.c b/alsaucm/usecase.c
> index 2db26f1..d1420c0 100644
> --- a/alsaucm/usecase.c
> +++ b/alsaucm/usecase.c
> @@ -94,8 +94,10 @@ static int list_verb_device_status(snd_use_case_mgr_t *uc_mgr)
>  	int i, enabled, num;
>  
>  	verb = snd_use_case_get_verb(uc_mgr);
> -	if (verb == NULL)
> +	if (verb == NULL) {
> +		printf(" no verb currently enabled.\n");
>  		return -ENODEV;
> +	}
>  
>  	num = snd_use_case_get_device_list(uc_mgr, verb, &device_list);
>  	if (num <= 0) {

Both Applied.

Jaroslav, are you going to apply these to your ucm branch ?

Thanks

Liam
-- 
Freelance Developer, SlimLogic Ltd
ASoC and Voltage Regulator Maintainer.
http://www.slimlogic.co.uk

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

end of thread, other threads:[~2010-10-10 10:45 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-10-08 21:11 [PATCH] alsaucm: Report failure to set a device due to missing verb setup Mark Brown
2010-10-10 10:45 ` Liam Girdwood

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox