From: <alexandru.tachici@analog.com>
To: <linux-hwmon@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
<devicetree@vger.kernel.org>
Cc: <robh+dt@kernel.org>, <linux@roeck-us.net>,
Alexandru Tachici <alexandru.tachici@analog.com>
Subject: [PATCH v4 4/7] hwmon: pmbus: adm1266: Add ioctl commands
Date: Tue, 23 Jun 2020 20:36:56 +0300 [thread overview]
Message-ID: <20200623173659.41358-5-alexandru.tachici@analog.com> (raw)
In-Reply-To: <20200623173659.41358-1-alexandru.tachici@analog.com>
From: Alexandru Tachici <alexandru.tachici@analog.com>
Add two ioctl commands for reading the current state
of the adm1266 sequencer and sending commands.
Signed-off-by: Alexandru Tachici <alexandru.tachici@analog.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@de.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
next prev parent reply other threads:[~2020-06-23 17:44 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-06-23 17:36 [PATCH v4 0/7] hwmon: pmbus: adm1266: add support alexandru.tachici
2020-06-23 17:36 ` [PATCH v4 1/7] " alexandru.tachici
2020-06-24 21:18 ` Guenter Roeck
2020-06-23 17:36 ` [PATCH v4 2/7] hwmon: pmbus: adm1266: Add Block process call alexandru.tachici
2020-06-24 21:28 ` Guenter Roeck
2020-06-23 17:36 ` [PATCH v4 3/7] hwmon: pmbus: adm1266: Add support for GPIOs alexandru.tachici
2020-06-24 21:35 ` Guenter Roeck
2020-06-23 17:36 ` alexandru.tachici [this message]
2020-06-23 22:43 ` [PATCH v4 4/7] hwmon: pmbus: adm1266: Add ioctl commands kernel test robot
2020-06-24 21:40 ` Guenter Roeck
2020-06-23 17:36 ` [PATCH v4 5/7] hwmon: pmbus: adm1266: read blackbox alexandru.tachici
2020-06-24 21:51 ` Guenter Roeck
2020-06-23 17:36 ` [PATCH v4 6/7] hwmon: pmbus: adm1266: debugfs for blackbox info alexandru.tachici
2020-06-23 17:36 ` [PATCH v4 7/7] dt-bindings: hwmon: Add bindings for ADM1266 alexandru.tachici
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20200623173659.41358-5-alexandru.tachici@analog.com \
--to=alexandru.tachici@analog.com \
--cc=devicetree@vger.kernel.org \
--cc=linux-hwmon@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux@roeck-us.net \
--cc=robh+dt@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).