[alsa-devel] [PATCH 1/1] ALSA: Tascam US-16x08 DSP mixer quirk

Detlef Urban onkel at paraair.de
Mon Jan 30 13:08:40 CET 2017


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 at 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



More information about the Alsa-devel mailing list