On Apr 22, 2007, at 17:39:59, Matthias Kaehlcke wrote:
quoted text
> use spinlock instead of binary mutex in idt77252 driver

I think you really meant: "Use mutex instead of binary semaphore in  
idt77252 driver", since this is a binary semaphore (not a mutex,  
which are always binary):
quoted text
> -	struct semaphore	mutex;

and this is a mutex, not a spinlock:
quoted text
> +	struct mutex		mutex;

Everything else looks good though

Cheers,
Kyle Moffett
-
unsubscribe notice
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/