Subject: Re: [PATCH resend] Renesas Ethernet AVB driver

From: Lino Sanfilippo <[email-masked]>
Date: Sat, 28 Mar 2015 06:17:13 +0100

Hi,

On 28.03.2015 00:13, Sergei Shtylyov wrote:

> +	spinlock_t lock;		/* Register access lock */

Do you really need one huge lock for all register accesses? Maybe you
should rethink which registers can actually be accessed concurrently and
implement some finer grained locking.

Regards,
Lino