Re: [PATCH v2 8/8] platform/chrome: cros_ec: sysfs: Modify error handling

2017-09-11 Thread Benson Leung
Hi Thierry, On Fri, Aug 11, 2017 at 04:23:05PM +0200, Thierry Escande wrote: > From: Gwendal Grignou > > When accessing a sysfs attribute, if the EC command fails, -EPROTO is > now returned instead of an error message as it is unlikely an app is > parsing the error message

Re: [PATCH v2 8/8] platform/chrome: cros_ec: sysfs: Modify error handling

2017-09-11 Thread Benson Leung
Hi Thierry, On Fri, Aug 11, 2017 at 04:23:05PM +0200, Thierry Escande wrote: > From: Gwendal Grignou > > When accessing a sysfs attribute, if the EC command fails, -EPROTO is > now returned instead of an error message as it is unlikely an app is > parsing the error message to do something

[PATCH v2 8/8] platform/chrome: cros_ec: sysfs: Modify error handling

2017-08-11 Thread Thierry Escande
From: Gwendal Grignou When accessing a sysfs attribute, if the EC command fails, -EPROTO is now returned instead of an error message as it is unlikely an app is parsing the error message to do something meaningful. Also, this patch makes use of cros_ec_cmd_xfer_status()

[PATCH v2 8/8] platform/chrome: cros_ec: sysfs: Modify error handling

2017-08-11 Thread Thierry Escande
From: Gwendal Grignou When accessing a sysfs attribute, if the EC command fails, -EPROTO is now returned instead of an error message as it is unlikely an app is parsing the error message to do something meaningful. Also, this patch makes use of cros_ec_cmd_xfer_status() instead of