LCID Fire wrote:
Attached are 3 patches which make the error messages in usbaudio.c more
useful and also some code optimizations.
Attached are the patches done via format patch.
>From cd0180399253b17ece4fc1177457b21b8b70fe8f Mon Sep 17 00:00:00 2001
From: Andreas Bergmeier <lcid-fire@xxxxxxx>
Date: Sat, 24 Jan 2009 12:22:13 +0100
Subject: [PATCH] Add device and control number so it's apparent for which control this error is showing up.
Signed-off-by: Andreas Bergmeier <lcid-fire@xxxxxxx>
---
sound/usb/usbaudio.c | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/sound/usb/usbaudio.c b/sound/usb/usbaudio.c
index 037435d..e578a5c 100644
--- a/sound/usb/usbaudio.c
+++ b/sound/usb/usbaudio.c
@@ -2867,12 +2867,12 @@ static int snd_usb_create_streams(struct snd_usb_audio *chip, int ctrlif)
/* find audiocontrol interface */
host_iface = &usb_ifnum_to_if(dev, ctrlif)->altsetting[0];
if (!(p1 = snd_usb_find_csint_desc(host_iface->extra, host_iface->extralen, NULL, HEADER))) {
- snd_printk(KERN_ERR "cannot find HEADER\n");
+ snd_printk(KERN_ERR "%d:%u: cannot find HEADER\n", dev->devnum, ctrlif);
return -EINVAL;
}
if (! p1[7] || p1[0] < 8 + p1[7]) {
- snd_printk(KERN_ERR "invalid HEADER\n");
+ snd_printk(KERN_ERR "%d:%u: invalid HEADER\n", dev->devnum, ctrlif);
return -EINVAL;
}
--
1.5.6.3
>From 191cf816a5a618a84cc3627c5f9f96355f5d22c4 Mon Sep 17 00:00:00 2001
From: Andreas Bergmeier <lcid-fire@xxxxxxx>
Date: Sat, 24 Jan 2009 12:17:01 +0100
Subject: [PATCH] Print device number which did not support low speed audio streaming
Signed-off-by: Andreas Bergmeier <lcid-fire@xxxxxxx>
---
sound/usb/usbaudio.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/sound/usb/usbaudio.c b/sound/usb/usbaudio.c
index 8b8eb9e..2cfb8a3 100644
--- a/sound/usb/usbaudio.c
+++ b/sound/usb/usbaudio.c
@@ -2929,7 +2929,7 @@ static int snd_usb_create_streams(struct snd_usb_audio *chip, int ctrlif)
continue;
}
if (snd_usb_get_speed(dev) == USB_SPEED_LOW) {
- snd_printk(KERN_ERR "low speed audio streaming not supported\n");
+ snd_printk(KERN_ERR "%d: low speed audio streaming not supported\n", dev->devnum);
continue;
}
if (! parse_audio_endpoints(chip, j)) {
--
1.5.6.3
>From c6d62cbaf6d36b252768ef109b8d85a036e1d451 Mon Sep 17 00:00:00 2001
From: Andreas Bergmeier <lcid-fire@xxxxxxx>
Date: Sat, 24 Jan 2009 12:16:16 +0100
Subject: [PATCH] Replaced 'return NULL' lines by break in snd_usb_find_desc so there is only one default error.
Signed-off-by: Andreas Bergmeier <lcid-fire@xxxxxxx>
---
sound/usb/usbaudio.c | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/sound/usb/usbaudio.c b/sound/usb/usbaudio.c
index d35b51c..8b8eb9e 100644
--- a/sound/usb/usbaudio.c
+++ b/sound/usb/usbaudio.c
@@ -2054,10 +2054,10 @@ void *snd_usb_find_desc(void *descstart, int desclen, void *after, u8 dtype)
end = p + desclen;
for (; p < end;) {
if (p[0] < 2)
- return NULL;
+ break;
next = p + p[0];
if (next > end)
- return NULL;
+ break;
if (p[1] == dtype && (!after || (void *)p > after)) {
return p;
}
--
1.5.6.3
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
