platform/x86: intel_scu_ipcutil: Convert to use new SCU IPC API

Convert the IPC util to use the new SCU IPC API where the SCU IPC
instance is passed to the functions.

Signed-off-by: Mika Westerberg <mika.westerberg@linux.intel.com>
Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: Lee Jones <lee.jones@linaro.org>

diff --git a/drivers/platform/x86/intel_scu_ipcutil.c b/drivers/platform/x86/intel_scu_ipcutil.c
index 8afe6fa06d7b8d45a1f9f68e5483f1e191f62e91..b7c10c15a3d633bf9b4c804f445935e2930c7d81 100644 (file)
--- a/drivers/platform/x86/intel_scu_ipcutil.c
+++ b/drivers/platform/x86/intel_scu_ipcutil.c
@@ -22,6 +22,9 @@
 
 static int major;
 
+struct intel_scu_ipc_dev *scu;
+static DEFINE_MUTEX(scu_lock);
+
 /* IOCTL commands */
 #define        INTE_SCU_IPC_REGISTER_READ      0
 #define INTE_SCU_IPC_REGISTER_WRITE    1
@@ -52,12 +55,12 @@ static int scu_reg_access(u32 cmd, struct scu_ipc_data  *data)
 
        switch (cmd) {
        case INTE_SCU_IPC_REGISTER_READ:
-               return intel_scu_ipc_readv(data->addr, data->data, count);
+               return intel_scu_ipc_dev_readv(scu, data->addr, data->data, count);
        case INTE_SCU_IPC_REGISTER_WRITE:
-               return intel_scu_ipc_writev(data->addr, data->data, count);
+               return intel_scu_ipc_dev_writev(scu, data->addr, data->data, count);
        case INTE_SCU_IPC_REGISTER_UPDATE:
-               return intel_scu_ipc_update_register(data->addr[0],
-                                                   data->data[0], data->mask);
+               return intel_scu_ipc_dev_update(scu, data->addr[0], data->data[0],
+                                               data->mask);
        default:
                return -ENOTTY;
        }
@@ -91,8 +94,40 @@ static long scu_ipc_ioctl(struct file *fp, unsigned int cmd,
        return 0;
 }
 
+static int scu_ipc_open(struct inode *inode, struct file *file)
+{
+       int ret = 0;
+
+       /* Only single open at the time */
+       mutex_lock(&scu_lock);
+       if (scu) {
+               ret = -EBUSY;
+               goto unlock;
+       }
+
+       scu = intel_scu_ipc_dev_get();
+       if (!scu)
+               ret = -ENODEV;
+
+unlock:
+       mutex_unlock(&scu_lock);
+       return ret;
+}
+
+static int scu_ipc_release(struct inode *inode, struct file *file)
+{
+       mutex_lock(&scu_lock);
+       intel_scu_ipc_dev_put(scu);
+       scu = NULL;
+       mutex_unlock(&scu_lock);
+
+       return 0;
+}
+
 static const struct file_operations scu_ipc_fops = {
        .unlocked_ioctl = scu_ipc_ioctl,
+       .open = scu_ipc_open,
+       .release = scu_ipc_release,
 };
 
 static int __init ipc_module_init(void)