Re: [PATCH v2 17/36] platform/x86: intel_scu_ipcutil: Convert to use new SCU IPC API

From: Andy Shevchenko
Date: Wed Jan 08 2020 - 11:45:40 EST


On Wed, Jan 08, 2020 at 02:41:42PM +0300, Mika Westerberg wrote:
> Convert the IPC util to use the new SCU IPC API where the SCU IPC
> instance is passed to the functions.
>

Reviewed-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>

> Signed-off-by: Mika Westerberg <mika.westerberg@xxxxxxxxxxxxxxx>
> ---
> drivers/platform/x86/intel_scu_ipcutil.c | 43 +++++++++++++++++++++---
> 1 file changed, 39 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/platform/x86/intel_scu_ipcutil.c b/drivers/platform/x86/intel_scu_ipcutil.c
> index 8afe6fa06d7b..b7c10c15a3d6 100644
> --- 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)
> --
> 2.24.1
>

--
With Best Regards,
Andy Shevchenko