mfd: cros-ec: copy the whole event in get_next_event_xfer
authorEmil Karlson <jekarlson@gmail.com>
Wed, 3 Oct 2018 18:43:18 +0000 (21:43 +0300)
committerBenson Leung <bleung@chromium.org>
Wed, 10 Oct 2018 03:57:30 +0000 (20:57 -0700)
commitd4d2313a3c17eff4aef9a544023c2df5b9f5bedc
tree18e21d847b5f4c1de71897149fe94b5b7f2cde5c
parent0238df646e6224016a45505d2c111a24669ebe21
mfd: cros-ec: copy the whole event in get_next_event_xfer

Commit 57e94c8b974db2d83c60e1139c89a70806abbea0 caused cros-ec keyboard events
be truncated on many chromebooks so that Left and Right keys on Column 12 were
always 0. Use ret as memcpy len to fix this.

The old code was using ec_dev->event_size, which is the event payload/data size
excluding event_type header, for the length of the memcpy operation. Use ret
as memcpy length to avoid the off by one and copy the whole msg->data.

Fixes: 57e94c8b974d ("mfd: cros-ec: Increase maximum mkbp event size")

Acked-by: Enric Balletbo i Serra <enric.balletbo@collabora.com>
Tested-by: Emil Renner Berthing <kernel@esmil.dk>
Signed-off-by: Emil Karlson <jekarlson@gmail.com>
Signed-off-by: Benson Leung <bleung@chromium.org>
drivers/platform/chrome/cros_ec_proto.c