Yair K. wrote:
> On Sunday 03 August 2008 15:02:37 Yair K. wrote:
>   
>> Hi,
>>
>>    The attached patch (all by Mario Goebbels, I only did trivial 
>> modifications to clean it a bit) adds support for the Asus Xonar DX cards to 
>> the oss_cmi878x driver.
>>     
>
> Oh, there should be a MUTEX_EXIT in the OSS_EIO case in two_wire_write which 
> isn't in the patch:
>
> --- oss_cmi878x.c     2008-08-03 20:37:31.439245476 +0300
> +++ oss_cmi878x.c     2008-08-03 20:34:56.417242953 +0300
> @@ -403,6 +403,7 @@
>    if(count == 0)
>    {
>       cmn_err(CE_WARN, "Time out on Two-Wire interface (busy).");
> +     MUTEX_EXIT(devc->dac_mutex, flags);
>       return OSS_EIO;
>    }
>    MUTEX_EXIT(devc->dac_mutex, flags);
>   
Ok. In the hg server now.

Best regards,

Hannu
_______________________________________________
oss-devel mailing list
oss-devel@mailman.opensound.com
http://mailman.opensound.com/mailman/listinfo/oss-devel

Reply via email to