Re: [Isisalsa-devel] Driver source code and irc support

[Pieter P. <ppa...@es...>]

Jean-Sébastien Valette wrote:

>	I'm a french coder who were  working in a same project like you.
>I'm looking at the code and it seem to me that the cvs contain lot's of stuff that isn't part of the useful code.
>I'm thinking useful code is only in the mai dir (the one containing isis.c) am i right ?
>
>By looking at the code I see that there is 2 spin lock for the dream chip (with and whitout irq).
>are they still both used and if so  are they locking the same thing or different part ?
>  
>
The driver, in it's current state, is of rather poor quality. It is 
therefore possible that there are some spinlocks that are defined but 
not used, or other issues. I don't even think that this driver would 
work on multiprocessor machines because the spinlocks aren't used 
correctly. To summarize: I don't know... the driver works acceptably 
good for the moment. I intend to do a complete rewrite of the driver.

The current driver is the result of a lot of experimenting with the ISIS 
to get it working (hence the tons of commented code). On top of that, 
it's my first expierence with ALSA driver writing, and (linux) driver 
developement in general. No wonder it's messy.

>I've also created  an irc channel on irc.freenode.net named isisalsa.
>Please join it as it is a good way to talk about driver code ans to do support for end user.
>
>  
>
Nice idea.

Pieter

>-------
>jeanseb
>I know it look like I'm insane
>Take a closer look I'm not to blame.
>If love is blind I guess I'll buy myself a cane
>	(slash / axl rose )
>
>
>-------------------------------------------------------
>This SF.Net email sponsored by: Free pre-built ASP.NET sites including
>Data Reports, E-commerce, Portals, and Forums are available now.
>Download today and enter to win an XBOX or Visual Studio .NET.
>http://aspnet.click-url.com/go/psa00100003ave/direct;at.aspnet_072303_01/01
>_______________________________________________
>Isisalsa-devel mailing list
>Isi...@li...
>https://lists.sourceforge.net/lists/listinfo/isisalsa-devel
>
>  
>