A bug in intel8x0.c

["freeman.liu" <freeman.liu@sun.com>]

Hi, guy,
   
    I'm sure I found a bug in intel8x0.c.
   
    At the end of the function snd_intel8x0_codec_semaphore. If we 
couldn't get the semaphore in
1 ms, we'll reset the register and return EBUSY. But to reset the 
register, we should do a
    iaputword ( chip, 0, 0 ), according to the datasheet, write any 
value the this register will reset the
register.
    But in the code, what we did is iagetword ( chip, 0 ). This bug 
hadn't be exclosed because this code
path had never been reached, for the semaphore error often means a 
hardware error, which is very
rare.
    So, is there anybody who can modify this ?


    Freeman




-------------------------------------------------------
The SF.Net email is sponsored by EclipseCon 2004
Premiere Conference on Open Tools Development and Integration
See the breadth of Eclipse activity. February 3-5 in Anaheim, CA.
http://www.eclipsecon.org/osdn