mfd: cros_ec: Switch to use the new cros-ec-chardev driver

With the purpose of remove the things that far extends the bounds of
what a MFD was designed to do, instantiate the new platform misc
cros-ec-chardev driver and get rid of all the unneeded code. After this
patch the misc chardev driver is a sub-device of the MFD, and all the
new file operations should be implemented there.

Signed-off-by: Enric Balletbo i Serra <enric.balletbo@collabora.com>
Acked-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Reviewed-by: Gwendal Grignou <gwendal@chromium.org>
Tested-by: Gwendal Grignou <gwendal@chromium.org>
Signed-off-by: Lee Jones <lee.jones@linaro.org>

2 files changed, 6 insertions, 216 deletions

diff --git a/drivers/mfd/cros_ec_dev.c b/drivers/mfd/cros_ec_dev.c
index 4c96445b1bf5..0c1c0ce3453e 100644
--- a/drivers/mfd/cros_ec_dev.c
+++ b/drivers/mfd/cros_ec_dev.c
@@ -5,76 +5,23 @@
  * Copyright (C) 2014 Google, Inc.
  */
 
-#include <linux/fs.h>
 #include <linux/mfd/core.h>
 #include <linux/mfd/cros_ec.h>
 #include <linux/mfd/cros_ec_commands.h>
 #include <linux/module.h>
 #include <linux/mod_devicetable.h>
-#include <linux/platform_data/cros_ec_chardev.h>
 #include <linux/of_platform.h>
 #include <linux/platform_device.h>
-#include <linux/pm.h>
+#include <linux/platform_data/cros_ec_chardev.h>
 #include <linux/slab.h>
-#include <linux/uaccess.h>
-
 
 #define DRV_NAME "cros-ec-dev"
 
-/* Device variables */
-#define CROS_MAX_DEV 128
-static int ec_major;
-
 static struct class cros_class = {
         .owner          = THIS_MODULE,
         .name           = "chromeos",
 };
 
-/* Basic communication */
-static int ec_get_version(struct cros_ec_dev *ec, char *str, int maxlen)
-{
-        struct ec_response_get_version *resp;
-        static const char * const current_image_name[] = {
-                "unknown", "read-only", "read-write", "invalid",
-        };
-        struct cros_ec_command *msg;
-        int ret;
-
-        msg = kmalloc(sizeof(*msg) + sizeof(*resp), GFP_KERNEL);
-        if (!msg)
-                return -ENOMEM;
-
-        msg->version = 0;
-        msg->command = EC_CMD_GET_VERSION + ec->cmd_offset;
-        msg->insize = sizeof(*resp);
-        msg->outsize = 0;
-
-        ret = cros_ec_cmd_xfer(ec->ec_dev, msg);
-        if (ret < 0)
-                goto exit;
-
-        if (msg->result != EC_RES_SUCCESS) {
-                snprintf(str, maxlen,
-                         "%s\nUnknown EC version: EC returned %d\n",
-                         CROS_EC_DEV_VERSION, msg->result);
-                ret = -EINVAL;
-                goto exit;
-        }
-
-        resp = (struct ec_response_get_version *)msg->data;
-        if (resp->current_image >= ARRAY_SIZE(current_image_name))
-                resp->current_image = 3; /* invalid */
-
-        snprintf(str, maxlen, "%s\n%s\n%s\n%s\n", CROS_EC_DEV_VERSION,
-                 resp->version_string_ro, resp->version_string_rw,
-                 current_image_name[resp->current_image]);
-
-        ret = 0;
-exit:
-        kfree(msg);
-        return ret;
-}
-
 static int cros_ec_check_features(struct cros_ec_dev *ec, int feature)
 {
         struct cros_ec_command *msg;
@@ -110,142 +57,6 @@ static int cros_ec_check_features(struct cros_ec_dev *ec, int feature)
         return ec->features[feature / 32] & EC_FEATURE_MASK_0(feature);
 }
 
-/* Device file ops */
-static int ec_device_open(struct inode *inode, struct file *filp)
-{
-        struct cros_ec_dev *ec = container_of(inode->i_cdev,
-                                              struct cros_ec_dev, cdev);
-        filp->private_data = ec;
-        nonseekable_open(inode, filp);
-        return 0;
-}
-
-static int ec_device_release(struct inode *inode, struct file *filp)
-{
-        return 0;
-}
-
-static ssize_t ec_device_read(struct file *filp, char __user *buffer,
-                              size_t length, loff_t *offset)
-{
-        struct cros_ec_dev *ec = filp->private_data;
-        char msg[sizeof(struct ec_response_get_version) +
-                 sizeof(CROS_EC_DEV_VERSION)];
-        size_t count;
-        int ret;
-
-        if (*offset != 0)
-                return 0;
-
-        ret = ec_get_version(ec, msg, sizeof(msg));
-        if (ret)
-                return ret;
-
-        count = min(length, strlen(msg));
-
-        if (copy_to_user(buffer, msg, count))
-                return -EFAULT;
-
-        *offset = count;
-        return count;
-}
-
-/* Ioctls */
-static long ec_device_ioctl_xcmd(struct cros_ec_dev *ec, void __user *arg)
-{
-        long ret;
-        struct cros_ec_command u_cmd;
-        struct cros_ec_command *s_cmd;
-
-        if (copy_from_user(&u_cmd, arg, sizeof(u_cmd)))
-                return -EFAULT;
-
-        if ((u_cmd.outsize > EC_MAX_MSG_BYTES) ||
-            (u_cmd.insize > EC_MAX_MSG_BYTES))
-                return -EINVAL;
-
-        s_cmd = kmalloc(sizeof(*s_cmd) + max(u_cmd.outsize, u_cmd.insize),
-                        GFP_KERNEL);
-        if (!s_cmd)
-                return -ENOMEM;
-
-        if (copy_from_user(s_cmd, arg, sizeof(*s_cmd) + u_cmd.outsize)) {
-                ret = -EFAULT;
-                goto exit;
-        }
-
-        if (u_cmd.outsize != s_cmd->outsize ||
-            u_cmd.insize != s_cmd->insize) {
-                ret = -EINVAL;
-                goto exit;
-        }
-
-        s_cmd->command += ec->cmd_offset;
-        ret = cros_ec_cmd_xfer(ec->ec_dev, s_cmd);
-        /* Only copy data to userland if data was received. */
-        if (ret < 0)
-                goto exit;
-
-        if (copy_to_user(arg, s_cmd, sizeof(*s_cmd) + s_cmd->insize))
-                ret = -EFAULT;
-exit:
-        kfree(s_cmd);
-        return ret;
-}
-
-static long ec_device_ioctl_readmem(struct cros_ec_dev *ec, void __user *arg)
-{
-        struct cros_ec_device *ec_dev = ec->ec_dev;
-        struct cros_ec_readmem s_mem = { };
-        long num;
-
-        /* Not every platform supports direct reads */
-        if (!ec_dev->cmd_readmem)
-                return -ENOTTY;
-
-        if (copy_from_user(&s_mem, arg, sizeof(s_mem)))
-                return -EFAULT;
-
-        num = ec_dev->cmd_readmem(ec_dev, s_mem.offset, s_mem.bytes,
-                                  s_mem.buffer);
-        if (num <= 0)
-                return num;
-
-        if (copy_to_user((void __user *)arg, &s_mem, sizeof(s_mem)))
-                return -EFAULT;
-
-        return num;
-}
-
-static long ec_device_ioctl(struct file *filp, unsigned int cmd,
-                            unsigned long arg)
-{
-        struct cros_ec_dev *ec = filp->private_data;
-
-        if (_IOC_TYPE(cmd) != CROS_EC_DEV_IOC)
-                return -ENOTTY;
-
-        switch (cmd) {
-        case CROS_EC_DEV_IOCXCMD:
-                return ec_device_ioctl_xcmd(ec, (void __user *)arg);