Re: [PATCH 09/10] mfd: cros_ec: Check result code from EC messages

2014-06-18 Thread Lee Jones
On Tue, 17 Jun 2014, Doug Anderson wrote: > Simon, > > On Tue, Jun 17, 2014 at 8:43 PM, Simon Glass wrote: > >> diff --git a/drivers/mfd/cros_ec_spi.c b/drivers/mfd/cros_ec_spi.c > >> index 09ca789..4d34f1c 100644 > >> --- a/drivers/mfd/cros_ec_spi.c > >> +++ b/drivers/mfd/cros_ec_spi.c > >> @@

Re: [PATCH 09/10] mfd: cros_ec: Check result code from EC messages

2014-06-17 Thread Doug Anderson
Simon, On Tue, Jun 17, 2014 at 8:43 PM, Simon Glass wrote: >> diff --git a/drivers/mfd/cros_ec_spi.c b/drivers/mfd/cros_ec_spi.c >> index 09ca789..4d34f1c 100644 >> --- a/drivers/mfd/cros_ec_spi.c >> +++ b/drivers/mfd/cros_ec_spi.c >> @@ -289,21 +289,23 @@ static int cros_ec_cmd_xfer_spi(struct c

Re: [PATCH 09/10] mfd: cros_ec: Check result code from EC messages

2014-06-17 Thread Simon Glass
Hi Doug, On 16 June 2014 14:39, Doug Anderson wrote: > From: Bill Richardson > > Just because the host was able to talk to the EC doesn't mean that the EC > was happy with what it was told. Errors in communincation are not the same > as error messages from the EC itself. > > This change lets the

[PATCH 09/10] mfd: cros_ec: Check result code from EC messages

2014-06-16 Thread Doug Anderson
From: Bill Richardson Just because the host was able to talk to the EC doesn't mean that the EC was happy with what it was told. Errors in communincation are not the same as error messages from the EC itself. This change lets the EC report its errors separately. Signed-off-by: Bill Richardson