[alsa-devel] [PATCH] ALSA: wss_lib: rework snd_ad1848_probe()

Rene Herman rene.herman at keyaccess.nl
Wed Aug 6 08:56:40 CEST 2008

On 06-08-08 08:42, Rene Herman wrote:

> This reworks snd_ad1848_probe(), making for IMO significantly more 
> straightforward reading.
> The original had a somewhat odd try-loop where it not only waited for 
> the codec to come out of init but also retried a failed basic detect 
> which seems to not make sense. Except for that, the detection logic 
> should be unchanged.
> I am by the way rather unsure what it is that the spin_lock is trying to 
> protect us from here. What's the error scenario if we just not take it? 
> We aren't live yet...
> Next step is folding this into snd_wss_probe() directly.

Oh, but now that I decided to do so not immediately, this first one 
should remeber to switch back to MODE 1, if only for cleanliness.

New version. Never mind the somewhat ugly goto. Disappears when folded 


-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: 0001-ALSA-wss_lib-rework-snd_ad1848_probe.patch
Url: http://mailman.alsa-project.org/pipermail/alsa-devel/attachments/20080806/9e5a220a/attachment.bat 

More information about the Alsa-devel mailing list