[alsa-devel] Patch - ESI Maya44 driver

Rainer Zimmermann mail at lightshed.de
Fri Mar 28 22:07:53 CET 2008


Hi Takashi,

sorry about this. here's the fixed patch, identical to the last except for the 
coding style fixes.

-Rainer


Takashi Iwai wrote:
> At Fri, 21 Mar 2008 12:27:49 +0100,
> Rainer Zimmermann wrote:
>> Hi Takashi,
>>
>> here's the maya44 patch.
>>
>> This is a patch against the current alsa-kernel tree.
> 
> Thanks.  This looks better than the previous version.
> 
>> I did add 2 definitions to ice1712.h (snd_ice1724_get_route_val and 
>> snd_ice1724_put_route_val), which allowed me to move some of the maya44 specific 
>> code to maya44.c .
> 
> We could do it in a bit cleaer way, but I think it's OK as is.
> 
>> There are still a few maya44 special cases in ice1724.c, because maya44 uses the 
>> "con" channel and routing switches differently. This is not really clean style I 
>> think, but I would suggest sorting this out later, as it would again require 
>> larger changes to ice1724.c and testing with other hardware.
> 
> Agreed.  This kind of clean up can be done later.
> 
>> Hope everything is ok with you, for now.
>>
>> This patch does *not* limit the sampling rate to 96kHz for capture, as discussed 
>> before.
> 
> The patch looks apparently not compliant to the kernel coding style.
> Could you fix at least the issues reported by checkpatch.pl (the
> latest one)?
> 
> 
> thanks,
> 
> Takashi
> _______________________________________________
> Alsa-devel mailing list
> Alsa-devel at alsa-project.org
> http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
> 


-- 
Lightshed IT Services
Löfflerstr. 27, 22765 Hamburg, Germany * mail at lightshed.de * www.lightshed.de
-------------- next part --------------
A non-text attachment was scrubbed...
Name: maya44_05.diff
Type: text/x-patch
Size: 55951 bytes
Desc: not available
Url : http://mailman.alsa-project.org/pipermail/alsa-devel/attachments/20080328/2c706d73/attachment-0001.diff 


More information about the Alsa-devel mailing list