Here my review on your reply.
Through a quick glance, the patch doesn't seem to follow the standard coding style. Please reformat all in a style defined in Documentation/process/coding-style.rst (or formerly Documentation/CodingStyle).
Fixed.
Also, you seem using some global variables. These are broken if you have multiple devices. That is, they have to be allocated dynamically.
Fixed.
Currently we have no private data field assigned to either usb audio card or mixer object, so we'll need a preliminary patch to add it beforehand.
I can't agree with that. In all cases the private data field I make use of is member of struct usb_mixer_elem_info. To me it seems to be a common approach to store device specific handling data. Please correct me if I'm wrong.
thanks,
Detlef
Am 26.01.2017 um 11:18 schrieb Takashi Iwai:
On Thu, 26 Jan 2017 09:37:23 +0100, Detlef Urban wrote:
Add mixer quirk for Tascam US-16x08 usb interface. Even that this an usb compliant device, the input channels and DSP functions (EQ/Compressor) arn't accessible by default.
Signed-off by: Detlef Urban onkel@paraair.de
Through a quick glance, the patch doesn't seem to follow the standard coding style. Please reformat all in a style defined in Documentation/process/coding-style.rst (or formerly Documentation/CodingStyle).
Also, you seem using some global variables. These are broken if you have multiple devices. That is, they have to be allocated dynamically.
Currently we have no private data field assigned to either usb audio card or mixer object, so we'll need a preliminary patch to add it beforehand.
thanks,
Takashi