| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Date: Tue, 23 Jun 2020 20:36:56 +0300
From: <alexandru.tachici@...log.com>
To: <linux-hwmon@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
<devicetree@...r.kernel.org>
CC: <robh+dt@...nel.org>, <linux@...ck-us.net>,
Alexandru Tachici <alexandru.tachici@...log.com>
Subject: [PATCH v4 4/7] hwmon: pmbus: adm1266: Add ioctl commands
From: Alexandru Tachici <alexandru.tachici@...log.com>
Add two ioctl commands for reading the current state
of the adm1266 sequencer and sending commands.
Signed-off-by: Alexandru Tachici <alexandru.tachici@...log.com>
---
Documentation/hwmon/adm1266.rst | 15 +++
.../userspace-api/ioctl/ioctl-number.rst | 1 +
drivers/hwmon/pmbus/adm1266.c | 97 +++++++++++++++++++
include/uapi/linux/adm1266.h | 16 +++
4 files changed, 129 insertions(+)
create mode 100644 include/uapi/linux/adm1266.h
diff --git a/Documentation/hwmon/adm1266.rst b/Documentation/hwmon/adm1266.rst
index 65662115750c..5dc05808db60 100644
--- a/Documentation/hwmon/adm1266.rst
+++ b/Documentation/hwmon/adm1266.rst
@@ -33,3 +33,18 @@ inX_min Minimum Voltage.
inX_max Maximum voltage.
inX_min_alarm Voltage low alarm.
inX_max_alarm Voltage high alarm.
+
+
+User API
+========
+
+ioctls
+------
+
+ADM1266_SET_GO_COMMAND:
+
+ Issue a GO_COMMAND to the device.
+
+ADM1266_GET_STATUS:
+
+ Returns state of the sequencer.
diff --git a/Documentation/userspace-api/ioctl/ioctl-number.rst b/Documentation/userspace-api/ioctl/ioctl-number.rst
index f759edafd938..db4d912e3d86 100644
--- a/Documentation/userspace-api/ioctl/ioctl-number.rst
+++ b/Documentation/userspace-api/ioctl/ioctl-number.rst
@@ -345,6 +345,7 @@ Code Seq# Include File Comments
0xCC 00-0F drivers/misc/ibmvmc.h pseries VMC driver
0xCD 01 linux/reiserfs_fs.h
0xCF 02 fs/cifs/ioctl.c
+0xD1 00-0F linux/adm1266.h
0xDB 00-0F drivers/char/mwave/mwavepub.h
0xDD 00-3F ZFCP device driver see drivers/s390/scsi/
<mailto:aherrman@...ibm.com>
diff --git a/drivers/hwmon/pmbus/adm1266.c b/drivers/hwmon/pmbus/adm1266.c
index 76bf2c78e737..0960ead8d96a 100644
--- a/drivers/hwmon/pmbus/adm1266.c
+++ b/drivers/hwmon/pmbus/adm1266.c
@@ -15,11 +15,16 @@
#include <linux/init.h>
#include <linux/kernel.h>
#include <linux/module.h>
+#include <linux/proc_fs.h>
#include <linux/slab.h>
+#include <linux/uaccess.h>
+#include <linux/adm1266.h>
#include "pmbus.h"
#define ADM1266_PDIO_CONFIG 0xD4
+#define ADM1266_GO_COMMAND 0xD8
+#define ADM1266_READ_STATE 0xD9
#define ADM1266_GPIO_CONFIG 0xE1
#define ADM1266_PDIO_STATUS 0xE9
#define ADM1266_GPIO_STATUS 0xEA
@@ -46,6 +51,7 @@ struct adm1266_data {
struct gpio_chip gc;
const char *gpio_names[ADM1266_GPIO_NR + ADM1266_PDIO_NR];
struct i2c_client *client;
+ struct mutex ioctl_mutex; /* lock ioctl access */
};
/* Different from Block Read as it sends data and waits for the slave to
@@ -311,6 +317,93 @@ static int adm1266_config_gpio(struct adm1266_data *data)
}
#endif
+static int adm1266_set_go_command_op(struct adm1266_data *data, u8 val)
+{
+ val = FIELD_GET(GENMASK(4, 0), val);
+
+ return i2c_smbus_write_word_data(data->client, ADM1266_GO_COMMAND, val);
+}
+
+static int adm1266_ioctl_unlocked(struct file *fp, unsigned int cmd,
+ unsigned long arg)
+{
+ int __user *argp = (int __user *)arg;
+ struct adm1266_data *data;
+ int val;
+ int ret;
+
+ data = fp->private_data;
+
+ if (!argp)
+ return -EINVAL;
+
+ switch (cmd) {
+ case ADM1266_SET_GO_COMMAND:
+ if (copy_from_user(&val, argp, sizeof(int)))
+ return -EFAULT;
+
+ return adm1266_set_go_command_op(data, val);
+ case ADM1266_GET_STATUS:
+ ret = i2c_smbus_read_word_data(data->client,
+ ADM1266_READ_STATE);
+
+ if (ret < 0)
+ return ret;
+
+ if (copy_to_user(argp, &ret, sizeof(int)))
+ return -EFAULT;
+
+ break;
+ default:
+ return -ENOTTY;
+ }
+
+ return 0;
+}
+
+static long adm1266_ioctl(struct file *fp, unsigned int cmd, unsigned long arg)
+{
+ struct adm1266_data *data;
+ long ret;
+
+ data = fp->private_data;
+
+ mutex_lock(&data->ioctl_mutex);
+ ret = adm1266_ioctl_unlocked(fp, cmd, arg);
+ mutex_unlock(&data->ioctl_mutex);
+
+ return ret;
+}
+
+static int adm1266_open(struct inode *inode, struct file *filp)
+{
+ filp->private_data = PDE_DATA(inode);
+
+ return 0;
+}
+
+static const struct proc_ops adm1266_proc_ops = {
+ .proc_open = adm1266_open,
+ .proc_ioctl = adm1266_ioctl,
+};
+
+static int adm1266_init_procfs(struct adm1266_data *data)
+{
+ struct proc_dir_entry *proc_dir;
+ u8 proc_fs_name[32];
+
+ mutex_init(&data->ioctl_mutex);
+
+ snprintf(proc_fs_name, 32, "adm1266-%x", data->client->addr);
+ proc_dir = proc_create_data(proc_fs_name, 0, NULL, &adm1266_proc_ops,
+ data);
+
+ if (!proc_dir)
+ return -EINVAL;
+
+ return 0;
+}
+
static int adm1266_probe(struct i2c_client *client,
const struct i2c_device_id *id)
{
@@ -333,6 +426,10 @@ static int adm1266_probe(struct i2c_client *client,
crc8_populate_msb(pmbus_crc_table, 0x7);
+ ret = adm1266_init_procfs(data);
+ if (ret < 0)
+ return ret;
+
info = &data->info;
info->pages = 17;
info->format[PSC_VOLTAGE_OUT] = linear;
diff --git a/include/uapi/linux/adm1266.h b/include/uapi/linux/adm1266.h
new file mode 100644
index 000000000000..0db3c1129293
--- /dev/null
+++ b/include/uapi/linux/adm1266.h
@@ -0,0 +1,16 @@
+/* SPDX-License-Identifier: GPL-2.0 */
+/*
+ * ADM1266 - Cascadable Super Sequencer with Margin
+ * Control and Fault Recording
+ *
+ * Copyright 2020 Analog Devices Inc.
+ */
+
+#ifndef _LINUX_ADM1266_H
+#define _LINUX_ADM1266_H
+
+/* ADM1266 ioctl commands */
+#define ADM1266_SET_GO_COMMAND _IOW(0xD1, 0x00, int)
+#define ADM1266_GET_STATUS _IOR(0xD1, 0x01, int)
+
+#endif
--
2.20.1
Powered by blists - more mailing lists