From: Benson Leung <bleung@google.com>
To: Enric Balletbo i Serra <enric.balletbo@collabora.com>
Cc: olof@lixom.net, bleung@chromium.org,
linux-kernel@vger.kernel.org, lee.jones@linaro.org,
Eric Caruso <ejcaruso@chromium.org>,
Guenter Roeck <groeck@chromium.org>
Subject: Re: [PATCH RESEND 10/13] platform/chrome: cros_ec_lightbar - Add lightbar program feature to sysfs
Date: Thu, 22 Jun 2017 16:43:07 -0700 [thread overview]
Message-ID: <20170622234307.GA126262@decatoncale.mtv.corp.google.com> (raw)
In-Reply-To: <20170516161319.13257-11-enric.balletbo@collabora.com>
[-- Attachment #1: Type: text/plain, Size: 5678 bytes --]
Hi Enric,
On Tue, May 16, 2017 at 06:13:16PM +0200, Enric Balletbo i Serra wrote:
> From: Eric Caruso <ejcaruso@chromium.org>
>
> Add a program feature so we can upload and run programs for lightbar
> sequences. We should be able to use this to shift sequences out of the
> EC and save space there.
>
> $ cat <suitable program bin> > /sys/devices/.../cros_ec/program
> $ echo program > /sys/devices/.../cros_ec/sequence
>
> Signed-off-by: Eric Caruso <ejcaruso@chromium.org>
> Signed-off-by: Guenter Roeck <groeck@chromium.org>
> Signed-off-by: Enric Balletbo i Serra <enric.balletbo@collabora.com>
> Acked-by: Lee Jones <lee.jones@linaro.org>
Signed-off-by: Benson Leung <bleung@chromium.org>
Applied.
> ---
> drivers/platform/chrome/cros_ec_lightbar.c | 69 +++++++++++++++++++++++++++++-
> include/linux/mfd/cros_ec_commands.h | 12 +++++-
> 2 files changed, 79 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/platform/chrome/cros_ec_lightbar.c b/drivers/platform/chrome/cros_ec_lightbar.c
> index 8df3d44..2667505 100644
> --- a/drivers/platform/chrome/cros_ec_lightbar.c
> +++ b/drivers/platform/chrome/cros_ec_lightbar.c
> @@ -295,7 +295,8 @@ static ssize_t led_rgb_store(struct device *dev, struct device_attribute *attr,
>
> static char const *seqname[] = {
> "ERROR", "S5", "S3", "S0", "S5S3", "S3S0",
> - "S0S3", "S3S5", "STOP", "RUN", "PULSE", "TEST", "KONAMI",
> + "S0S3", "S3S5", "STOP", "RUN", "KONAMI",
> + "TAP", "PROGRAM",
> };
>
> static ssize_t sequence_show(struct device *dev,
> @@ -390,6 +391,69 @@ static ssize_t sequence_store(struct device *dev, struct device_attribute *attr,
> return ret;
> }
>
> +static ssize_t program_store(struct device *dev, struct device_attribute *attr,
> + const char *buf, size_t count)
> +{
> + int extra_bytes, max_size, ret;
> + struct ec_params_lightbar *param;
> + struct cros_ec_command *msg;
> + struct cros_ec_dev *ec = container_of(dev, struct cros_ec_dev,
> + class_dev);
> +
> + /*
> + * We might need to reject the program for size reasons. The EC
> + * enforces a maximum program size, but we also don't want to try
> + * and send a program that is too big for the protocol. In order
> + * to ensure the latter, we also need to ensure we have extra bytes
> + * to represent the rest of the packet.
> + */
> + extra_bytes = sizeof(*param) - sizeof(param->set_program.data);
> + max_size = min(EC_LB_PROG_LEN, ec->ec_dev->max_request - extra_bytes);
> + if (count > max_size) {
> + dev_err(dev, "Program is %u bytes, too long to send (max: %u)",
> + (unsigned int)count, max_size);
> +
> + return -EINVAL;
> + }
> +
> + msg = alloc_lightbar_cmd_msg(ec);
> + if (!msg)
> + return -ENOMEM;
> +
> + ret = lb_throttle();
> + if (ret)
> + goto exit;
> +
> + dev_info(dev, "Copying %zu byte program to EC", count);
> +
> + param = (struct ec_params_lightbar *)msg->data;
> + param->cmd = LIGHTBAR_CMD_SET_PROGRAM;
> +
> + param->set_program.size = count;
> + memcpy(param->set_program.data, buf, count);
> +
> + /*
> + * We need to set the message size manually or else it will use
> + * EC_LB_PROG_LEN. This might be too long, and the program
> + * is unlikely to use all of the space.
> + */
> + msg->outsize = count + extra_bytes;
> +
> + ret = cros_ec_cmd_xfer(ec->ec_dev, msg);
> + if (ret < 0)
> + goto exit;
> + if (msg->result != EC_RES_SUCCESS) {
> + ret = -EINVAL;
> + goto exit;
> + }
> +
> + ret = count;
> +exit:
> + kfree(msg);
> +
> + return ret;
> +}
> +
> /* Module initialization */
>
> static DEVICE_ATTR_RW(interval_msec);
> @@ -397,12 +461,15 @@ static DEVICE_ATTR_RO(version);
> static DEVICE_ATTR_WO(brightness);
> static DEVICE_ATTR_WO(led_rgb);
> static DEVICE_ATTR_RW(sequence);
> +static DEVICE_ATTR_WO(program);
> +
> static struct attribute *__lb_cmds_attrs[] = {
> &dev_attr_interval_msec.attr,
> &dev_attr_version.attr,
> &dev_attr_brightness.attr,
> &dev_attr_led_rgb.attr,
> &dev_attr_sequence.attr,
> + &dev_attr_program.attr,
> NULL,
> };
>
> diff --git a/include/linux/mfd/cros_ec_commands.h b/include/linux/mfd/cros_ec_commands.h
> index 1b19e424..dbea580 100644
> --- a/include/linux/mfd/cros_ec_commands.h
> +++ b/include/linux/mfd/cros_ec_commands.h
> @@ -1162,6 +1162,13 @@ struct lightbar_params_v1 {
> struct rgb_s color[8]; /* 0-3 are Google colors */
> } __packed;
>
> +/* Lightbar program */
> +#define EC_LB_PROG_LEN 192
> +struct lightbar_program {
> + uint8_t size;
> + uint8_t data[EC_LB_PROG_LEN];
> +};
> +
> struct ec_params_lightbar {
> uint8_t cmd; /* Command (see enum lightbar_command) */
> union {
> @@ -1188,6 +1195,7 @@ struct ec_params_lightbar {
>
> struct lightbar_params_v0 set_params_v0;
> struct lightbar_params_v1 set_params_v1;
> + struct lightbar_program set_program;
> };
> } __packed;
>
> @@ -1220,7 +1228,8 @@ struct ec_response_lightbar {
> struct {
> /* no return params */
> } off, on, init, set_brightness, seq, reg, set_rgb,
> - demo, set_params_v0, set_params_v1;
> + demo, set_params_v0, set_params_v1,
> + set_program;
> };
> } __packed;
>
> @@ -1244,6 +1253,7 @@ enum lightbar_command {
> LIGHTBAR_CMD_GET_DEMO = 15,
> LIGHTBAR_CMD_GET_PARAMS_V1 = 16,
> LIGHTBAR_CMD_SET_PARAMS_V1 = 17,
> + LIGHTBAR_CMD_SET_PROGRAM = 18,
> LIGHTBAR_NUM_CMDS
> };
>
> --
> 2.9.3
>
--
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 --]
next prev parent reply other threads:[~2017-06-22 23:43 UTC|newest]
Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-05-16 16:13 [PATCH RESEND 00/13] platform/chrome: Add console debugfs, lpc and lightbar programming support Enric Balletbo i Serra
2017-05-16 16:13 ` [PATCH RESEND 01/13] mfd: cros_ec: Add helper for event notifier Enric Balletbo i Serra
2017-06-16 20:45 ` Benson Leung
2017-05-16 16:13 ` [PATCH RESEND 02/13] mfd: cros_ec: Add EC console read structures definitions Enric Balletbo i Serra
2017-06-16 20:47 ` Benson Leung
2017-05-16 16:13 ` [PATCH RESEND 03/13] mfd: cros_ec: add debugfs, console log file Enric Balletbo i Serra
2017-06-16 19:42 ` Benson Leung
2017-05-16 16:13 ` [PATCH RESEND 04/13] mfd: cros_ec: Add support for dumping panic information Enric Balletbo i Serra
2017-06-22 23:38 ` Benson Leung
2017-05-16 16:13 ` [PATCH RESEND 05/13] platform/chrome: cros_ec_lpc: Add R/W helpers to LPC protocol variants Enric Balletbo i Serra
2017-05-22 11:09 ` Lee Jones
2017-06-22 19:02 ` Benson Leung
2017-05-16 16:13 ` [PATCH RESEND 06/13] platform/chrome: cros_ec_lpc: Add support for mec1322 EC Enric Balletbo i Serra
2017-05-22 11:09 ` Lee Jones
2017-06-22 19:01 ` Benson Leung
2017-05-16 16:13 ` [PATCH RESEND 07/13] platform/chrome: cros_ec_lpc: Add support for GOOG004 ACPI device Enric Balletbo i Serra
2017-06-22 19:39 ` Benson Leung
2017-05-16 16:13 ` [PATCH RESEND 08/13] platform/chrome: cros_ec_lpc: Add power management ops Enric Balletbo i Serra
2017-06-22 19:46 ` Benson Leung
2017-05-16 16:13 ` [PATCH RESEND 09/13] platform/chrome: cros_ec_lpc: Add MKBP events support over ACPI Enric Balletbo i Serra
2017-06-22 19:35 ` Benson Leung
2017-06-23 7:35 ` Thierry Escande
2017-06-23 17:54 ` Benson Leung
2017-05-16 16:13 ` [PATCH RESEND 10/13] platform/chrome: cros_ec_lightbar - Add lightbar program feature to sysfs Enric Balletbo i Serra
2017-06-22 23:43 ` Benson Leung [this message]
2017-05-16 16:13 ` [PATCH RESEND 11/13] platform/chrome: cros_ec_lightbar - Control of suspend/resume lightbar sequence Enric Balletbo i Serra
2017-06-23 22:08 ` Benson Leung
2017-05-16 16:13 ` [PATCH RESEND 12/13] platform/chrome: cros_ec_lightbar - Add userspace lightbar control bit to EC Enric Balletbo i Serra
2017-06-23 22:12 ` Benson Leung
2017-05-16 16:13 ` [PATCH RESEND 13/13] platform/chrome: cros_ec_lightbar - Avoid I2C xfer to EC during suspend Enric Balletbo i Serra
2017-06-23 22:22 ` Benson Leung
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20170622234307.GA126262@decatoncale.mtv.corp.google.com \
--to=bleung@google.com \
--cc=bleung@chromium.org \
--cc=ejcaruso@chromium.org \
--cc=enric.balletbo@collabora.com \
--cc=groeck@chromium.org \
--cc=lee.jones@linaro.org \
--cc=linux-kernel@vger.kernel.org \
--cc=olof@lixom.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).