WS cleanup: remove SPACE(s) followed by TAB
[platform/kernel/u-boot.git] / drivers / misc / cros_ec.c
index feaa5d8..c627c1d 100644 (file)
@@ -1,9 +1,8 @@
+// SPDX-License-Identifier: GPL-2.0+
 /*
  * Chromium OS cros_ec driver
  *
  * Copyright (c) 2012 The Chromium OS Authors.
- *
- * SPDX-License-Identifier:    GPL-2.0+
  */
 
 /*
  * is not reset.
  */
 
+#define LOG_CATEGORY UCLASS_CROS_EC
+
 #include <common.h>
 #include <command.h>
 #include <dm.h>
+#include <flash.h>
 #include <i2c.h>
 #include <cros_ec.h>
 #include <fdtdec.h>
+#include <log.h>
 #include <malloc.h>
 #include <spi.h>
+#include <linux/delay.h>
 #include <linux/errno.h>
 #include <asm/io.h>
 #include <asm-generic/gpio.h>
@@ -40,9 +44,59 @@ enum {
        CROS_EC_CMD_TIMEOUT_MS  = 5000,
        /* Timeout waiting for a synchronous hash to be recomputed */
        CROS_EC_CMD_HASH_TIMEOUT_MS = 2000,
+
+       /* Wait 10 ms between attempts to check if EC's hash is ready */
+       CROS_EC_HASH_CHECK_DELAY_MS = 10,
+
 };
 
-DECLARE_GLOBAL_DATA_PTR;
+#define INVALID_HCMD 0xFF
+
+/*
+ * Map UHEPI masks to non UHEPI commands in order to support old EC FW
+ * which does not support UHEPI command.
+ */
+static const struct {
+       u8 set_cmd;
+       u8 clear_cmd;
+       u8 get_cmd;
+} event_map[] = {
+       [EC_HOST_EVENT_MAIN] = {
+               INVALID_HCMD, EC_CMD_HOST_EVENT_CLEAR,
+               INVALID_HCMD,
+       },
+       [EC_HOST_EVENT_B] = {
+               INVALID_HCMD, EC_CMD_HOST_EVENT_CLEAR_B,
+               EC_CMD_HOST_EVENT_GET_B,
+       },
+       [EC_HOST_EVENT_SCI_MASK] = {
+               EC_CMD_HOST_EVENT_SET_SCI_MASK, INVALID_HCMD,
+               EC_CMD_HOST_EVENT_GET_SCI_MASK,
+       },
+       [EC_HOST_EVENT_SMI_MASK] = {
+               EC_CMD_HOST_EVENT_SET_SMI_MASK, INVALID_HCMD,
+               EC_CMD_HOST_EVENT_GET_SMI_MASK,
+       },
+       [EC_HOST_EVENT_ALWAYS_REPORT_MASK] = {
+               INVALID_HCMD, INVALID_HCMD, INVALID_HCMD,
+       },
+       [EC_HOST_EVENT_ACTIVE_WAKE_MASK] = {
+               EC_CMD_HOST_EVENT_SET_WAKE_MASK, INVALID_HCMD,
+               EC_CMD_HOST_EVENT_GET_WAKE_MASK,
+       },
+       [EC_HOST_EVENT_LAZY_WAKE_MASK_S0IX] = {
+               EC_CMD_HOST_EVENT_SET_WAKE_MASK, INVALID_HCMD,
+               EC_CMD_HOST_EVENT_GET_WAKE_MASK,
+       },
+       [EC_HOST_EVENT_LAZY_WAKE_MASK_S3] = {
+               EC_CMD_HOST_EVENT_SET_WAKE_MASK, INVALID_HCMD,
+               EC_CMD_HOST_EVENT_GET_WAKE_MASK,
+       },
+       [EC_HOST_EVENT_LAZY_WAKE_MASK_S5] = {
+               EC_CMD_HOST_EVENT_SET_WAKE_MASK, INVALID_HCMD,
+               EC_CMD_HOST_EVENT_GET_WAKE_MASK,
+       },
+};
 
 void cros_ec_dump_data(const char *name, int cmd, const uint8_t *data, int len)
 {
@@ -86,15 +140,15 @@ int cros_ec_calc_checksum(const uint8_t *data, int size)
  * @param dout_len      Size of output data in bytes
  * @return packet size in bytes, or <0 if error.
  */
-static int create_proto3_request(struct cros_ec_dev *dev,
+static int create_proto3_request(struct cros_ec_dev *cdev,
                                 int cmd, int cmd_version,
                                 const void *dout, int dout_len)
 {
-       struct ec_host_request *rq = (struct ec_host_request *)dev->dout;
+       struct ec_host_request *rq = (struct ec_host_request *)cdev->dout;
        int out_bytes = dout_len + sizeof(*rq);
 
        /* Fail if output size is too big */
-       if (out_bytes > (int)sizeof(dev->dout)) {
+       if (out_bytes > (int)sizeof(cdev->dout)) {
                debug("%s: Cannot send %d bytes\n", __func__, dout_len);
                return -EC_RES_REQUEST_TRUNCATED;
        }
@@ -111,9 +165,9 @@ static int create_proto3_request(struct cros_ec_dev *dev,
        memcpy(rq + 1, dout, dout_len);
 
        /* Write checksum field so the entire packet sums to 0 */
-       rq->checksum = (uint8_t)(-cros_ec_calc_checksum(dev->dout, out_bytes));
+       rq->checksum = (uint8_t)(-cros_ec_calc_checksum(cdev->dout, out_bytes));
 
-       cros_ec_dump_data("out", cmd, dev->dout, out_bytes);
+       cros_ec_dump_data("out", cmd, cdev->dout, out_bytes);
 
        /* Return size of request packet */
        return out_bytes;
@@ -126,12 +180,12 @@ static int create_proto3_request(struct cros_ec_dev *dev,
  * @param din_len       Maximum size of response in bytes
  * @return maximum expected number of bytes in response, or <0 if error.
  */
-static int prepare_proto3_response_buffer(struct cros_ec_dev *dev, int din_len)
+static int prepare_proto3_response_buffer(struct cros_ec_dev *cdev, int din_len)
 {
        int in_bytes = din_len + sizeof(struct ec_host_response);
 
        /* Fail if input size is too big */
-       if (in_bytes > (int)sizeof(dev->din)) {
+       if (in_bytes > (int)sizeof(cdev->din)) {
                debug("%s: Cannot receive %d bytes\n", __func__, din_len);
                return -EC_RES_RESPONSE_TOO_BIG;
        }
@@ -200,7 +254,7 @@ static int handle_proto3_response(struct cros_ec_dev *dev,
        return rs->data_len;
 }
 
-static int send_command_proto3(struct cros_ec_dev *dev,
+static int send_command_proto3(struct cros_ec_dev *cdev,
                               int cmd, int cmd_version,
                               const void *dout, int dout_len,
                               uint8_t **dinp, int din_len)
@@ -210,26 +264,27 @@ static int send_command_proto3(struct cros_ec_dev *dev,
        int rv;
 
        /* Create request packet */
-       out_bytes = create_proto3_request(dev, cmd, cmd_version,
+       out_bytes = create_proto3_request(cdev, cmd, cmd_version,
                                          dout, dout_len);
        if (out_bytes < 0)
                return out_bytes;
 
        /* Prepare response buffer */
-       in_bytes = prepare_proto3_response_buffer(dev, din_len);
+       in_bytes = prepare_proto3_response_buffer(cdev, din_len);
        if (in_bytes < 0)
                return in_bytes;
 
-       ops = dm_cros_ec_get_ops(dev->dev);
-       rv = ops->packet ? ops->packet(dev->dev, out_bytes, in_bytes) : -ENOSYS;
+       ops = dm_cros_ec_get_ops(cdev->dev);
+       rv = ops->packet ? ops->packet(cdev->dev, out_bytes, in_bytes) :
+                       -ENOSYS;
        if (rv < 0)
                return rv;
 
        /* Process the response */
-       return handle_proto3_response(dev, dinp, din_len);
+       return handle_proto3_response(cdev, dinp, din_len);
 }
 
-static int send_command(struct cros_ec_dev *dev, uint8_t cmd, int cmd_version,
+static int send_command(struct cros_ec_dev *dev, uint cmd, int cmd_version,
                        const void *dout, int dout_len,
                        uint8_t **dinp, int din_len)
 {
@@ -265,15 +320,16 @@ static int send_command(struct cros_ec_dev *dev, uint8_t cmd, int cmd_version,
  * @param din_len       Maximum size of response in bytes
  * @return number of bytes in response, or -ve on error
  */
-static int ec_command_inptr(struct cros_ec_dev *dev, uint8_t cmd,
-               int cmd_version, const void *dout, int dout_len, uint8_t **dinp,
-               int din_len)
+static int ec_command_inptr(struct udevice *dev, uint cmd,
+                           int cmd_version, const void *dout, int dout_len,
+                           uint8_t **dinp, int din_len)
 {
+       struct cros_ec_dev *cdev = dev_get_uclass_priv(dev);
        uint8_t *din = NULL;
        int len;
 
-       len = send_command(dev, cmd, cmd_version, dout, dout_len,
-                               &din, din_len);
+       len = send_command(cdev, cmd, cmd_version, dout, dout_len, &din,
+                          din_len);
 
        /* If the command doesn't complete, wait a while */
        if (len == -EC_RES_IN_PROGRESS) {
@@ -286,9 +342,9 @@ static int ec_command_inptr(struct cros_ec_dev *dev, uint8_t cmd,
                        int ret;
 
                        mdelay(50);     /* Insert some reasonable delay */
-                       ret = send_command(dev, EC_CMD_GET_COMMS_STATUS, 0,
-                                       NULL, 0,
-                                       (uint8_t **)&resp, sizeof(*resp));
+                       ret = send_command(cdev, EC_CMD_GET_COMMS_STATUS, 0,
+                                          NULL, 0,
+                                          (uint8_t **)&resp, sizeof(*resp));
                        if (ret < 0)
                                return ret;
 
@@ -301,8 +357,8 @@ static int ec_command_inptr(struct cros_ec_dev *dev, uint8_t cmd,
 
                /* OK it completed, so read the status response */
                /* not sure why it was 0 for the last argument */
-               len = send_command(dev, EC_CMD_RESEND_RESPONSE, 0,
-                               NULL, 0, &din, din_len);
+               len = send_command(cdev, EC_CMD_RESEND_RESPONSE, 0, NULL, 0,
+                                  &din, din_len);
        }
 
        debug("%s: len=%d, din=%p\n", __func__, len, din);
@@ -331,7 +387,7 @@ static int ec_command_inptr(struct cros_ec_dev *dev, uint8_t cmd,
  * @param din_len       Maximum size of response in bytes
  * @return number of bytes in response, or -ve on error
  */
-static int ec_command(struct cros_ec_dev *dev, uint8_t cmd, int cmd_version,
+static int ec_command(struct udevice *dev, uint cmd, int cmd_version,
                      const void *dout, int dout_len,
                      void *din, int din_len)
 {
@@ -340,7 +396,7 @@ static int ec_command(struct cros_ec_dev *dev, uint8_t cmd, int cmd_version,
 
        assert((din_len == 0) || din);
        len = ec_command_inptr(dev, cmd, cmd_version, dout, dout_len,
-                       &in_buffer, din_len);
+                              &in_buffer, din_len);
        if (len > 0) {
                /*
                 * If we were asked to put it somewhere, do so, otherwise just
@@ -348,6 +404,8 @@ static int ec_command(struct cros_ec_dev *dev, uint8_t cmd, int cmd_version,
                 */
                if (din && in_buffer) {
                        assert(len <= din_len);
+                       if (len > din_len)
+                               return -ENOSPC;
                        memmove(din, in_buffer, len);
                }
        }
@@ -356,22 +414,39 @@ static int ec_command(struct cros_ec_dev *dev, uint8_t cmd, int cmd_version,
 
 int cros_ec_scan_keyboard(struct udevice *dev, struct mbkp_keyscan *scan)
 {
-       struct cros_ec_dev *cdev = dev_get_uclass_priv(dev);
-
-       if (ec_command(cdev, EC_CMD_MKBP_STATE, 0, NULL, 0, scan,
+       if (ec_command(dev, EC_CMD_MKBP_STATE, 0, NULL, 0, scan,
                       sizeof(scan->data)) != sizeof(scan->data))
                return -1;
 
        return 0;
 }
 
-int cros_ec_read_id(struct cros_ec_dev *dev, char *id, int maxlen)
+int cros_ec_get_next_event(struct udevice *dev,
+                          struct ec_response_get_next_event *event)
+{
+       int ret;
+
+       ret = ec_command(dev, EC_CMD_GET_NEXT_EVENT, 0, NULL, 0,
+                        event, sizeof(*event));
+       if (ret < 0)
+               return ret;
+       else if (ret != sizeof(*event))
+               return -EC_RES_INVALID_RESPONSE;
+
+       return 0;
+}
+
+int cros_ec_read_id(struct udevice *dev, char *id, int maxlen)
 {
        struct ec_response_get_version *r;
+       int ret;
 
-       if (ec_command_inptr(dev, EC_CMD_GET_VERSION, 0, NULL, 0,
-                       (uint8_t **)&r, sizeof(*r)) != sizeof(*r))
+       ret = ec_command_inptr(dev, EC_CMD_GET_VERSION, 0, NULL, 0,
+                              (uint8_t **)&r, sizeof(*r));
+       if (ret != sizeof(*r)) {
+               log_err("Got rc %d, expected %u\n", ret, (uint)sizeof(*r));
                return -1;
+       }
 
        if (maxlen > (int)sizeof(r->version_string_ro))
                maxlen = sizeof(r->version_string_ro);
@@ -384,6 +459,7 @@ int cros_ec_read_id(struct cros_ec_dev *dev, char *id, int maxlen)
                memcpy(id, r->version_string_rw, maxlen);
                break;
        default:
+               log_err("Invalid EC image %d\n", r->current_image);
                return -1;
        }
 
@@ -391,8 +467,8 @@ int cros_ec_read_id(struct cros_ec_dev *dev, char *id, int maxlen)
        return 0;
 }
 
-int cros_ec_read_version(struct cros_ec_dev *dev,
-                      struct ec_response_get_version **versionp)
+int cros_ec_read_version(struct udevice *dev,
+                        struct ec_response_get_version **versionp)
 {
        if (ec_command_inptr(dev, EC_CMD_GET_VERSION, 0, NULL, 0,
                        (uint8_t **)versionp, sizeof(**versionp))
@@ -402,7 +478,7 @@ int cros_ec_read_version(struct cros_ec_dev *dev,
        return 0;
 }
 
-int cros_ec_read_build_info(struct cros_ec_dev *dev, char **strp)
+int cros_ec_read_build_info(struct udevice *dev, char **strp)
 {
        if (ec_command_inptr(dev, EC_CMD_GET_BUILD_INFO, 0, NULL, 0,
                        (uint8_t **)strp, EC_PROTO2_MAX_PARAM_SIZE) < 0)
@@ -411,8 +487,8 @@ int cros_ec_read_build_info(struct cros_ec_dev *dev, char **strp)
        return 0;
 }
 
-int cros_ec_read_current_image(struct cros_ec_dev *dev,
-               enum ec_current_image *image)
+int cros_ec_read_current_image(struct udevice *dev,
+                              enum ec_current_image *image)
 {
        struct ec_response_get_version *r;
 
@@ -424,19 +500,20 @@ int cros_ec_read_current_image(struct cros_ec_dev *dev,
        return 0;
 }
 
-static int cros_ec_wait_on_hash_done(struct cros_ec_dev *dev,
-                                 struct ec_response_vboot_hash *hash)
+static int cros_ec_wait_on_hash_done(struct udevice *dev,
+                                    struct ec_params_vboot_hash *p,
+                                    struct ec_response_vboot_hash *hash)
 {
-       struct ec_params_vboot_hash p;
        ulong start;
 
        start = get_timer(0);
        while (hash->status == EC_VBOOT_HASH_STATUS_BUSY) {
-               mdelay(50);     /* Insert some reasonable delay */
+               mdelay(CROS_EC_HASH_CHECK_DELAY_MS);
 
-               p.cmd = EC_VBOOT_HASH_GET;
-               if (ec_command(dev, EC_CMD_VBOOT_HASH, 0, &p, sizeof(p),
-                      hash, sizeof(*hash)) < 0)
+               p->cmd = EC_VBOOT_HASH_GET;
+
+               if (ec_command(dev, EC_CMD_VBOOT_HASH, 0, p, sizeof(*p), hash,
+                              sizeof(*hash)) < 0)
                        return -1;
 
                if (get_timer(start) > CROS_EC_CMD_HASH_TIMEOUT_MS) {
@@ -447,20 +524,20 @@ static int cros_ec_wait_on_hash_done(struct cros_ec_dev *dev,
        return 0;
 }
 
-
-int cros_ec_read_hash(struct cros_ec_dev *dev,
-               struct ec_response_vboot_hash *hash)
+int cros_ec_read_hash(struct udevice *dev, uint hash_offset,
+                     struct ec_response_vboot_hash *hash)
 {
        struct ec_params_vboot_hash p;
        int rv;
 
        p.cmd = EC_VBOOT_HASH_GET;
+       p.offset = hash_offset;
        if (ec_command(dev, EC_CMD_VBOOT_HASH, 0, &p, sizeof(p),
                       hash, sizeof(*hash)) < 0)
                return -1;
 
        /* If the EC is busy calculating the hash, fidget until it's done. */
-       rv = cros_ec_wait_on_hash_done(dev, hash);
+       rv = cros_ec_wait_on_hash_done(dev, &p, hash);
        if (rv)
                return rv;
 
@@ -477,22 +554,26 @@ int cros_ec_read_hash(struct cros_ec_dev *dev,
        p.cmd = EC_VBOOT_HASH_START;
        p.hash_type = EC_VBOOT_HASH_TYPE_SHA256;
        p.nonce_size = 0;
-       p.offset = EC_VBOOT_HASH_OFFSET_RW;
+       p.offset = hash_offset;
 
        if (ec_command(dev, EC_CMD_VBOOT_HASH, 0, &p, sizeof(p),
                       hash, sizeof(*hash)) < 0)
                return -1;
 
-       rv = cros_ec_wait_on_hash_done(dev, hash);
+       rv = cros_ec_wait_on_hash_done(dev, &p, hash);
        if (rv)
                return rv;
+       if (hash->status != EC_VBOOT_HASH_STATUS_DONE) {
+               log_err("Hash did not complete, status=%d\n", hash->status);
+               return -EIO;
+       }
 
        debug("%s: hash done\n", __func__);
 
        return 0;
 }
 
-static int cros_ec_invalidate_hash(struct cros_ec_dev *dev)
+static int cros_ec_invalidate_hash(struct udevice *dev)
 {
        struct ec_params_vboot_hash p;
        struct ec_response_vboot_hash *hash;
@@ -517,8 +598,26 @@ static int cros_ec_invalidate_hash(struct cros_ec_dev *dev)
        return 0;
 }
 
-int cros_ec_reboot(struct cros_ec_dev *dev, enum ec_reboot_cmd cmd,
-               uint8_t flags)
+int cros_ec_hello(struct udevice *dev, uint *handshakep)
+{
+       struct ec_params_hello req;
+       struct ec_response_hello *resp;
+
+       req.in_data = 0x12345678;
+       if (ec_command_inptr(dev, EC_CMD_HELLO, 0, &req, sizeof(req),
+                            (uint8_t **)&resp, sizeof(*resp)) < 0)
+               return -EIO;
+       if (resp->out_data != req.in_data + 0x01020304) {
+               printf("Received invalid handshake %x\n", resp->out_data);
+               if (handshakep)
+                       *handshakep = req.in_data;
+               return -ENOTSYNC;
+       }
+
+       return 0;
+}
+
+int cros_ec_reboot(struct udevice *dev, enum ec_reboot_cmd cmd, uint8_t flags)
 {
        struct ec_params_reboot_ec p;
 
@@ -530,18 +629,23 @@ int cros_ec_reboot(struct cros_ec_dev *dev, enum ec_reboot_cmd cmd,
                return -1;
 
        if (!(flags & EC_REBOOT_FLAG_ON_AP_SHUTDOWN)) {
+               ulong start;
+
                /*
                 * EC reboot will take place immediately so delay to allow it
                 * to complete.  Note that some reboot types (EC_REBOOT_COLD)
                 * will reboot the AP as well, in which case we won't actually
                 * get to this point.
                 */
-               /*
-                * TODO(rspangler@chromium.org): Would be nice if we had a
-                * better way to determine when the reboot is complete.  Could
-                * we poll a memory-mapped LPC value?
-                */
-               udelay(50000);
+               mdelay(50);
+               start = get_timer(0);
+               while (cros_ec_hello(dev, NULL)) {
+                       if (get_timer(start) > 3000) {
+                               log_err("EC did not return from reboot\n");
+                               return -ETIMEDOUT;
+                       }
+                       mdelay(5);
+               }
        }
 
        return 0;
@@ -558,7 +662,7 @@ int cros_ec_interrupt_pending(struct udevice *dev)
        return dm_gpio_get_value(&cdev->ec_int);
 }
 
-int cros_ec_info(struct cros_ec_dev *dev, struct ec_response_mkbp_info *info)
+int cros_ec_info(struct udevice *dev, struct ec_response_mkbp_info *info)
 {
        if (ec_command(dev, EC_CMD_MKBP_INFO, 0, NULL, 0, info,
                       sizeof(*info)) != sizeof(*info))
@@ -567,7 +671,37 @@ int cros_ec_info(struct cros_ec_dev *dev, struct ec_response_mkbp_info *info)
        return 0;
 }
 
-int cros_ec_get_host_events(struct cros_ec_dev *dev, uint32_t *events_ptr)
+int cros_ec_get_event_mask(struct udevice *dev, uint type, uint32_t *mask)
+{
+       struct ec_response_host_event_mask rsp;
+       int ret;
+
+       ret = ec_command(dev, type, 0, NULL, 0, &rsp, sizeof(rsp));
+       if (ret < 0)
+               return ret;
+       else if (ret != sizeof(rsp))
+               return -EINVAL;
+
+       *mask = rsp.mask;
+
+       return 0;
+}
+
+int cros_ec_set_event_mask(struct udevice *dev, uint type, uint32_t mask)
+{
+       struct ec_params_host_event_mask req;
+       int ret;
+
+       req.mask = mask;
+
+       ret = ec_command(dev, type, 0, &req, sizeof(req), NULL, 0);
+       if (ret < 0)
+               return ret;
+
+       return 0;
+}
+
+int cros_ec_get_host_events(struct udevice *dev, uint32_t *events_ptr)
 {
        struct ec_response_host_event_mask *resp;
 
@@ -586,7 +720,7 @@ int cros_ec_get_host_events(struct cros_ec_dev *dev, uint32_t *events_ptr)
        return 0;
 }
 
-int cros_ec_clear_host_events(struct cros_ec_dev *dev, uint32_t events)
+int cros_ec_clear_host_events(struct udevice *dev, uint32_t events)
 {
        struct ec_params_host_event_mask params;
 
@@ -603,9 +737,9 @@ int cros_ec_clear_host_events(struct cros_ec_dev *dev, uint32_t events)
        return 0;
 }
 
-int cros_ec_flash_protect(struct cros_ec_dev *dev,
-                      uint32_t set_mask, uint32_t set_flags,
-                      struct ec_response_flash_protect *resp)
+int cros_ec_flash_protect(struct udevice *dev, uint32_t set_mask,
+                         uint32_t set_flags,
+                         struct ec_response_flash_protect *resp)
 {
        struct ec_params_flash_protect params;
 
@@ -620,17 +754,17 @@ int cros_ec_flash_protect(struct cros_ec_dev *dev,
        return 0;
 }
 
-static int cros_ec_check_version(struct cros_ec_dev *dev)
+static int cros_ec_check_version(struct udevice *dev)
 {
+       struct cros_ec_dev *cdev = dev_get_uclass_priv(dev);
        struct ec_params_hello req;
-       struct ec_response_hello *resp;
 
        struct dm_cros_ec_ops *ops;
        int ret;
 
-       ops = dm_cros_ec_get_ops(dev->dev);
+       ops = dm_cros_ec_get_ops(dev);
        if (ops->check_version) {
-               ret = ops->check_version(dev->dev);
+               ret = ops->check_version(dev);
                if (ret)
                        return ret;
        }
@@ -650,19 +784,17 @@ static int cros_ec_check_version(struct cros_ec_dev *dev)
         */
 
        /* Try sending a version 3 packet */
-       dev->protocol_version = 3;
+       cdev->protocol_version = 3;
        req.in_data = 0;
-       if (ec_command_inptr(dev, EC_CMD_HELLO, 0, &req, sizeof(req),
-                            (uint8_t **)&resp, sizeof(*resp)) > 0) {
+       ret = cros_ec_hello(dev, NULL);
+       if (!ret || ret == -ENOTSYNC)
                return 0;
-       }
 
        /* Try sending a version 2 packet */
-       dev->protocol_version = 2;
-       if (ec_command_inptr(dev, EC_CMD_HELLO, 0, &req, sizeof(req),
-                      (uint8_t **)&resp, sizeof(*resp)) > 0) {
+       cdev->protocol_version = 2;
+       ret = cros_ec_hello(dev, NULL);
+       if (!ret || ret == -ENOTSYNC)
                return 0;
-       }
 
        /*
         * Fail if we're still here, since the EC doesn't understand any
@@ -670,31 +802,29 @@ static int cros_ec_check_version(struct cros_ec_dev *dev)
         * version is no longer supported, and we don't know about any new
         * protocol versions.
         */
-       dev->protocol_version = 0;
+       cdev->protocol_version = 0;
        printf("%s: ERROR: old EC interface not supported\n", __func__);
        return -1;
 }
 
-int cros_ec_test(struct cros_ec_dev *dev)
+int cros_ec_test(struct udevice *dev)
 {
-       struct ec_params_hello req;
-       struct ec_response_hello *resp;
+       uint out_data;
+       int ret;
 
-       req.in_data = 0x12345678;
-       if (ec_command_inptr(dev, EC_CMD_HELLO, 0, &req, sizeof(req),
-                      (uint8_t **)&resp, sizeof(*resp)) < sizeof(*resp)) {
+       ret = cros_ec_hello(dev, &out_data);
+       if (ret == -ENOTSYNC) {
+               printf("Received invalid handshake %x\n", out_data);
+               return ret;
+       } else if (ret) {
                printf("ec_command_inptr() returned error\n");
-               return -1;
-       }
-       if (resp->out_data != req.in_data + 0x01020304) {
-               printf("Received invalid handshake %x\n", resp->out_data);
-               return -1;
+               return ret;
        }
 
        return 0;
 }
 
-int cros_ec_flash_offset(struct cros_ec_dev *dev, enum ec_flash_region region,
+int cros_ec_flash_offset(struct udevice *dev, enum ec_flash_region region,
                      uint32_t *offset, uint32_t *size)
 {
        struct ec_params_flash_region_info p;
@@ -716,7 +846,7 @@ int cros_ec_flash_offset(struct cros_ec_dev *dev, enum ec_flash_region region,
        return 0;
 }
 
-int cros_ec_flash_erase(struct cros_ec_dev *dev, uint32_t offset, uint32_t size)
+int cros_ec_flash_erase(struct udevice *dev, uint32_t offset, uint32_t size)
 {
        struct ec_params_flash_erase p;
 
@@ -744,8 +874,8 @@ int cros_ec_flash_erase(struct cros_ec_dev *dev, uint32_t offset, uint32_t size)
  * @param size         Number of bytes to write
  * @return 0 if ok, -1 on error
  */
-static int cros_ec_flash_write_block(struct cros_ec_dev *dev,
-               const uint8_t *data, uint32_t offset, uint32_t size)
+static int cros_ec_flash_write_block(struct udevice *dev, const uint8_t *data,
+                                    uint32_t offset, uint32_t size)
 {
        struct ec_params_flash_write *p;
        int ret;
@@ -770,7 +900,7 @@ static int cros_ec_flash_write_block(struct cros_ec_dev *dev,
 /**
  * Return optimal flash write burst size
  */
-static int cros_ec_flash_write_burst_size(struct cros_ec_dev *dev)
+static int cros_ec_flash_write_burst_size(struct udevice *dev)
 {
        return EC_FLASH_WRITE_VER0_SIZE;
 }
@@ -804,8 +934,8 @@ static int cros_ec_data_is_erased(const uint32_t *data, int size)
  * @param dev  Pointer to device
  * @param info Pointer to output flash info struct
  */
-int cros_ec_read_flashinfo(struct cros_ec_dev *dev,
-                         struct ec_response_flash_info *info)
+int cros_ec_read_flashinfo(struct udevice *dev,
+                          struct ec_response_flash_info *info)
 {
        int ret;
 
@@ -817,13 +947,17 @@ int cros_ec_read_flashinfo(struct cros_ec_dev *dev,
        return ret < sizeof(*info) ? -1 : 0;
 }
 
-int cros_ec_flash_write(struct cros_ec_dev *dev, const uint8_t *data,
-                    uint32_t offset, uint32_t size)
+int cros_ec_flash_write(struct udevice *dev, const uint8_t *data,
+                       uint32_t offset, uint32_t size)
 {
+       struct cros_ec_dev *cdev = dev_get_uclass_priv(dev);
        uint32_t burst = cros_ec_flash_write_burst_size(dev);
        uint32_t end, off;
        int ret;
 
+       if (!burst)
+               return -EINVAL;
+
        /*
         * TODO: round up to the nearest multiple of write size.  Can get away
         * without that on link right now because its write size is 4 bytes.
@@ -834,8 +968,8 @@ int cros_ec_flash_write(struct cros_ec_dev *dev, const uint8_t *data,
 
                /* If the data is empty, there is no point in programming it */
                todo = min(end - off, burst);
-               if (dev->optimise_flash_write &&
-                               cros_ec_data_is_erased((uint32_t *)data, todo))
+               if (cdev->optimise_flash_write &&
+                   cros_ec_data_is_erased((uint32_t *)data, todo))
                        continue;
 
                ret = cros_ec_flash_write_block(dev, data, off, todo);
@@ -847,6 +981,35 @@ int cros_ec_flash_write(struct cros_ec_dev *dev, const uint8_t *data,
 }
 
 /**
+ * Run verification on a slot
+ *
+ * @param me     CrosEc instance
+ * @param region Region to run verification on
+ * @return 0 if success or not applicable. Non-zero if verification failed.
+ */
+int cros_ec_efs_verify(struct udevice *dev, enum ec_flash_region region)
+{
+       struct ec_params_efs_verify p;
+       int rv;
+
+       log_info("EFS: EC is verifying updated image...\n");
+       p.region = region;
+
+       rv = ec_command(dev, EC_CMD_EFS_VERIFY, 0, &p, sizeof(p), NULL, 0);
+       if (rv >= 0) {
+               log_info("EFS: Verification success\n");
+               return 0;
+       }
+       if (rv == -EC_RES_INVALID_COMMAND) {
+               log_info("EFS: EC doesn't support EFS_VERIFY command\n");
+               return 0;
+       }
+       log_info("EFS: Verification failed\n");
+
+       return rv;
+}
+
+/**
  * Read a single block from the flash
  *
  * Read a block of data from the EC flash. The size must not exceed the flash
@@ -861,8 +1024,8 @@ int cros_ec_flash_write(struct cros_ec_dev *dev, const uint8_t *data,
  * @param size         Number of bytes to read
  * @return 0 if ok, -1 on error
  */
-static int cros_ec_flash_read_block(struct cros_ec_dev *dev, uint8_t *data,
-                                uint32_t offset, uint32_t size)
+static int cros_ec_flash_read_block(struct udevice *dev, uint8_t *data,
+                                   uint32_t offset, uint32_t size)
 {
        struct ec_params_flash_read p;
 
@@ -873,8 +1036,8 @@ static int cros_ec_flash_read_block(struct cros_ec_dev *dev, uint8_t *data,
                          &p, sizeof(p), data, size) >= 0 ? 0 : -1;
 }
 
-int cros_ec_flash_read(struct cros_ec_dev *dev, uint8_t *data, uint32_t offset,
-                   uint32_t size)
+int cros_ec_flash_read(struct udevice *dev, uint8_t *data, uint32_t offset,
+                      uint32_t size)
 {
        uint32_t burst = cros_ec_flash_write_burst_size(dev);
        uint32_t end, off;
@@ -891,13 +1054,14 @@ int cros_ec_flash_read(struct cros_ec_dev *dev, uint8_t *data, uint32_t offset,
        return 0;
 }
 
-int cros_ec_flash_update_rw(struct cros_ec_dev *dev,
-                        const uint8_t *image, int image_size)
+int cros_ec_flash_update_rw(struct udevice *dev, const uint8_t *image,
+                           int image_size)
 {
        uint32_t rw_offset, rw_size;
        int ret;
 
-       if (cros_ec_flash_offset(dev, EC_FLASH_REGION_RW, &rw_offset, &rw_size))
+       if (cros_ec_flash_offset(dev, EC_FLASH_REGION_ACTIVE, &rw_offset,
+               &rw_size))
                return -1;
        if (image_size > (int)rw_size)
                return -1;
@@ -930,46 +1094,96 @@ int cros_ec_flash_update_rw(struct cros_ec_dev *dev,
        return 0;
 }
 
-int cros_ec_read_vbnvcontext(struct cros_ec_dev *dev, uint8_t *block)
+int cros_ec_get_sku_id(struct udevice *dev)
+{
+       struct ec_sku_id_info *r;
+       int ret;
+
+       ret = ec_command_inptr(dev, EC_CMD_GET_SKU_ID, 0, NULL, 0,
+                              (uint8_t **)&r, sizeof(*r));
+       if (ret != sizeof(*r))
+               return -ret;
+
+       return r->sku_id;
+}
+
+int cros_ec_read_nvdata(struct udevice *dev, uint8_t *block, int size)
 {
        struct ec_params_vbnvcontext p;
        int len;
 
+       if (size != EC_VBNV_BLOCK_SIZE && size != EC_VBNV_BLOCK_SIZE_V2)
+               return -EINVAL;
+
        p.op = EC_VBNV_CONTEXT_OP_READ;
 
        len = ec_command(dev, EC_CMD_VBNV_CONTEXT, EC_VER_VBNV_CONTEXT,
-                       &p, sizeof(p), block, EC_VBNV_BLOCK_SIZE);
-       if (len < EC_VBNV_BLOCK_SIZE)
-               return -1;
+                        &p, sizeof(uint32_t) + size, block, size);
+       if (len != size) {
+               log_err("Expected %d bytes, got %d\n", size, len);
+               return -EIO;
+       }
 
        return 0;
 }
 
-int cros_ec_write_vbnvcontext(struct cros_ec_dev *dev, const uint8_t *block)
+int cros_ec_write_nvdata(struct udevice *dev, const uint8_t *block, int size)
 {
        struct ec_params_vbnvcontext p;
        int len;
 
+       if (size != EC_VBNV_BLOCK_SIZE && size != EC_VBNV_BLOCK_SIZE_V2)
+               return -EINVAL;
        p.op = EC_VBNV_CONTEXT_OP_WRITE;
-       memcpy(p.block, block, sizeof(p.block));
+       memcpy(p.block, block, size);
 
        len = ec_command_inptr(dev, EC_CMD_VBNV_CONTEXT, EC_VER_VBNV_CONTEXT,
-                       &p, sizeof(p), NULL, 0);
+                       &p, sizeof(uint32_t) + size, NULL, 0);
+       if (len < 0)
+               return -1;
+
+       return 0;
+}
+
+int cros_ec_battery_cutoff(struct udevice *dev, uint8_t flags)
+{
+       struct ec_params_battery_cutoff p;
+       int len;
+
+       p.flags = flags;
+       len = ec_command(dev, EC_CMD_BATTERY_CUT_OFF, 1, &p, sizeof(p),
+                        NULL, 0);
+
        if (len < 0)
                return -1;
+       return 0;
+}
+
+int cros_ec_set_pwm_duty(struct udevice *dev, uint8_t index, uint16_t duty)
+{
+       struct ec_params_pwm_set_duty p;
+       int ret;
+
+       p.duty = duty;
+       p.pwm_type = EC_PWM_TYPE_GENERIC;
+       p.index = index;
+
+       ret = ec_command(dev, EC_CMD_PWM_SET_DUTY, 0, &p, sizeof(p),
+                        NULL, 0);
+       if (ret < 0)
+               return ret;
 
        return 0;
 }
 
 int cros_ec_set_ldo(struct udevice *dev, uint8_t index, uint8_t state)
 {
-       struct cros_ec_dev *cdev = dev_get_uclass_priv(dev);
        struct ec_params_ldo_set params;
 
        params.index = index;
        params.state = state;
 
-       if (ec_command_inptr(cdev, EC_CMD_LDO_SET, 0, &params, sizeof(params),
+       if (ec_command_inptr(dev, EC_CMD_LDO_SET, 0, &params, sizeof(params),
                             NULL, 0))
                return -1;
 
@@ -978,13 +1192,12 @@ int cros_ec_set_ldo(struct udevice *dev, uint8_t index, uint8_t state)
 
 int cros_ec_get_ldo(struct udevice *dev, uint8_t index, uint8_t *state)
 {
-       struct cros_ec_dev *cdev = dev_get_uclass_priv(dev);
        struct ec_params_ldo_get params;
        struct ec_response_ldo_get *resp;
 
        params.index = index;
 
-       if (ec_command_inptr(cdev, EC_CMD_LDO_GET, 0, &params, sizeof(params),
+       if (ec_command_inptr(dev, EC_CMD_LDO_GET, 0, &params, sizeof(params),
                             (uint8_t **)&resp, sizeof(*resp)) !=
                             sizeof(*resp))
                return -1;
@@ -1004,12 +1217,12 @@ int cros_ec_register(struct udevice *dev)
                             GPIOD_IS_IN);
        cdev->optimise_flash_write = dev_read_bool(dev, "optimise-flash-write");
 
-       if (cros_ec_check_version(cdev)) {
+       if (cros_ec_check_version(dev)) {
                debug("%s: Could not detect CROS-EC version\n", __func__);
                return -CROS_EC_ERR_CHECK_VERSION;
        }
 
-       if (cros_ec_read_id(cdev, id, sizeof(id))) {
+       if (cros_ec_read_id(dev, id, sizeof(id))) {
                debug("%s: Could not read KBC ID\n", __func__);
                return -CROS_EC_ERR_READ_ID;
        }
@@ -1031,22 +1244,21 @@ int cros_ec_decode_ec_flash(struct udevice *dev, struct fdt_cros_ec *config)
                return -1;
        }
 
-       if (of_read_fmap_entry(flash_node, "flash", &config->flash)) {
+       if (ofnode_read_fmap_entry(flash_node,  &config->flash)) {
                debug("Failed to decode flash node in chrome-ec\n");
                return -1;
        }
 
        config->flash_erase_value = ofnode_read_s32_default(flash_node,
                                                            "erase-value", -1);
-       for (node = ofnode_first_subnode(flash_node); ofnode_valid(node);
-            node = ofnode_next_subnode(node)) {
+       ofnode_for_each_subnode(node, flash_node) {
                const char *name = ofnode_get_name(node);
                enum ec_flash_region region;
 
                if (0 == strcmp(name, "ro")) {
                        region = EC_FLASH_REGION_RO;
                } else if (0 == strcmp(name, "rw")) {
-                       region = EC_FLASH_REGION_RW;
+                       region = EC_FLASH_REGION_ACTIVE;
                } else if (0 == strcmp(name, "wp-ro")) {
                        region = EC_FLASH_REGION_WP_RO;
                } else {
@@ -1054,7 +1266,7 @@ int cros_ec_decode_ec_flash(struct udevice *dev, struct fdt_cros_ec *config)
                        return -1;
                }
 
-               if (of_read_fmap_entry(node, "reg", &config->region[region])) {
+               if (ofnode_read_fmap_entry(node, &config->region[region])) {
                        debug("Failed to decode flash region in chrome-ec'\n");
                        return -1;
                }
@@ -1066,7 +1278,6 @@ int cros_ec_decode_ec_flash(struct udevice *dev, struct fdt_cros_ec *config)
 int cros_ec_i2c_tunnel(struct udevice *dev, int port, struct i2c_msg *in,
                       int nmsgs)
 {
-       struct cros_ec_dev *cdev = dev_get_uclass_priv(dev);
        union {
                struct ec_params_i2c_passthru p;
                uint8_t outbuf[EC_PROTO2_MAX_PARAM_SIZE];
@@ -1116,7 +1327,7 @@ int cros_ec_i2c_tunnel(struct udevice *dev, int port, struct i2c_msg *in,
                }
        }
 
-       rv = ec_command(cdev, EC_CMD_I2C_PASSTHRU, 0, p, pdata - (uint8_t *)p,
+       rv = ec_command(dev, EC_CMD_I2C_PASSTHRU, 0, p, pdata - (uint8_t *)p,
                        r, sizeof(*r) + read_len);
        if (rv < 0)
                return rv;
@@ -1139,9 +1350,329 @@ int cros_ec_i2c_tunnel(struct udevice *dev, int port, struct i2c_msg *in,
        return 0;
 }
 
+int cros_ec_get_features(struct udevice *dev, u64 *featuresp)
+{
+       struct ec_response_get_features r;
+       int rv;
+
+       rv = ec_command(dev, EC_CMD_GET_FEATURES, 0, NULL, 0, &r, sizeof(r));
+       if (rv != sizeof(r))
+               return -EIO;
+       *featuresp = r.flags[0] | (u64)r.flags[1] << 32;
+
+       return 0;
+}
+
+int cros_ec_check_feature(struct udevice *dev, uint feature)
+{
+       struct ec_response_get_features r;
+       int rv;
+
+       rv = ec_command(dev, EC_CMD_GET_FEATURES, 0, NULL, 0, &r, sizeof(r));
+       if (rv != sizeof(r))
+               return -EIO;
+
+       if (feature >= 8 * sizeof(r.flags))
+               return -EINVAL;
+
+       return r.flags[feature / 32] & EC_FEATURE_MASK_0(feature) ? true :
+                false;
+}
+
+/*
+ * Query the EC for specified mask indicating enabled events.
+ * The EC maintains separate event masks for SMI, SCI and WAKE.
+ */
+static int cros_ec_uhepi_cmd(struct udevice *dev, uint mask, uint action,
+                            uint64_t *value)
+{
+       int ret;
+       struct ec_params_host_event req;
+       struct ec_response_host_event rsp;
+
+       req.action = action;
+       req.mask_type = mask;
+       if (action != EC_HOST_EVENT_GET)
+               req.value = *value;
+       else
+               *value = 0;
+       ret = ec_command(dev, EC_CMD_HOST_EVENT, 0, &req, sizeof(req), &rsp,
+                        sizeof(rsp));
+
+       if (action != EC_HOST_EVENT_GET)
+               return ret;
+       if (ret == 0)
+               *value = rsp.value;
+
+       return ret;
+}
+
+static int cros_ec_handle_non_uhepi_cmd(struct udevice *dev, uint hcmd,
+                                       uint action, uint64_t *value)
+{
+       int ret = -1;
+       struct ec_params_host_event_mask req;
+       struct ec_response_host_event_mask rsp;
+
+       if (hcmd == INVALID_HCMD)
+               return ret;
+
+       if (action != EC_HOST_EVENT_GET)
+               req.mask = (uint32_t)*value;
+       else
+               *value = 0;
+
+       ret = ec_command(dev, hcmd, 0, &req, sizeof(req), &rsp, sizeof(rsp));
+       if (action != EC_HOST_EVENT_GET)
+               return ret;
+       if (ret == 0)
+               *value = rsp.mask;
+
+       return ret;
+}
+
+bool cros_ec_is_uhepi_supported(struct udevice *dev)
+{
+#define UHEPI_SUPPORTED 1
+#define UHEPI_NOT_SUPPORTED 2
+       static int uhepi_support;
+
+       if (!uhepi_support) {
+               uhepi_support = cros_ec_check_feature(dev,
+                       EC_FEATURE_UNIFIED_WAKE_MASKS) > 0 ? UHEPI_SUPPORTED :
+                       UHEPI_NOT_SUPPORTED;
+               log_debug("Chrome EC: UHEPI %s\n",
+                         uhepi_support == UHEPI_SUPPORTED ? "supported" :
+                         "not supported");
+       }
+       return uhepi_support == UHEPI_SUPPORTED;
+}
+
+static int cros_ec_get_mask(struct udevice *dev, uint type)
+{
+       u64 value = 0;
+
+       if (cros_ec_is_uhepi_supported(dev)) {
+               cros_ec_uhepi_cmd(dev, type, EC_HOST_EVENT_GET, &value);
+       } else {
+               assert(type < ARRAY_SIZE(event_map));
+               cros_ec_handle_non_uhepi_cmd(dev, event_map[type].get_cmd,
+                                            EC_HOST_EVENT_GET, &value);
+       }
+       return value;
+}
+
+static int cros_ec_clear_mask(struct udevice *dev, uint type, u64 mask)
+{
+       if (cros_ec_is_uhepi_supported(dev))
+               return cros_ec_uhepi_cmd(dev, type, EC_HOST_EVENT_CLEAR, &mask);
+
+       assert(type < ARRAY_SIZE(event_map));
+
+       return cros_ec_handle_non_uhepi_cmd(dev, event_map[type].clear_cmd,
+                                           EC_HOST_EVENT_CLEAR, &mask);
+}
+
+uint64_t cros_ec_get_events_b(struct udevice *dev)
+{
+       return cros_ec_get_mask(dev, EC_HOST_EVENT_B);
+}
+
+int cros_ec_clear_events_b(struct udevice *dev, uint64_t mask)
+{
+       log_debug("Chrome EC: clear events_b mask to 0x%016llx\n", mask);
+
+       return cros_ec_clear_mask(dev, EC_HOST_EVENT_B, mask);
+}
+
+int cros_ec_read_limit_power(struct udevice *dev, int *limit_powerp)
+{
+       struct ec_params_charge_state p;
+       struct ec_response_charge_state r;
+       int ret;
+
+       p.cmd = CHARGE_STATE_CMD_GET_PARAM;
+       p.get_param.param = CS_PARAM_LIMIT_POWER;
+       ret = ec_command(dev, EC_CMD_CHARGE_STATE, 0, &p, sizeof(p),
+                        &r, sizeof(r));
+
+       /*
+        * If our EC doesn't support the LIMIT_POWER parameter, assume that
+        * LIMIT_POWER is not requested.
+        */
+       if (ret == -EC_RES_INVALID_PARAM || ret == -EC_RES_INVALID_COMMAND) {
+               log_warning("PARAM_LIMIT_POWER not supported by EC\n");
+               return -ENOSYS;
+       }
+
+       if (ret != sizeof(r.get_param))
+               return -EINVAL;
+
+       *limit_powerp = r.get_param.value;
+       return 0;
+}
+
+int cros_ec_config_powerbtn(struct udevice *dev, uint32_t flags)
+{
+       struct ec_params_config_power_button params;
+       int ret;
+
+       params.flags = flags;
+       ret = ec_command(dev, EC_CMD_CONFIG_POWER_BUTTON, 0,
+                        &params, sizeof(params), NULL, 0);
+       if (ret < 0)
+               return ret;
+
+       return 0;
+}
+
+int cros_ec_get_lid_shutdown_mask(struct udevice *dev)
+{
+       u32 mask;
+       int ret;
+
+       ret = cros_ec_get_event_mask(dev, EC_CMD_HOST_EVENT_GET_SMI_MASK,
+                                    &mask);
+       if (ret < 0)
+               return ret;
+
+       return !!(mask & EC_HOST_EVENT_MASK(EC_HOST_EVENT_LID_CLOSED));
+}
+
+int cros_ec_set_lid_shutdown_mask(struct udevice *dev, int enable)
+{
+       u32 mask;
+       int ret;
+
+       ret = cros_ec_get_event_mask(dev, EC_CMD_HOST_EVENT_GET_SMI_MASK,
+                                    &mask);
+       if (ret < 0)
+               return ret;
+
+       /* Set lid close event state in the EC SMI event mask */
+       if (enable)
+               mask |= EC_HOST_EVENT_MASK(EC_HOST_EVENT_LID_CLOSED);
+       else
+               mask &= ~EC_HOST_EVENT_MASK(EC_HOST_EVENT_LID_CLOSED);
+
+       ret = cros_ec_set_event_mask(dev, EC_CMD_HOST_EVENT_SET_SMI_MASK, mask);
+       if (ret < 0)
+               return ret;
+
+       printf("EC: %sabled lid close event\n", enable ? "en" : "dis");
+       return 0;
+}
+
+int cros_ec_vstore_supported(struct udevice *dev)
+{
+       return cros_ec_check_feature(dev, EC_FEATURE_VSTORE);
+}
+
+int cros_ec_vstore_info(struct udevice *dev, u32 *lockedp)
+{
+       struct ec_response_vstore_info *resp;
+
+       if (ec_command_inptr(dev, EC_CMD_VSTORE_INFO, 0, NULL, 0,
+                            (uint8_t **)&resp, sizeof(*resp)) != sizeof(*resp))
+               return -EIO;
+
+       if (lockedp)
+               *lockedp = resp->slot_locked;
+
+       return resp->slot_count;
+}
+
+/*
+ * cros_ec_vstore_read - Read data from EC vstore slot
+ *
+ * @slot: vstore slot to read from
+ * @data: buffer to store read data, must be EC_VSTORE_SLOT_SIZE bytes
+ */
+int cros_ec_vstore_read(struct udevice *dev, int slot, uint8_t *data)
+{
+       struct ec_params_vstore_read req;
+       struct ec_response_vstore_read *resp;
+
+       req.slot = slot;
+       if (ec_command_inptr(dev, EC_CMD_VSTORE_READ, 0, &req, sizeof(req),
+                            (uint8_t **)&resp, sizeof(*resp)) != sizeof(*resp))
+               return -EIO;
+
+       if (!data || req.slot >= EC_VSTORE_SLOT_MAX)
+               return -EINVAL;
+
+       memcpy(data, resp->data, sizeof(resp->data));
+
+       return 0;
+}
+
+/*
+ * cros_ec_vstore_write - Save data into EC vstore slot
+ *
+ * @slot: vstore slot to write into
+ * @data: data to write
+ * @size: size of data in bytes
+ *
+ * Maximum size of data is EC_VSTORE_SLOT_SIZE.  It is the callers
+ * responsibility to check the number of implemented slots by
+ * querying the vstore info.
+ */
+int cros_ec_vstore_write(struct udevice *dev, int slot, const uint8_t *data,
+                        size_t size)
+{
+       struct ec_params_vstore_write req;
+
+       if (slot >= EC_VSTORE_SLOT_MAX || size > EC_VSTORE_SLOT_SIZE)
+               return -EINVAL;
+
+       req.slot = slot;
+       memcpy(req.data, data, size);
+
+       if (ec_command(dev, EC_CMD_VSTORE_WRITE, 0, &req, sizeof(req), NULL, 0))
+               return -EIO;
+
+       return 0;
+}
+
+int cros_ec_get_switches(struct udevice *dev)
+{
+       struct dm_cros_ec_ops *ops;
+       int ret;
+
+       ops = dm_cros_ec_get_ops(dev);
+       if (!ops->get_switches)
+               return -ENOSYS;
+
+       ret = ops->get_switches(dev);
+       if (ret < 0)
+               return log_msg_ret("get", ret);
+
+       return ret;
+}
+
+int cros_ec_read_batt_charge(struct udevice *dev, uint *chargep)
+{
+       struct ec_params_charge_state req;
+       struct ec_response_charge_state resp;
+       int ret;
+
+       req.cmd = CHARGE_STATE_CMD_GET_STATE;
+       ret = ec_command(dev, EC_CMD_CHARGE_STATE, 0, &req, sizeof(req),
+                        &resp, sizeof(resp));
+       if (ret)
+               return log_msg_ret("read", ret);
+
+       *chargep = resp.get_state.batt_state_of_charge;
+
+       return 0;
+}
+
 UCLASS_DRIVER(cros_ec) = {
        .id             = UCLASS_CROS_EC,
-       .name           = "cros_ec",
-       .per_device_auto_alloc_size = sizeof(struct cros_ec_dev),
+       .name           = "cros-ec",
+       .per_device_auto        = sizeof(struct cros_ec_dev),
+#if CONFIG_IS_ENABLED(OF_REAL)
        .post_bind      = dm_scan_fdt_dev,
+#endif
+       .flags          = DM_UC_FLAG_ALLOC_PRIV_DMA,
 };