Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753115AbdGFHMm (ORCPT ); Thu, 6 Jul 2017 03:12:42 -0400 Received: from mail-wr0-f174.google.com ([209.85.128.174]:33530 "EHLO mail-wr0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752939AbdGFHMk (ORCPT ); Thu, 6 Jul 2017 03:12:40 -0400 Date: Thu, 6 Jul 2017 08:12:37 +0100 From: Lee Jones To: Benson Leung Cc: Thierry Escande , Benson Leung , linux-kernel@vger.kernel.org, enric.balletbo@collabora.com Subject: Re: [PATCH v2 3/3] cros_ec: Don't signal wake event for non-wake host events Message-ID: <20170706071237.wtsveat75iqak3t3@dell> References: <1487102282-12259-1-git-send-email-thierry.escande@collabora.com> <1487102282-12259-4-git-send-email-thierry.escande@collabora.com> <20170627162353.GC39605@decatoncale.mtv.corp.google.com> <20170703114641.2zivoeqiv7tk4bqu@dell> <20170705201104.GC77970@decatoncale.mtv.corp.google.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20170705201104.GC77970@decatoncale.mtv.corp.google.com> User-Agent: NeoMutt/20170113 (1.7.2) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2036 Lines: 49 On Wed, 05 Jul 2017, Benson Leung wrote: > On Mon, Jul 03, 2017 at 12:46:41PM +0100, Lee Jones wrote: > > On Tue, 27 Jun 2017, Benson Leung wrote: > > > > > Hi Thierry, > > > > > > On Tue, Feb 14, 2017 at 08:58:02PM +0100, Thierry Escande wrote: > > > > From: Shawn Nematbakhsh > > > > > > > > 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 > > > > Signed-off-by: Thierry Escande > > > > --- > > > > 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. > > > > No need to change anything this time, but please note the "for my own > > reference" in my Ack in future. It typically means that I plan on > > pushing it through the MFD tree or that it requires further discussion > > (in the case that another Maintainer would prefer it go through their > > tree). > > Sorry about that. Thanks for the advice! I'm still getting my sea legs for > maintainership. It's not your fault. The tag I use is non-standard, so I expect some trivial issues like this to occur from time to time. But it solves more problems than it creates, so it's still work me using it. Thanks for your understanding. -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog