Hi Oliver,

> +     mutex_lock(&radio->lock);
> +     mutex_unlock(&radio->lock);

You are adding a lot of mutexes around si470x rds register accesses, which I 
don't think is necessary.
Parallel accesses to the registers will never harm anything or cannnot cause 
any data corruption.
I used radio->lock just for RDS buffer modifications, so multiple read/write 
attempts do not interfere.

But anyway, I applied anything else.
For testing it, I finally have to setup a kernel and environment, that support 
suspend and resume.
I try to give a feedback soon.

Bye,
  Toby
-
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to