* [PATCH] Add a quirk for Plantronics DA45
@ 2016-02-19 13:01 denk
2016-03-01 10:19 ` denk
0 siblings, 1 reply; 8+ messages in thread
From: denk @ 2016-02-19 13:01 UTC (permalink / raw)
To: alsa-devel
Plantronics DA45 does not support reading the sample rate which leads to many lines of "cannot get freq at ep 0x4" and "cannot get freq at ep 0x84". This patch adds the USB ID of the DA45 to quirks.c and avoids those error messages.
---
sound/usb/quirks.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/sound/usb/quirks.c b/sound/usb/quirks.c
index 2585c17..5b03296 100644
--- a/sound/usb/quirks.c
+++ b/sound/usb/quirks.c
@@ -1123,6 +1123,7 @@ bool snd_usb_get_sample_rate_quirk(struct snd_usb_audio *chip)
case USB_ID(0x045E, 0x076F): /* MS Lifecam HD-6000 */
case USB_ID(0x045E, 0x0772): /* MS Lifecam Studio */
case USB_ID(0x045E, 0x0779): /* MS Lifecam HD-3000 */
+ case USB_ID(0x047F, 0xAA05): /* Plantronics DA45 */
case USB_ID(0x04D8, 0xFEEA): /* Benchmark DAC1 Pre */
case USB_ID(0x074D, 0x3553): /* Outlaw RR2150 (Micronas UAC3553B) */
case USB_ID(0x21B4, 0x0081): /* AudioQuest DragonFly */
---
Signed-off-by: Dennis Kadioglu <denk@post.com>
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH] Add a quirk for Plantronics DA45
2016-02-19 13:01 [PATCH] Add a quirk for Plantronics DA45 denk
@ 2016-03-01 10:19 ` denk
2016-03-01 10:23 ` Takashi Iwai
0 siblings, 1 reply; 8+ messages in thread
From: denk @ 2016-03-01 10:19 UTC (permalink / raw)
To: alsa-devel
As I've got no response by now, I would like to politely ask, if the patch can be accepted now, after the requested modifications?
Thank you very much and best
denk
----- Ursprüngliche Mail -----
Von: "denk" <denk@post.com>
An: alsa-devel@alsa-project.org
Gesendet: Freitag, 19. Februar 2016 14:01:02
Betreff: [PATCH] Add a quirk for Plantronics DA45
Plantronics DA45 does not support reading the sample rate which leads to many lines of "cannot get freq at ep 0x4" and "cannot get freq at ep 0x84". This patch adds the USB ID of the DA45 to quirks.c and avoids those error messages.
---
sound/usb/quirks.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/sound/usb/quirks.c b/sound/usb/quirks.c
index 2585c17..5b03296 100644
--- a/sound/usb/quirks.c
+++ b/sound/usb/quirks.c
@@ -1123,6 +1123,7 @@ bool snd_usb_get_sample_rate_quirk(struct snd_usb_audio *chip)
case USB_ID(0x045E, 0x076F): /* MS Lifecam HD-6000 */
case USB_ID(0x045E, 0x0772): /* MS Lifecam Studio */
case USB_ID(0x045E, 0x0779): /* MS Lifecam HD-3000 */
+ case USB_ID(0x047F, 0xAA05): /* Plantronics DA45 */
case USB_ID(0x04D8, 0xFEEA): /* Benchmark DAC1 Pre */
case USB_ID(0x074D, 0x3553): /* Outlaw RR2150 (Micronas UAC3553B) */
case USB_ID(0x21B4, 0x0081): /* AudioQuest DragonFly */
---
Signed-off-by: Dennis Kadioglu <denk@post.com>
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH] Add a quirk for Plantronics DA45
2016-03-01 10:19 ` denk
@ 2016-03-01 10:23 ` Takashi Iwai
2016-03-01 11:37 ` denk
2016-03-01 11:54 ` David Henningsson
0 siblings, 2 replies; 8+ messages in thread
From: Takashi Iwai @ 2016-03-01 10:23 UTC (permalink / raw)
To: denk; +Cc: alsa-devel
On Tue, 01 Mar 2016 11:19:48 +0100,
denk wrote:
>
> As I've got no response by now, I would like to politely ask, if the patch can be accepted now, after the requested modifications?
I thought I replied you, suggesting to give a proper sign-off line.
Without your own sign-off, we can't merge any patch from you.
See Documentation/SubmittingPatches for details.
So, please resubmit with it.
thanks,
Takashi
>
> Thank you very much and best
> denk
>
> ----- Ursprüngliche Mail -----
> Von: "denk" <denk@post.com>
> An: alsa-devel@alsa-project.org
> Gesendet: Freitag, 19. Februar 2016 14:01:02
> Betreff: [PATCH] Add a quirk for Plantronics DA45
>
> Plantronics DA45 does not support reading the sample rate which leads to many lines of "cannot get freq at ep 0x4" and "cannot get freq at ep 0x84". This patch adds the USB ID of the DA45 to quirks.c and avoids those error messages.
>
> ---
> sound/usb/quirks.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/sound/usb/quirks.c b/sound/usb/quirks.c
> index 2585c17..5b03296 100644
> --- a/sound/usb/quirks.c
> +++ b/sound/usb/quirks.c
> @@ -1123,6 +1123,7 @@ bool snd_usb_get_sample_rate_quirk(struct snd_usb_audio *chip)
> case USB_ID(0x045E, 0x076F): /* MS Lifecam HD-6000 */
> case USB_ID(0x045E, 0x0772): /* MS Lifecam Studio */
> case USB_ID(0x045E, 0x0779): /* MS Lifecam HD-3000 */
> + case USB_ID(0x047F, 0xAA05): /* Plantronics DA45 */
> case USB_ID(0x04D8, 0xFEEA): /* Benchmark DAC1 Pre */
> case USB_ID(0x074D, 0x3553): /* Outlaw RR2150 (Micronas UAC3553B) */
> case USB_ID(0x21B4, 0x0081): /* AudioQuest DragonFly */
> ---
> Signed-off-by: Dennis Kadioglu <denk@post.com>
> _______________________________________________
> Alsa-devel mailing list
> Alsa-devel@alsa-project.org
> http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] Add a quirk for Plantronics DA45
2016-03-01 10:23 ` Takashi Iwai
@ 2016-03-01 11:37 ` denk
2016-03-01 11:54 ` David Henningsson
1 sibling, 0 replies; 8+ messages in thread
From: denk @ 2016-03-01 11:37 UTC (permalink / raw)
To: Takashi Iwai; +Cc: denk, alsa-devel
I'm sorry, maybe I've misunderstood something. As far as I understood https://www.kernel.org/doc/Documentation/SubmittingPatches, it should be sufficient to close the patch mail with Signed-off-by: Dennis Kadioglu <denk@post.com>, which I did in my second mail (see below). Is there anything else I've missed?
----- Ursprüngliche Mail -----
Von: "Takashi Iwai" <tiwai@suse.de>
An: "denk" <denk@post.com>
CC: alsa-devel@alsa-project.org
Gesendet: Dienstag, 1. März 2016 11:23:19
Betreff: Re: [alsa-devel] [PATCH] Add a quirk for Plantronics DA45
On Tue, 01 Mar 2016 11:19:48 +0100,
denk wrote:
>
> As I've got no response by now, I would like to politely ask, if the patch can be accepted now, after the requested modifications?
I thought I replied you, suggesting to give a proper sign-off line.
Without your own sign-off, we can't merge any patch from you.
See Documentation/SubmittingPatches for details.
So, please resubmit with it.
thanks,
Takashi
>
> Thank you very much and best
> denk
>
> ----- Ursprüngliche Mail -----
> Von: "denk" <denk@post.com>
> An: alsa-devel@alsa-project.org
> Gesendet: Freitag, 19. Februar 2016 14:01:02
> Betreff: [PATCH] Add a quirk for Plantronics DA45
>
> Plantronics DA45 does not support reading the sample rate which leads to many lines of "cannot get freq at ep 0x4" and "cannot get freq at ep 0x84". This patch adds the USB ID of the DA45 to quirks.c and avoids those error messages.
>
> ---
> sound/usb/quirks.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/sound/usb/quirks.c b/sound/usb/quirks.c
> index 2585c17..5b03296 100644
> --- a/sound/usb/quirks.c
> +++ b/sound/usb/quirks.c
> @@ -1123,6 +1123,7 @@ bool snd_usb_get_sample_rate_quirk(struct snd_usb_audio *chip)
> case USB_ID(0x045E, 0x076F): /* MS Lifecam HD-6000 */
> case USB_ID(0x045E, 0x0772): /* MS Lifecam Studio */
> case USB_ID(0x045E, 0x0779): /* MS Lifecam HD-3000 */
> + case USB_ID(0x047F, 0xAA05): /* Plantronics DA45 */
> case USB_ID(0x04D8, 0xFEEA): /* Benchmark DAC1 Pre */
> case USB_ID(0x074D, 0x3553): /* Outlaw RR2150 (Micronas UAC3553B) */
> case USB_ID(0x21B4, 0x0081): /* AudioQuest DragonFly */
> ---
> Signed-off-by: Dennis Kadioglu <denk@post.com>
> _______________________________________________
> Alsa-devel mailing list
> Alsa-devel@alsa-project.org
> http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] Add a quirk for Plantronics DA45
2016-03-01 10:23 ` Takashi Iwai
2016-03-01 11:37 ` denk
@ 2016-03-01 11:54 ` David Henningsson
2016-03-01 13:23 ` Dennis Kadioglu
1 sibling, 1 reply; 8+ messages in thread
From: David Henningsson @ 2016-03-01 11:54 UTC (permalink / raw)
To: Takashi Iwai, denk; +Cc: alsa-devel
Actually, there is a Signed-off-by line, but it's at the wrong place.
Also the Author/From line should state your full name.
Denk, please see below:
On 2016-03-01 11:23, Takashi Iwai wrote:
> On Tue, 01 Mar 2016 11:19:48 +0100,
> denk wrote:
>>
>> As I've got no response by now, I would like to politely ask, if the patch can be accepted now, after the requested modifications?
>
> I thought I replied you, suggesting to give a proper sign-off line.
> Without your own sign-off, we can't merge any patch from you.
> See Documentation/SubmittingPatches for details.
>
> So, please resubmit with it.
>
>
> thanks,
>
> Takashi
>
>
>>
>> Thank you very much and best
>> denk
>>
>> ----- Ursprüngliche Mail -----
>> Von: "denk" <denk@post.com>
>> An: alsa-devel@alsa-project.org
>> Gesendet: Freitag, 19. Februar 2016 14:01:02
>> Betreff: [PATCH] Add a quirk for Plantronics DA45
>>
>> Plantronics DA45 does not support reading the sample rate which leads to many lines of "cannot get freq at ep 0x4" and "cannot get freq at ep 0x84". This patch adds the USB ID of the DA45 to quirks.c and avoids those error messages.
Here is where sign-off-line should be: after the commit message but
before the "---" line.
>>
>> ---
>> sound/usb/quirks.c | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/sound/usb/quirks.c b/sound/usb/quirks.c
>> index 2585c17..5b03296 100644
>> --- a/sound/usb/quirks.c
>> +++ b/sound/usb/quirks.c
>> @@ -1123,6 +1123,7 @@ bool snd_usb_get_sample_rate_quirk(struct snd_usb_audio *chip)
>> case USB_ID(0x045E, 0x076F): /* MS Lifecam HD-6000 */
>> case USB_ID(0x045E, 0x0772): /* MS Lifecam Studio */
>> case USB_ID(0x045E, 0x0779): /* MS Lifecam HD-3000 */
>> + case USB_ID(0x047F, 0xAA05): /* Plantronics DA45 */
>> case USB_ID(0x04D8, 0xFEEA): /* Benchmark DAC1 Pre */
>> case USB_ID(0x074D, 0x3553): /* Outlaw RR2150 (Micronas UAC3553B) */
>> case USB_ID(0x21B4, 0x0081): /* AudioQuest DragonFly */
>> ---
>> Signed-off-by: Dennis Kadioglu <denk@post.com>
And here where it actually is; down here it won't be picked up correctly.
>> _______________________________________________
>> Alsa-devel mailing list
>> Alsa-devel@alsa-project.org
>> http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
> _______________________________________________
> Alsa-devel mailing list
> Alsa-devel@alsa-project.org
> http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
>
--
David Henningsson, Canonical Ltd.
https://launchpad.net/~diwic
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] Add a quirk for Plantronics DA45
2016-03-01 11:54 ` David Henningsson
@ 2016-03-01 13:23 ` Dennis Kadioglu
0 siblings, 0 replies; 8+ messages in thread
From: Dennis Kadioglu @ 2016-03-01 13:23 UTC (permalink / raw)
To: David Henningsson; +Cc: Takashi Iwai, alsa-devel
OK, thanks for the clarification and sorry for the mistake.
I will send it again including the mentioned fixes.
----- Ursprüngliche Mail -----
Von: "David Henningsson" <david.henningsson@canonical.com>
An: "Takashi Iwai" <tiwai@suse.de>, "denk" <denk@post.com>
CC: alsa-devel@alsa-project.org
Gesendet: Dienstag, 1. März 2016 12:54:39
Betreff: Re: [alsa-devel] [PATCH] Add a quirk for Plantronics DA45
Actually, there is a Signed-off-by line, but it's at the wrong place.
Also the Author/From line should state your full name.
Denk, please see below:
On 2016-03-01 11:23, Takashi Iwai wrote:
> On Tue, 01 Mar 2016 11:19:48 +0100,
> denk wrote:
>>
>> As I've got no response by now, I would like to politely ask, if the patch can be accepted now, after the requested modifications?
>
> I thought I replied you, suggesting to give a proper sign-off line.
> Without your own sign-off, we can't merge any patch from you.
> See Documentation/SubmittingPatches for details.
>
> So, please resubmit with it.
>
>
> thanks,
>
> Takashi
>
>
>>
>> Thank you very much and best
>> denk
>>
>> ----- Ursprüngliche Mail -----
>> Von: "denk" <denk@post.com>
>> An: alsa-devel@alsa-project.org
>> Gesendet: Freitag, 19. Februar 2016 14:01:02
>> Betreff: [PATCH] Add a quirk for Plantronics DA45
>>
>> Plantronics DA45 does not support reading the sample rate which leads to many lines of "cannot get freq at ep 0x4" and "cannot get freq at ep 0x84". This patch adds the USB ID of the DA45 to quirks.c and avoids those error messages.
Here is where sign-off-line should be: after the commit message but
before the "---" line.
>>
>> ---
>> sound/usb/quirks.c | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/sound/usb/quirks.c b/sound/usb/quirks.c
>> index 2585c17..5b03296 100644
>> --- a/sound/usb/quirks.c
>> +++ b/sound/usb/quirks.c
>> @@ -1123,6 +1123,7 @@ bool snd_usb_get_sample_rate_quirk(struct snd_usb_audio *chip)
>> case USB_ID(0x045E, 0x076F): /* MS Lifecam HD-6000 */
>> case USB_ID(0x045E, 0x0772): /* MS Lifecam Studio */
>> case USB_ID(0x045E, 0x0779): /* MS Lifecam HD-3000 */
>> + case USB_ID(0x047F, 0xAA05): /* Plantronics DA45 */
>> case USB_ID(0x04D8, 0xFEEA): /* Benchmark DAC1 Pre */
>> case USB_ID(0x074D, 0x3553): /* Outlaw RR2150 (Micronas UAC3553B) */
>> case USB_ID(0x21B4, 0x0081): /* AudioQuest DragonFly */
>> ---
>> Signed-off-by: Dennis Kadioglu <denk@post.com>
And here where it actually is; down here it won't be picked up correctly.
>> _______________________________________________
>> Alsa-devel mailing list
>> Alsa-devel@alsa-project.org
>> http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
> _______________________________________________
> Alsa-devel mailing list
> Alsa-devel@alsa-project.org
> http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
>
--
David Henningsson, Canonical Ltd.
https://launchpad.net/~diwic
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH] Add a quirk for Plantronics DA45
@ 2016-03-01 13:23 Dennis Kadioglu
2016-03-01 13:56 ` Takashi Iwai
0 siblings, 1 reply; 8+ messages in thread
From: Dennis Kadioglu @ 2016-03-01 13:23 UTC (permalink / raw)
To: alsa-devel
Plantronics DA45 does not support reading the sample rate which leads to many lines of "cannot get freq at ep 0x4" and "cannot get freq at ep 0x84". This patch adds the USB ID of the DA45 to quirks.c and avoids those error messages.
Signed-off-by: Dennis Kadioglu <denk@post.com>
---
sound/usb/quirks.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/sound/usb/quirks.c b/sound/usb/quirks.c
index 2585c17..5b03296 100644
--- a/sound/usb/quirks.c
+++ b/sound/usb/quirks.c
@@ -1123,6 +1123,7 @@ bool snd_usb_get_sample_rate_quirk(struct snd_usb_audio *chip)
case USB_ID(0x045E, 0x076F): /* MS Lifecam HD-6000 */
case USB_ID(0x045E, 0x0772): /* MS Lifecam Studio */
case USB_ID(0x045E, 0x0779): /* MS Lifecam HD-3000 */
+ case USB_ID(0x047F, 0xAA05): /* Plantronics DA45 */
case USB_ID(0x04D8, 0xFEEA): /* Benchmark DAC1 Pre */
case USB_ID(0x074D, 0x3553): /* Outlaw RR2150 (Micronas UAC3553B) */
case USB_ID(0x21B4, 0x0081): /* AudioQuest DragonFly */
---
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH] Add a quirk for Plantronics DA45
2016-03-01 13:23 Dennis Kadioglu
@ 2016-03-01 13:56 ` Takashi Iwai
0 siblings, 0 replies; 8+ messages in thread
From: Takashi Iwai @ 2016-03-01 13:56 UTC (permalink / raw)
To: Dennis Kadioglu; +Cc: alsa-devel
On Tue, 01 Mar 2016 14:23:29 +0100,
Dennis Kadioglu wrote:
>
> Plantronics DA45 does not support reading the sample rate which leads to many lines of "cannot get freq at ep 0x4" and "cannot get freq at ep 0x84". This patch adds the USB ID of the DA45 to quirks.c and avoids those error messages.
>
>
> Signed-off-by: Dennis Kadioglu <denk@post.com>
Applied, thanks.
Takashi
> ---
> sound/usb/quirks.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/sound/usb/quirks.c b/sound/usb/quirks.c
> index 2585c17..5b03296 100644
> --- a/sound/usb/quirks.c
> +++ b/sound/usb/quirks.c
> @@ -1123,6 +1123,7 @@ bool snd_usb_get_sample_rate_quirk(struct snd_usb_audio *chip)
> case USB_ID(0x045E, 0x076F): /* MS Lifecam HD-6000 */
> case USB_ID(0x045E, 0x0772): /* MS Lifecam Studio */
> case USB_ID(0x045E, 0x0779): /* MS Lifecam HD-3000 */
> + case USB_ID(0x047F, 0xAA05): /* Plantronics DA45 */
> case USB_ID(0x04D8, 0xFEEA): /* Benchmark DAC1 Pre */
> case USB_ID(0x074D, 0x3553): /* Outlaw RR2150 (Micronas UAC3553B) */
> case USB_ID(0x21B4, 0x0081): /* AudioQuest DragonFly */
> ---
> _______________________________________________
> Alsa-devel mailing list
> Alsa-devel@alsa-project.org
> http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
>
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2016-03-01 13:56 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-02-19 13:01 [PATCH] Add a quirk for Plantronics DA45 denk
2016-03-01 10:19 ` denk
2016-03-01 10:23 ` Takashi Iwai
2016-03-01 11:37 ` denk
2016-03-01 11:54 ` David Henningsson
2016-03-01 13:23 ` Dennis Kadioglu
-- strict thread matches above, loose matches on Subject: below --
2016-03-01 13:23 Dennis Kadioglu
2016-03-01 13:56 ` Takashi Iwai
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.