Re: [PATCH v2 3/3] cros_ec: Don't signal wake event for non-wake host events

[Benson Leung <bleung@google.com>]

[-- Attachment #1: Type: text/plain, Size: 1113 bytes --]

Hi Thierry,

On Tue, Feb 14, 2017 at 08:58:02PM +0100, Thierry Escande wrote:
> From: Shawn Nematbakhsh <shawnn@chromium.org>
> 
> The subset of wake-enabled host events is defined by the EC, but the EC
> may still send non-wake host events if we're in the process of
> suspending. Get the mask of wake-enabled host events from the EC and
> filter out non-wake events to prevent spurious aborted suspend
> attempts.
> 
> Signed-off-by: Shawn Nematbakhsh <shawnn@chromium.org>
> Signed-off-by: Thierry Escande <thierry.escande@collabora.com>
> ---
>  drivers/mfd/cros_ec.c                   | 13 ++++--
>  drivers/platform/chrome/cros_ec_proto.c | 76 ++++++++++++++++++++++++++++++---
>  include/linux/mfd/cros_ec.h             |  5 ++-
>  3 files changed, 85 insertions(+), 9 deletions(-)
> 

Another use of cros_ec_get_next_event was introduced in cros_ec_lpc.c since
this patch was posted, so I went ahead and modified that too.

Applied. Thanks.


-- 
Benson Leung
Staff Software Engineer
Chrome OS Kernel
Google Inc.
bleung@google.com
Chromium OS Project
bleung@chromium.org

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]