At Thu, 30 Oct 2014 08:53:04 -0200, Mauro Carvalho Chehab wrote: > > From: Mauro Carvalho Chehab <m.chehab@xxxxxxxxxxx> > > Add a macro to simplify au0828 quirk table. That makes easier > to check it against the USB IDs at drivers/media/usb/au0828-card.c > > Cc: stable@xxxxxxxxxxxxxxx > Signed-off-by: Mauro Carvalho Chehab <m.chehab@xxxxxxxxxxx> > Signed-off-by: Mauro Carvalho Chehab <mchehab@xxxxxxxxxxxxxxx> Which sign-off should I take? Judging from the author line, the former one? The second patch had only s-o-b from @samsung.com. thanks, Takashi > > diff --git a/sound/usb/quirks-table.h b/sound/usb/quirks-table.h > index c657752a420c..5ae1d02d17a3 100644 > --- a/sound/usb/quirks-table.h > +++ b/sound/usb/quirks-table.h > @@ -2804,133 +2804,37 @@ YAMAHA_DEVICE(0x7010, "UB99"), > } > }, > > -/* Hauppauge HVR-950Q and HVR-850 */ > -{ > - USB_DEVICE_VENDOR_SPEC(0x2040, 0x7200), > - .match_flags = USB_DEVICE_ID_MATCH_DEVICE | > - USB_DEVICE_ID_MATCH_INT_CLASS | > - USB_DEVICE_ID_MATCH_INT_SUBCLASS, > - .bInterfaceClass = USB_CLASS_AUDIO, > - .bInterfaceSubClass = USB_SUBCLASS_AUDIOCONTROL, > - .driver_info = (unsigned long) &(const struct snd_usb_audio_quirk) { > - .vendor_name = "Hauppauge", > - .product_name = "HVR-950Q", > - .ifnum = QUIRK_ANY_INTERFACE, > - .type = QUIRK_AUDIO_ALIGN_TRANSFER, > - } > -}, > -{ > - USB_DEVICE_VENDOR_SPEC(0x2040, 0x7210), > - .match_flags = USB_DEVICE_ID_MATCH_DEVICE | > - USB_DEVICE_ID_MATCH_INT_CLASS | > - USB_DEVICE_ID_MATCH_INT_SUBCLASS, > - .bInterfaceClass = USB_CLASS_AUDIO, > - .bInterfaceSubClass = USB_SUBCLASS_AUDIOCONTROL, > - .driver_info = (unsigned long) &(const struct snd_usb_audio_quirk) { > - .vendor_name = "Hauppauge", > - .product_name = "HVR-950Q", > - .ifnum = QUIRK_ANY_INTERFACE, > - .type = QUIRK_AUDIO_ALIGN_TRANSFER, > - } > -}, > -{ > - USB_DEVICE_VENDOR_SPEC(0x2040, 0x7217), > - .match_flags = USB_DEVICE_ID_MATCH_DEVICE | > - USB_DEVICE_ID_MATCH_INT_CLASS | > - USB_DEVICE_ID_MATCH_INT_SUBCLASS, > - .bInterfaceClass = USB_CLASS_AUDIO, > - .bInterfaceSubClass = USB_SUBCLASS_AUDIOCONTROL, > - .driver_info = (unsigned long) &(const struct snd_usb_audio_quirk) { > - .vendor_name = "Hauppauge", > - .product_name = "HVR-950Q", > - .ifnum = QUIRK_ANY_INTERFACE, > - .type = QUIRK_AUDIO_ALIGN_TRANSFER, > - } > -}, > -{ > - USB_DEVICE_VENDOR_SPEC(0x2040, 0x721b), > - .match_flags = USB_DEVICE_ID_MATCH_DEVICE | > - USB_DEVICE_ID_MATCH_INT_CLASS | > - USB_DEVICE_ID_MATCH_INT_SUBCLASS, > - .bInterfaceClass = USB_CLASS_AUDIO, > - .bInterfaceSubClass = USB_SUBCLASS_AUDIOCONTROL, > - .driver_info = (unsigned long) &(const struct snd_usb_audio_quirk) { > - .vendor_name = "Hauppauge", > - .product_name = "HVR-950Q", > - .ifnum = QUIRK_ANY_INTERFACE, > - .type = QUIRK_AUDIO_ALIGN_TRANSFER, > - } > -}, > -{ > - USB_DEVICE_VENDOR_SPEC(0x2040, 0x721e), > - .match_flags = USB_DEVICE_ID_MATCH_DEVICE | > - USB_DEVICE_ID_MATCH_INT_CLASS | > - USB_DEVICE_ID_MATCH_INT_SUBCLASS, > - .bInterfaceClass = USB_CLASS_AUDIO, > - .bInterfaceSubClass = USB_SUBCLASS_AUDIOCONTROL, > - .driver_info = (unsigned long) &(const struct snd_usb_audio_quirk) { > - .vendor_name = "Hauppauge", > - .product_name = "HVR-950Q", > - .ifnum = QUIRK_ANY_INTERFACE, > - .type = QUIRK_AUDIO_ALIGN_TRANSFER, > - } > -}, > -{ > - USB_DEVICE_VENDOR_SPEC(0x2040, 0x721f), > - .match_flags = USB_DEVICE_ID_MATCH_DEVICE | > - USB_DEVICE_ID_MATCH_INT_CLASS | > - USB_DEVICE_ID_MATCH_INT_SUBCLASS, > - .bInterfaceClass = USB_CLASS_AUDIO, > - .bInterfaceSubClass = USB_SUBCLASS_AUDIOCONTROL, > - .driver_info = (unsigned long) &(const struct snd_usb_audio_quirk) { > - .vendor_name = "Hauppauge", > - .product_name = "HVR-950Q", > - .ifnum = QUIRK_ANY_INTERFACE, > - .type = QUIRK_AUDIO_ALIGN_TRANSFER, > - } > -}, > -{ > - USB_DEVICE_VENDOR_SPEC(0x2040, 0x7240), > - .match_flags = USB_DEVICE_ID_MATCH_DEVICE | > - USB_DEVICE_ID_MATCH_INT_CLASS | > - USB_DEVICE_ID_MATCH_INT_SUBCLASS, > - .bInterfaceClass = USB_CLASS_AUDIO, > - .bInterfaceSubClass = USB_SUBCLASS_AUDIOCONTROL, > - .driver_info = (unsigned long) &(const struct snd_usb_audio_quirk) { > - .vendor_name = "Hauppauge", > - .product_name = "HVR-850", > - .ifnum = QUIRK_ANY_INTERFACE, > - .type = QUIRK_AUDIO_ALIGN_TRANSFER, > - } > -}, > -{ > - USB_DEVICE_VENDOR_SPEC(0x2040, 0x7280), > - .match_flags = USB_DEVICE_ID_MATCH_DEVICE | > - USB_DEVICE_ID_MATCH_INT_CLASS | > - USB_DEVICE_ID_MATCH_INT_SUBCLASS, > - .bInterfaceClass = USB_CLASS_AUDIO, > - .bInterfaceSubClass = USB_SUBCLASS_AUDIOCONTROL, > - .driver_info = (unsigned long) &(const struct snd_usb_audio_quirk) { > - .vendor_name = "Hauppauge", > - .product_name = "HVR-950Q", > - .ifnum = QUIRK_ANY_INTERFACE, > - .type = QUIRK_AUDIO_ALIGN_TRANSFER, > - } > -}, > -{ > - USB_DEVICE_VENDOR_SPEC(0x0fd9, 0x0008), > - .match_flags = USB_DEVICE_ID_MATCH_DEVICE | > - USB_DEVICE_ID_MATCH_INT_CLASS | > - USB_DEVICE_ID_MATCH_INT_SUBCLASS, > - .bInterfaceClass = USB_CLASS_AUDIO, > - .bInterfaceSubClass = USB_SUBCLASS_AUDIOCONTROL, > - .driver_info = (unsigned long) &(const struct snd_usb_audio_quirk) { > - .vendor_name = "Hauppauge", > - .product_name = "HVR-950Q", > - .ifnum = QUIRK_ANY_INTERFACE, > - .type = QUIRK_AUDIO_ALIGN_TRANSFER, > - } > -}, > +/* > + * Auvitek au0828 devices with audio interface. > + * This should be kept in sync with drivers/media/usb/au0828-card.c > + * Please notice that some drivers are DVB only, and don't need to be > + * here. That's the case, for example, of DVICO_FUSIONHDTV7. > + */ > + > +#define AU0828_DEVICE(vid, pid, vname, pname) { \ > + USB_DEVICE_VENDOR_SPEC(vid, pid), \ > + .match_flags = USB_DEVICE_ID_MATCH_DEVICE | \ > + USB_DEVICE_ID_MATCH_INT_CLASS | \ > + USB_DEVICE_ID_MATCH_INT_SUBCLASS, \ > + .bInterfaceClass = USB_CLASS_AUDIO, \ > + .bInterfaceSubClass = USB_SUBCLASS_AUDIOCONTROL, \ > + .driver_info = (unsigned long) &(const struct snd_usb_audio_quirk) { \ > + .vendor_name = vname, \ > + .product_name = pname, \ > + .ifnum = QUIRK_ANY_INTERFACE, \ > + .type = QUIRK_AUDIO_ALIGN_TRANSFER, \ > + } \ > +} > + > +AU0828_DEVICE(0x2040, 0x7200, "Hauppauge", "HVR-950Q"), > +AU0828_DEVICE(0x2040, 0x7210, "Hauppauge", "HVR-950Q"), > +AU0828_DEVICE(0x2040, 0x7217, "Hauppauge", "HVR-950Q"), > +AU0828_DEVICE(0x2040, 0x721b, "Hauppauge", "HVR-950Q"), > +AU0828_DEVICE(0x2040, 0x721e, "Hauppauge", "HVR-950Q"), > +AU0828_DEVICE(0x2040, 0x721f, "Hauppauge", "HVR-950Q"), > +AU0828_DEVICE(0x2040, 0x7240, "Hauppauge", "HVR-850"), > +AU0828_DEVICE(0x2040, 0x7280, "Hauppauge", "HVR-950Q"), > +AU0828_DEVICE(0x0fd9, 0x0008, "Hauppauge", "HVR-950Q"), > > /* Digidesign Mbox */ > { > -- > 1.9.3 > -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html