mfd: cros_ec_dev: Return number of bytes read with CROS_EC_DEV_IOCRDMEM
authorGuenter Roeck <linux@roeck-us.net>
Wed, 6 Feb 2019 17:39:59 +0000 (18:39 +0100)
committerLee Jones <lee.jones@linaro.org>
Thu, 7 Feb 2019 10:44:24 +0000 (10:44 +0000)
commitc1778e5819a9784b5551cf8f7abd66571c1239e2
tree195e122b9ecf24edc85eddd729df532449f83b9d
parent66265e719b4603ef9a1b8a6c876bcb542c021496
mfd: cros_ec_dev: Return number of bytes read with CROS_EC_DEV_IOCRDMEM

While ioctls normally return a negative error or 0 on success, this is not
the case for CROS_EC_DEV_IOCXCMD in the cros_ec driver, which returns the
number of bytes read instead. However, CROS_EC_DEV_IOCRDMEM in the same
driver does not return the number of byte read on success. This is both
inconsistent and problematic, since the amount of returned data can be
dynamic. Have it return the number of data bytes for consistency.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Signed-off-by: Enric Balletbo i Serra <enric.balletbo@collabora.com>
Reviewed-by: Benson Leung <bleung@chromium.org>
Signed-off-by: Lee Jones <lee.jones@linaro.org>
drivers/mfd/cros_ec_dev.c