From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755988AbcHWVPR (ORCPT ); Tue, 23 Aug 2016 17:15:17 -0400 Received: from mail-qk0-f170.google.com ([209.85.220.170]:34984 "EHLO mail-qk0-f170.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753073AbcHWVPP (ORCPT ); Tue, 23 Aug 2016 17:15:15 -0400 MIME-Version: 1.0 In-Reply-To: <1471926859-21263-2-git-send-email-drinkcat@chromium.org> References: <1471926859-21263-1-git-send-email-drinkcat@chromium.org> <1471926859-21263-2-git-send-email-drinkcat@chromium.org> From: Guenter Roeck Date: Tue, 23 Aug 2016 14:15:14 -0700 Message-ID: Subject: Re: [PATCH 1/3] mfd: cros_ec: Add EC console read structures definitions To: Nicolas Boichat Cc: Lee Jones , Gwendal Grignou , ejcaruso@chromium.org, Olof Johansson , linux-kernel , Guenter Roeck Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Aug 22, 2016 at 9:34 PM, Nicolas Boichat wrote: > ec_params_console_read_v1 is used to capture EC logs from kernel, > and ec_params_get_cmd_versions_v1 is used to probe whether EC > supports that command. > > Signed-off-by: Nicolas Boichat Reviewed-by: Guenter Roeck > --- > include/linux/mfd/cros_ec_commands.h | 21 ++++++++++++++++++++- > 1 file changed, 20 insertions(+), 1 deletion(-) > > diff --git a/include/linux/mfd/cros_ec_commands.h b/include/linux/mfd/cros_ec_commands.h > index 7e7a8d4..74c84e82 100644 > --- a/include/linux/mfd/cros_ec_commands.h > +++ b/include/linux/mfd/cros_ec_commands.h > @@ -625,6 +625,10 @@ struct ec_params_get_cmd_versions { > uint8_t cmd; /* Command to check */ > } __packed; > > +struct ec_params_get_cmd_versions_v1 { > + uint16_t cmd; /* Command to check */ > +} __packed; > + > struct ec_response_get_cmd_versions { > /* > * Mask of supported versions; use EC_VER_MASK() to compare with a > @@ -2003,13 +2007,28 @@ struct ec_params_charge_control { > #define EC_CMD_CONSOLE_SNAPSHOT 0x97 > > /* > - * Read next chunk of data from saved snapshot. > + * Read data from the saved snapshot. If the subcmd parameter is > + * CONSOLE_READ_NEXT, this will return data starting from the beginning of > + * the latest snapshot. If it is CONSOLE_READ_RECENT, it will start from the > + * end of the previous snapshot. > + * > + * The params are only looked at in version >= 1 of this command. Prior > + * versions will just default to CONSOLE_READ_NEXT behavior. > * > * Response is null-terminated string. Empty string, if there is no more > * remaining output. > */ > #define EC_CMD_CONSOLE_READ 0x98 > > +enum ec_console_read_subcmd { > + CONSOLE_READ_NEXT = 0, > + CONSOLE_READ_RECENT > +}; > + > +struct ec_params_console_read_v1 { > + uint8_t subcmd; /* enum ec_console_read_subcmd */ > +} __packed; > + > /*****************************************************************************/ > > /* > -- > 2.8.0.rc3.226.g39d4020 >