Re: [PATCH 2/7] mfd: cros_ec: spi: Add mutex to cros_ec_spi

2014-04-18 Thread Simon Glass
Hi Doug, On 18 April 2014 15:15, Doug Anderson wrote: > > Simon, > > On Fri, Apr 18, 2014 at 10:28 AM, Simon Glass wrote: > > Hi Doug, > > > > On 17 April 2014 11:59, Doug Anderson wrote: > >> The main transfer function for cros_ec_spi can be called by more than > >> one client at a time. Make

Re: [PATCH 2/7] mfd: cros_ec: spi: Add mutex to cros_ec_spi

2014-04-18 Thread Doug Anderson
Simon, On Fri, Apr 18, 2014 at 10:28 AM, Simon Glass wrote: > Hi Doug, > > On 17 April 2014 11:59, Doug Anderson wrote: >> The main transfer function for cros_ec_spi can be called by more than >> one client at a time. Make sure that those clients don't stomp on >> each other by locking the bus

Re: [PATCH 2/7] mfd: cros_ec: spi: Add mutex to cros_ec_spi

2014-04-18 Thread Simon Glass
Hi Doug, On 17 April 2014 11:59, Doug Anderson wrote: > The main transfer function for cros_ec_spi can be called by more than > one client at a time. Make sure that those clients don't stomp on > each other by locking the bus for the duration of the transfer > function. Is there no lock at the

[PATCH 2/7] mfd: cros_ec: spi: Add mutex to cros_ec_spi

2014-04-17 Thread Doug Anderson
The main transfer function for cros_ec_spi can be called by more than one client at a time. Make sure that those clients don't stomp on each other by locking the bus for the duration of the transfer function. Signed-off-by: Doug Anderson --- drivers/mfd/cros_ec_spi.c | 26 +-