Re: [PATCH 1/1] Added functionality for E-mu 0404USB/0202USB/TrackerPre

[Takashi Iwai <tiwai@suse.de>]

At Mon, 21 Dec 2009 12:39:27 -0800 (PST),
Zerg Cannibal wrote:
> 
> Added functionality:
> 1) Extension Units support (all XU settings now available at alsamixer, kmix, etc):
>  - "AnalogueIn soft limiter" switch;
>  - "Sample rate" selector (values 0,1,2,3,4,5 corresponds to 44.1 48 ... 192 kHz);
>  - "DigitalIn CLK source" selector (internal/external) (**);
>  - "DigitalOut format SPDIF/AC3" switch (**);
> (**)E-mu-0404usb only.
> 
> 2) Automatic device sample rate adjustment depending on substream samplerate for both capture and playback substream. 
>   
> Patch is based on Mac OS-X driver (http://sourceforge.net/projects/zaudiodrivermac/) and successfully tested by Ubuntu community users for 0404usb and 0202usb.
> 
> Signed-off-by: CannibalZerg <cnb_zerg@yahoo.com>

Thanks for the patch.  Through a quick look, it's fine.
But, could you fix the following and repost?

- Fix warnings suggested by scripts/checkpatch.pl.
  Some 80-chars warnings can be ignored if it makes worse, though.

- Please give the right name and address matching with your from
  address.

Also, the sample rate control could be an enum at best.  But, this
needs more another changes, so let's merge this patch first.


thanks,

Takashi