WS cleanup: remove SPACE(s) followed by TAB
[platform/kernel/u-boot.git] / drivers / misc / cros_ec.c
index e0f3dfc..c627c1d 100644 (file)
 #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>
@@ -41,6 +44,58 @@ 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,
+
+};
+
+#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)
@@ -265,7 +320,7 @@ static int send_command(struct cros_ec_dev *dev, uint 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 udevice *dev, uint8_t cmd,
+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)
 {
@@ -349,6 +404,8 @@ static int ec_command(struct udevice *dev, uint cmd, int cmd_version,
                 */
                if (din && in_buffer) {
                        assert(len <= din_len);
+                       if (len > din_len)
+                               return -ENOSPC;
                        memmove(din, in_buffer, len);
                }
        }
@@ -357,13 +414,28 @@ static int ec_command(struct udevice *dev, uint cmd, int cmd_version,
 
 int cros_ec_scan_keyboard(struct udevice *dev, struct mbkp_keyscan *scan)
 {
-       if (ec_command(dev, 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_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;
@@ -372,7 +444,7 @@ int cros_ec_read_id(struct udevice *dev, char *id, int maxlen)
        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 %d\n", ret, sizeof(*r));
+               log_err("Got rc %d, expected %u\n", ret, (uint)sizeof(*r));
                return -1;
        }
 
@@ -429,18 +501,19 @@ int cros_ec_read_current_image(struct udevice *dev,
 }
 
 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) {
@@ -464,7 +537,7 @@ int cros_ec_read_hash(struct udevice *dev, uint hash_offset,
                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;
 
@@ -487,9 +560,13 @@ int cros_ec_read_hash(struct udevice *dev, uint hash_offset,
                       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__);
 
@@ -521,6 +598,25 @@ static int cros_ec_invalidate_hash(struct udevice *dev)
        return 0;
 }
 
+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;
@@ -533,18 +629,23 @@ int cros_ec_reboot(struct udevice *dev, enum ec_reboot_cmd cmd, uint8_t flags)
                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;
@@ -570,6 +671,36 @@ int cros_ec_info(struct udevice *dev, struct ec_response_mkbp_info *info)
        return 0;
 }
 
+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;
@@ -627,7 +758,6 @@ 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;
@@ -656,14 +786,14 @@ static int cros_ec_check_version(struct udevice *dev)
        /* Try sending a version 3 packet */
        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 */
        cdev->protocol_version = 2;
-       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;
 
        /*
@@ -679,18 +809,16 @@ static int cros_ec_check_version(struct udevice *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;
@@ -827,6 +955,9 @@ int cros_ec_flash_write(struct udevice *dev, const uint8_t *data,
        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.
@@ -850,6 +981,35 @@ int cros_ec_flash_write(struct udevice *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
@@ -934,20 +1094,35 @@ int cros_ec_flash_update_rw(struct udevice *dev, const uint8_t *image,
        return 0;
 }
 
+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)
+       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)
+                        &p, sizeof(uint32_t) + size, block, size);
+       if (len != size) {
+               log_err("Expected %d bytes, got %d\n", size, len);
                return -EIO;
+       }
 
        return 0;
 }
@@ -957,19 +1132,50 @@ 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)
+       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 ec_params_ldo_set params;
@@ -1144,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,
 };