* [PATCH v2 4/6] Add Advantech iManager I2C driver
@ 2016-01-10 9:11 ` richard.dorsch
2016-01-10 10:34 ` kbuild test robot
0 siblings, 1 reply; 43+ messages in thread
From: richard.dorsch @ 2016-01-10 9:11 UTC (permalink / raw)
To: linux-kernel
Cc: lm-sensors, linux-i2c, linux-watchdog, linux-gpio, lee.jones,
jdelvare, linux, wim, jo.sunga, Richard Vidal-Dorsch
From: Richard Vidal-Dorsch <richard.dorsch@gmail.com>
Signed-off-by: Richard Vidal-Dorsch <richard.dorsch@gmail.com>
---
Documentation/i2c/busses/i2c-imanager | 48 ++++
drivers/i2c/busses/Kconfig | 11 +
drivers/i2c/busses/Makefile | 2 +
drivers/i2c/busses/imanager-ec-i2c.c | 466 ++++++++++++++++++++++++++++++++++
drivers/i2c/busses/imanager-i2c.c | 240 +++++++++++++++++
include/linux/mfd/imanager/i2c.h | 55 ++++
6 files changed, 822 insertions(+)
create mode 100644 Documentation/i2c/busses/i2c-imanager
create mode 100644 drivers/i2c/busses/imanager-ec-i2c.c
create mode 100644 drivers/i2c/busses/imanager-i2c.c
create mode 100644 include/linux/mfd/imanager/i2c.h
diff --git a/Documentation/i2c/busses/i2c-imanager b/Documentation/i2c/busses/i2c-imanager
new file mode 100644
index 0000000..d149fbf
--- /dev/null
+++ b/Documentation/i2c/busses/i2c-imanager
@@ -0,0 +1,48 @@
+Kernel driver imanager_i2c
+==========================
+
+This platform driver provides support for iManager I2C/SMBus.
+
+This driver depends on imanager (mfd).
+
+Module Parameters
+-----------------
+
+* bus_frequency (unsigned short)
+Set desired bus frequency. Valid values (kHz) are:
+ 50 Slow
+ 100 Standard (default)
+ 400 Fast
+
+
+Description
+-----------
+
+The Advantech iManager provides up to four SMBus controllers. One of them
+is configured for I2C compatibility.
+
+
+Process Call Support
+--------------------
+
+Not supported.
+
+
+I2C Block Read Support
+----------------------
+
+I2C block read is supported.
+
+
+SMBus 2.0 Support
+-----------------
+
+Several SMBus 2.0 features are supported.
+No PEC support.
+
+
+Interrupt Support
+-----------------
+
+No interrupt support available
+
diff --git a/drivers/i2c/busses/Kconfig b/drivers/i2c/busses/Kconfig
index 7b0aa82..4c401a4 100644
--- a/drivers/i2c/busses/Kconfig
+++ b/drivers/i2c/busses/Kconfig
@@ -42,6 +42,17 @@ config I2C_ALI15X3
This driver can also be built as a module. If so, the module
will be called i2c-ali15x3.
+config I2C_IMANAGER
+ tristate "Advantech iManager I2C Interface"
+ depends on MFD_IMANAGER
+ help
+ This enables support for Advantech iManager I2C of some
+ Advantech SOM, MIO, AIMB, and PCM modules/boards.
+ Requires mfd-core and imanager-core to function properly.
+
+ This driver can also be built as a module. If so, the module
+ will be called i2c-imanager.
+
config I2C_AMD756
tristate "AMD 756/766/768/8111 and nVidia nForce"
depends on PCI
diff --git a/drivers/i2c/busses/Makefile b/drivers/i2c/busses/Makefile
index 37f2819..da76404 100644
--- a/drivers/i2c/busses/Makefile
+++ b/drivers/i2c/busses/Makefile
@@ -15,6 +15,8 @@ obj-$(CONFIG_I2C_AMD8111) += i2c-amd8111.o
obj-$(CONFIG_I2C_I801) += i2c-i801.o
obj-$(CONFIG_I2C_ISCH) += i2c-isch.o
obj-$(CONFIG_I2C_ISMT) += i2c-ismt.o
+i2c-imanager-objs := imanager-i2c.o imanager-ec-i2c.o
+obj-$(CONFIG_I2C_IMANAGER) += i2c-imanager.o
obj-$(CONFIG_I2C_NFORCE2) += i2c-nforce2.o
obj-$(CONFIG_I2C_NFORCE2_S4985) += i2c-nforce2-s4985.o
obj-$(CONFIG_I2C_PIIX4) += i2c-piix4.o
diff --git a/drivers/i2c/busses/imanager-ec-i2c.c b/drivers/i2c/busses/imanager-ec-i2c.c
new file mode 100644
index 0000000..f8839dc
--- /dev/null
+++ b/drivers/i2c/busses/imanager-ec-i2c.c
@@ -0,0 +1,466 @@
+/*
+ * Advantech iManager I2C bus core
+ *
+ * Copyright (C) 2016 Advantech Co., Ltd., Irvine, CA, USA
+ * Author: Richard Vidal-Dorsch <richard.dorsch@advantech.com>
+ *
+ * This program is free software; you can redistribute it and/or modify it
+ * under the terms of the GNU General Public License as published by the
+ * Free Software Foundation; either version 2 of the License, or (at your
+ * option) any later version.
+ */
+
+#include <linux/types.h>
+#include <linux/errno.h>
+#include <linux/bug.h>
+#include <linux/io.h>
+#include <linux/delay.h>
+#include <linux/string.h>
+#include <linux/byteorder/generic.h>
+#include <linux/mfd/imanager/ec.h>
+#include <linux/mfd/imanager/i2c.h>
+
+#define I2C_SMBUS_BLOCK_SIZE 32
+
+#define EVAL_WR_SIZE(x) \
+ (x < I2C_SMBUS_BLOCK_SIZE ? x : I2C_SMBUS_BLOCK_SIZE - 1)
+#define EVAL_RD_SIZE(x) \
+ (x && (x <= I2C_SMBUS_BLOCK_SIZE) ? x : I2C_SMBUS_BLOCK_SIZE)
+
+#define EC_HWRAM_OFFSET_STATUS 0UL
+
+#define I2C_ERR_PROTO 0x19UL
+#define I2C_ERR_TIMEOUT 0x18UL
+#define I2C_ERR_ACCESS 0x17UL
+#define I2C_ERR_UNKNOWN 0x13UL
+#define I2C_ERR_ADDR_NACK 0x10UL
+
+#define EC_SMB_DID(N) (0x28 + N)
+
+struct ec_i2c_status {
+ u32 error : 7;
+ u32 complete : 1;
+};
+
+static const struct imanager_i2c_device *i2c;
+
+static int i2c_core_eval_status(u8 _status)
+{
+ struct ec_i2c_status *status = (struct ec_i2c_status *)&_status;
+ int err = 0;
+
+ switch (status->error) {
+ case 0:
+ break;
+ case I2C_ERR_ADDR_NACK:
+ err = -ENODEV;
+ break;
+ case I2C_ERR_ACCESS:
+ case I2C_ERR_UNKNOWN:
+ err = -EAGAIN;
+ break;
+ case I2C_ERR_TIMEOUT:
+ err = -ETIME;
+ break;
+ case I2C_ERR_PROTO:
+ err = -EPROTO;
+ break;
+ default:
+ pr_err("Undefined status code 0x%02X\n", status->error);
+ err = -EIO;
+ break;
+ }
+
+ return err;
+}
+
+static inline int i2c_send_message(u8 cmd, u8 param, struct ec_message *msg)
+{
+ int err;
+
+ err = imanager_msg_write(cmd, param, msg);
+ if (err)
+ return i2c_core_eval_status(err);
+
+ return 0;
+}
+
+static int i2c_core_blk_wr_rw_combined(u8 proto, struct ec_message *msg)
+{
+ int err;
+
+ if (WARN_ON(!msg))
+ return -EINVAL;
+
+ err = imanager_wait_proc_complete(EC_HWRAM_OFFSET_STATUS, 0);
+ if (err)
+ return err;
+
+ err = i2c_send_message(proto, i2c->i2coem->did, msg);
+ if (err)
+ return err;
+
+ if (msg->rlen) {
+ if (msg->rlen == 1)
+ return msg->u.data[0];
+ else if (msg->rlen == 2)
+ return (msg->u.data[1] << 8) | msg->u.data[0];
+ else
+ return msg->rlen;
+ }
+
+ return 0;
+}
+
+/* Write-Read and Read-Write wrappers */
+static inline int i2c_core_wr_combined(struct ec_message *msg)
+{
+ return i2c_core_blk_wr_rw_combined(EC_CMD_I2C_WR, msg);
+}
+
+static inline int i2c_core_rw_combined(struct ec_message *msg)
+{
+ return i2c_core_blk_wr_rw_combined(EC_CMD_I2C_RW, msg);
+}
+
+/*
+ * iManager I2C core API
+ */
+int i2c_core_write_quick(u16 addr)
+{
+ struct ec_message msg = {
+ .rlen = 0,
+ .wlen = sizeof(struct ec_message_header),
+ .u = {
+ .smb.hdr = {
+ .addr_low = LOADDR16(addr),
+ .addr_high = HIADDR16(addr),
+ .rlen = 0,
+ .wlen = 1,
+ },
+ },
+ };
+
+ return i2c_core_wr_combined(&msg);
+}
+
+int i2c_core_read_byte(u16 addr)
+{
+ struct ec_message msg = {
+ .rlen = 1,
+ .wlen = sizeof(struct ec_message_header),
+ .u = {
+ .smb.hdr = {
+ .addr_low = LOADDR16(addr),
+ .addr_high = HIADDR16(addr),
+ .rlen = 1,
+ .wlen = 0,
+ },
+ },
+ };
+
+ return i2c_core_rw_combined(&msg);
+}
+
+int i2c_core_write_byte(u16 addr, u8 cmd)
+{
+ struct ec_message msg = {
+ .rlen = 1,
+ .wlen = sizeof(struct ec_message_header),
+ .u = {
+ .smb.hdr = {
+ .addr_low = LOADDR16(addr),
+ .addr_high = HIADDR16(addr),
+ .rlen = 1,
+ .wlen = 1,
+ .cmd = cmd,
+ },
+ },
+ };
+
+ return i2c_core_wr_combined(&msg);
+}
+
+int i2c_core_read_byte_data(u16 addr, u8 cmd)
+{
+ struct ec_message msg = {
+ .rlen = 1,
+ .wlen = sizeof(struct ec_message_header),
+ .u = {
+ .smb.hdr = {
+ .addr_low = LOADDR16(addr),
+ .addr_high = HIADDR16(addr),
+ .rlen = 1,
+ .wlen = 1,
+ .cmd = cmd,
+ },
+ },
+ };
+
+ return i2c_core_wr_combined(&msg);
+}
+
+int i2c_core_write_byte_data(u16 addr, u8 cmd, u8 value)
+{
+ struct ec_message msg = {
+ .rlen = 1,
+ .wlen = sizeof(struct ec_message_header) + 1,
+ .u = {
+ .smb.hdr = {
+ .addr_low = LOADDR16(addr),
+ .addr_high = HIADDR16(addr),
+ .rlen = 0,
+ .wlen = 2,
+ .cmd = cmd,
+ },
+ .smb.data[0] = value,
+ },
+ };
+
+ return i2c_core_wr_combined(&msg);
+}
+
+int i2c_core_read_word_data(u16 addr, u8 cmd)
+{
+ struct ec_message msg = {
+ .rlen = 2,
+ .wlen = sizeof(struct ec_message_header),
+ .u = {
+ .smb.hdr = {
+ .addr_low = LOADDR16(addr),
+ .addr_high = HIADDR16(addr),
+ .rlen = 2,
+ .wlen = 1,
+ .cmd = cmd,
+ },
+ },
+ };
+
+ return i2c_core_wr_combined(&msg);
+}
+
+int i2c_core_write_word_data(u16 addr, u8 cmd, u16 value)
+{
+ struct ec_message msg = {
+ .rlen = 1,
+ .wlen = sizeof(struct ec_message_header) + 2,
+ .u = {
+ .smb.hdr = {
+ .addr_low = LOADDR16(addr),
+ .addr_high = HIADDR16(addr),
+ .rlen = 0,
+ .wlen = 3,
+ .cmd = cmd,
+ },
+ .smb.data[0] = LOBYTE16(value),
+ .smb.data[1] = HIBYTE16(value),
+ },
+ };
+
+ return i2c_core_wr_combined(&msg);
+}
+
+int i2c_core_write_block_data(u16 addr, u8 cmd, u8 *buf)
+{
+ int i;
+ struct ec_message msg = {
+ .rlen = 1,
+ .wlen = sizeof(struct ec_message_header) +
+ EVAL_WR_SIZE(buf[0]),
+ .u = {
+ .smb.hdr = {
+ .addr_low = LOADDR16(addr),
+ .addr_high = HIADDR16(addr),
+ .rlen = 0,
+ .wlen = 1 + EVAL_WR_SIZE(buf[0]),
+ .cmd = cmd,
+ },
+ },
+ };
+
+ if ((buf[0] == 0) || (buf[0] >= I2C_MAX_WRITE_BYTES)) {
+ pr_err("Invalid I2C write length %d\n", buf[0]);
+ return -EINVAL;
+ }
+
+ for (i = 0; i < EVAL_WR_SIZE(buf[0]); i++)
+ msg.u.data[i + sizeof(struct ec_message_header)] = buf[i + 1];
+
+ return i2c_core_wr_combined(&msg);
+}
+
+int i2c_core_read_block_data(u16 addr, u8 cmd, u8 *buf)
+{
+ int i;
+ int ret;
+ struct ec_message msg = {
+ .rlen = EVAL_RD_SIZE(buf[0]),
+ .wlen = sizeof(struct ec_message_header),
+ .u = {
+ .smb.hdr = {
+ .addr_low = LOADDR16(addr),
+ .addr_high = HIADDR16(addr),
+ .rlen = EVAL_RD_SIZE(buf[0]),
+ .wlen = 1,
+ .cmd = cmd,
+ },
+ },
+ };
+
+ /*
+ * If buf[0] == 0 EC will read I2C_MAX_READ_BYTES
+ */
+ ret = i2c_core_wr_combined(&msg);
+ if (ret < 0) {
+ pr_err("I2C transaction failed\n");
+ return ret;
+ }
+
+ buf[0] = ret;
+ for (i = 0; i < ret; i++)
+ buf[i + 1] = msg.u.data[i];
+
+ return 0;
+}
+
+int i2c_core_read_i2c_block_data(u16 addr, u8 cmd, u8 *buf)
+{
+ int i;
+ int ret;
+ struct ec_message msg = {
+ .rlen = EVAL_RD_SIZE(buf[0]),
+ .wlen = sizeof(struct ec_message_header),
+ .u = {
+ .smb.hdr = {
+ .addr_low = LOADDR16(addr),
+ .addr_high = HIADDR16(addr),
+ .rlen = EVAL_RD_SIZE(buf[0]),
+ .wlen = 1,
+ .cmd = cmd,
+ },
+ },
+ };
+
+ if ((buf[0] == 0) || (buf[0] > I2C_MAX_READ_BYTES)) {
+ pr_err("Invalid I2C read length\n");
+ return -EINVAL;
+ }
+
+ ret = i2c_core_wr_combined(&msg);
+ if (ret < 0) {
+ pr_err("I2C transaction failed\n");
+ return ret;
+ }
+
+ buf[0] = ret;
+ for (i = 0; i < ret; i++)
+ buf[i + 1] = msg.u.data[i];
+
+ return 0;
+}
+
+int i2c_core_write_i2c_block_data(u16 addr, u8 cmd, u8 *buf)
+{
+ if (WARN_ON(!buf))
+ return -EINVAL;
+
+ return i2c_core_write_block_data(addr, cmd, buf);
+}
+
+int i2c_core_smb_get_freq(u32 bus_id)
+{
+ int ret = 0, f;
+ int freq_id, freq;
+
+ if (WARN_ON(bus_id > I2C_OEM_1))
+ return -EINVAL;
+
+ switch (i2c->ecdev->id) {
+ case IT8518:
+ case IT8528:
+ ret = imanager_read_word(EC_CMD_SMB_FREQ_RD,
+ EC_SMB_DID(bus_id));
+ if (ret < 0) {
+ pr_err("Failed to get bus frequency\n");
+ return ret;
+ }
+
+ freq_id = HIBYTE16(ret);
+ f = LOBYTE16(ret);
+ switch (freq_id) {
+ case 0:
+ freq = f;
+ break;
+ case 1:
+ freq = 50;
+ break;
+ case 2:
+ freq = 100;
+ break;
+ case 3:
+ freq = 400;
+ break;
+ default:
+ return -EINVAL;
+ }
+ break;
+ default:
+ pr_err("EC version not supported!\n");
+ return -ENODEV;
+ }
+
+ return freq;
+}
+
+int i2c_core_smb_set_freq(u32 bus_id, u32 freq)
+{
+ int err;
+ u16 val;
+
+ if (WARN_ON(bus_id > I2C_OEM_1))
+ return -EINVAL;
+
+ switch (i2c->ecdev->id) {
+ case IT8518:
+ case IT8528:
+ switch (freq) {
+ case 50:
+ val = 0x0100;
+ break;
+ case 100:
+ val = 0x0200;
+ break;
+ case 400:
+ val = 0x0300;
+ break;
+ default:
+ if (freq < 50)
+ val = freq;
+ else
+ return -EINVAL;
+ }
+
+ err = imanager_write_word(EC_CMD_SMB_FREQ_WR,
+ EC_SMB_DID(bus_id), val);
+ if (err) {
+ pr_err("Failed to set I2C bus frequency\n");
+ return err;
+ }
+ break;
+ default:
+ pr_err("EC version not supported!\n");
+ return -ENODEV;
+ }
+
+ return 0;
+}
+
+int i2c_core_init(void)
+{
+ i2c = imanager_get_i2c_device();
+ if (!i2c)
+ return -ENODEV;
+
+ return 0;
+}
+
diff --git a/drivers/i2c/busses/imanager-i2c.c b/drivers/i2c/busses/imanager-i2c.c
new file mode 100644
index 0000000..bbfd453
--- /dev/null
+++ b/drivers/i2c/busses/imanager-i2c.c
@@ -0,0 +1,240 @@
+/*
+ * Advantech iManager I2C bus driver
+ *
+ * Copyright (C) 2016 Advantech Co., Ltd., Irvine, CA, USA
+ * Author: Richard Vidal-Dorsch <richard.dorsch@advantech.com>
+ *
+ * This program is free software; you can redistribute it and/or modify it
+ * under the terms of the GNU General Public License as published by the
+ * Free Software Foundation; either version 2 of the License, or (at your
+ * option) any later version.
+ */
+
+#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
+
+#include <linux/kernel.h>
+#include <linux/module.h>
+#include <linux/slab.h>
+#include <linux/platform_device.h>
+#include <linux/i2c.h>
+#include <linux/mfd/imanager/core.h>
+#include <linux/mfd/imanager/i2c.h>
+
+static uint bus_frequency = 100;
+module_param(bus_frequency, uint, 0);
+MODULE_PARM_DESC(bus_frequency,
+ "I2C bus frequency [50, 100, 400]kHz (defaults to 100kHz)");
+
+struct imanager_i2c_data {
+ struct imanager_device_data *idev;
+ struct i2c_adapter adapter;
+};
+
+static int imanager_smb_access(struct i2c_adapter *adap, u16 addr,
+ unsigned short flags, char read_write, u8 command,
+ int size, union i2c_smbus_data *smb_data)
+{
+ struct imanager_i2c_data *data = i2c_get_adapdata(adap);
+ struct device *dev = data->adapter.dev.parent;
+ int ret = 0;
+ int val = 0;
+
+ if (!data)
+ return -ENODEV;
+
+ addr <<= 1;
+
+ mutex_lock(&data->idev->lock);
+
+ switch (size) {
+ case I2C_SMBUS_QUICK:
+ ret = i2c_core_write_quick(addr);
+ break;
+ case I2C_SMBUS_BYTE:
+ if (read_write == I2C_SMBUS_WRITE) /* NOT tested */
+ val = i2c_core_write_byte(addr, command);
+ else
+ val = i2c_core_read_byte(addr);
+
+ if (val < 0)
+ ret = val;
+ else
+ smb_data->byte = val;
+ break;
+ case I2C_SMBUS_BYTE_DATA:
+ if (read_write == I2C_SMBUS_WRITE)
+ val = i2c_core_write_byte_data(addr, command,
+ smb_data->byte);
+ else
+ val = i2c_core_read_byte_data(addr, command);
+
+ if (val < 0)
+ ret = val;
+ else
+ smb_data->byte = val;
+ break;
+ case I2C_SMBUS_WORD_DATA:
+ if (read_write == I2C_SMBUS_WRITE)
+ val = i2c_core_write_word_data(addr, command,
+ smb_data->word);
+ else
+ val = i2c_core_read_word_data(addr, command);
+
+ if (val < 0)
+ ret = val;
+ else
+ smb_data->word = val;
+ break;
+ case I2C_SMBUS_BLOCK_DATA:
+ if (read_write == I2C_SMBUS_WRITE)
+ ret = i2c_core_write_block_data(addr, command,
+ smb_data->block);
+ else
+ ret = i2c_core_read_block_data(addr, command,
+ smb_data->block);
+ break;
+ case I2C_SMBUS_I2C_BLOCK_DATA:
+ if (read_write == I2C_SMBUS_WRITE)
+ ret = i2c_core_write_i2c_block_data(addr, command,
+ smb_data->block);
+ else
+ ret = i2c_core_read_i2c_block_data(addr, command,
+ smb_data->block);
+ break;
+ default:
+ dev_err(dev, "Unsupported SMB transaction %d\n", size);
+ ret = -EOPNOTSUPP;
+ }
+
+ mutex_unlock(&data->idev->lock);
+
+ return ret;
+}
+
+static int imanager_i2c_access(struct i2c_adapter *adap, struct i2c_msg *msg,
+ int num)
+{
+ struct imanager_i2c_data *data = i2c_get_adapdata(adap);
+ struct device *dev = data->adapter.dev.parent;
+
+ /*
+ * To be implemented
+ */
+
+ dev_info(dev, "i2c_access() is not yet implemented. msg=%p, num=%d\n",
+ msg, num);
+
+ return 0;
+}
+
+static u32 imanager_smb_i2c_func(struct i2c_adapter *adapter)
+{
+ return I2C_FUNC_SMBUS_QUICK | I2C_FUNC_SMBUS_BYTE |
+ I2C_FUNC_SMBUS_BYTE_DATA | I2C_FUNC_SMBUS_WORD_DATA |
+ I2C_FUNC_SMBUS_BLOCK_DATA |
+ I2C_FUNC_SMBUS_I2C_BLOCK | I2C_FUNC_I2C;
+}
+
+static const struct i2c_algorithm imanager_algorithm = {
+ .smbus_xfer = imanager_smb_access,
+ .master_xfer = imanager_i2c_access,
+ .functionality = imanager_smb_i2c_func,
+};
+
+static int imanager_i2c_probe(struct platform_device *pdev)
+{
+ struct device *dev = &pdev->dev;
+ struct imanager_device_data *idev = dev_get_drvdata(dev->parent);
+ struct imanager_i2c_data *i2c;
+ int ret;
+
+ if (!idev) {
+ dev_err(dev, "Invalid platform data\n");
+ return -EINVAL;
+ }
+
+ ret = i2c_core_init();
+ if (ret) {
+ dev_err(dev, "Failed to initialize I2C core\n");
+ return -EIO;
+ }
+
+ if (bus_frequency > 100)
+ bus_frequency = 400;
+ else if (bus_frequency < 50)
+ bus_frequency = 50;
+ else
+ bus_frequency = 100;
+
+ ret = i2c_core_smb_set_freq(I2C_OEM_1, bus_frequency);
+ if (ret < 0) {
+ dev_err(dev, "Failed to set I2C bus frequency to %d kHz\n",
+ bus_frequency);
+ return ret;
+ }
+
+ ret = i2c_core_smb_get_freq(I2C_OEM_1);
+ if (ret < 0) {
+ dev_err(dev, "Failed to get I2C bus frequency\n");
+ return ret;
+ }
+ bus_frequency = ret;
+ dev_info(dev, "Bus frequency: %d kHz\n", bus_frequency);
+
+ i2c = devm_kzalloc(dev, sizeof(struct imanager_i2c_data), GFP_KERNEL);
+ if (!i2c)
+ return -ENOMEM;
+
+ platform_set_drvdata(pdev, i2c);
+ i2c_set_adapdata(&i2c->adapter, i2c);
+
+ i2c->idev = idev;
+
+ i2c->adapter.owner = THIS_MODULE;
+ i2c->adapter.class = I2C_CLASS_HWMON | I2C_CLASS_SPD;
+ i2c->adapter.algo = &imanager_algorithm;
+
+ /* set up the sysfs linkage to our parent device */
+ i2c->adapter.dev.parent = dev;
+
+ /* Retry up to 3 times on lost arbitration */
+ i2c->adapter.retries = 3;
+
+ snprintf(i2c->adapter.name, sizeof(i2c->adapter.name),
+ "iManager I2C driver");
+
+ ret = i2c_add_adapter(&i2c->adapter);
+ if (ret) {
+ dev_err(dev, "Failed to add SMBus adapter\n");
+ return ret;
+ }
+
+ return 0;
+}
+
+static int imanager_i2c_remove(struct platform_device *pdev)
+{
+ struct device *dev = &pdev->dev;
+ struct imanager_i2c_data *i2c = dev_get_drvdata(dev);
+
+ i2c_del_adapter(&i2c->adapter);
+ i2c_set_adapdata(&i2c->adapter, NULL);
+
+ return 0;
+}
+
+static struct platform_driver imanager_i2c_driver = {
+ .driver = {
+ .owner = THIS_MODULE,
+ .name = "imanager_i2c",
+ },
+ .probe = imanager_i2c_probe,
+ .remove = imanager_i2c_remove,
+};
+
+module_platform_driver(imanager_i2c_driver);
+
+MODULE_DESCRIPTION("Advantech iManager I2C Driver");
+MODULE_AUTHOR("Richard Vidal-Dorsch <richard.dorsch at advantech.com>");
+MODULE_LICENSE("GPL");
+MODULE_ALIAS("platform:imanager_i2c");
diff --git a/include/linux/mfd/imanager/i2c.h b/include/linux/mfd/imanager/i2c.h
new file mode 100644
index 0000000..a8ef6c2
--- /dev/null
+++ b/include/linux/mfd/imanager/i2c.h
@@ -0,0 +1,55 @@
+/*
+ * Advantech iManager I2C bus core
+ *
+ * Copyright (C) 2016 Advantech Co., Ltd., Irvine, CA, USA
+ * Author: Richard Vidal-Dorsch <richard.dorsch@advantech.com>
+ *
+ * This program is free software; you can redistribute it and/or modify it
+ * under the terms of the GNU General Public License as published by the
+ * Free Software Foundation; either version 2 of the License, or (at your
+ * option) any later version.
+ */
+
+#ifndef __I2C_H__
+#define __I2C_H__
+
+#include <linux/types.h>
+
+#define I2C_MAX_READ_BYTES 32
+#define I2C_MAX_WRITE_BYTES 32
+
+/* Only for setting SMBus frequency */
+enum smb_bus_id {
+ SMB_OEM_0,
+ SMB_OEM_1,
+ SMB_OEM_2,
+ SMB_EEPROM,
+ SMB_TH_0,
+ SMB_TH_1,
+ SMB_SECURITY_EEPROM,
+ I2C_OEM_1,
+};
+
+int i2c_core_init(void);
+
+int i2c_core_write_quick(u16 addr);
+
+int i2c_core_read_byte(u16 addr);
+int i2c_core_write_byte(u16 addr, u8 cmd);
+
+int i2c_core_write_byte_data(u16 addr, u8 cmd, u8 value);
+int i2c_core_read_byte_data(u16 addr, u8 cmd);
+
+int i2c_core_write_word_data(u16 addr, u8 cmd, u16 value);
+int i2c_core_read_word_data(u16 addr, u8 cmd);
+
+int i2c_core_write_block_data(u16 addr, u8 cmd, u8 *buf);
+int i2c_core_read_block_data(u16 addr, u8 cmd, u8 *buf);
+
+int i2c_core_write_i2c_block_data(u16 addr, u8 cmd, u8 *buf);
+int i2c_core_read_i2c_block_data(u16 addr, u8 cmd, u8 *buf);
+
+int i2c_core_smb_get_freq(u32 bus_id);
+int i2c_core_smb_set_freq(u32 bus_id, u32 freq);
+
+#endif
--
2.6.4
^ permalink raw reply related [flat|nested] 43+ messages in thread
* [PATCH] fix platform_no_drv_owner.cocci warnings
2016-01-10 9:11 ` richard.dorsch
@ 2016-01-10 10:34 ` kbuild test robot
0 siblings, 0 replies; 43+ messages in thread
From: kbuild test robot @ 2016-01-10 10:34 UTC (permalink / raw)
Cc: kbuild-all, linux-kernel, lm-sensors, linux-i2c, linux-watchdog,
linux-gpio, lee.jones, jdelvare, linux, wim, jo.sunga,
Richard Vidal-Dorsch
drivers/i2c/busses/imanager-i2c.c:228:3-8: No need to set .owner here. The core will do it.
Remove .owner field if calls are used which set it automatically
Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
CC: Richard Vidal-Dorsch <richard.dorsch@gmail.com>
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
---
imanager-i2c.c | 1 -
1 file changed, 1 deletion(-)
--- a/drivers/i2c/busses/imanager-i2c.c
+++ b/drivers/i2c/busses/imanager-i2c.c
@@ -225,7 +225,6 @@ static int imanager_i2c_remove(struct pl
static struct platform_driver imanager_i2c_driver = {
.driver = {
- .owner = THIS_MODULE,
.name = "imanager_i2c",
},
.probe = imanager_i2c_probe,
^ permalink raw reply [flat|nested] 43+ messages in thread
* Re: [PATCH v2 4/6] Add Advantech iManager I2C driver
2016-01-10 9:11 ` richard.dorsch
@ 2016-01-10 10:34 ` kbuild test robot
0 siblings, 0 replies; 43+ messages in thread
From: kbuild test robot @ 2016-01-10 10:34 UTC (permalink / raw)
Cc: kbuild-all, linux-kernel, lm-sensors, linux-i2c, linux-watchdog,
linux-gpio, lee.jones, jdelvare, linux, wim, jo.sunga,
Richard Vidal-Dorsch
Hi Richard,
[auto build test WARNING on hwmon/hwmon-next]
[also build test WARNING on v4.4-rc8 next-20160108]
[if your patch is applied to the wrong git tree, please drop us a note to help improving the system]
url: https://github.com/0day-ci/linux/commits/richard-dorsch-gmail-com/Add-Advantech-iManager-EC-driver-set/20160110-171635
base: https://git.kernel.org/pub/scm/linux/kernel/git/groeck/linux-staging.git hwmon-next
coccinelle warnings: (new ones prefixed by >>)
>> drivers/i2c/busses/imanager-i2c.c:228:3-8: No need to set .owner here. The core will do it.
Please review and possibly fold the followup patch.
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
^ permalink raw reply [flat|nested] 43+ messages in thread
* [PATCH] fix platform_no_drv_owner.cocci warnings
@ 2016-01-10 10:34 ` kbuild test robot
0 siblings, 0 replies; 43+ messages in thread
From: kbuild test robot @ 2016-01-10 10:34 UTC (permalink / raw)
To: richard.dorsch
Cc: kbuild-all, linux-kernel, lm-sensors, linux-i2c, linux-watchdog,
linux-gpio, lee.jones, jdelvare, linux, wim, jo.sunga,
Richard Vidal-Dorsch
drivers/i2c/busses/imanager-i2c.c:228:3-8: No need to set .owner here. The core will do it.
Remove .owner field if calls are used which set it automatically
Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
CC: Richard Vidal-Dorsch <richard.dorsch@gmail.com>
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
---
imanager-i2c.c | 1 -
1 file changed, 1 deletion(-)
--- a/drivers/i2c/busses/imanager-i2c.c
+++ b/drivers/i2c/busses/imanager-i2c.c
@@ -225,7 +225,6 @@ static int imanager_i2c_remove(struct pl
static struct platform_driver imanager_i2c_driver = {
.driver = {
- .owner = THIS_MODULE,
.name = "imanager_i2c",
},
.probe = imanager_i2c_probe,
^ permalink raw reply [flat|nested] 43+ messages in thread
* Re: [PATCH v2 4/6] Add Advantech iManager I2C driver
@ 2016-01-10 10:34 ` kbuild test robot
0 siblings, 0 replies; 43+ messages in thread
From: kbuild test robot @ 2016-01-10 10:34 UTC (permalink / raw)
To: richard.dorsch
Cc: kbuild-all, linux-kernel, lm-sensors, linux-i2c, linux-watchdog,
linux-gpio, lee.jones, jdelvare, linux, wim, jo.sunga,
Richard Vidal-Dorsch
Hi Richard,
[auto build test WARNING on hwmon/hwmon-next]
[also build test WARNING on v4.4-rc8 next-20160108]
[if your patch is applied to the wrong git tree, please drop us a note to help improving the system]
url: https://github.com/0day-ci/linux/commits/richard-dorsch-gmail-com/Add-Advantech-iManager-EC-driver-set/20160110-171635
base: https://git.kernel.org/pub/scm/linux/kernel/git/groeck/linux-staging.git hwmon-next
coccinelle warnings: (new ones prefixed by >>)
>> drivers/i2c/busses/imanager-i2c.c:228:3-8: No need to set .owner here. The core will do it.
Please review and possibly fold the followup patch.
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
^ permalink raw reply [flat|nested] 43+ messages in thread
* [PATCH] fix platform_no_drv_owner.cocci warnings
[not found] <201906240241.S3kwAGar%lkp@intel.com>
@ 2019-06-23 18:50 ` kbuild test robot
0 siblings, 0 replies; 43+ messages in thread
From: kbuild test robot @ 2019-06-23 18:50 UTC (permalink / raw)
To: Asmaa Mnebhi
Cc: kbuild-all, Corey Minyard, Arnd Bergmann, Greg Kroah-Hartman,
openipmi-developer, linux-kernel
From: kbuild test robot <lkp@intel.com>
drivers/char/ipmi/ipmb_dev_int.c:352:3-8: No need to set .owner here. The core will do it.
Remove .owner field if calls are used which set it automatically
Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
Fixes: 51bd6f291583 ("Add support for IPMB driver")
CC: Asmaa Mnebhi <Asmaa@mellanox.com>
Signed-off-by: kbuild test robot <lkp@intel.com>
---
tree: https://github.com/cminyard/linux-ipmi for-next
head: 21feadd024e747123bc0aaba8d8cbef2b5eda058
commit: 51bd6f291583684f495ea498984dfc22049d7fd2 [5/7] Add support for IPMB driver
ipmb_dev_int.c | 1 -
1 file changed, 1 deletion(-)
--- a/drivers/char/ipmi/ipmb_dev_int.c
+++ b/drivers/char/ipmi/ipmb_dev_int.c
@@ -349,7 +349,6 @@ MODULE_DEVICE_TABLE(acpi, acpi_ipmb_id);
static struct i2c_driver ipmb_driver = {
.driver = {
- .owner = THIS_MODULE,
.name = "ipmb-dev",
.acpi_match_table = ACPI_PTR(acpi_ipmb_id),
},
^ permalink raw reply [flat|nested] 43+ messages in thread
* [PATCH] fix platform_no_drv_owner.cocci warnings
2018-12-30 5:10 [nomadik:ixp4 4/9] drivers/gpio/gw_i2c_pld.c:25:10: fatal error: linux/i2c/gw_i2c_pld.h: No such file or directory kbuild test robot
@ 2018-12-30 5:10 ` kbuild test robot
0 siblings, 0 replies; 43+ messages in thread
From: kbuild test robot @ 2018-12-30 5:10 UTC (permalink / raw)
To: Linus Walleij; +Cc: kbuild-all, linux-arm-kernel
From: kbuild test robot <fengguang.wu@intel.com>
drivers/gpio/gw_i2c_pld.c:351:3-8: No need to set .owner here. The core will do it.
Remove .owner field if calls are used which set it automatically
Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
Fixes: 443f1c782cc7 ("GW i2c PLF GPIO")
Signed-off-by: kbuild test robot <fengguang.wu@intel.com>
---
tree: https://git.kernel.org/pub/scm/linux/kernel/git/linusw/linux-nomadik.git ixp4
head: fd04bd59327ab42cfa79c1a5aaa3160fc4821402
commit: 443f1c782cc742e40a1ef4cb2bca0afa2fea9baf [4/9] GW i2c PLF GPIO
gw_i2c_pld.c | 1 -
1 file changed, 1 deletion(-)
--- a/drivers/gpio/gw_i2c_pld.c
+++ b/drivers/gpio/gw_i2c_pld.c
@@ -348,7 +348,6 @@ static int gw_i2c_pld_remove(struct i2c_
static struct i2c_driver gw_i2c_pld_driver = {
.driver = {
.name = "gw_i2c_pld",
- .owner = THIS_MODULE,
},
.probe = gw_i2c_pld_probe,
.remove = gw_i2c_pld_remove,
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 43+ messages in thread
* [PATCH] fix platform_no_drv_owner.cocci warnings
2017-02-20 10:33 ` Patrick Vogelaar
@ 2017-02-20 12:05 ` kbuild test robot
0 siblings, 0 replies; 43+ messages in thread
From: kbuild test robot @ 2017-02-20 12:05 UTC (permalink / raw)
To: Patrick Vogelaar
Cc: kbuild-all, dmitry.torokhov, linux-kernel, devicetree,
linux-input, Patrick Vogelaar
drivers/input/misc/cy8cmbr3102.c:210:4-9: No need to set .owner here. The core will do it.
Remove .owner field if calls are used which set it automatically
Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
CC: Patrick Vogelaar <Patrick.Vogelaar@gigatronik.com>
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
---
cy8cmbr3102.c | 1 -
1 file changed, 1 deletion(-)
--- a/drivers/input/misc/cy8cmbr3102.c
+++ b/drivers/input/misc/cy8cmbr3102.c
@@ -207,7 +207,6 @@ MODULE_DEVICE_TABLE(of, of_cy8cmbr3102_m
static struct i2c_driver cy8cmbr3102_driver = {
.driver = {
.name = "cy8cmbr3102",
- .owner = THIS_MODULE,
.of_match_table = of_match_ptr(of_cy8cmbr3102_match),
},
.probe = cy8cmbr3102_probe,
^ permalink raw reply [flat|nested] 43+ messages in thread
* [PATCH] fix platform_no_drv_owner.cocci warnings
@ 2017-02-20 12:05 ` kbuild test robot
0 siblings, 0 replies; 43+ messages in thread
From: kbuild test robot @ 2017-02-20 12:05 UTC (permalink / raw)
Cc: kbuild-all, dmitry.torokhov, linux-kernel, devicetree,
linux-input, Patrick Vogelaar
drivers/input/misc/cy8cmbr3102.c:210:4-9: No need to set .owner here. The core will do it.
Remove .owner field if calls are used which set it automatically
Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
CC: Patrick Vogelaar <Patrick.Vogelaar@gigatronik.com>
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
---
cy8cmbr3102.c | 1 -
1 file changed, 1 deletion(-)
--- a/drivers/input/misc/cy8cmbr3102.c
+++ b/drivers/input/misc/cy8cmbr3102.c
@@ -207,7 +207,6 @@ MODULE_DEVICE_TABLE(of, of_cy8cmbr3102_m
static struct i2c_driver cy8cmbr3102_driver = {
.driver = {
.name = "cy8cmbr3102",
- .owner = THIS_MODULE,
.of_match_table = of_match_ptr(of_cy8cmbr3102_match),
},
.probe = cy8cmbr3102_probe,
^ permalink raw reply [flat|nested] 43+ messages in thread
* [PATCH] fix platform_no_drv_owner.cocci warnings
2016-11-23 4:57 ` Ryan Lee
@ 2016-11-23 18:13 ` kbuild test robot
0 siblings, 0 replies; 43+ messages in thread
From: kbuild test robot @ 2016-11-23 18:13 UTC (permalink / raw)
To: Ryan Lee
Cc: kbuild-all, lgirdwood, broonie, robh+dt, mark.rutland, perex,
tiwai, arnd, michael, oder_chiou, yesanishhere, jacob,
Damien.Horsley, bardliao, kuninori.morimoto.gx, petr, lars, nh6z,
ryans.lee, alsa-devel, devicetree, linux-kernel
sound/soc/codecs/max98927.c:941:3-8: No need to set .owner here. The core will do it.
Remove .owner field if calls are used which set it automatically
Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
CC: Ryan Lee <RyanS.Lee@maximintegrated.com>
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
---
max98927.c | 1 -
1 file changed, 1 deletion(-)
--- a/sound/soc/codecs/max98927.c
+++ b/sound/soc/codecs/max98927.c
@@ -938,7 +938,6 @@ MODULE_DEVICE_TABLE(of, max98927_of_matc
static struct i2c_driver max98927_i2c_driver = {
.driver = {
.name = "max98927",
- .owner = THIS_MODULE,
.of_match_table = of_match_ptr(max98927_of_match),
.pm = NULL,
},
^ permalink raw reply [flat|nested] 43+ messages in thread
* Re: [PATCH] fix platform_no_drv_owner.cocci warnings
2016-07-27 14:41 ` Julia Lawall
@ 2016-08-08 5:47 ` Vinod Koul
-1 siblings, 0 replies; 43+ messages in thread
From: Vinod Koul @ 2016-08-08 5:47 UTC (permalink / raw)
To: Julia Lawall
Cc: Michal Simek, Dan Williams, Sören Brinkmann, Hyun Kwon,
kbuild-all, linux-arm-kernel, dmaengine, linux-kernel
On Wed, Jul 27, 2016 at 04:41:16PM +0200, Julia Lawall wrote:
> No need to set .owner here. The core will do it.
And which tree was this generated against :-)
Upstream doesn't have _dpdma.c
>
> Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
>
> Signed-off-by: Julia Lawall <julia.lawall@lip6.fr>
> Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
> ---
>
> xilinx_dpdma.c | 1 -
> 1 file changed, 1 deletion(-)
>
> --- a/drivers/dma/xilinx/xilinx_dpdma.c
> +++ b/drivers/dma/xilinx/xilinx_dpdma.c
> @@ -1944,7 +1944,6 @@ static struct platform_driver xilinx_dpd
> .remove = xilinx_dpdma_remove,
> .driver = {
> .name = "xilinx-dpdma",
> - .owner = THIS_MODULE,
> .of_match_table = xilinx_dpdma_of_match,
> },
> };
--
~Vinod
^ permalink raw reply [flat|nested] 43+ messages in thread
* [PATCH] fix platform_no_drv_owner.cocci warnings
@ 2016-08-08 5:47 ` Vinod Koul
0 siblings, 0 replies; 43+ messages in thread
From: Vinod Koul @ 2016-08-08 5:47 UTC (permalink / raw)
To: linux-arm-kernel
On Wed, Jul 27, 2016 at 04:41:16PM +0200, Julia Lawall wrote:
> No need to set .owner here. The core will do it.
And which tree was this generated against :-)
Upstream doesn't have _dpdma.c
>
> Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
>
> Signed-off-by: Julia Lawall <julia.lawall@lip6.fr>
> Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
> ---
>
> xilinx_dpdma.c | 1 -
> 1 file changed, 1 deletion(-)
>
> --- a/drivers/dma/xilinx/xilinx_dpdma.c
> +++ b/drivers/dma/xilinx/xilinx_dpdma.c
> @@ -1944,7 +1944,6 @@ static struct platform_driver xilinx_dpd
> .remove = xilinx_dpdma_remove,
> .driver = {
> .name = "xilinx-dpdma",
> - .owner = THIS_MODULE,
> .of_match_table = xilinx_dpdma_of_match,
> },
> };
--
~Vinod
^ permalink raw reply [flat|nested] 43+ messages in thread
* [PATCH] fix platform_no_drv_owner.cocci warnings
2016-08-06 15:03 ` Gerard H. Pille
@ 2016-08-06 22:00 ` kbuild test robot
0 siblings, 0 replies; 43+ messages in thread
From: kbuild test robot @ 2016-08-06 22:00 UTC (permalink / raw)
To: Gerard H. Pille; +Cc: kbuild-all, linux-media, Gerard H. Pille
drivers/media/dvb-frontends/si2165.c:1192:3-8: No need to set .owner here. The core will do it.
Remove .owner field if calls are used which set it automatically
Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
CC: Gerard H. Pille <g.h.p@skynet.be>
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
---
si2165.c | 1 -
1 file changed, 1 deletion(-)
--- a/drivers/media/dvb-frontends/si2165.c
+++ b/drivers/media/dvb-frontends/si2165.c
@@ -1189,7 +1189,6 @@ MODULE_DEVICE_TABLE(i2c, si2165_id_table
static struct i2c_driver si2165_driver = {
.driver = {
- .owner = THIS_MODULE,
.name = "si2165",
},
.probe = si2165_probe,
^ permalink raw reply [flat|nested] 43+ messages in thread
* [PATCH] fix platform_no_drv_owner.cocci warnings
@ 2016-07-27 14:41 ` Julia Lawall
0 siblings, 0 replies; 43+ messages in thread
From: Julia Lawall @ 2016-07-27 14:41 UTC (permalink / raw)
To: Michal Simek, Dan Williams, Vinod Koul, Sören Brinkmann, Hyun Kwon
Cc: kbuild-all, linux-arm-kernel, dmaengine, linux-kernel
No need to set .owner here. The core will do it.
Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
Signed-off-by: Julia Lawall <julia.lawall@lip6.fr>
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
---
xilinx_dpdma.c | 1 -
1 file changed, 1 deletion(-)
--- a/drivers/dma/xilinx/xilinx_dpdma.c
+++ b/drivers/dma/xilinx/xilinx_dpdma.c
@@ -1944,7 +1944,6 @@ static struct platform_driver xilinx_dpd
.remove = xilinx_dpdma_remove,
.driver = {
.name = "xilinx-dpdma",
- .owner = THIS_MODULE,
.of_match_table = xilinx_dpdma_of_match,
},
};
^ permalink raw reply [flat|nested] 43+ messages in thread
* [PATCH] fix platform_no_drv_owner.cocci warnings
@ 2016-07-27 14:41 ` Julia Lawall
0 siblings, 0 replies; 43+ messages in thread
From: Julia Lawall @ 2016-07-27 14:41 UTC (permalink / raw)
To: linux-arm-kernel
No need to set .owner here. The core will do it.
Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
Signed-off-by: Julia Lawall <julia.lawall@lip6.fr>
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
---
xilinx_dpdma.c | 1 -
1 file changed, 1 deletion(-)
--- a/drivers/dma/xilinx/xilinx_dpdma.c
+++ b/drivers/dma/xilinx/xilinx_dpdma.c
@@ -1944,7 +1944,6 @@ static struct platform_driver xilinx_dpd
.remove = xilinx_dpdma_remove,
.driver = {
.name = "xilinx-dpdma",
- .owner = THIS_MODULE,
.of_match_table = xilinx_dpdma_of_match,
},
};
^ permalink raw reply [flat|nested] 43+ messages in thread
* [PATCH] fix platform_no_drv_owner.cocci warnings
2016-02-15 6:08 ` info
@ 2016-02-15 7:34 ` kbuild test robot
0 siblings, 0 replies; 43+ messages in thread
From: kbuild test robot @ 2016-02-15 7:34 UTC (permalink / raw)
To: info
Cc: kbuild-all, linux-media,
Буди
Романто,
AreMa Inc, linux-kernel, crope, m.chehab, mchehab, hdegoede,
laurent.pinchart, mkrufky, sylvester.nawrocki, g.liakhovetski,
peter.senna
drivers/media/dvb-frontends/tc90522.c:271:3-8: No need to set .owner here. The core will do it.
Remove .owner field if calls are used which set it automatically
Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
CC: Буди Романто, AreMa Inc <knightrider@are.ma>
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
---
tc90522.c | 1 -
1 file changed, 1 deletion(-)
--- a/drivers/media/dvb-frontends/tc90522.c
+++ b/drivers/media/dvb-frontends/tc90522.c
@@ -268,7 +268,6 @@ MODULE_DEVICE_TABLE(i2c, tc90522_id);
static struct i2c_driver tc90522_driver = {
.driver = {
- .owner = THIS_MODULE,
.name = tc90522_id->name,
},
.probe = tc90522_probe,
^ permalink raw reply [flat|nested] 43+ messages in thread
* [PATCH] fix platform_no_drv_owner.cocci warnings
@ 2016-01-10 10:44 ` kbuild test robot
0 siblings, 0 replies; 43+ messages in thread
From: kbuild test robot @ 2016-01-10 10:44 UTC (permalink / raw)
Cc: kbuild-all-JC7UmRfGjtg, linux-kernel-u79uwXL29TY76Z2rM5mHXA,
lm-sensors-GZX6beZjE8VD60Wz+7aTrA,
linux-i2c-u79uwXL29TY76Z2rM5mHXA,
linux-watchdog-u79uwXL29TY76Z2rM5mHXA,
linux-gpio-u79uwXL29TY76Z2rM5mHXA,
lee.jones-QSEj5FYQhm4dnm+yROfE0A, jdelvare-IBi9RG/b67k,
linux-0h96xk9xTtrk1uMJSBkQmQ, wim-IQzOog9fTRqzQB+pC5nmwQ,
jo.sunga-ELdSlb/RfAS1Z/+hSey0Gg, Richard Vidal-Dorsch
drivers/video/backlight/imanager-bl.c:187:3-8: No need to set .owner here. The core will do it.
Remove .owner field if calls are used which set it automatically
Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
CC: Richard Vidal-Dorsch <richard.dorsch-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
Signed-off-by: Fengguang Wu <fengguang.wu-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
---
imanager-bl.c | 1 -
1 file changed, 1 deletion(-)
--- a/drivers/video/backlight/imanager-bl.c
+++ b/drivers/video/backlight/imanager-bl.c
@@ -184,7 +184,6 @@ static int imanager_backlight_remove(str
static struct platform_driver imanager_backlight_driver = {
.driver = {
- .owner = THIS_MODULE,
.name = "imanager_backlight",
},
.probe = imanager_backlight_probe,
--
To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 43+ messages in thread
* [PATCH] fix platform_no_drv_owner.cocci warnings
@ 2016-01-10 10:44 ` kbuild test robot
0 siblings, 0 replies; 43+ messages in thread
From: kbuild test robot @ 2016-01-10 10:44 UTC (permalink / raw)
To: richard.dorsch
Cc: kbuild-all, linux-kernel, lm-sensors, linux-i2c, linux-watchdog,
linux-gpio, lee.jones, jdelvare, linux, wim, jo.sunga,
Richard Vidal-Dorsch
drivers/video/backlight/imanager-bl.c:187:3-8: No need to set .owner here. The core will do it.
Remove .owner field if calls are used which set it automatically
Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
CC: Richard Vidal-Dorsch <richard.dorsch@gmail.com>
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
---
imanager-bl.c | 1 -
1 file changed, 1 deletion(-)
--- a/drivers/video/backlight/imanager-bl.c
+++ b/drivers/video/backlight/imanager-bl.c
@@ -184,7 +184,6 @@ static int imanager_backlight_remove(str
static struct platform_driver imanager_backlight_driver = {
.driver = {
- .owner = THIS_MODULE,
.name = "imanager_backlight",
},
.probe = imanager_backlight_probe,
^ permalink raw reply [flat|nested] 43+ messages in thread
* [PATCH] fix platform_no_drv_owner.cocci warnings
2016-01-10 9:11 ` richard.dorsch
@ 2016-01-10 10:25 ` kbuild test robot
0 siblings, 0 replies; 43+ messages in thread
From: kbuild test robot @ 2016-01-10 10:25 UTC (permalink / raw)
Cc: kbuild-all, linux-kernel, lm-sensors, linux-i2c, linux-watchdog,
linux-gpio, lee.jones, jdelvare, linux, wim, jo.sunga,
Richard Vidal-Dorsch
drivers/hwmon/imanager-hwmon.c:1047:3-8: No need to set .owner here. The core will do it.
Remove .owner field if calls are used which set it automatically
Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
CC: Richard Vidal-Dorsch <richard.dorsch@gmail.com>
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
---
imanager-hwmon.c | 1 -
1 file changed, 1 deletion(-)
--- a/drivers/hwmon/imanager-hwmon.c
+++ b/drivers/hwmon/imanager-hwmon.c
@@ -1044,7 +1044,6 @@ static int imanager_hwmon_probe(struct p
static struct platform_driver imanager_hwmon_driver = {
.driver = {
- .owner = THIS_MODULE,
.name = "imanager_hwmon",
},
.probe = imanager_hwmon_probe,
^ permalink raw reply [flat|nested] 43+ messages in thread
* [PATCH] fix platform_no_drv_owner.cocci warnings
@ 2016-01-10 10:25 ` kbuild test robot
0 siblings, 0 replies; 43+ messages in thread
From: kbuild test robot @ 2016-01-10 10:25 UTC (permalink / raw)
To: richard.dorsch
Cc: kbuild-all, linux-kernel, lm-sensors, linux-i2c, linux-watchdog,
linux-gpio, lee.jones, jdelvare, linux, wim, jo.sunga,
Richard Vidal-Dorsch
drivers/hwmon/imanager-hwmon.c:1047:3-8: No need to set .owner here. The core will do it.
Remove .owner field if calls are used which set it automatically
Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
CC: Richard Vidal-Dorsch <richard.dorsch@gmail.com>
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
---
imanager-hwmon.c | 1 -
1 file changed, 1 deletion(-)
--- a/drivers/hwmon/imanager-hwmon.c
+++ b/drivers/hwmon/imanager-hwmon.c
@@ -1044,7 +1044,6 @@ static int imanager_hwmon_probe(struct p
static struct platform_driver imanager_hwmon_driver = {
.driver = {
- .owner = THIS_MODULE,
.name = "imanager_hwmon",
},
.probe = imanager_hwmon_probe,
^ permalink raw reply [flat|nested] 43+ messages in thread
* [PATCH] fix platform_no_drv_owner.cocci warnings
2016-01-10 9:10 ` richard.dorsch-Re5JQEeQqe8AvxtiuMwx3w
@ 2016-01-10 10:11 ` kbuild test robot
0 siblings, 0 replies; 43+ messages in thread
From: kbuild test robot @ 2016-01-10 10:11 UTC (permalink / raw)
Cc: kbuild-all, linux-kernel, lm-sensors, linux-i2c, linux-watchdog,
linux-gpio, lee.jones, jdelvare, linux, wim, jo.sunga,
Richard Vidal-Dorsch
drivers/mfd/imanager-core.c:248:3-8: No need to set .owner here. The core will do it.
Remove .owner field if calls are used which set it automatically
Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
CC: Richard Vidal-Dorsch <richard.dorsch@gmail.com>
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
---
imanager-core.c | 1 -
1 file changed, 1 deletion(-)
--- a/drivers/mfd/imanager-core.c
+++ b/drivers/mfd/imanager-core.c
@@ -245,7 +245,6 @@ static int imanager_remove(struct platfo
static struct platform_driver imanager_driver = {
.driver = {
- .owner = THIS_MODULE,
.name = "imanager-core",
},
.probe = imanager_probe,
^ permalink raw reply [flat|nested] 43+ messages in thread
* [PATCH] fix platform_no_drv_owner.cocci warnings
@ 2016-01-10 10:11 ` kbuild test robot
0 siblings, 0 replies; 43+ messages in thread
From: kbuild test robot @ 2016-01-10 10:11 UTC (permalink / raw)
To: richard.dorsch
Cc: kbuild-all, linux-kernel, lm-sensors, linux-i2c, linux-watchdog,
linux-gpio, lee.jones, jdelvare, linux, wim, jo.sunga,
Richard Vidal-Dorsch
drivers/mfd/imanager-core.c:248:3-8: No need to set .owner here. The core will do it.
Remove .owner field if calls are used which set it automatically
Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
CC: Richard Vidal-Dorsch <richard.dorsch@gmail.com>
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
---
imanager-core.c | 1 -
1 file changed, 1 deletion(-)
--- a/drivers/mfd/imanager-core.c
+++ b/drivers/mfd/imanager-core.c
@@ -245,7 +245,6 @@ static int imanager_remove(struct platfo
static struct platform_driver imanager_driver = {
.driver = {
- .owner = THIS_MODULE,
.name = "imanager-core",
},
.probe = imanager_probe,
^ permalink raw reply [flat|nested] 43+ messages in thread
* [PATCH] fix platform_no_drv_owner.cocci warnings
@ 2016-01-09 2:02 ` kbuild test robot
0 siblings, 0 replies; 43+ messages in thread
From: kbuild test robot @ 2016-01-09 2:02 UTC (permalink / raw)
Cc: kbuild-all-JC7UmRfGjtg, linux-kernel-u79uwXL29TY76Z2rM5mHXA,
lm-sensors-GZX6beZjE8VD60Wz+7aTrA,
linux-i2c-u79uwXL29TY76Z2rM5mHXA,
linux-watchdog-u79uwXL29TY76Z2rM5mHXA,
linux-gpio-u79uwXL29TY76Z2rM5mHXA,
lee.jones-QSEj5FYQhm4dnm+yROfE0A, jdelvare-IBi9RG/b67k,
linux-0h96xk9xTtrk1uMJSBkQmQ, wim-IQzOog9fTRqzQB+pC5nmwQ,
jo.sunga-ELdSlb/RfAS1Z/+hSey0Gg, Richard Vidal-Dorsch
drivers/watchdog/imanager-wdt.c:322:3-8: No need to set .owner here. The core will do it.
Remove .owner field if calls are used which set it automatically
Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
CC: Richard Vidal-Dorsch <richard.dorsch-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
Signed-off-by: Fengguang Wu <fengguang.wu-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
---
imanager-wdt.c | 1 -
1 file changed, 1 deletion(-)
--- a/drivers/watchdog/imanager-wdt.c
+++ b/drivers/watchdog/imanager-wdt.c
@@ -319,7 +319,6 @@ static int imanager_wdt_remove(struct pl
static struct platform_driver imanager_wdt_driver = {
.driver = {
- .owner = THIS_MODULE,
.name = "imanager_wdt",
},
.probe = imanager_wdt_probe,
--
To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 43+ messages in thread
* [PATCH] fix platform_no_drv_owner.cocci warnings
@ 2016-01-09 2:02 ` kbuild test robot
0 siblings, 0 replies; 43+ messages in thread
From: kbuild test robot @ 2016-01-09 2:02 UTC (permalink / raw)
To: richard.dorsch
Cc: kbuild-all, linux-kernel, lm-sensors, linux-i2c, linux-watchdog,
linux-gpio, lee.jones, jdelvare, linux, wim, jo.sunga,
Richard Vidal-Dorsch
drivers/watchdog/imanager-wdt.c:322:3-8: No need to set .owner here. The core will do it.
Remove .owner field if calls are used which set it automatically
Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
CC: Richard Vidal-Dorsch <richard.dorsch@gmail.com>
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
---
imanager-wdt.c | 1 -
1 file changed, 1 deletion(-)
--- a/drivers/watchdog/imanager-wdt.c
+++ b/drivers/watchdog/imanager-wdt.c
@@ -319,7 +319,6 @@ static int imanager_wdt_remove(struct pl
static struct platform_driver imanager_wdt_driver = {
.driver = {
- .owner = THIS_MODULE,
.name = "imanager_wdt",
},
.probe = imanager_wdt_probe,
^ permalink raw reply [flat|nested] 43+ messages in thread
* [PATCH] fix platform_no_drv_owner.cocci warnings
2016-01-08 22:29 ` richard.dorsch
@ 2016-01-09 0:50 ` kbuild test robot
0 siblings, 0 replies; 43+ messages in thread
From: kbuild test robot @ 2016-01-09 0:50 UTC (permalink / raw)
Cc: kbuild-all, linux-kernel, lm-sensors, linux-i2c, linux-watchdog,
linux-gpio, lee.jones, jdelvare, linux, wim, jo.sunga,
Richard Vidal-Dorsch
drivers/gpio/imanager-gpio.c:170:3-8: No need to set .owner here. The core will do it.
Remove .owner field if calls are used which set it automatically
Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
CC: Richard Vidal-Dorsch <richard.dorsch@gmail.com>
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
---
imanager-gpio.c | 1 -
1 file changed, 1 deletion(-)
--- a/drivers/gpio/imanager-gpio.c
+++ b/drivers/gpio/imanager-gpio.c
@@ -167,7 +167,6 @@ static int imanager_remove(struct platfo
static struct platform_driver imanager_gpio_driver = {
.driver = {
- .owner = THIS_MODULE,
.name = "imanager_gpio",
},
.probe = imanager_gpio_probe,
^ permalink raw reply [flat|nested] 43+ messages in thread
* [PATCH] fix platform_no_drv_owner.cocci warnings
@ 2016-01-09 0:50 ` kbuild test robot
0 siblings, 0 replies; 43+ messages in thread
From: kbuild test robot @ 2016-01-09 0:50 UTC (permalink / raw)
To: richard.dorsch
Cc: kbuild-all, linux-kernel, lm-sensors, linux-i2c, linux-watchdog,
linux-gpio, lee.jones, jdelvare, linux, wim, jo.sunga,
Richard Vidal-Dorsch
drivers/gpio/imanager-gpio.c:170:3-8: No need to set .owner here. The core will do it.
Remove .owner field if calls are used which set it automatically
Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
CC: Richard Vidal-Dorsch <richard.dorsch@gmail.com>
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
---
imanager-gpio.c | 1 -
1 file changed, 1 deletion(-)
--- a/drivers/gpio/imanager-gpio.c
+++ b/drivers/gpio/imanager-gpio.c
@@ -167,7 +167,6 @@ static int imanager_remove(struct platfo
static struct platform_driver imanager_gpio_driver = {
.driver = {
- .owner = THIS_MODULE,
.name = "imanager_gpio",
},
.probe = imanager_gpio_probe,
^ permalink raw reply [flat|nested] 43+ messages in thread
* [PATCH] fix platform_no_drv_owner.cocci warnings
2015-11-13 12:06 ` [PATCHv3] " Joao Pinto
@ 2015-11-13 11:36 ` kbuild test robot
0 siblings, 0 replies; 43+ messages in thread
From: kbuild test robot @ 2015-11-13 11:36 UTC (permalink / raw)
To: Joao Pinto; +Cc: kbuild-all, bhelgaas, CARLOS.PALMINHA, linux-pci, Joao Pinto
drivers/pci/host/pcie-snpsdev.c:323:3-8: No need to set .owner here. The core will do it.
Remove .owner field if calls are used which set it automatically
Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
CC: Joao Pinto <Joao.Pinto@synopsys.com>
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
---
pcie-snpsdev.c | 1 -
1 file changed, 1 deletion(-)
--- a/drivers/pci/host/pcie-snpsdev.c
+++ b/drivers/pci/host/pcie-snpsdev.c
@@ -320,7 +320,6 @@ static struct platform_driver snpsdev_pc
.remove = __exit_p(snpsdev_pcie_rc_remove),
.driver = {
.name = "pcie-snpsdev",
- .owner = THIS_MODULE,
.of_match_table = snpsdev_pcie_rc_of_match,
},
};
^ permalink raw reply [flat|nested] 43+ messages in thread
* Re: [PATCH] fix platform_no_drv_owner.cocci warnings
2015-04-07 13:09 ` Linus Walleij
@ 2015-04-08 1:24 ` Fengguang Wu
0 siblings, 0 replies; 43+ messages in thread
From: Fengguang Wu @ 2015-04-08 1:24 UTC (permalink / raw)
To: Linus Walleij
Cc: Thierry Reding, Alexandre Courbot, kbuild-all, linux-gpio, linux-kernel
On Tue, Apr 07, 2015 at 03:09:46PM +0200, Linus Walleij wrote:
> On Mon, Mar 30, 2015 at 3:05 PM, Thierry Reding <treding@nvidia.com> wrote:
> > On Sun, Mar 29, 2015 at 03:49:20PM +0800, kbuild test robot wrote:
> >> drivers/pinctrl/pinctrl-max77620.c:472:3-8: No need to set .owner here. The core will do it.
> >>
> >> Remove .owner field if calls are used which set it automatically
> >>
> >> Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
> >>
> >> CC: Alexandre Courbot <acourbot@nvidia.com>
> >> Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
> >> ---
> >>
> >> pinctrl-max77620.c | 1 -
> >> 1 file changed, 1 deletion(-)
> >
> > Hi Linus,
> >
> > please ignore this. It's from a staging tree and against a driver that
> > doesn't exist upstream yet. I have for now removed these branches from
> > my github tree completely until we can figure out a way to keep the 0-
> > day builder from generating these patches.
>
> Bah no big deal, I think I managed to fire off a similar thing on
> a PWM driver down your path :P
>
> Fengguang said he'd fixed it though IIRC.
Yes sorry for the noise! I've listed Thierry's tree as private report
tree (the black list way). Perhaps would be better to make the logic
white list based -- then it will be completely noise free.
Thanks,
Fengguang
^ permalink raw reply [flat|nested] 43+ messages in thread
* Re: [PATCH] fix platform_no_drv_owner.cocci warnings
2015-03-30 13:05 ` Thierry Reding
(?)
@ 2015-04-07 13:09 ` Linus Walleij
2015-04-08 1:24 ` Fengguang Wu
-1 siblings, 1 reply; 43+ messages in thread
From: Linus Walleij @ 2015-04-07 13:09 UTC (permalink / raw)
To: Thierry Reding
Cc: kbuild test robot, Alexandre Courbot, kbuild-all, linux-gpio,
linux-kernel
On Mon, Mar 30, 2015 at 3:05 PM, Thierry Reding <treding@nvidia.com> wrote:
> On Sun, Mar 29, 2015 at 03:49:20PM +0800, kbuild test robot wrote:
>> drivers/pinctrl/pinctrl-max77620.c:472:3-8: No need to set .owner here. The core will do it.
>>
>> Remove .owner field if calls are used which set it automatically
>>
>> Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
>>
>> CC: Alexandre Courbot <acourbot@nvidia.com>
>> Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
>> ---
>>
>> pinctrl-max77620.c | 1 -
>> 1 file changed, 1 deletion(-)
>
> Hi Linus,
>
> please ignore this. It's from a staging tree and against a driver that
> doesn't exist upstream yet. I have for now removed these branches from
> my github tree completely until we can figure out a way to keep the 0-
> day builder from generating these patches.
Bah no big deal, I think I managed to fire off a similar thing on
a PWM driver down your path :P
Fengguang said he'd fixed it though IIRC.
Yours,
Linus Walleij
^ permalink raw reply [flat|nested] 43+ messages in thread
* Re: [PATCH] fix platform_no_drv_owner.cocci warnings
2015-03-29 7:49 ` kbuild test robot
@ 2015-03-30 13:05 ` Thierry Reding
0 siblings, 0 replies; 43+ messages in thread
From: Thierry Reding @ 2015-03-30 13:05 UTC (permalink / raw)
To: kbuild test robot
Cc: Alexandre Courbot, kbuild-all, Linus Walleij, linux-gpio, linux-kernel
[-- Attachment #1: Type: text/plain, Size: 1160 bytes --]
On Sun, Mar 29, 2015 at 03:49:20PM +0800, kbuild test robot wrote:
> drivers/pinctrl/pinctrl-max77620.c:472:3-8: No need to set .owner here. The core will do it.
>
> Remove .owner field if calls are used which set it automatically
>
> Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
>
> CC: Alexandre Courbot <acourbot@nvidia.com>
> Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
> ---
>
> pinctrl-max77620.c | 1 -
> 1 file changed, 1 deletion(-)
Hi Linus,
please ignore this. It's from a staging tree and against a driver that
doesn't exist upstream yet. I have for now removed these branches from
my github tree completely until we can figure out a way to keep the 0-
day builder from generating these patches.
Thierry
> --- a/drivers/pinctrl/pinctrl-max77620.c
> +++ b/drivers/pinctrl/pinctrl-max77620.c
> @@ -469,7 +469,6 @@ static int max77620_pinctrl_remove(struc
> static struct platform_driver max77620_pinctrl_driver = {
> .driver = {
> .name = "max77620-pinctrl",
> - .owner = THIS_MODULE,
> },
> .probe = max77620_pinctrl_probe,
> .remove = max77620_pinctrl_remove,
[-- Attachment #2: Type: application/pgp-signature, Size: 819 bytes --]
^ permalink raw reply [flat|nested] 43+ messages in thread
* Re: [PATCH] fix platform_no_drv_owner.cocci warnings
@ 2015-03-30 13:05 ` Thierry Reding
0 siblings, 0 replies; 43+ messages in thread
From: Thierry Reding @ 2015-03-30 13:05 UTC (permalink / raw)
To: kbuild test robot
Cc: Alexandre Courbot, kbuild-all, Linus Walleij, linux-gpio, linux-kernel
[-- Attachment #1: Type: text/plain, Size: 1160 bytes --]
On Sun, Mar 29, 2015 at 03:49:20PM +0800, kbuild test robot wrote:
> drivers/pinctrl/pinctrl-max77620.c:472:3-8: No need to set .owner here. The core will do it.
>
> Remove .owner field if calls are used which set it automatically
>
> Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
>
> CC: Alexandre Courbot <acourbot@nvidia.com>
> Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
> ---
>
> pinctrl-max77620.c | 1 -
> 1 file changed, 1 deletion(-)
Hi Linus,
please ignore this. It's from a staging tree and against a driver that
doesn't exist upstream yet. I have for now removed these branches from
my github tree completely until we can figure out a way to keep the 0-
day builder from generating these patches.
Thierry
> --- a/drivers/pinctrl/pinctrl-max77620.c
> +++ b/drivers/pinctrl/pinctrl-max77620.c
> @@ -469,7 +469,6 @@ static int max77620_pinctrl_remove(struc
> static struct platform_driver max77620_pinctrl_driver = {
> .driver = {
> .name = "max77620-pinctrl",
> - .owner = THIS_MODULE,
> },
> .probe = max77620_pinctrl_probe,
> .remove = max77620_pinctrl_remove,
[-- Attachment #2: Type: application/pgp-signature, Size: 819 bytes --]
^ permalink raw reply [flat|nested] 43+ messages in thread
* Re: [PATCH] fix platform_no_drv_owner.cocci warnings
2015-03-29 16:24 ` Mark Brown
@ 2015-03-30 12:52 ` Thierry Reding
0 siblings, 0 replies; 43+ messages in thread
From: Thierry Reding @ 2015-03-30 12:52 UTC (permalink / raw)
To: Mark Brown
Cc: kbuild test robot, Paul Walmsley, kbuild-all, Liam Girdwood,
linux-kernel
[-- Attachment #1: Type: text/plain, Size: 580 bytes --]
On Sun, Mar 29, 2015 at 09:24:13AM -0700, Mark Brown wrote:
> * PGP Signed by an unknown key
>
> On Sun, Mar 29, 2015 at 03:33:33PM +0800, kbuild test robot wrote:
> > drivers/regulator/max77620-regulator.c:800:3-8: No need to set .owner here. The core will do it.
>
> This doesn't apply against current code, this file is not present.
Hi Mark,
Sorry about this. The tree that this was generated against is heavily
work in progress and should certainly not have patches for mainline
generated against it.
I'm working with Fengguang to resolve this.
Thierry
[-- Attachment #2: Type: application/pgp-signature, Size: 819 bytes --]
^ permalink raw reply [flat|nested] 43+ messages in thread
* Re: [PATCH] fix platform_no_drv_owner.cocci warnings
2015-03-29 7:42 ` kbuild test robot
@ 2015-03-29 16:27 ` Mark Brown
0 siblings, 0 replies; 43+ messages in thread
From: Mark Brown @ 2015-03-29 16:27 UTC (permalink / raw)
To: kbuild test robot
Cc: Alexandre Courbot, kbuild-all, Thierry Reding, Liam Girdwood,
linux-kernel
[-- Attachment #1: Type: text/plain, Size: 310 bytes --]
On Sun, Mar 29, 2015 at 03:42:46PM +0800, kbuild test robot wrote:
> drivers/regulator/regulator-pwm.c:333:3-8: No need to set .owner here. The core will do it.
>
> Remove .owner field if calls are used which set it automatically
This doesn't apply against current code, the above file isn't present.
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 473 bytes --]
^ permalink raw reply [flat|nested] 43+ messages in thread
* Re: [PATCH] fix platform_no_drv_owner.cocci warnings
2015-03-29 7:33 ` kbuild test robot
@ 2015-03-29 16:24 ` Mark Brown
2015-03-30 12:52 ` Thierry Reding
0 siblings, 1 reply; 43+ messages in thread
From: Mark Brown @ 2015-03-29 16:24 UTC (permalink / raw)
To: kbuild test robot
Cc: Paul Walmsley, kbuild-all, Thierry Reding, Liam Girdwood, linux-kernel
[-- Attachment #1: Type: text/plain, Size: 234 bytes --]
On Sun, Mar 29, 2015 at 03:33:33PM +0800, kbuild test robot wrote:
> drivers/regulator/max77620-regulator.c:800:3-8: No need to set .owner here. The core will do it.
This doesn't apply against current code, this file is not present.
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 473 bytes --]
^ permalink raw reply [flat|nested] 43+ messages in thread
* [PATCH] fix platform_no_drv_owner.cocci warnings
[not found] <201503291519.K9Z6Qptb%fengguang.wu@intel.com>
@ 2015-03-29 7:49 ` kbuild test robot
2015-03-30 13:05 ` Thierry Reding
0 siblings, 1 reply; 43+ messages in thread
From: kbuild test robot @ 2015-03-29 7:49 UTC (permalink / raw)
To: Alexandre Courbot
Cc: kbuild-all, Thierry Reding, Linus Walleij, linux-gpio, linux-kernel
drivers/pinctrl/pinctrl-max77620.c:472:3-8: No need to set .owner here. The core will do it.
Remove .owner field if calls are used which set it automatically
Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
CC: Alexandre Courbot <acourbot@nvidia.com>
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
---
pinctrl-max77620.c | 1 -
1 file changed, 1 deletion(-)
--- a/drivers/pinctrl/pinctrl-max77620.c
+++ b/drivers/pinctrl/pinctrl-max77620.c
@@ -469,7 +469,6 @@ static int max77620_pinctrl_remove(struc
static struct platform_driver max77620_pinctrl_driver = {
.driver = {
.name = "max77620-pinctrl",
- .owner = THIS_MODULE,
},
.probe = max77620_pinctrl_probe,
.remove = max77620_pinctrl_remove,
^ permalink raw reply [flat|nested] 43+ messages in thread
* [PATCH] fix platform_no_drv_owner.cocci warnings
[not found] <201503291545.My1tADgz%fengguang.wu@intel.com>
@ 2015-03-29 7:42 ` kbuild test robot
2015-03-29 16:27 ` Mark Brown
0 siblings, 1 reply; 43+ messages in thread
From: kbuild test robot @ 2015-03-29 7:42 UTC (permalink / raw)
To: Alexandre Courbot
Cc: kbuild-all, Thierry Reding, Liam Girdwood, Mark Brown, linux-kernel
drivers/regulator/regulator-pwm.c:333:3-8: No need to set .owner here. The core will do it.
Remove .owner field if calls are used which set it automatically
Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
CC: Alexandre Courbot <acourbot@nvidia.com>
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
---
regulator-pwm.c | 1 -
1 file changed, 1 deletion(-)
--- a/drivers/regulator/regulator-pwm.c
+++ b/drivers/regulator/regulator-pwm.c
@@ -330,7 +330,6 @@ MODULE_DEVICE_TABLE(of, pwm_regulator_of
static struct platform_driver pwm_regulator_driver = {
.driver = {
.name = "regulator-pwm",
- .owner = THIS_MODULE,
.pm = &pwm_regulator_pm_ops,
.of_match_table = pwm_regulator_of_match,
},
^ permalink raw reply [flat|nested] 43+ messages in thread
* [PATCH] fix platform_no_drv_owner.cocci warnings
[not found] <201503291531.GP28FwKy%fengguang.wu@intel.com>
@ 2015-03-29 7:33 ` kbuild test robot
2015-03-29 16:24 ` Mark Brown
0 siblings, 1 reply; 43+ messages in thread
From: kbuild test robot @ 2015-03-29 7:33 UTC (permalink / raw)
To: Paul Walmsley
Cc: kbuild-all, Thierry Reding, Liam Girdwood, Mark Brown, linux-kernel
drivers/regulator/max77620-regulator.c:800:3-8: No need to set .owner here. The core will do it.
Remove .owner field if calls are used which set it automatically
Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
CC: Paul Walmsley <pwalmsley@nvidia.com>
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
---
max77620-regulator.c | 1 -
1 file changed, 1 deletion(-)
--- a/drivers/regulator/max77620-regulator.c
+++ b/drivers/regulator/max77620-regulator.c
@@ -797,7 +797,6 @@ static struct platform_driver max77620_r
.probe = max77620_regulator_probe,
.driver = {
.name = "max77620-pmic",
- .owner = THIS_MODULE,
},
};
^ permalink raw reply [flat|nested] 43+ messages in thread
* Re: [PATCH] fix platform_no_drv_owner.cocci warnings
2015-03-24 10:43 ` Thierry Reding
@ 2015-03-25 1:56 ` Fengguang Wu
0 siblings, 0 replies; 43+ messages in thread
From: Fengguang Wu @ 2015-03-25 1:56 UTC (permalink / raw)
To: Thierry Reding; +Cc: Linus Walleij, kbuild-all, linux-pwm, linux-kernel
On Tue, Mar 24, 2015 at 11:43:41AM +0100, Thierry Reding wrote:
> On Wed, Mar 18, 2015 at 02:38:18AM +0100, Linus Walleij wrote:
> > On Wed, Mar 11, 2015 at 12:05 PM, Thierry Reding
> > <thierry.reding@gmail.com> wrote:
> > > On Fri, Jan 16, 2015 at 09:06:14AM +0800, kbuild test robot wrote:
> > >> drivers/pwm/pwm-stmpe.c:99:3-8: No need to set .owner here. The core will do it.
> > >>
> > >> Remove .owner field if calls are used which set it automatically
> > >>
> > >> Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
> > >>
> > >> Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
> > >> ---
> > >>
> > >> pwm-stmpe.c | 1 -
> > >> 1 file changed, 1 deletion(-)
> > >
> > > Erm... this driver doesn't exist in my tree. What's going on?
> >
> > What's going on is that the 0day build robot builds everything it
> > finds, including experimental branches.
> >
> > I dunno why it's mailing you though, probably I already added your
> > CC on the patch... stupid me. A bit much to think of.
>
> I guess the robot generates this patch and then runs get_maintainers.pl
> on it, so it's doing the right thing. Anyway, I'll know to ignore these
> in the future if drivers aren't in my tree yet.
Sorry I've added some heuristics to avoid adding CC from
get_maintainers.pl for the likely private branches. It should reduce
such confusions and noises considerably in future.
Thanks,
Fengguang
^ permalink raw reply [flat|nested] 43+ messages in thread
* Re: [PATCH] fix platform_no_drv_owner.cocci warnings
2015-03-18 1:38 ` Linus Walleij
@ 2015-03-24 10:43 ` Thierry Reding
2015-03-25 1:56 ` Fengguang Wu
0 siblings, 1 reply; 43+ messages in thread
From: Thierry Reding @ 2015-03-24 10:43 UTC (permalink / raw)
To: Linus Walleij; +Cc: kbuild test robot, kbuild-all, linux-pwm, linux-kernel
[-- Attachment #1: Type: text/plain, Size: 1140 bytes --]
On Wed, Mar 18, 2015 at 02:38:18AM +0100, Linus Walleij wrote:
> On Wed, Mar 11, 2015 at 12:05 PM, Thierry Reding
> <thierry.reding@gmail.com> wrote:
> > On Fri, Jan 16, 2015 at 09:06:14AM +0800, kbuild test robot wrote:
> >> drivers/pwm/pwm-stmpe.c:99:3-8: No need to set .owner here. The core will do it.
> >>
> >> Remove .owner field if calls are used which set it automatically
> >>
> >> Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
> >>
> >> Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
> >> ---
> >>
> >> pwm-stmpe.c | 1 -
> >> 1 file changed, 1 deletion(-)
> >
> > Erm... this driver doesn't exist in my tree. What's going on?
>
> What's going on is that the 0day build robot builds everything it
> finds, including experimental branches.
>
> I dunno why it's mailing you though, probably I already added your
> CC on the patch... stupid me. A bit much to think of.
I guess the robot generates this patch and then runs get_maintainers.pl
on it, so it's doing the right thing. Anyway, I'll know to ignore these
in the future if drivers aren't in my tree yet.
Thierry
[-- Attachment #2: Type: application/pgp-signature, Size: 819 bytes --]
^ permalink raw reply [flat|nested] 43+ messages in thread
* Re: [PATCH] fix platform_no_drv_owner.cocci warnings
2015-03-11 11:05 ` Thierry Reding
@ 2015-03-18 1:38 ` Linus Walleij
2015-03-24 10:43 ` Thierry Reding
0 siblings, 1 reply; 43+ messages in thread
From: Linus Walleij @ 2015-03-18 1:38 UTC (permalink / raw)
To: Thierry Reding; +Cc: kbuild test robot, kbuild-all, linux-pwm, linux-kernel
On Wed, Mar 11, 2015 at 12:05 PM, Thierry Reding
<thierry.reding@gmail.com> wrote:
> On Fri, Jan 16, 2015 at 09:06:14AM +0800, kbuild test robot wrote:
>> drivers/pwm/pwm-stmpe.c:99:3-8: No need to set .owner here. The core will do it.
>>
>> Remove .owner field if calls are used which set it automatically
>>
>> Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
>>
>> Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
>> ---
>>
>> pwm-stmpe.c | 1 -
>> 1 file changed, 1 deletion(-)
>
> Erm... this driver doesn't exist in my tree. What's going on?
What's going on is that the 0day build robot builds everything it
finds, including experimental branches.
I dunno why it's mailing you though, probably I already added your
CC on the patch... stupid me. A bit much to think of.
Yours,
Linus Walleij
^ permalink raw reply [flat|nested] 43+ messages in thread
* Re: [PATCH] fix platform_no_drv_owner.cocci warnings
2015-01-16 1:06 ` kbuild test robot
@ 2015-03-11 11:05 ` Thierry Reding
2015-03-18 1:38 ` Linus Walleij
0 siblings, 1 reply; 43+ messages in thread
From: Thierry Reding @ 2015-03-11 11:05 UTC (permalink / raw)
To: kbuild test robot; +Cc: Linus Walleij, kbuild-all, linux-pwm, linux-kernel
[-- Attachment #1: Type: text/plain, Size: 503 bytes --]
On Fri, Jan 16, 2015 at 09:06:14AM +0800, kbuild test robot wrote:
> drivers/pwm/pwm-stmpe.c:99:3-8: No need to set .owner here. The core will do it.
>
> Remove .owner field if calls are used which set it automatically
>
> Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
>
> Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
> ---
>
> pwm-stmpe.c | 1 -
> 1 file changed, 1 deletion(-)
Erm... this driver doesn't exist in my tree. What's going on?
Thierry
[-- Attachment #2: Type: application/pgp-signature, Size: 819 bytes --]
^ permalink raw reply [flat|nested] 43+ messages in thread
* [PATCH] fix platform_no_drv_owner.cocci warnings
[not found] <201502131224.TGzEj1Ve%fengguang.wu@intel.com>
@ 2015-02-13 4:49 ` kbuild test robot
0 siblings, 0 replies; 43+ messages in thread
From: kbuild test robot @ 2015-02-13 4:49 UTC (permalink / raw)
To: Rob Clark
Cc: kbuild-all, Mike Turquette, Stephen Boyd, Srinivas Kandagatla,
linux-kernel
drivers/clk/qcom/clk-rpm.c:262:3-8: No need to set .owner here. The core will do it.
Remove .owner field if calls are used which set it automatically
Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
---
clk-rpm.c | 1 -
1 file changed, 1 deletion(-)
--- a/drivers/clk/qcom/clk-rpm.c
+++ b/drivers/clk/qcom/clk-rpm.c
@@ -259,7 +259,6 @@ static struct platform_driver rpm_clk_dr
.probe = rpm_clk_probe,
.driver = {
.name = "qcom-rpm-clk",
- .owner = THIS_MODULE,
.of_match_table = of_match_ptr(clk_rpm_of_match),
},
};
^ permalink raw reply [flat|nested] 43+ messages in thread
* [PATCH] fix platform_no_drv_owner.cocci warnings
[not found] <201501160912.S502Nlmz%fengguang.wu@intel.com>
@ 2015-01-16 1:06 ` kbuild test robot
2015-03-11 11:05 ` Thierry Reding
0 siblings, 1 reply; 43+ messages in thread
From: kbuild test robot @ 2015-01-16 1:06 UTC (permalink / raw)
To: Linus Walleij; +Cc: kbuild-all, Thierry Reding, linux-pwm, linux-kernel
drivers/pwm/pwm-stmpe.c:99:3-8: No need to set .owner here. The core will do it.
Remove .owner field if calls are used which set it automatically
Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
---
pwm-stmpe.c | 1 -
1 file changed, 1 deletion(-)
--- a/drivers/pwm/pwm-stmpe.c
+++ b/drivers/pwm/pwm-stmpe.c
@@ -96,7 +96,6 @@ static int stmpe_pwm_remove(struct platf
static struct platform_driver stmpe_pwm_driver = {
.driver = {
.name = "stmpe-pwm",
- .owner = THIS_MODULE,
},
.probe = stmpe_pwm_probe,
.remove = stmpe_pwm_remove,
^ permalink raw reply [flat|nested] 43+ messages in thread
end of thread, other threads:[~2019-06-23 18:50 UTC | newest]
Thread overview: 43+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-01-10 10:34 [PATCH v2 4/6] Add Advantech iManager I2C driver kbuild test robot
2016-01-10 10:34 ` kbuild test robot
2016-01-10 9:11 ` richard.dorsch
2016-01-10 10:34 ` [PATCH] fix platform_no_drv_owner.cocci warnings kbuild test robot
2016-01-10 10:34 ` kbuild test robot
[not found] <201906240241.S3kwAGar%lkp@intel.com>
2019-06-23 18:50 ` kbuild test robot
-- strict thread matches above, loose matches on Subject: below --
2018-12-30 5:10 [nomadik:ixp4 4/9] drivers/gpio/gw_i2c_pld.c:25:10: fatal error: linux/i2c/gw_i2c_pld.h: No such file or directory kbuild test robot
2018-12-30 5:10 ` [PATCH] fix platform_no_drv_owner.cocci warnings kbuild test robot
2017-02-20 12:05 [PATCH 1/2] add driver for cypress cy8cmbr3102 kbuild test robot
2017-02-20 10:33 ` Patrick Vogelaar
2017-02-20 12:05 ` [PATCH] fix platform_no_drv_owner.cocci warnings kbuild test robot
2017-02-20 12:05 ` kbuild test robot
2016-11-23 18:13 [PATCH] ALSA SoC MAX98927 driver - Initial release kbuild test robot
2016-11-23 4:57 ` Ryan Lee
2016-11-23 18:13 ` [PATCH] fix platform_no_drv_owner.cocci warnings kbuild test robot
2016-08-06 22:00 [PATCH] This patch allows the Terratec Cinergy HTC Stick HD (0ccb:0101) to be used to watch DVB-T kbuild test robot
2016-08-06 15:03 ` Gerard H. Pille
2016-08-06 22:00 ` [PATCH] fix platform_no_drv_owner.cocci warnings kbuild test robot
2016-07-27 14:41 Julia Lawall
2016-07-27 14:41 ` Julia Lawall
2016-08-08 5:47 ` Vinod Koul
2016-08-08 5:47 ` Vinod Koul
2016-02-15 7:34 [media 7/7] PCI bridge driver for PT3 & PXQ3PE kbuild test robot
2016-02-15 6:08 ` info
2016-02-15 7:34 ` [PATCH] fix platform_no_drv_owner.cocci warnings kbuild test robot
2016-01-10 10:44 [PATCH v2 5/6] Add Advantech iManager Backlight driver kbuild test robot
[not found] ` <1452417098-28667-1-git-send-email-richard.dorsch-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2016-01-10 10:44 ` [PATCH] fix platform_no_drv_owner.cocci warnings kbuild test robot
2016-01-10 10:44 ` kbuild test robot
2016-01-10 10:25 [PATCH v2 3/6] Add Advantech iManager HWmon driver kbuild test robot
2016-01-10 9:11 ` richard.dorsch
2016-01-10 10:25 ` [PATCH] fix platform_no_drv_owner.cocci warnings kbuild test robot
2016-01-10 10:25 ` kbuild test robot
2016-01-10 10:11 [PATCH v2 1/6] Add Advantech iManager MFD core driver kbuild test robot
2016-01-10 9:10 ` richard.dorsch-Re5JQEeQqe8AvxtiuMwx3w
2016-01-10 10:11 ` [PATCH] fix platform_no_drv_owner.cocci warnings kbuild test robot
2016-01-10 10:11 ` kbuild test robot
2016-01-09 2:02 [PATCH 6/6] Add Advantech iManager Watchdog driver kbuild test robot
[not found] ` <1452292166-20118-7-git-send-email-richard.dorsch-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2016-01-09 2:02 ` [PATCH] fix platform_no_drv_owner.cocci warnings kbuild test robot
2016-01-09 2:02 ` kbuild test robot
2016-01-09 0:50 [PATCH 2/6] Add Advantech iManager GPIO driver kbuild test robot
2016-01-08 22:29 ` richard.dorsch
2016-01-09 0:50 ` [PATCH] fix platform_no_drv_owner.cocci warnings kbuild test robot
2016-01-09 0:50 ` kbuild test robot
2015-11-13 11:36 [PATCH] add new platform driver for PCI RC kbuild test robot
2015-11-13 12:06 ` [PATCHv3] " Joao Pinto
2015-11-13 11:36 ` [PATCH] fix platform_no_drv_owner.cocci warnings kbuild test robot
[not found] <201503291519.K9Z6Qptb%fengguang.wu@intel.com>
2015-03-29 7:49 ` kbuild test robot
2015-03-30 13:05 ` Thierry Reding
2015-03-30 13:05 ` Thierry Reding
2015-04-07 13:09 ` Linus Walleij
2015-04-08 1:24 ` Fengguang Wu
[not found] <201503291545.My1tADgz%fengguang.wu@intel.com>
2015-03-29 7:42 ` kbuild test robot
2015-03-29 16:27 ` Mark Brown
[not found] <201503291531.GP28FwKy%fengguang.wu@intel.com>
2015-03-29 7:33 ` kbuild test robot
2015-03-29 16:24 ` Mark Brown
2015-03-30 12:52 ` Thierry Reding
[not found] <201502131224.TGzEj1Ve%fengguang.wu@intel.com>
2015-02-13 4:49 ` kbuild test robot
[not found] <201501160912.S502Nlmz%fengguang.wu@intel.com>
2015-01-16 1:06 ` kbuild test robot
2015-03-11 11:05 ` Thierry Reding
2015-03-18 1:38 ` Linus Walleij
2015-03-24 10:43 ` Thierry Reding
2015-03-25 1:56 ` Fengguang Wu
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.