* [PATCH] i2c-designware: Add support for ACPI i2c device
[not found] <carlpeng008@gmail.com>
@ 2014-08-29 3:01 ` Carl Peng
2014-08-29 3:09 ` Carl Peng
1 sibling, 0 replies; 4+ messages in thread
From: Carl Peng @ 2014-08-29 3:01 UTC (permalink / raw)
To: linux-i2c-u79uwXL29TY76Z2rM5mHXA,
carlpeng008-Re5JQEeQqe8AvxtiuMwx3w
AMD platform i2c bus controllers are ACPI devices,
this patch is to add a ACPI glue for Designware
core, make it support i2c bus controller with
ACPI interface.
Signed-off-by: Carl Peng <carlpeng008-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
---
drivers/i2c/busses/Kconfig | 12 +
drivers/i2c/busses/Makefile | 1 +
drivers/i2c/busses/i2c-designware-acpidrv.c | 355 ++++++++++++++++++++++++++++
drivers/i2c/busses/i2c-designware-core.h | 4 +
4 files changed, 372 insertions(+)
create mode 100644 drivers/i2c/busses/i2c-designware-acpidrv.c
diff --git a/drivers/i2c/busses/Kconfig b/drivers/i2c/busses/Kconfig
index 2ac87fa..974700c 100644
--- a/drivers/i2c/busses/Kconfig
+++ b/drivers/i2c/busses/Kconfig
@@ -441,6 +441,18 @@ config I2C_DESIGNWARE_PCI
This driver can also be built as a module. If so, the module
will be called i2c-designware-pci.
+config I2C_DESIGNWARE_ACPI
+ tristate "Synopsys DesignWare ACPI"
+ depends on ACPI
+ select I2C_DESIGNWARE_CORE
+ help
+ AMD platform i2c bus controller is ACPI device, this driver is
+ to add ACPI glue for designware core, make it support ACPI
+ interface.
+
+ If you say yes to this option, support will be included for the
+ Synopsys DesignWare I2C adapter. Only master mode is supported.
+
config I2C_EFM32
tristate "EFM32 I2C controller"
depends on ARCH_EFM32 || COMPILE_TEST
diff --git a/drivers/i2c/busses/Makefile b/drivers/i2c/busses/Makefile
index 49bf07e..ce86081 100644
--- a/drivers/i2c/busses/Makefile
+++ b/drivers/i2c/busses/Makefile
@@ -42,6 +42,7 @@ obj-$(CONFIG_I2C_DESIGNWARE_PLATFORM) += i2c-designware-platform.o
i2c-designware-platform-objs := i2c-designware-platdrv.o
obj-$(CONFIG_I2C_DESIGNWARE_PCI) += i2c-designware-pci.o
i2c-designware-pci-objs := i2c-designware-pcidrv.o
+obj-$(CONFIG_I2C_DESIGNWARE_ACPI) += i2c-designware-acpidrv.o
obj-$(CONFIG_I2C_EFM32) += i2c-efm32.o
obj-$(CONFIG_I2C_EG20T) += i2c-eg20t.o
obj-$(CONFIG_I2C_EXYNOS5) += i2c-exynos5.o
diff --git a/drivers/i2c/busses/i2c-designware-acpidrv.c b/drivers/i2c/busses/i2c-designware-acpidrv.c
new file mode 100644
index 0000000..ebe3a51
--- /dev/null
+++ b/drivers/i2c/busses/i2c-designware-acpidrv.c
@@ -0,0 +1,355 @@
+/*
+ * Synopsys DesignWare I2C adapter driver (master only).
+ *
+ * Based on the TI DAVINCI I2C adapter driver.
+ *
+ * Copyright (C) 2006 Texas Instruments.
+ * Copyright (C) 2007 MontaVista Software Inc.
+ * Copyright (C) 2009 Provigent Ltd.
+ * Copyright (C) 2011 Intel corporation.
+ *
+ * ----------------------------------------------------------------------------
+ *
+ * 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.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ * ----------------------------------------------------------------------------
+ *
+ */
+
+#include <linux/kernel.h>
+#include <linux/module.h>
+#include <linux/delay.h>
+#include <linux/i2c.h>
+#include <linux/errno.h>
+#include <linux/sched.h>
+#include <linux/err.h>
+#include <linux/interrupt.h>
+#include <linux/io.h>
+#include <linux/slab.h>
+#include <linux/pm_runtime.h>
+#include <linux/acpi.h>
+
+#include "i2c-designware-core.h"
+
+#define CLK_KHZ (133 * 1024)
+#define TX_FIFO_DEPTH 128
+#define RX_FIFO_DEPTH 128
+
+#define DRIVER_NAME "i2c-designware-acpi"
+
+#define INTEL_MID_STD_CFG (DW_IC_CON_MASTER | \
+ DW_IC_CON_SLAVE_DISABLE | \
+ DW_IC_CON_RESTART_EN)
+
+enum dw_acpi_ctl_id_t {
+ I2C_BUS_A = 0,
+ I2C_BUS_B,
+ I2C_BUS_C,
+ I2C_BUS_D,
+};
+
+struct dw_acpi_controller {
+ u32 bus_num;
+ u32 bus_cfg;
+ u32 tx_fifo_depth;
+ u32 rx_fifo_depth;
+ u32 clk_khz;
+};
+
+/*i2c controller parameter*/
+static struct dw_acpi_controller dw_acpi_controllers[] = {
+ [I2C_BUS_A] = {
+ .bus_num = 1,
+ .bus_cfg = INTEL_MID_STD_CFG | DW_IC_CON_SPEED_STD,
+ .tx_fifo_depth = TX_FIFO_DEPTH,
+ .rx_fifo_depth = RX_FIFO_DEPTH,
+ .clk_khz = CLK_KHZ,
+ },
+ [I2C_BUS_B] = {
+ .bus_num = 2,
+ .bus_cfg = INTEL_MID_STD_CFG | DW_IC_CON_SPEED_STD,
+ .tx_fifo_depth = TX_FIFO_DEPTH,
+ .rx_fifo_depth = RX_FIFO_DEPTH,
+ .clk_khz = CLK_KHZ,
+ },
+ [I2C_BUS_C] = {
+ .bus_num = 3,
+ .bus_cfg = INTEL_MID_STD_CFG | DW_IC_CON_SPEED_STD,
+ .tx_fifo_depth = TX_FIFO_DEPTH,
+ .rx_fifo_depth = RX_FIFO_DEPTH,
+ .clk_khz = CLK_KHZ,
+ },
+ [I2C_BUS_D] = {
+ .bus_num = 4,
+ .bus_cfg = INTEL_MID_STD_CFG | DW_IC_CON_SPEED_STD,
+ .tx_fifo_depth = TX_FIFO_DEPTH,
+ .rx_fifo_depth = RX_FIFO_DEPTH,
+ .clk_khz = CLK_KHZ,
+ },
+};
+
+static struct i2c_algorithm i2c_dw_algo = {
+ .master_xfer = i2c_dw_xfer,
+ .functionality = i2c_dw_func,
+};
+
+static int i2c_dw_acpi_suspend(struct device *dev)
+{
+ struct dw_i2c_dev *i2c = acpi_driver_data(to_acpi_device(dev));
+
+ i2c_dw_disable(i2c);
+
+ return 0;
+}
+
+static int i2c_dw_acpi_resume(struct device *dev)
+{
+ struct dw_i2c_dev *i2c = acpi_driver_data(to_acpi_device(dev));
+ u32 enabled;
+
+ enabled = i2c_dw_is_enabled(i2c);
+ if (enabled)
+ return 0;
+
+ i2c_dw_init(i2c);
+
+ return 0;
+}
+
+static int i2c_dw_acpi_runtime_idle(struct device *dev)
+{
+ int err = pm_schedule_suspend(dev, 500);
+
+ dev_dbg(dev, "runtime_idle called\n");
+
+ if (err != 0)
+ return 0;
+
+ return -EBUSY;
+}
+
+static int get_irq_flag(int trigger, int polarity, int share)
+{
+ int flags;
+
+ if (trigger == ACPI_LEVEL_SENSITIVE)
+ flags = (polarity == ACPI_ACTIVE_LOW) ? IRQF_TRIGGER_LOW
+ : IRQF_TRIGGER_HIGH;
+ else
+ flags = (polarity == ACPI_ACTIVE_LOW) ? IRQF_TRIGGER_FALLING
+ : IRQF_TRIGGER_RISING;
+
+ if (share == ACPI_SHARED)
+ flags |= IRQF_SHARED;
+
+ return flags;
+}
+
+static acpi_status
+acpi_i2c_parse_resource(struct acpi_resource *res, void *context)
+{
+ int ret;
+ struct dw_i2c_dev *dev = context;
+ struct acpi_resource_fixed_memory32 *fixmem32;
+
+ switch (res->type) {
+ case ACPI_RESOURCE_TYPE_FIXED_MEMORY32:
+ fixmem32 = &res->data.fixed_memory32;
+ if (!fixmem32)
+ return AE_NO_MEMORY;
+
+ dev->io_base = fixmem32->address;
+ dev->io_length = fixmem32->address_length;
+ ret = AE_OK;
+ break;
+
+ case ACPI_RESOURCE_TYPE_IRQ:
+ dev->irq = res->data.irq.interrupts[0];
+ dev->irq_flag = get_irq_flag(res->data.irq.triggering,
+ res->data.irq.polarity, res->data.irq.sharable);
+ ret = AE_OK;
+ break;
+
+ default:
+ if (dev->io_base && dev->irq)
+ ret = AE_OK;
+ else
+ ret = AE_NOT_FOUND;
+ break;
+ }
+
+ return ret;
+}
+
+static const struct dev_pm_ops i2c_dw_pm_ops = {
+ .resume = i2c_dw_acpi_resume,
+ .suspend = i2c_dw_acpi_suspend,
+ SET_RUNTIME_PM_OPS(i2c_dw_acpi_suspend, i2c_dw_acpi_resume,
+ i2c_dw_acpi_runtime_idle)
+};
+
+static u32 i2c_dw_get_clk_rate_khz(struct dw_i2c_dev *dev)
+{
+ return dev->acpi_controller->clk_khz;
+}
+
+static u32 i2c_get_uid(struct acpi_device *dev)
+{
+ return kstrtoul(dev->pnp.unique_id, 10, NULL);
+}
+
+static int i2c_dw_acpi_add(struct acpi_device *pdev)
+{
+ int r;
+ acpi_status status;
+ struct dw_i2c_dev *dev;
+ struct i2c_adapter *adap;
+ struct dw_acpi_controller *controller;
+ u32 controller_id;
+
+ controller_id = i2c_get_uid(pdev);
+
+ if (controller_id >= ARRAY_SIZE(dw_acpi_controllers)) {
+ dev_err(&pdev->dev, "invalid controller number %d\n",
+ controller_id);
+ return -EINVAL;
+ }
+
+ controller = &dw_acpi_controllers[controller_id];
+
+ dev = devm_kzalloc(&pdev->dev, sizeof(struct dw_i2c_dev), GFP_KERNEL);
+ if (!dev)
+ return -ENOMEM;
+
+ status = acpi_walk_resources(pdev->handle, METHOD_NAME__CRS,
+ acpi_i2c_parse_resource, dev);
+ if (ACPI_FAILURE(status) || !dev->io_base || !dev->irq) {
+ dev_err(&pdev->dev, "failure getting resource\n");
+ return -ENODEV;
+ }
+
+ init_completion(&dev->cmd_complete);
+ mutex_init(&dev->lock);
+ dev->clk = NULL;
+ dev->acpi_controller = controller;
+ dev->get_clk_rate_khz = i2c_dw_get_clk_rate_khz;
+ dev->base = devm_ioremap(&pdev->dev, dev->io_base, dev->io_length);
+ if (dev->base == NULL) {
+ dev_err(&pdev->dev, "failure mapping I/O memory\n");
+ return -EBUSY;
+ }
+ dev->dev = &pdev->dev;
+ dev->functionality =
+ I2C_FUNC_I2C |
+ I2C_FUNC_SMBUS_BYTE |
+ I2C_FUNC_SMBUS_BYTE_DATA |
+ I2C_FUNC_SMBUS_WORD_DATA |
+ I2C_FUNC_SMBUS_I2C_BLOCK;
+ dev->master_cfg = controller->bus_cfg;
+
+ pdev->driver_data = dev;
+
+ dev->tx_fifo_depth = controller->tx_fifo_depth;
+ dev->rx_fifo_depth = controller->rx_fifo_depth;
+
+ r = i2c_dw_init(dev);
+ if (r) {
+ dev_err(&pdev->dev, "failure initiating i2c controller\n");
+ return -EINVAL;
+ }
+
+ adap = &dev->adapter;
+ i2c_set_adapdata(adap, dev);
+ adap->owner = THIS_MODULE;
+ adap->class = I2C_CLASS_HWMON;
+ adap->algo = &i2c_dw_algo;
+ adap->dev.parent = &pdev->dev;
+ ACPI_COMPANION_SET(adap->dev.parent, pdev);
+ adap->nr = controller->bus_num;
+ snprintf(adap->name, sizeof(adap->name), "i2c-designware-acpi-%d",
+ adap->nr);
+
+ r = devm_request_irq(&pdev->dev, dev->irq, i2c_dw_isr, dev->irq_flag,
+ adap->name, dev);
+ if (r) {
+ dev_err(&pdev->dev, "failure requesting irq %d\n", dev->irq);
+ return r;
+ }
+
+ i2c_dw_disable_int(dev);
+ i2c_dw_clear_int(dev);
+
+ r = i2c_add_numbered_adapter(adap);
+ if (r) {
+ dev_err(&pdev->dev, "failure adding adapter\n");
+ return r;
+ }
+
+ pm_runtime_set_autosuspend_delay(&pdev->dev, 1000);
+ pm_runtime_use_autosuspend(&pdev->dev);
+ pm_runtime_allow(&pdev->dev);
+
+ return 0;
+}
+
+static int i2c_dw_acpi_remove(struct acpi_device *pdev)
+{
+ struct dw_i2c_dev *dev = acpi_driver_data(pdev);
+
+ i2c_dw_disable(dev);
+ pm_runtime_forbid(&pdev->dev);
+ pm_runtime_get_noresume(&pdev->dev);
+
+ i2c_del_adapter(&dev->adapter);
+
+ return 0;
+}
+
+/* work with hotplug and coldplug */
+MODULE_ALIAS("i2c_designware-acpi");
+
+static const struct acpi_device_id i2_designware_acpi_ids[] = {
+ { "IFC0000", 0},
+ { "AMD0010", 0},
+ { "", 0},
+};
+MODULE_DEVICE_TABLE(acpi, i2_designware_acpi_ids);
+
+static struct acpi_driver dw_i2c_driver = {
+ .name = DRIVER_NAME,
+ .class = DRIVER_NAME,
+ .ids = i2_designware_acpi_ids,
+ .ops = {
+ .add = i2c_dw_acpi_add,
+ .remove = i2c_dw_acpi_remove,
+ },
+ .drv.pm = &i2c_dw_pm_ops,
+};
+
+static int __init dw_i2c_module_init(void)
+{
+ int ret;
+
+ ret = acpi_bus_register_driver(&dw_i2c_driver);
+
+ return ret;
+}
+module_init(dw_i2c_module_init);
+
+static void __exit dw_i2c_module_exit(void)
+{
+ acpi_bus_unregister_driver(&dw_i2c_driver);
+}
+module_exit(dw_i2c_module_exit);
+
+MODULE_AUTHOR("Carl Peng <carlpeng008-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>");
+MODULE_DESCRIPTION("Synopsys DesignWare ACPI I2C bus adapter");
+MODULE_LICENSE("GPL");
diff --git a/drivers/i2c/busses/i2c-designware-core.h b/drivers/i2c/busses/i2c-designware-core.h
index d66b6cb..b88dbb8 100644
--- a/drivers/i2c/busses/i2c-designware-core.h
+++ b/drivers/i2c/busses/i2c-designware-core.h
@@ -73,11 +73,14 @@
struct dw_i2c_dev {
struct device *dev;
void __iomem *base;
+ resource_size_t io_base;
+ u32 io_length;
struct completion cmd_complete;
struct mutex lock;
struct clk *clk;
u32 (*get_clk_rate_khz) (struct dw_i2c_dev *dev);
struct dw_pci_controller *controller;
+ struct dw_acpi_controller *acpi_controller;
int cmd_err;
struct i2c_msg *msgs;
int msgs_num;
@@ -91,6 +94,7 @@ struct dw_i2c_dev {
unsigned int status;
u32 abort_source;
int irq;
+ unsigned int irq_flag;
u32 accessor_flags;
struct i2c_adapter adapter;
u32 functionality;
--
1.9.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [PATCH] i2c-designware: Add support for ACPI i2c device
[not found] <carlpeng008@gmail.com>
2014-08-29 3:01 ` [PATCH] i2c-designware: Add support for ACPI i2c device Carl Peng
@ 2014-08-29 3:09 ` Carl Peng
[not found] ` <1409281752-1473-1-git-send-email-carlpeng008-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
1 sibling, 1 reply; 4+ messages in thread
From: Carl Peng @ 2014-08-29 3:09 UTC (permalink / raw)
To: linux-i2c-u79uwXL29TY76Z2rM5mHXA; +Cc: Carl Peng
AMD platform i2c bus controllers are ACPI devices,
this patch is to add a ACPI glue for Designware
core, make it support i2c bus controller with
ACPI interface.
Signed-off-by: Carl Peng <carlpeng008-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
---
drivers/i2c/busses/Kconfig | 12 +
drivers/i2c/busses/Makefile | 1 +
drivers/i2c/busses/i2c-designware-acpidrv.c | 355 ++++++++++++++++++++++++++++
drivers/i2c/busses/i2c-designware-core.h | 4 +
4 files changed, 372 insertions(+)
create mode 100644 drivers/i2c/busses/i2c-designware-acpidrv.c
diff --git a/drivers/i2c/busses/Kconfig b/drivers/i2c/busses/Kconfig
index 2ac87fa..974700c 100644
--- a/drivers/i2c/busses/Kconfig
+++ b/drivers/i2c/busses/Kconfig
@@ -441,6 +441,18 @@ config I2C_DESIGNWARE_PCI
This driver can also be built as a module. If so, the module
will be called i2c-designware-pci.
+config I2C_DESIGNWARE_ACPI
+ tristate "Synopsys DesignWare ACPI"
+ depends on ACPI
+ select I2C_DESIGNWARE_CORE
+ help
+ AMD platform i2c bus controller is ACPI device, this driver is
+ to add ACPI glue for designware core, make it support ACPI
+ interface.
+
+ If you say yes to this option, support will be included for the
+ Synopsys DesignWare I2C adapter. Only master mode is supported.
+
config I2C_EFM32
tristate "EFM32 I2C controller"
depends on ARCH_EFM32 || COMPILE_TEST
diff --git a/drivers/i2c/busses/Makefile b/drivers/i2c/busses/Makefile
index 49bf07e..ce86081 100644
--- a/drivers/i2c/busses/Makefile
+++ b/drivers/i2c/busses/Makefile
@@ -42,6 +42,7 @@ obj-$(CONFIG_I2C_DESIGNWARE_PLATFORM) += i2c-designware-platform.o
i2c-designware-platform-objs := i2c-designware-platdrv.o
obj-$(CONFIG_I2C_DESIGNWARE_PCI) += i2c-designware-pci.o
i2c-designware-pci-objs := i2c-designware-pcidrv.o
+obj-$(CONFIG_I2C_DESIGNWARE_ACPI) += i2c-designware-acpidrv.o
obj-$(CONFIG_I2C_EFM32) += i2c-efm32.o
obj-$(CONFIG_I2C_EG20T) += i2c-eg20t.o
obj-$(CONFIG_I2C_EXYNOS5) += i2c-exynos5.o
diff --git a/drivers/i2c/busses/i2c-designware-acpidrv.c b/drivers/i2c/busses/i2c-designware-acpidrv.c
new file mode 100644
index 0000000..ebe3a51
--- /dev/null
+++ b/drivers/i2c/busses/i2c-designware-acpidrv.c
@@ -0,0 +1,355 @@
+/*
+ * Synopsys DesignWare I2C adapter driver (master only).
+ *
+ * Based on the TI DAVINCI I2C adapter driver.
+ *
+ * Copyright (C) 2006 Texas Instruments.
+ * Copyright (C) 2007 MontaVista Software Inc.
+ * Copyright (C) 2009 Provigent Ltd.
+ * Copyright (C) 2011 Intel corporation.
+ *
+ * ----------------------------------------------------------------------------
+ *
+ * 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.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ * ----------------------------------------------------------------------------
+ *
+ */
+
+#include <linux/kernel.h>
+#include <linux/module.h>
+#include <linux/delay.h>
+#include <linux/i2c.h>
+#include <linux/errno.h>
+#include <linux/sched.h>
+#include <linux/err.h>
+#include <linux/interrupt.h>
+#include <linux/io.h>
+#include <linux/slab.h>
+#include <linux/pm_runtime.h>
+#include <linux/acpi.h>
+
+#include "i2c-designware-core.h"
+
+#define CLK_KHZ (133 * 1024)
+#define TX_FIFO_DEPTH 128
+#define RX_FIFO_DEPTH 128
+
+#define DRIVER_NAME "i2c-designware-acpi"
+
+#define INTEL_MID_STD_CFG (DW_IC_CON_MASTER | \
+ DW_IC_CON_SLAVE_DISABLE | \
+ DW_IC_CON_RESTART_EN)
+
+enum dw_acpi_ctl_id_t {
+ I2C_BUS_A = 0,
+ I2C_BUS_B,
+ I2C_BUS_C,
+ I2C_BUS_D,
+};
+
+struct dw_acpi_controller {
+ u32 bus_num;
+ u32 bus_cfg;
+ u32 tx_fifo_depth;
+ u32 rx_fifo_depth;
+ u32 clk_khz;
+};
+
+/*i2c controller parameter*/
+static struct dw_acpi_controller dw_acpi_controllers[] = {
+ [I2C_BUS_A] = {
+ .bus_num = 1,
+ .bus_cfg = INTEL_MID_STD_CFG | DW_IC_CON_SPEED_STD,
+ .tx_fifo_depth = TX_FIFO_DEPTH,
+ .rx_fifo_depth = RX_FIFO_DEPTH,
+ .clk_khz = CLK_KHZ,
+ },
+ [I2C_BUS_B] = {
+ .bus_num = 2,
+ .bus_cfg = INTEL_MID_STD_CFG | DW_IC_CON_SPEED_STD,
+ .tx_fifo_depth = TX_FIFO_DEPTH,
+ .rx_fifo_depth = RX_FIFO_DEPTH,
+ .clk_khz = CLK_KHZ,
+ },
+ [I2C_BUS_C] = {
+ .bus_num = 3,
+ .bus_cfg = INTEL_MID_STD_CFG | DW_IC_CON_SPEED_STD,
+ .tx_fifo_depth = TX_FIFO_DEPTH,
+ .rx_fifo_depth = RX_FIFO_DEPTH,
+ .clk_khz = CLK_KHZ,
+ },
+ [I2C_BUS_D] = {
+ .bus_num = 4,
+ .bus_cfg = INTEL_MID_STD_CFG | DW_IC_CON_SPEED_STD,
+ .tx_fifo_depth = TX_FIFO_DEPTH,
+ .rx_fifo_depth = RX_FIFO_DEPTH,
+ .clk_khz = CLK_KHZ,
+ },
+};
+
+static struct i2c_algorithm i2c_dw_algo = {
+ .master_xfer = i2c_dw_xfer,
+ .functionality = i2c_dw_func,
+};
+
+static int i2c_dw_acpi_suspend(struct device *dev)
+{
+ struct dw_i2c_dev *i2c = acpi_driver_data(to_acpi_device(dev));
+
+ i2c_dw_disable(i2c);
+
+ return 0;
+}
+
+static int i2c_dw_acpi_resume(struct device *dev)
+{
+ struct dw_i2c_dev *i2c = acpi_driver_data(to_acpi_device(dev));
+ u32 enabled;
+
+ enabled = i2c_dw_is_enabled(i2c);
+ if (enabled)
+ return 0;
+
+ i2c_dw_init(i2c);
+
+ return 0;
+}
+
+static int i2c_dw_acpi_runtime_idle(struct device *dev)
+{
+ int err = pm_schedule_suspend(dev, 500);
+
+ dev_dbg(dev, "runtime_idle called\n");
+
+ if (err != 0)
+ return 0;
+
+ return -EBUSY;
+}
+
+static int get_irq_flag(int trigger, int polarity, int share)
+{
+ int flags;
+
+ if (trigger == ACPI_LEVEL_SENSITIVE)
+ flags = (polarity == ACPI_ACTIVE_LOW) ? IRQF_TRIGGER_LOW
+ : IRQF_TRIGGER_HIGH;
+ else
+ flags = (polarity == ACPI_ACTIVE_LOW) ? IRQF_TRIGGER_FALLING
+ : IRQF_TRIGGER_RISING;
+
+ if (share == ACPI_SHARED)
+ flags |= IRQF_SHARED;
+
+ return flags;
+}
+
+static acpi_status
+acpi_i2c_parse_resource(struct acpi_resource *res, void *context)
+{
+ int ret;
+ struct dw_i2c_dev *dev = context;
+ struct acpi_resource_fixed_memory32 *fixmem32;
+
+ switch (res->type) {
+ case ACPI_RESOURCE_TYPE_FIXED_MEMORY32:
+ fixmem32 = &res->data.fixed_memory32;
+ if (!fixmem32)
+ return AE_NO_MEMORY;
+
+ dev->io_base = fixmem32->address;
+ dev->io_length = fixmem32->address_length;
+ ret = AE_OK;
+ break;
+
+ case ACPI_RESOURCE_TYPE_IRQ:
+ dev->irq = res->data.irq.interrupts[0];
+ dev->irq_flag = get_irq_flag(res->data.irq.triggering,
+ res->data.irq.polarity, res->data.irq.sharable);
+ ret = AE_OK;
+ break;
+
+ default:
+ if (dev->io_base && dev->irq)
+ ret = AE_OK;
+ else
+ ret = AE_NOT_FOUND;
+ break;
+ }
+
+ return ret;
+}
+
+static const struct dev_pm_ops i2c_dw_pm_ops = {
+ .resume = i2c_dw_acpi_resume,
+ .suspend = i2c_dw_acpi_suspend,
+ SET_RUNTIME_PM_OPS(i2c_dw_acpi_suspend, i2c_dw_acpi_resume,
+ i2c_dw_acpi_runtime_idle)
+};
+
+static u32 i2c_dw_get_clk_rate_khz(struct dw_i2c_dev *dev)
+{
+ return dev->acpi_controller->clk_khz;
+}
+
+static u32 i2c_get_uid(struct acpi_device *dev)
+{
+ return kstrtoul(dev->pnp.unique_id, 10, NULL);
+}
+
+static int i2c_dw_acpi_add(struct acpi_device *pdev)
+{
+ int r;
+ acpi_status status;
+ struct dw_i2c_dev *dev;
+ struct i2c_adapter *adap;
+ struct dw_acpi_controller *controller;
+ u32 controller_id;
+
+ controller_id = i2c_get_uid(pdev);
+
+ if (controller_id >= ARRAY_SIZE(dw_acpi_controllers)) {
+ dev_err(&pdev->dev, "invalid controller number %d\n",
+ controller_id);
+ return -EINVAL;
+ }
+
+ controller = &dw_acpi_controllers[controller_id];
+
+ dev = devm_kzalloc(&pdev->dev, sizeof(struct dw_i2c_dev), GFP_KERNEL);
+ if (!dev)
+ return -ENOMEM;
+
+ status = acpi_walk_resources(pdev->handle, METHOD_NAME__CRS,
+ acpi_i2c_parse_resource, dev);
+ if (ACPI_FAILURE(status) || !dev->io_base || !dev->irq) {
+ dev_err(&pdev->dev, "failure getting resource\n");
+ return -ENODEV;
+ }
+
+ init_completion(&dev->cmd_complete);
+ mutex_init(&dev->lock);
+ dev->clk = NULL;
+ dev->acpi_controller = controller;
+ dev->get_clk_rate_khz = i2c_dw_get_clk_rate_khz;
+ dev->base = devm_ioremap(&pdev->dev, dev->io_base, dev->io_length);
+ if (dev->base == NULL) {
+ dev_err(&pdev->dev, "failure mapping I/O memory\n");
+ return -EBUSY;
+ }
+ dev->dev = &pdev->dev;
+ dev->functionality =
+ I2C_FUNC_I2C |
+ I2C_FUNC_SMBUS_BYTE |
+ I2C_FUNC_SMBUS_BYTE_DATA |
+ I2C_FUNC_SMBUS_WORD_DATA |
+ I2C_FUNC_SMBUS_I2C_BLOCK;
+ dev->master_cfg = controller->bus_cfg;
+
+ pdev->driver_data = dev;
+
+ dev->tx_fifo_depth = controller->tx_fifo_depth;
+ dev->rx_fifo_depth = controller->rx_fifo_depth;
+
+ r = i2c_dw_init(dev);
+ if (r) {
+ dev_err(&pdev->dev, "failure initiating i2c controller\n");
+ return -EINVAL;
+ }
+
+ adap = &dev->adapter;
+ i2c_set_adapdata(adap, dev);
+ adap->owner = THIS_MODULE;
+ adap->class = I2C_CLASS_HWMON;
+ adap->algo = &i2c_dw_algo;
+ adap->dev.parent = &pdev->dev;
+ ACPI_COMPANION_SET(adap->dev.parent, pdev);
+ adap->nr = controller->bus_num;
+ snprintf(adap->name, sizeof(adap->name), "i2c-designware-acpi-%d",
+ adap->nr);
+
+ r = devm_request_irq(&pdev->dev, dev->irq, i2c_dw_isr, dev->irq_flag,
+ adap->name, dev);
+ if (r) {
+ dev_err(&pdev->dev, "failure requesting irq %d\n", dev->irq);
+ return r;
+ }
+
+ i2c_dw_disable_int(dev);
+ i2c_dw_clear_int(dev);
+
+ r = i2c_add_numbered_adapter(adap);
+ if (r) {
+ dev_err(&pdev->dev, "failure adding adapter\n");
+ return r;
+ }
+
+ pm_runtime_set_autosuspend_delay(&pdev->dev, 1000);
+ pm_runtime_use_autosuspend(&pdev->dev);
+ pm_runtime_allow(&pdev->dev);
+
+ return 0;
+}
+
+static int i2c_dw_acpi_remove(struct acpi_device *pdev)
+{
+ struct dw_i2c_dev *dev = acpi_driver_data(pdev);
+
+ i2c_dw_disable(dev);
+ pm_runtime_forbid(&pdev->dev);
+ pm_runtime_get_noresume(&pdev->dev);
+
+ i2c_del_adapter(&dev->adapter);
+
+ return 0;
+}
+
+/* work with hotplug and coldplug */
+MODULE_ALIAS("i2c_designware-acpi");
+
+static const struct acpi_device_id i2_designware_acpi_ids[] = {
+ { "IFC0000", 0},
+ { "AMD0010", 0},
+ { "", 0},
+};
+MODULE_DEVICE_TABLE(acpi, i2_designware_acpi_ids);
+
+static struct acpi_driver dw_i2c_driver = {
+ .name = DRIVER_NAME,
+ .class = DRIVER_NAME,
+ .ids = i2_designware_acpi_ids,
+ .ops = {
+ .add = i2c_dw_acpi_add,
+ .remove = i2c_dw_acpi_remove,
+ },
+ .drv.pm = &i2c_dw_pm_ops,
+};
+
+static int __init dw_i2c_module_init(void)
+{
+ int ret;
+
+ ret = acpi_bus_register_driver(&dw_i2c_driver);
+
+ return ret;
+}
+module_init(dw_i2c_module_init);
+
+static void __exit dw_i2c_module_exit(void)
+{
+ acpi_bus_unregister_driver(&dw_i2c_driver);
+}
+module_exit(dw_i2c_module_exit);
+
+MODULE_AUTHOR("Carl Peng <carlpeng008-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>");
+MODULE_DESCRIPTION("Synopsys DesignWare ACPI I2C bus adapter");
+MODULE_LICENSE("GPL");
diff --git a/drivers/i2c/busses/i2c-designware-core.h b/drivers/i2c/busses/i2c-designware-core.h
index d66b6cb..b88dbb8 100644
--- a/drivers/i2c/busses/i2c-designware-core.h
+++ b/drivers/i2c/busses/i2c-designware-core.h
@@ -73,11 +73,14 @@
struct dw_i2c_dev {
struct device *dev;
void __iomem *base;
+ resource_size_t io_base;
+ u32 io_length;
struct completion cmd_complete;
struct mutex lock;
struct clk *clk;
u32 (*get_clk_rate_khz) (struct dw_i2c_dev *dev);
struct dw_pci_controller *controller;
+ struct dw_acpi_controller *acpi_controller;
int cmd_err;
struct i2c_msg *msgs;
int msgs_num;
@@ -91,6 +94,7 @@ struct dw_i2c_dev {
unsigned int status;
u32 abort_source;
int irq;
+ unsigned int irq_flag;
u32 accessor_flags;
struct i2c_adapter adapter;
u32 functionality;
--
1.9.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] i2c-designware: Add support for ACPI i2c device
[not found] ` <1409281752-1473-1-git-send-email-carlpeng008-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
@ 2014-08-29 8:39 ` Wolfram Sang
2014-08-29 8:52 ` Mika Westerberg
0 siblings, 1 reply; 4+ messages in thread
From: Wolfram Sang @ 2014-08-29 8:39 UTC (permalink / raw)
To: Carl Peng; +Cc: linux-i2c-u79uwXL29TY76Z2rM5mHXA, Mika Westerberg
[-- Attachment #1: Type: text/plain, Size: 13583 bytes --]
On Fri, Aug 29, 2014 at 11:09:12AM +0800, Carl Peng wrote:
> AMD platform i2c bus controllers are ACPI devices,
> this patch is to add a ACPI glue for Designware
> core, make it support i2c bus controller with
> ACPI interface.
>
> Signed-off-by: Carl Peng <carlpeng008-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
i2c-designware-platdrv.c has ACPI support, can't you reuse that?
CCing Mika for ACPI expertise...
> ---
> drivers/i2c/busses/Kconfig | 12 +
> drivers/i2c/busses/Makefile | 1 +
> drivers/i2c/busses/i2c-designware-acpidrv.c | 355 ++++++++++++++++++++++++++++
> drivers/i2c/busses/i2c-designware-core.h | 4 +
> 4 files changed, 372 insertions(+)
> create mode 100644 drivers/i2c/busses/i2c-designware-acpidrv.c
>
> diff --git a/drivers/i2c/busses/Kconfig b/drivers/i2c/busses/Kconfig
> index 2ac87fa..974700c 100644
> --- a/drivers/i2c/busses/Kconfig
> +++ b/drivers/i2c/busses/Kconfig
> @@ -441,6 +441,18 @@ config I2C_DESIGNWARE_PCI
> This driver can also be built as a module. If so, the module
> will be called i2c-designware-pci.
>
> +config I2C_DESIGNWARE_ACPI
> + tristate "Synopsys DesignWare ACPI"
> + depends on ACPI
> + select I2C_DESIGNWARE_CORE
> + help
> + AMD platform i2c bus controller is ACPI device, this driver is
> + to add ACPI glue for designware core, make it support ACPI
> + interface.
> +
> + If you say yes to this option, support will be included for the
> + Synopsys DesignWare I2C adapter. Only master mode is supported.
> +
> config I2C_EFM32
> tristate "EFM32 I2C controller"
> depends on ARCH_EFM32 || COMPILE_TEST
> diff --git a/drivers/i2c/busses/Makefile b/drivers/i2c/busses/Makefile
> index 49bf07e..ce86081 100644
> --- a/drivers/i2c/busses/Makefile
> +++ b/drivers/i2c/busses/Makefile
> @@ -42,6 +42,7 @@ obj-$(CONFIG_I2C_DESIGNWARE_PLATFORM) += i2c-designware-platform.o
> i2c-designware-platform-objs := i2c-designware-platdrv.o
> obj-$(CONFIG_I2C_DESIGNWARE_PCI) += i2c-designware-pci.o
> i2c-designware-pci-objs := i2c-designware-pcidrv.o
> +obj-$(CONFIG_I2C_DESIGNWARE_ACPI) += i2c-designware-acpidrv.o
> obj-$(CONFIG_I2C_EFM32) += i2c-efm32.o
> obj-$(CONFIG_I2C_EG20T) += i2c-eg20t.o
> obj-$(CONFIG_I2C_EXYNOS5) += i2c-exynos5.o
> diff --git a/drivers/i2c/busses/i2c-designware-acpidrv.c b/drivers/i2c/busses/i2c-designware-acpidrv.c
> new file mode 100644
> index 0000000..ebe3a51
> --- /dev/null
> +++ b/drivers/i2c/busses/i2c-designware-acpidrv.c
> @@ -0,0 +1,355 @@
> +/*
> + * Synopsys DesignWare I2C adapter driver (master only).
> + *
> + * Based on the TI DAVINCI I2C adapter driver.
> + *
> + * Copyright (C) 2006 Texas Instruments.
> + * Copyright (C) 2007 MontaVista Software Inc.
> + * Copyright (C) 2009 Provigent Ltd.
> + * Copyright (C) 2011 Intel corporation.
> + *
> + * ----------------------------------------------------------------------------
> + *
> + * 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.
> + *
> + * This program is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> + * GNU General Public License for more details.
> + *
> + * ----------------------------------------------------------------------------
> + *
> + */
> +
> +#include <linux/kernel.h>
> +#include <linux/module.h>
> +#include <linux/delay.h>
> +#include <linux/i2c.h>
> +#include <linux/errno.h>
> +#include <linux/sched.h>
> +#include <linux/err.h>
> +#include <linux/interrupt.h>
> +#include <linux/io.h>
> +#include <linux/slab.h>
> +#include <linux/pm_runtime.h>
> +#include <linux/acpi.h>
> +
> +#include "i2c-designware-core.h"
> +
> +#define CLK_KHZ (133 * 1024)
> +#define TX_FIFO_DEPTH 128
> +#define RX_FIFO_DEPTH 128
> +
> +#define DRIVER_NAME "i2c-designware-acpi"
> +
> +#define INTEL_MID_STD_CFG (DW_IC_CON_MASTER | \
> + DW_IC_CON_SLAVE_DISABLE | \
> + DW_IC_CON_RESTART_EN)
> +
> +enum dw_acpi_ctl_id_t {
> + I2C_BUS_A = 0,
> + I2C_BUS_B,
> + I2C_BUS_C,
> + I2C_BUS_D,
> +};
> +
> +struct dw_acpi_controller {
> + u32 bus_num;
> + u32 bus_cfg;
> + u32 tx_fifo_depth;
> + u32 rx_fifo_depth;
> + u32 clk_khz;
> +};
> +
> +/*i2c controller parameter*/
> +static struct dw_acpi_controller dw_acpi_controllers[] = {
> + [I2C_BUS_A] = {
> + .bus_num = 1,
> + .bus_cfg = INTEL_MID_STD_CFG | DW_IC_CON_SPEED_STD,
> + .tx_fifo_depth = TX_FIFO_DEPTH,
> + .rx_fifo_depth = RX_FIFO_DEPTH,
> + .clk_khz = CLK_KHZ,
> + },
> + [I2C_BUS_B] = {
> + .bus_num = 2,
> + .bus_cfg = INTEL_MID_STD_CFG | DW_IC_CON_SPEED_STD,
> + .tx_fifo_depth = TX_FIFO_DEPTH,
> + .rx_fifo_depth = RX_FIFO_DEPTH,
> + .clk_khz = CLK_KHZ,
> + },
> + [I2C_BUS_C] = {
> + .bus_num = 3,
> + .bus_cfg = INTEL_MID_STD_CFG | DW_IC_CON_SPEED_STD,
> + .tx_fifo_depth = TX_FIFO_DEPTH,
> + .rx_fifo_depth = RX_FIFO_DEPTH,
> + .clk_khz = CLK_KHZ,
> + },
> + [I2C_BUS_D] = {
> + .bus_num = 4,
> + .bus_cfg = INTEL_MID_STD_CFG | DW_IC_CON_SPEED_STD,
> + .tx_fifo_depth = TX_FIFO_DEPTH,
> + .rx_fifo_depth = RX_FIFO_DEPTH,
> + .clk_khz = CLK_KHZ,
> + },
> +};
> +
> +static struct i2c_algorithm i2c_dw_algo = {
> + .master_xfer = i2c_dw_xfer,
> + .functionality = i2c_dw_func,
> +};
> +
> +static int i2c_dw_acpi_suspend(struct device *dev)
> +{
> + struct dw_i2c_dev *i2c = acpi_driver_data(to_acpi_device(dev));
> +
> + i2c_dw_disable(i2c);
> +
> + return 0;
> +}
> +
> +static int i2c_dw_acpi_resume(struct device *dev)
> +{
> + struct dw_i2c_dev *i2c = acpi_driver_data(to_acpi_device(dev));
> + u32 enabled;
> +
> + enabled = i2c_dw_is_enabled(i2c);
> + if (enabled)
> + return 0;
> +
> + i2c_dw_init(i2c);
> +
> + return 0;
> +}
> +
> +static int i2c_dw_acpi_runtime_idle(struct device *dev)
> +{
> + int err = pm_schedule_suspend(dev, 500);
> +
> + dev_dbg(dev, "runtime_idle called\n");
> +
> + if (err != 0)
> + return 0;
> +
> + return -EBUSY;
> +}
> +
> +static int get_irq_flag(int trigger, int polarity, int share)
> +{
> + int flags;
> +
> + if (trigger == ACPI_LEVEL_SENSITIVE)
> + flags = (polarity == ACPI_ACTIVE_LOW) ? IRQF_TRIGGER_LOW
> + : IRQF_TRIGGER_HIGH;
> + else
> + flags = (polarity == ACPI_ACTIVE_LOW) ? IRQF_TRIGGER_FALLING
> + : IRQF_TRIGGER_RISING;
> +
> + if (share == ACPI_SHARED)
> + flags |= IRQF_SHARED;
> +
> + return flags;
> +}
> +
> +static acpi_status
> +acpi_i2c_parse_resource(struct acpi_resource *res, void *context)
> +{
> + int ret;
> + struct dw_i2c_dev *dev = context;
> + struct acpi_resource_fixed_memory32 *fixmem32;
> +
> + switch (res->type) {
> + case ACPI_RESOURCE_TYPE_FIXED_MEMORY32:
> + fixmem32 = &res->data.fixed_memory32;
> + if (!fixmem32)
> + return AE_NO_MEMORY;
> +
> + dev->io_base = fixmem32->address;
> + dev->io_length = fixmem32->address_length;
> + ret = AE_OK;
> + break;
> +
> + case ACPI_RESOURCE_TYPE_IRQ:
> + dev->irq = res->data.irq.interrupts[0];
> + dev->irq_flag = get_irq_flag(res->data.irq.triggering,
> + res->data.irq.polarity, res->data.irq.sharable);
> + ret = AE_OK;
> + break;
> +
> + default:
> + if (dev->io_base && dev->irq)
> + ret = AE_OK;
> + else
> + ret = AE_NOT_FOUND;
> + break;
> + }
> +
> + return ret;
> +}
> +
> +static const struct dev_pm_ops i2c_dw_pm_ops = {
> + .resume = i2c_dw_acpi_resume,
> + .suspend = i2c_dw_acpi_suspend,
> + SET_RUNTIME_PM_OPS(i2c_dw_acpi_suspend, i2c_dw_acpi_resume,
> + i2c_dw_acpi_runtime_idle)
> +};
> +
> +static u32 i2c_dw_get_clk_rate_khz(struct dw_i2c_dev *dev)
> +{
> + return dev->acpi_controller->clk_khz;
> +}
> +
> +static u32 i2c_get_uid(struct acpi_device *dev)
> +{
> + return kstrtoul(dev->pnp.unique_id, 10, NULL);
> +}
> +
> +static int i2c_dw_acpi_add(struct acpi_device *pdev)
> +{
> + int r;
> + acpi_status status;
> + struct dw_i2c_dev *dev;
> + struct i2c_adapter *adap;
> + struct dw_acpi_controller *controller;
> + u32 controller_id;
> +
> + controller_id = i2c_get_uid(pdev);
> +
> + if (controller_id >= ARRAY_SIZE(dw_acpi_controllers)) {
> + dev_err(&pdev->dev, "invalid controller number %d\n",
> + controller_id);
> + return -EINVAL;
> + }
> +
> + controller = &dw_acpi_controllers[controller_id];
> +
> + dev = devm_kzalloc(&pdev->dev, sizeof(struct dw_i2c_dev), GFP_KERNEL);
> + if (!dev)
> + return -ENOMEM;
> +
> + status = acpi_walk_resources(pdev->handle, METHOD_NAME__CRS,
> + acpi_i2c_parse_resource, dev);
> + if (ACPI_FAILURE(status) || !dev->io_base || !dev->irq) {
> + dev_err(&pdev->dev, "failure getting resource\n");
> + return -ENODEV;
> + }
> +
> + init_completion(&dev->cmd_complete);
> + mutex_init(&dev->lock);
> + dev->clk = NULL;
> + dev->acpi_controller = controller;
> + dev->get_clk_rate_khz = i2c_dw_get_clk_rate_khz;
> + dev->base = devm_ioremap(&pdev->dev, dev->io_base, dev->io_length);
> + if (dev->base == NULL) {
> + dev_err(&pdev->dev, "failure mapping I/O memory\n");
> + return -EBUSY;
> + }
> + dev->dev = &pdev->dev;
> + dev->functionality =
> + I2C_FUNC_I2C |
> + I2C_FUNC_SMBUS_BYTE |
> + I2C_FUNC_SMBUS_BYTE_DATA |
> + I2C_FUNC_SMBUS_WORD_DATA |
> + I2C_FUNC_SMBUS_I2C_BLOCK;
> + dev->master_cfg = controller->bus_cfg;
> +
> + pdev->driver_data = dev;
> +
> + dev->tx_fifo_depth = controller->tx_fifo_depth;
> + dev->rx_fifo_depth = controller->rx_fifo_depth;
> +
> + r = i2c_dw_init(dev);
> + if (r) {
> + dev_err(&pdev->dev, "failure initiating i2c controller\n");
> + return -EINVAL;
> + }
> +
> + adap = &dev->adapter;
> + i2c_set_adapdata(adap, dev);
> + adap->owner = THIS_MODULE;
> + adap->class = I2C_CLASS_HWMON;
> + adap->algo = &i2c_dw_algo;
> + adap->dev.parent = &pdev->dev;
> + ACPI_COMPANION_SET(adap->dev.parent, pdev);
> + adap->nr = controller->bus_num;
> + snprintf(adap->name, sizeof(adap->name), "i2c-designware-acpi-%d",
> + adap->nr);
> +
> + r = devm_request_irq(&pdev->dev, dev->irq, i2c_dw_isr, dev->irq_flag,
> + adap->name, dev);
> + if (r) {
> + dev_err(&pdev->dev, "failure requesting irq %d\n", dev->irq);
> + return r;
> + }
> +
> + i2c_dw_disable_int(dev);
> + i2c_dw_clear_int(dev);
> +
> + r = i2c_add_numbered_adapter(adap);
> + if (r) {
> + dev_err(&pdev->dev, "failure adding adapter\n");
> + return r;
> + }
> +
> + pm_runtime_set_autosuspend_delay(&pdev->dev, 1000);
> + pm_runtime_use_autosuspend(&pdev->dev);
> + pm_runtime_allow(&pdev->dev);
> +
> + return 0;
> +}
> +
> +static int i2c_dw_acpi_remove(struct acpi_device *pdev)
> +{
> + struct dw_i2c_dev *dev = acpi_driver_data(pdev);
> +
> + i2c_dw_disable(dev);
> + pm_runtime_forbid(&pdev->dev);
> + pm_runtime_get_noresume(&pdev->dev);
> +
> + i2c_del_adapter(&dev->adapter);
> +
> + return 0;
> +}
> +
> +/* work with hotplug and coldplug */
> +MODULE_ALIAS("i2c_designware-acpi");
> +
> +static const struct acpi_device_id i2_designware_acpi_ids[] = {
> + { "IFC0000", 0},
> + { "AMD0010", 0},
> + { "", 0},
> +};
> +MODULE_DEVICE_TABLE(acpi, i2_designware_acpi_ids);
> +
> +static struct acpi_driver dw_i2c_driver = {
> + .name = DRIVER_NAME,
> + .class = DRIVER_NAME,
> + .ids = i2_designware_acpi_ids,
> + .ops = {
> + .add = i2c_dw_acpi_add,
> + .remove = i2c_dw_acpi_remove,
> + },
> + .drv.pm = &i2c_dw_pm_ops,
> +};
> +
> +static int __init dw_i2c_module_init(void)
> +{
> + int ret;
> +
> + ret = acpi_bus_register_driver(&dw_i2c_driver);
> +
> + return ret;
> +}
> +module_init(dw_i2c_module_init);
> +
> +static void __exit dw_i2c_module_exit(void)
> +{
> + acpi_bus_unregister_driver(&dw_i2c_driver);
> +}
> +module_exit(dw_i2c_module_exit);
> +
> +MODULE_AUTHOR("Carl Peng <carlpeng008-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>");
> +MODULE_DESCRIPTION("Synopsys DesignWare ACPI I2C bus adapter");
> +MODULE_LICENSE("GPL");
> diff --git a/drivers/i2c/busses/i2c-designware-core.h b/drivers/i2c/busses/i2c-designware-core.h
> index d66b6cb..b88dbb8 100644
> --- a/drivers/i2c/busses/i2c-designware-core.h
> +++ b/drivers/i2c/busses/i2c-designware-core.h
> @@ -73,11 +73,14 @@
> struct dw_i2c_dev {
> struct device *dev;
> void __iomem *base;
> + resource_size_t io_base;
> + u32 io_length;
> struct completion cmd_complete;
> struct mutex lock;
> struct clk *clk;
> u32 (*get_clk_rate_khz) (struct dw_i2c_dev *dev);
> struct dw_pci_controller *controller;
> + struct dw_acpi_controller *acpi_controller;
> int cmd_err;
> struct i2c_msg *msgs;
> int msgs_num;
> @@ -91,6 +94,7 @@ struct dw_i2c_dev {
> unsigned int status;
> u32 abort_source;
> int irq;
> + unsigned int irq_flag;
> u32 accessor_flags;
> struct i2c_adapter adapter;
> u32 functionality;
> --
> 1.9.1
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
> the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] i2c-designware: Add support for ACPI i2c device
2014-08-29 8:39 ` Wolfram Sang
@ 2014-08-29 8:52 ` Mika Westerberg
0 siblings, 0 replies; 4+ messages in thread
From: Mika Westerberg @ 2014-08-29 8:52 UTC (permalink / raw)
To: Wolfram Sang; +Cc: Carl Peng, linux-i2c-u79uwXL29TY76Z2rM5mHXA
On Fri, Aug 29, 2014 at 10:39:17AM +0200, Wolfram Sang wrote:
> On Fri, Aug 29, 2014 at 11:09:12AM +0800, Carl Peng wrote:
> > AMD platform i2c bus controllers are ACPI devices,
> > this patch is to add a ACPI glue for Designware
> > core, make it support i2c bus controller with
> > ACPI interface.
> >
> > Signed-off-by: Carl Peng <carlpeng008-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
>
> i2c-designware-platdrv.c has ACPI support, can't you reuse that?
> CCing Mika for ACPI expertise...
Indeed, you should just update the i2c-designware-platdrv.c with your
ACPI IDs.
>
> > ---
> > drivers/i2c/busses/Kconfig | 12 +
> > drivers/i2c/busses/Makefile | 1 +
> > drivers/i2c/busses/i2c-designware-acpidrv.c | 355 ++++++++++++++++++++++++++++
> > drivers/i2c/busses/i2c-designware-core.h | 4 +
> > 4 files changed, 372 insertions(+)
> > create mode 100644 drivers/i2c/busses/i2c-designware-acpidrv.c
> >
> > diff --git a/drivers/i2c/busses/Kconfig b/drivers/i2c/busses/Kconfig
> > index 2ac87fa..974700c 100644
> > --- a/drivers/i2c/busses/Kconfig
> > +++ b/drivers/i2c/busses/Kconfig
> > @@ -441,6 +441,18 @@ config I2C_DESIGNWARE_PCI
> > This driver can also be built as a module. If so, the module
> > will be called i2c-designware-pci.
> >
> > +config I2C_DESIGNWARE_ACPI
> > + tristate "Synopsys DesignWare ACPI"
> > + depends on ACPI
> > + select I2C_DESIGNWARE_CORE
> > + help
> > + AMD platform i2c bus controller is ACPI device, this driver is
> > + to add ACPI glue for designware core, make it support ACPI
> > + interface.
> > +
> > + If you say yes to this option, support will be included for the
> > + Synopsys DesignWare I2C adapter. Only master mode is supported.
> > +
> > config I2C_EFM32
> > tristate "EFM32 I2C controller"
> > depends on ARCH_EFM32 || COMPILE_TEST
> > diff --git a/drivers/i2c/busses/Makefile b/drivers/i2c/busses/Makefile
> > index 49bf07e..ce86081 100644
> > --- a/drivers/i2c/busses/Makefile
> > +++ b/drivers/i2c/busses/Makefile
> > @@ -42,6 +42,7 @@ obj-$(CONFIG_I2C_DESIGNWARE_PLATFORM) += i2c-designware-platform.o
> > i2c-designware-platform-objs := i2c-designware-platdrv.o
> > obj-$(CONFIG_I2C_DESIGNWARE_PCI) += i2c-designware-pci.o
> > i2c-designware-pci-objs := i2c-designware-pcidrv.o
> > +obj-$(CONFIG_I2C_DESIGNWARE_ACPI) += i2c-designware-acpidrv.o
> > obj-$(CONFIG_I2C_EFM32) += i2c-efm32.o
> > obj-$(CONFIG_I2C_EG20T) += i2c-eg20t.o
> > obj-$(CONFIG_I2C_EXYNOS5) += i2c-exynos5.o
> > diff --git a/drivers/i2c/busses/i2c-designware-acpidrv.c b/drivers/i2c/busses/i2c-designware-acpidrv.c
> > new file mode 100644
> > index 0000000..ebe3a51
> > --- /dev/null
> > +++ b/drivers/i2c/busses/i2c-designware-acpidrv.c
> > @@ -0,0 +1,355 @@
> > +/*
> > + * Synopsys DesignWare I2C adapter driver (master only).
> > + *
> > + * Based on the TI DAVINCI I2C adapter driver.
> > + *
> > + * Copyright (C) 2006 Texas Instruments.
> > + * Copyright (C) 2007 MontaVista Software Inc.
> > + * Copyright (C) 2009 Provigent Ltd.
> > + * Copyright (C) 2011 Intel corporation.
> > + *
> > + * ----------------------------------------------------------------------------
> > + *
> > + * 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.
> > + *
> > + * This program is distributed in the hope that it will be useful,
> > + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> > + * GNU General Public License for more details.
> > + *
> > + * ----------------------------------------------------------------------------
> > + *
> > + */
> > +
> > +#include <linux/kernel.h>
> > +#include <linux/module.h>
> > +#include <linux/delay.h>
> > +#include <linux/i2c.h>
> > +#include <linux/errno.h>
> > +#include <linux/sched.h>
> > +#include <linux/err.h>
> > +#include <linux/interrupt.h>
> > +#include <linux/io.h>
> > +#include <linux/slab.h>
> > +#include <linux/pm_runtime.h>
> > +#include <linux/acpi.h>
> > +
> > +#include "i2c-designware-core.h"
> > +
> > +#define CLK_KHZ (133 * 1024)
> > +#define TX_FIFO_DEPTH 128
> > +#define RX_FIFO_DEPTH 128
> > +
> > +#define DRIVER_NAME "i2c-designware-acpi"
> > +
> > +#define INTEL_MID_STD_CFG (DW_IC_CON_MASTER | \
> > + DW_IC_CON_SLAVE_DISABLE | \
> > + DW_IC_CON_RESTART_EN)
> > +
> > +enum dw_acpi_ctl_id_t {
> > + I2C_BUS_A = 0,
> > + I2C_BUS_B,
> > + I2C_BUS_C,
> > + I2C_BUS_D,
> > +};
> > +
> > +struct dw_acpi_controller {
> > + u32 bus_num;
> > + u32 bus_cfg;
> > + u32 tx_fifo_depth;
> > + u32 rx_fifo_depth;
> > + u32 clk_khz;
> > +};
> > +
> > +/*i2c controller parameter*/
> > +static struct dw_acpi_controller dw_acpi_controllers[] = {
> > + [I2C_BUS_A] = {
> > + .bus_num = 1,
> > + .bus_cfg = INTEL_MID_STD_CFG | DW_IC_CON_SPEED_STD,
> > + .tx_fifo_depth = TX_FIFO_DEPTH,
> > + .rx_fifo_depth = RX_FIFO_DEPTH,
> > + .clk_khz = CLK_KHZ,
> > + },
> > + [I2C_BUS_B] = {
> > + .bus_num = 2,
> > + .bus_cfg = INTEL_MID_STD_CFG | DW_IC_CON_SPEED_STD,
> > + .tx_fifo_depth = TX_FIFO_DEPTH,
> > + .rx_fifo_depth = RX_FIFO_DEPTH,
> > + .clk_khz = CLK_KHZ,
> > + },
> > + [I2C_BUS_C] = {
> > + .bus_num = 3,
> > + .bus_cfg = INTEL_MID_STD_CFG | DW_IC_CON_SPEED_STD,
> > + .tx_fifo_depth = TX_FIFO_DEPTH,
> > + .rx_fifo_depth = RX_FIFO_DEPTH,
> > + .clk_khz = CLK_KHZ,
> > + },
> > + [I2C_BUS_D] = {
> > + .bus_num = 4,
> > + .bus_cfg = INTEL_MID_STD_CFG | DW_IC_CON_SPEED_STD,
> > + .tx_fifo_depth = TX_FIFO_DEPTH,
> > + .rx_fifo_depth = RX_FIFO_DEPTH,
> > + .clk_khz = CLK_KHZ,
> > + },
> > +};
> > +
> > +static struct i2c_algorithm i2c_dw_algo = {
> > + .master_xfer = i2c_dw_xfer,
> > + .functionality = i2c_dw_func,
> > +};
> > +
> > +static int i2c_dw_acpi_suspend(struct device *dev)
> > +{
> > + struct dw_i2c_dev *i2c = acpi_driver_data(to_acpi_device(dev));
> > +
> > + i2c_dw_disable(i2c);
> > +
> > + return 0;
> > +}
> > +
> > +static int i2c_dw_acpi_resume(struct device *dev)
> > +{
> > + struct dw_i2c_dev *i2c = acpi_driver_data(to_acpi_device(dev));
> > + u32 enabled;
> > +
> > + enabled = i2c_dw_is_enabled(i2c);
> > + if (enabled)
> > + return 0;
> > +
> > + i2c_dw_init(i2c);
> > +
> > + return 0;
> > +}
> > +
> > +static int i2c_dw_acpi_runtime_idle(struct device *dev)
> > +{
> > + int err = pm_schedule_suspend(dev, 500);
> > +
> > + dev_dbg(dev, "runtime_idle called\n");
> > +
> > + if (err != 0)
> > + return 0;
> > +
> > + return -EBUSY;
> > +}
> > +
> > +static int get_irq_flag(int trigger, int polarity, int share)
> > +{
> > + int flags;
> > +
> > + if (trigger == ACPI_LEVEL_SENSITIVE)
> > + flags = (polarity == ACPI_ACTIVE_LOW) ? IRQF_TRIGGER_LOW
> > + : IRQF_TRIGGER_HIGH;
> > + else
> > + flags = (polarity == ACPI_ACTIVE_LOW) ? IRQF_TRIGGER_FALLING
> > + : IRQF_TRIGGER_RISING;
> > +
> > + if (share == ACPI_SHARED)
> > + flags |= IRQF_SHARED;
> > +
> > + return flags;
> > +}
> > +
> > +static acpi_status
> > +acpi_i2c_parse_resource(struct acpi_resource *res, void *context)
> > +{
> > + int ret;
> > + struct dw_i2c_dev *dev = context;
> > + struct acpi_resource_fixed_memory32 *fixmem32;
> > +
> > + switch (res->type) {
> > + case ACPI_RESOURCE_TYPE_FIXED_MEMORY32:
> > + fixmem32 = &res->data.fixed_memory32;
> > + if (!fixmem32)
> > + return AE_NO_MEMORY;
> > +
> > + dev->io_base = fixmem32->address;
> > + dev->io_length = fixmem32->address_length;
> > + ret = AE_OK;
> > + break;
> > +
> > + case ACPI_RESOURCE_TYPE_IRQ:
> > + dev->irq = res->data.irq.interrupts[0];
> > + dev->irq_flag = get_irq_flag(res->data.irq.triggering,
> > + res->data.irq.polarity, res->data.irq.sharable);
> > + ret = AE_OK;
> > + break;
> > +
> > + default:
> > + if (dev->io_base && dev->irq)
> > + ret = AE_OK;
> > + else
> > + ret = AE_NOT_FOUND;
> > + break;
> > + }
> > +
> > + return ret;
> > +}
> > +
> > +static const struct dev_pm_ops i2c_dw_pm_ops = {
> > + .resume = i2c_dw_acpi_resume,
> > + .suspend = i2c_dw_acpi_suspend,
> > + SET_RUNTIME_PM_OPS(i2c_dw_acpi_suspend, i2c_dw_acpi_resume,
> > + i2c_dw_acpi_runtime_idle)
> > +};
> > +
> > +static u32 i2c_dw_get_clk_rate_khz(struct dw_i2c_dev *dev)
> > +{
> > + return dev->acpi_controller->clk_khz;
> > +}
> > +
> > +static u32 i2c_get_uid(struct acpi_device *dev)
> > +{
> > + return kstrtoul(dev->pnp.unique_id, 10, NULL);
> > +}
> > +
> > +static int i2c_dw_acpi_add(struct acpi_device *pdev)
> > +{
> > + int r;
> > + acpi_status status;
> > + struct dw_i2c_dev *dev;
> > + struct i2c_adapter *adap;
> > + struct dw_acpi_controller *controller;
> > + u32 controller_id;
> > +
> > + controller_id = i2c_get_uid(pdev);
> > +
> > + if (controller_id >= ARRAY_SIZE(dw_acpi_controllers)) {
> > + dev_err(&pdev->dev, "invalid controller number %d\n",
> > + controller_id);
> > + return -EINVAL;
> > + }
> > +
> > + controller = &dw_acpi_controllers[controller_id];
> > +
> > + dev = devm_kzalloc(&pdev->dev, sizeof(struct dw_i2c_dev), GFP_KERNEL);
> > + if (!dev)
> > + return -ENOMEM;
> > +
> > + status = acpi_walk_resources(pdev->handle, METHOD_NAME__CRS,
> > + acpi_i2c_parse_resource, dev);
> > + if (ACPI_FAILURE(status) || !dev->io_base || !dev->irq) {
> > + dev_err(&pdev->dev, "failure getting resource\n");
> > + return -ENODEV;
> > + }
> > +
> > + init_completion(&dev->cmd_complete);
> > + mutex_init(&dev->lock);
> > + dev->clk = NULL;
> > + dev->acpi_controller = controller;
> > + dev->get_clk_rate_khz = i2c_dw_get_clk_rate_khz;
> > + dev->base = devm_ioremap(&pdev->dev, dev->io_base, dev->io_length);
> > + if (dev->base == NULL) {
> > + dev_err(&pdev->dev, "failure mapping I/O memory\n");
> > + return -EBUSY;
> > + }
> > + dev->dev = &pdev->dev;
> > + dev->functionality =
> > + I2C_FUNC_I2C |
> > + I2C_FUNC_SMBUS_BYTE |
> > + I2C_FUNC_SMBUS_BYTE_DATA |
> > + I2C_FUNC_SMBUS_WORD_DATA |
> > + I2C_FUNC_SMBUS_I2C_BLOCK;
> > + dev->master_cfg = controller->bus_cfg;
> > +
> > + pdev->driver_data = dev;
> > +
> > + dev->tx_fifo_depth = controller->tx_fifo_depth;
> > + dev->rx_fifo_depth = controller->rx_fifo_depth;
> > +
> > + r = i2c_dw_init(dev);
> > + if (r) {
> > + dev_err(&pdev->dev, "failure initiating i2c controller\n");
> > + return -EINVAL;
> > + }
> > +
> > + adap = &dev->adapter;
> > + i2c_set_adapdata(adap, dev);
> > + adap->owner = THIS_MODULE;
> > + adap->class = I2C_CLASS_HWMON;
> > + adap->algo = &i2c_dw_algo;
> > + adap->dev.parent = &pdev->dev;
> > + ACPI_COMPANION_SET(adap->dev.parent, pdev);
> > + adap->nr = controller->bus_num;
> > + snprintf(adap->name, sizeof(adap->name), "i2c-designware-acpi-%d",
> > + adap->nr);
> > +
> > + r = devm_request_irq(&pdev->dev, dev->irq, i2c_dw_isr, dev->irq_flag,
> > + adap->name, dev);
> > + if (r) {
> > + dev_err(&pdev->dev, "failure requesting irq %d\n", dev->irq);
> > + return r;
> > + }
> > +
> > + i2c_dw_disable_int(dev);
> > + i2c_dw_clear_int(dev);
> > +
> > + r = i2c_add_numbered_adapter(adap);
> > + if (r) {
> > + dev_err(&pdev->dev, "failure adding adapter\n");
> > + return r;
> > + }
> > +
> > + pm_runtime_set_autosuspend_delay(&pdev->dev, 1000);
> > + pm_runtime_use_autosuspend(&pdev->dev);
> > + pm_runtime_allow(&pdev->dev);
> > +
> > + return 0;
> > +}
> > +
> > +static int i2c_dw_acpi_remove(struct acpi_device *pdev)
> > +{
> > + struct dw_i2c_dev *dev = acpi_driver_data(pdev);
> > +
> > + i2c_dw_disable(dev);
> > + pm_runtime_forbid(&pdev->dev);
> > + pm_runtime_get_noresume(&pdev->dev);
> > +
> > + i2c_del_adapter(&dev->adapter);
> > +
> > + return 0;
> > +}
> > +
> > +/* work with hotplug and coldplug */
> > +MODULE_ALIAS("i2c_designware-acpi");
> > +
> > +static const struct acpi_device_id i2_designware_acpi_ids[] = {
> > + { "IFC0000", 0},
> > + { "AMD0010", 0},
> > + { "", 0},
> > +};
> > +MODULE_DEVICE_TABLE(acpi, i2_designware_acpi_ids);
> > +
> > +static struct acpi_driver dw_i2c_driver = {
> > + .name = DRIVER_NAME,
> > + .class = DRIVER_NAME,
> > + .ids = i2_designware_acpi_ids,
> > + .ops = {
> > + .add = i2c_dw_acpi_add,
> > + .remove = i2c_dw_acpi_remove,
> > + },
> > + .drv.pm = &i2c_dw_pm_ops,
> > +};
> > +
> > +static int __init dw_i2c_module_init(void)
> > +{
> > + int ret;
> > +
> > + ret = acpi_bus_register_driver(&dw_i2c_driver);
> > +
> > + return ret;
> > +}
> > +module_init(dw_i2c_module_init);
> > +
> > +static void __exit dw_i2c_module_exit(void)
> > +{
> > + acpi_bus_unregister_driver(&dw_i2c_driver);
> > +}
> > +module_exit(dw_i2c_module_exit);
> > +
> > +MODULE_AUTHOR("Carl Peng <carlpeng008-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>");
> > +MODULE_DESCRIPTION("Synopsys DesignWare ACPI I2C bus adapter");
> > +MODULE_LICENSE("GPL");
> > diff --git a/drivers/i2c/busses/i2c-designware-core.h b/drivers/i2c/busses/i2c-designware-core.h
> > index d66b6cb..b88dbb8 100644
> > --- a/drivers/i2c/busses/i2c-designware-core.h
> > +++ b/drivers/i2c/busses/i2c-designware-core.h
> > @@ -73,11 +73,14 @@
> > struct dw_i2c_dev {
> > struct device *dev;
> > void __iomem *base;
> > + resource_size_t io_base;
> > + u32 io_length;
> > struct completion cmd_complete;
> > struct mutex lock;
> > struct clk *clk;
> > u32 (*get_clk_rate_khz) (struct dw_i2c_dev *dev);
> > struct dw_pci_controller *controller;
> > + struct dw_acpi_controller *acpi_controller;
> > int cmd_err;
> > struct i2c_msg *msgs;
> > int msgs_num;
> > @@ -91,6 +94,7 @@ struct dw_i2c_dev {
> > unsigned int status;
> > u32 abort_source;
> > int irq;
> > + unsigned int irq_flag;
> > u32 accessor_flags;
> > struct i2c_adapter adapter;
> > u32 functionality;
> > --
> > 1.9.1
> >
> > --
> > To unsubscribe from this list: send the line "unsubscribe linux-i2c" 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] 4+ messages in thread
end of thread, other threads:[~2014-08-29 8:52 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <carlpeng008@gmail.com>
2014-08-29 3:01 ` [PATCH] i2c-designware: Add support for ACPI i2c device Carl Peng
2014-08-29 3:09 ` Carl Peng
[not found] ` <1409281752-1473-1-git-send-email-carlpeng008-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2014-08-29 8:39 ` Wolfram Sang
2014-08-29 8:52 ` Mika Westerberg
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox