Re: [RESEND PATCH 6/8] mfd: cros_ec: rev cros_ec_commands.h

[Gwendal Grignou]

Reviewed-by: Gwendal Grignou <gwendal@xxxxxxxxxxxx>
Tested-by: Gwendal Grignou <gwendal@xxxxxxxxxxxx>

On Mon, Apr 6, 2015 at 9:15 AM, Javier Martinez Canillas
<javier.martinez@xxxxxxxxxxxxxxx> wrote:
> From: Stephen Barber <smbarber@xxxxxxxxxxxx>
>
> Update cros_ec_commands.h to the latest version in the EC
> firmware sources and add power domain and passthru commands.
>
> Also, update lightbar to use new command names.
>
> Signed-off-by: Stephen Barber <smbarber@xxxxxxxxxxxx>
> Reviewed-by: Randall Spangler <rspangler@xxxxxxxxxxxx>
> Signed-off-by: Javier Martinez Canillas <javier.martinez@xxxxxxxxxxxxxxx>
> ---
>  drivers/platform/chrome/cros_ec_lightbar.c |  14 +-
>  include/linux/mfd/cros_ec_commands.h       | 277 ++++++++++++++++++++++++++---
>  2 files changed, 262 insertions(+), 29 deletions(-)
>
> diff --git a/drivers/platform/chrome/cros_ec_lightbar.c b/drivers/platform/chrome/cros_ec_lightbar.c
> index 8ffec6649ead..38a9e58a9d2c 100644
> --- a/drivers/platform/chrome/cros_ec_lightbar.c
> +++ b/drivers/platform/chrome/cros_ec_lightbar.c
> @@ -196,8 +196,8 @@ static ssize_t brightness_store(struct device *dev,
>                 return -ENOMEM;
>
>         param = (struct ec_params_lightbar *)msg->data;
> -       param->cmd = LIGHTBAR_CMD_BRIGHTNESS;
> -       param->brightness.num = val;
> +       param->cmd = LIGHTBAR_CMD_SET_BRIGHTNESS;
> +       param->set_brightness.num = val;
>         ret = lb_throttle();
>         if (ret)
>                 goto exit;
> @@ -252,11 +252,11 @@ static ssize_t led_rgb_store(struct device *dev, struct device_attribute *attr,
>
>                 if (i == 4) {
>                         param = (struct ec_params_lightbar *)msg->data;
> -                       param->cmd = LIGHTBAR_CMD_RGB;
> -                       param->rgb.led = val[0];
> -                       param->rgb.red = val[1];
> -                       param->rgb.green = val[2];
> -                       param->rgb.blue = val[3];
> +                       param->cmd = LIGHTBAR_CMD_SET_RGB;
> +                       param->set_rgb.led = val[0];
> +                       param->set_rgb.red = val[1];
> +                       param->set_rgb.green = val[2];
> +                       param->set_rgb.blue = val[3];
>                         /*
>                          * Throttle only the first of every four transactions,
>                          * so that the user can update all four LEDs at once.
> diff --git a/include/linux/mfd/cros_ec_commands.h b/include/linux/mfd/cros_ec_commands.h
> index a49cd41feea7..13b630c10d4c 100644
> --- a/include/linux/mfd/cros_ec_commands.h
> +++ b/include/linux/mfd/cros_ec_commands.h
> @@ -515,7 +515,7 @@ struct ec_host_response {
>  /*
>   * Notes on commands:
>   *
> - * Each command is an 8-byte command value.  Commands which take params or
> + * Each command is an 16-bit command value.  Commands which take params or
>   * return response data specify structs for that data.  If no struct is
>   * specified, the command does not input or output data, respectively.
>   * Parameter/response length is implicit in the structs.  Some underlying
> @@ -966,7 +966,7 @@ struct rgb_s {
>  /* List of tweakable parameters. NOTE: It's __packed so it can be sent in a
>   * host command, but the alignment is the same regardless. Keep it that way.
>   */
> -struct lightbar_params {
> +struct lightbar_params_v0 {
>         /* Timing */
>         int32_t google_ramp_up;
>         int32_t google_ramp_down;
> @@ -1000,32 +1000,81 @@ struct lightbar_params {
>         struct rgb_s color[8];                  /* 0-3 are Google colors */
>  } __packed;
>
> +struct lightbar_params_v1 {
> +       /* Timing */
> +       int32_t google_ramp_up;
> +       int32_t google_ramp_down;
> +       int32_t s3s0_ramp_up;
> +       int32_t s0_tick_delay[2];               /* AC=0/1 */
> +       int32_t s0a_tick_delay[2];              /* AC=0/1 */
> +       int32_t s0s3_ramp_down;
> +       int32_t s3_sleep_for;
> +       int32_t s3_ramp_up;
> +       int32_t s3_ramp_down;
> +       int32_t tap_tick_delay;
> +       int32_t tap_display_time;
> +
> +       /* Tap-for-battery params */
> +       uint8_t tap_pct_red;
> +       uint8_t tap_pct_green;
> +       uint8_t tap_seg_min_on;
> +       uint8_t tap_seg_max_on;
> +       uint8_t tap_seg_osc;
> +       uint8_t tap_idx[3];
> +
> +       /* Oscillation */
> +       uint8_t osc_min[2];                     /* AC=0/1 */
> +       uint8_t osc_max[2];                     /* AC=0/1 */
> +       uint8_t w_ofs[2];                       /* AC=0/1 */
> +
> +       /* Brightness limits based on the backlight and AC. */
> +       uint8_t bright_bl_off_fixed[2];         /* AC=0/1 */
> +       uint8_t bright_bl_on_min[2];            /* AC=0/1 */
> +       uint8_t bright_bl_on_max[2];            /* AC=0/1 */
> +
> +       /* Battery level thresholds */
> +       uint8_t battery_threshold[LB_BATTERY_LEVELS - 1];
> +
> +       /* Map [AC][battery_level] to color index */
> +       uint8_t s0_idx[2][LB_BATTERY_LEVELS];   /* AP is running */
> +       uint8_t s3_idx[2][LB_BATTERY_LEVELS];   /* AP is sleeping */
> +
> +       /* Color palette */
> +       struct rgb_s color[8];                  /* 0-3 are Google colors */
> +} __packed;
> +
>  struct ec_params_lightbar {
>         uint8_t cmd;                  /* Command (see enum lightbar_command) */
>         union {
>                 struct {
>                         /* no args */
> -               } dump, off, on, init, get_seq, get_params, version;
> +               } dump, off, on, init, get_seq, get_params_v0, get_params_v1,
> +                       version, get_brightness, get_demo;
>
> -               struct num {
> +               struct {
>                         uint8_t num;
> -               } brightness, seq, demo;
> +               } set_brightness, seq, demo;
>
> -               struct reg {
> +               struct {
>                         uint8_t ctrl, reg, value;
>                 } reg;
>
> -               struct rgb {
> +               struct {
>                         uint8_t led, red, green, blue;
> -               } rgb;
> +               } set_rgb;
> +
> +               struct {
> +                       uint8_t led;
> +               } get_rgb;
>
> -               struct lightbar_params set_params;
> +               struct lightbar_params_v0 set_params_v0;
> +               struct lightbar_params_v1 set_params_v1;
>         };
>  } __packed;
>
>  struct ec_response_lightbar {
>         union {
> -               struct dump {
> +               struct {
>                         struct {
>                                 uint8_t reg;
>                                 uint8_t ic0;
> @@ -1033,20 +1082,26 @@ struct ec_response_lightbar {
>                         } vals[23];
>                 } dump;
>
> -               struct get_seq {
> +               struct  {
>                         uint8_t num;
> -               } get_seq;
> +               } get_seq, get_brightness, get_demo;
>
> -               struct lightbar_params get_params;
> +               struct lightbar_params_v0 get_params_v0;
> +               struct lightbar_params_v1 get_params_v1;
>
> -               struct version {
> +               struct {
>                         uint32_t num;
>                         uint32_t flags;
>                 } version;
>
>                 struct {
> +                       uint8_t red, green, blue;
> +               } get_rgb;
> +
> +               struct {
>                         /* no return params */
> -               } off, on, init, brightness, seq, reg, rgb, demo, set_params;
> +               } off, on, init, set_brightness, seq, reg, set_rgb,
> +                       demo, set_params_v0, set_params_v1;
>         };
>  } __packed;
>
> @@ -1056,15 +1111,20 @@ enum lightbar_command {
>         LIGHTBAR_CMD_OFF = 1,
>         LIGHTBAR_CMD_ON = 2,
>         LIGHTBAR_CMD_INIT = 3,
> -       LIGHTBAR_CMD_BRIGHTNESS = 4,
> +       LIGHTBAR_CMD_SET_BRIGHTNESS = 4,
>         LIGHTBAR_CMD_SEQ = 5,
>         LIGHTBAR_CMD_REG = 6,
> -       LIGHTBAR_CMD_RGB = 7,
> +       LIGHTBAR_CMD_SET_RGB = 7,
>         LIGHTBAR_CMD_GET_SEQ = 8,
>         LIGHTBAR_CMD_DEMO = 9,
> -       LIGHTBAR_CMD_GET_PARAMS = 10,
> -       LIGHTBAR_CMD_SET_PARAMS = 11,
> +       LIGHTBAR_CMD_GET_PARAMS_V0 = 10,
> +       LIGHTBAR_CMD_SET_PARAMS_V0 = 11,
>         LIGHTBAR_CMD_VERSION = 12,
> +       LIGHTBAR_CMD_GET_BRIGHTNESS = 13,
> +       LIGHTBAR_CMD_GET_RGB = 14,
> +       LIGHTBAR_CMD_GET_DEMO = 15,
> +       LIGHTBAR_CMD_GET_PARAMS_V1 = 16,
> +       LIGHTBAR_CMD_SET_PARAMS_V1 = 17,
>         LIGHTBAR_NUM_CMDS
>  };
>
> @@ -1421,8 +1481,40 @@ struct ec_response_rtc {
>  /*****************************************************************************/
>  /* Port80 log access */
>
> +/* Maximum entries that can be read/written in a single command */
> +#define EC_PORT80_SIZE_MAX 32
> +
>  /* Get last port80 code from previous boot */
>  #define EC_CMD_PORT80_LAST_BOOT 0x48
> +#define EC_CMD_PORT80_READ 0x48
> +
> +enum ec_port80_subcmd {
> +       EC_PORT80_GET_INFO = 0,
> +       EC_PORT80_READ_BUFFER,
> +};
> +
> +struct ec_params_port80_read {
> +       uint16_t subcmd;
> +       union {
> +               struct {
> +                       uint32_t offset;
> +                       uint32_t num_entries;
> +               } read_buffer;
> +       };
> +} __packed;
> +
> +struct ec_response_port80_read {
> +       union {
> +               struct {
> +                       uint32_t writes;
> +                       uint32_t history_size;
> +                       uint32_t last_boot;
> +               } get_info;
> +               struct {
> +                       uint16_t codes[EC_PORT80_SIZE_MAX];
> +               } data;
> +       };
> +} __packed;
>
>  struct ec_response_port80_last_boot {
>         uint16_t code;
> @@ -1782,6 +1874,7 @@ struct ec_params_gpio_set {
>  /* Get GPIO value */
>  #define EC_CMD_GPIO_GET 0x93
>
> +/* Version 0 of input params and response */
>  struct ec_params_gpio_get {
>         char name[32];
>  } __packed;
> @@ -1789,6 +1882,38 @@ struct ec_response_gpio_get {
>         uint8_t val;
>  } __packed;
>
> +/* Version 1 of input params and response */
> +struct ec_params_gpio_get_v1 {
> +       uint8_t subcmd;
> +       union {
> +               struct {
> +                       char name[32];
> +               } get_value_by_name;
> +               struct {
> +                       uint8_t index;
> +               } get_info;
> +       };
> +} __packed;
> +
> +struct ec_response_gpio_get_v1 {
> +       union {
> +               struct {
> +                       uint8_t val;
> +               } get_value_by_name, get_count;
> +               struct {
> +                       uint8_t val;
> +                       char name[32];
> +                       uint32_t flags;
> +               } get_info;
> +       };
> +} __packed;
> +
> +enum gpio_get_subcmd {
> +       EC_GPIO_GET_BY_NAME = 0,
> +       EC_GPIO_GET_COUNT = 1,
> +       EC_GPIO_GET_INFO = 2,
> +};
> +
>  /*****************************************************************************/
>  /* I2C commands. Only available when flash write protect is unlocked. */
>
> @@ -1857,13 +1982,21 @@ struct ec_params_charge_control {
>  /*****************************************************************************/
>
>  /*
> - * Cut off battery power output if the battery supports.
> + * Cut off battery power immediately or after the host has shut down.
>   *
> - * For unsupported battery, just don't implement this command and lets EC
> - * return EC_RES_INVALID_COMMAND.
> + * return EC_RES_INVALID_COMMAND if unsupported by a board/battery.
> + *       EC_RES_SUCCESS if the command was successful.
> + *       EC_RES_ERROR if the cut off command failed.
>   */
> +
>  #define EC_CMD_BATTERY_CUT_OFF 0x99
>
> +#define EC_BATTERY_CUTOFF_FLAG_AT_SHUTDOWN     (1 << 0)
> +
> +struct ec_params_battery_cutoff {
> +       uint8_t flags;
> +} __packed;
> +
>  /*****************************************************************************/
>  /* USB port mux control. */
>
> @@ -2142,6 +2275,32 @@ struct ec_params_sb_wr_block {
>  } __packed;
>
>  /*****************************************************************************/
> +/* Battery vendor parameters
> + *
> + * Get or set vendor-specific parameters in the battery. Implementations may
> + * differ between boards or batteries. On a set operation, the response
> + * contains the actual value set, which may be rounded or clipped from the
> + * requested value.
> + */
> +
> +#define EC_CMD_BATTERY_VENDOR_PARAM 0xb4
> +
> +enum ec_battery_vendor_param_mode {
> +       BATTERY_VENDOR_PARAM_MODE_GET = 0,
> +       BATTERY_VENDOR_PARAM_MODE_SET,
> +};
> +
> +struct ec_params_battery_vendor_param {
> +       uint32_t param;
> +       uint32_t value;
> +       uint8_t mode;
> +} __packed;
> +
> +struct ec_response_battery_vendor_param {
> +       uint32_t value;
> +} __packed;
> +
> +/*****************************************************************************/
>  /* System commands */
>
>  /*
> @@ -2338,6 +2497,80 @@ struct ec_params_reboot_ec {
>
>  /*****************************************************************************/
>  /*
> + * PD commands
> + *
> + * These commands are for PD MCU communication.
> + */
> +
> +/* EC to PD MCU exchange status command */
> +#define EC_CMD_PD_EXCHANGE_STATUS 0x100
> +
> +/* Status of EC being sent to PD */
> +struct ec_params_pd_status {
> +       int8_t batt_soc; /* battery state of charge */
> +} __packed;
> +
> +/* Status of PD being sent back to EC */
> +struct ec_response_pd_status {
> +       int8_t status;        /* PD MCU status */
> +       uint32_t curr_lim_ma; /* input current limit */
> +} __packed;
> +
> +/* Set USB type-C port role and muxes */
> +#define EC_CMD_USB_PD_CONTROL 0x101
> +
> +enum usb_pd_control_role {
> +       USB_PD_CTRL_ROLE_NO_CHANGE = 0,
> +       USB_PD_CTRL_ROLE_TOGGLE_ON = 1, /* == AUTO */
> +       USB_PD_CTRL_ROLE_TOGGLE_OFF = 2,
> +       USB_PD_CTRL_ROLE_FORCE_SINK = 3,
> +       USB_PD_CTRL_ROLE_FORCE_SOURCE = 4,
> +};
> +
> +enum usb_pd_control_mux {
> +       USB_PD_CTRL_MUX_NO_CHANGE = 0,
> +       USB_PD_CTRL_MUX_NONE = 1,
> +       USB_PD_CTRL_MUX_USB = 2,
> +       USB_PD_CTRL_MUX_DP = 3,
> +       USB_PD_CTRL_MUX_DOCK = 4,
> +       USB_PD_CTRL_MUX_AUTO = 5,
> +};
> +
> +struct ec_params_usb_pd_control {
> +       uint8_t port;
> +       uint8_t role;
> +       uint8_t mux;
> +} __packed;
> +
> +/*****************************************************************************/
> +/*
> + * Passthru commands
> + *
> + * Some platforms have sub-processors chained to each other.  For example.
> + *
> + *     AP <--> EC <--> PD MCU
> + *
> + * The top 2 bits of the command number are used to indicate which device the
> + * command is intended for.  Device 0 is always the device receiving the
> + * command; other device mapping is board-specific.
> + *
> + * When a device receives a command to be passed to a sub-processor, it passes
> + * it on with the device number set back to 0.  This allows the sub-processor
> + * to remain blissfully unaware of whether the command originated on the next
> + * device up the chain, or was passed through from the AP.
> + *
> + * In the above example, if the AP wants to send command 0x0002 to the PD MCU,
> + *     AP sends command 0x4002 to the EC
> + *     EC sends command 0x0002 to the PD MCU
> + *     EC forwards PD MCU response back to the AP
> + */
> +
> +/* Offset and max command number for sub-device n */
> +#define EC_CMD_PASSTHRU_OFFSET(n) (0x4000 * (n))
> +#define EC_CMD_PASSTHRU_MAX(n) (EC_CMD_PASSTHRU_OFFSET(n) + 0x3fff)
> +
> +/*****************************************************************************/
> +/*
>   * Deprecated constants. These constants have been renamed for clarity. The
>   * meaning and size has not changed. Programs that use the old names should
>   * switch to the new names soon, as the old names may not be carried forward
> --
> 2.1.4
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/