Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753564AbcDKLXJ (ORCPT ); Mon, 11 Apr 2016 07:23:09 -0400 Received: from mail-wm0-f49.google.com ([74.125.82.49]:34024 "EHLO mail-wm0-f49.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752532AbcDKLXI (ORCPT ); Mon, 11 Apr 2016 07:23:08 -0400 Date: Mon, 11 Apr 2016 12:23:03 +0100 From: Lee Jones To: Tomeu Vizoso Cc: linux-kernel@vger.kernel.org, Sameer Nanda , Javier Martinez Canillas , Benson Leung , Enric =?iso-8859-1?Q?Balletb=F2?= , Vic Yang , Stephen Boyd , Vincent Palatin , Randall Spangler , Todd Broch , Gwendal Grignou , Olof Johansson Subject: Re: [RESEND PATCH v7 3/6] mfd: cros_ec: Add cros_ec_get_host_event Message-ID: <20160411112303.GI8094@x1> References: <1459842789-13852-1-git-send-email-tomeu.vizoso@collabora.com> <1459842789-13852-4-git-send-email-tomeu.vizoso@collabora.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <1459842789-13852-4-git-send-email-tomeu.vizoso@collabora.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2926 Lines: 93 On Tue, 05 Apr 2016, Tomeu Vizoso wrote: > This function returns the code for the host event that triggered the > interrupt that is being currently handled. > > Is to be used by observers of the event_notifier in the EC device. > > Signed-off-by: Tomeu Vizoso > --- > > Changes in v7: None > Changes in v6: > - Return 0 if the EC doesn't support MKBP, as expected by callers. > > Changes in v5: None > Changes in v4: None > Changes in v3: None > Changes in v2: None > > drivers/platform/chrome/cros_ec_proto.c | 23 +++++++++++++++++++++++ > include/linux/mfd/cros_ec.h | 10 ++++++++++ > 2 files changed, 33 insertions(+) > > diff --git a/drivers/platform/chrome/cros_ec_proto.c b/drivers/platform/chrome/cros_ec_proto.c > index aaccdde1c9d5..3e1e83874edf 100644 > --- a/drivers/platform/chrome/cros_ec_proto.c > +++ b/drivers/platform/chrome/cros_ec_proto.c > @@ -486,3 +486,26 @@ int cros_ec_cmd_xfer_status(struct cros_ec_device *ec_dev, > return ret; > } > EXPORT_SYMBOL(cros_ec_cmd_xfer_status); > + > +u32 cros_ec_get_host_event(struct cros_ec_device *ec_dev) > +{ > + u32 host_event; > + > + if (!ec_dev->mkbp_event_supported) { > + dev_warn(ec_dev->dev, > + "This EC does not support EC_MKBP_EVENT_HOST_EVENT"); > + return 0; > + } > + > + if (ec_dev->event_data.event_type != EC_MKBP_EVENT_HOST_EVENT) > + return 0; > + > + if (ec_dev->event_size != sizeof(host_event)) { > + dev_warn(ec_dev->dev, "Invalid host event size\n"); > + return 0; > + } > + > + host_event = get_unaligned_le32(&ec_dev->event_data.data.host_event); This assignment is pointless. Just return right away. > + return host_event; > +} > +EXPORT_SYMBOL(cros_ec_get_host_event); > diff --git a/include/linux/mfd/cros_ec.h b/include/linux/mfd/cros_ec.h > index e4c4c0480c14..9dd2c6711ea9 100644 > --- a/include/linux/mfd/cros_ec.h > +++ b/include/linux/mfd/cros_ec.h > @@ -304,6 +304,16 @@ int cros_ec_query_all(struct cros_ec_device *ec_dev); > */ > int cros_ec_get_next_event(struct cros_ec_device *ec_dev); > > +/** > + * cros_ec_get_host_event - Return a mask of event set by the EC. > + * > + * Returns the mask of the event that caused the last interrupt to be raised. > + * > + * @ec_dev: Device to fetch event from > + * @return event mask, 0 on error s/@return/Returns:/ "Event mask on success" I have a few problems with "0 on error". Please reword so it doesn't looks like we're returning "success on error". I suggest, if you're not returning >0, that it's not really an error, so drop the word. > + */ > +uint32_t cros_ec_get_host_event(struct cros_ec_device *ec_dev); > + > /* sysfs stuff */ > extern struct attribute_group cros_ec_attr_group; > extern struct attribute_group cros_ec_lightbar_attr_group; -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog