Re: [PATCH] Add a quirk for Plantronics DA45

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxx>
An: alsa-devel@xxxxxxxxxxxxxxxx
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@xxxxxxxx>

And here where it actually is; down here it won't be picked up correctly.

_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel


--
David Henningsson, Canonical Ltd.
https://launchpad.net/~diwic
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel




[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux