powerpc/83xx: Move mcu_mpc8349emitx driver out of drivers/i2c/chips/
authorAnton Vorontsov <avorontsov@ru.mvista.com>
Sun, 11 Jan 2009 16:55:39 +0000 (19:55 +0300)
committerKumar Gala <galak@kernel.crashing.org>
Tue, 13 Jan 2009 23:46:31 +0000 (17:46 -0600)
This patch is used to help Jean Delvare to get rid of drivers/i2c/chips/
directory. The new location suggested by Kumar Gala: as the driver is
83xx specific it's placed into arch/powerpc/platforms/83xx/.

Signed-off-by: Anton Vorontsov <avorontsov@ru.mvista.com>
Acked-by: Jean Delvare <khali@linux-fr.org>
Signed-off-by: Kumar Gala <galak@kernel.crashing.org>
arch/powerpc/platforms/83xx/Makefile
arch/powerpc/platforms/83xx/mcu_mpc8349emitx.c [new file with mode: 0644]
arch/powerpc/platforms/Kconfig
drivers/i2c/chips/Kconfig
drivers/i2c/chips/Makefile
drivers/i2c/chips/mcu_mpc8349emitx.c [deleted file]

index ba5028e2989065a7484733b0833fda94fdf78354..051777c542c74ac741225f7a070968fd56c31514 100644 (file)
@@ -3,6 +3,7 @@
 #
 obj-y                          := misc.o usb.o
 obj-$(CONFIG_SUSPEND)          += suspend.o suspend-asm.o
+obj-$(CONFIG_MCU_MPC8349EMITX) += mcu_mpc8349emitx.o
 obj-$(CONFIG_MPC831x_RDB)      += mpc831x_rdb.o
 obj-$(CONFIG_MPC832x_RDB)      += mpc832x_rdb.o
 obj-$(CONFIG_MPC834x_MDS)      += mpc834x_mds.o
diff --git a/arch/powerpc/platforms/83xx/mcu_mpc8349emitx.c b/arch/powerpc/platforms/83xx/mcu_mpc8349emitx.c
new file mode 100644 (file)
index 0000000..82a9bcb
--- /dev/null
@@ -0,0 +1,209 @@
+/*
+ * Power Management and GPIO expander driver for MPC8349E-mITX-compatible MCU
+ *
+ * Copyright (c) 2008  MontaVista Software, Inc.
+ *
+ * Author: Anton Vorontsov <avorontsov@ru.mvista.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/init.h>
+#include <linux/kernel.h>
+#include <linux/module.h>
+#include <linux/device.h>
+#include <linux/mutex.h>
+#include <linux/i2c.h>
+#include <linux/gpio.h>
+#include <linux/of.h>
+#include <linux/of_gpio.h>
+#include <asm/prom.h>
+#include <asm/machdep.h>
+
+/*
+ * I don't have specifications for the MCU firmware, I found this register
+ * and bits positions by the trial&error method.
+ */
+#define MCU_REG_CTRL   0x20
+#define MCU_CTRL_POFF  0x40
+
+#define MCU_NUM_GPIO   2
+
+struct mcu {
+       struct mutex lock;
+       struct device_node *np;
+       struct i2c_client *client;
+       struct of_gpio_chip of_gc;
+       u8 reg_ctrl;
+};
+
+static struct mcu *glob_mcu;
+
+static void mcu_power_off(void)
+{
+       struct mcu *mcu = glob_mcu;
+
+       pr_info("Sending power-off request to the MCU...\n");
+       mutex_lock(&mcu->lock);
+       i2c_smbus_write_byte_data(glob_mcu->client, MCU_REG_CTRL,
+                                 mcu->reg_ctrl | MCU_CTRL_POFF);
+       mutex_unlock(&mcu->lock);
+}
+
+static void mcu_gpio_set(struct gpio_chip *gc, unsigned int gpio, int val)
+{
+       struct of_gpio_chip *of_gc = to_of_gpio_chip(gc);
+       struct mcu *mcu = container_of(of_gc, struct mcu, of_gc);
+       u8 bit = 1 << (4 + gpio);
+
+       mutex_lock(&mcu->lock);
+       if (val)
+               mcu->reg_ctrl &= ~bit;
+       else
+               mcu->reg_ctrl |= bit;
+
+       i2c_smbus_write_byte_data(mcu->client, MCU_REG_CTRL, mcu->reg_ctrl);
+       mutex_unlock(&mcu->lock);
+}
+
+static int mcu_gpio_dir_out(struct gpio_chip *gc, unsigned int gpio, int val)
+{
+       mcu_gpio_set(gc, gpio, val);
+       return 0;
+}
+
+static int mcu_gpiochip_add(struct mcu *mcu)
+{
+       struct device_node *np;
+       struct of_gpio_chip *of_gc = &mcu->of_gc;
+       struct gpio_chip *gc = &of_gc->gc;
+       int ret;
+
+       np = of_find_compatible_node(NULL, NULL, "fsl,mcu-mpc8349emitx");
+       if (!np)
+               return -ENODEV;
+
+       gc->owner = THIS_MODULE;
+       gc->label = np->full_name;
+       gc->can_sleep = 1;
+       gc->ngpio = MCU_NUM_GPIO;
+       gc->base = -1;
+       gc->set = mcu_gpio_set;
+       gc->direction_output = mcu_gpio_dir_out;
+       of_gc->gpio_cells = 2;
+       of_gc->xlate = of_gpio_simple_xlate;
+
+       np->data = of_gc;
+       mcu->np = np;
+
+       /*
+        * We don't want to lose the node, its ->data and ->full_name...
+        * So, if succeeded, we don't put the node here.
+        */
+       ret = gpiochip_add(gc);
+       if (ret)
+               of_node_put(np);
+       return ret;
+}
+
+static int mcu_gpiochip_remove(struct mcu *mcu)
+{
+       int ret;
+
+       ret = gpiochip_remove(&mcu->of_gc.gc);
+       if (ret)
+               return ret;
+       of_node_put(mcu->np);
+
+       return 0;
+}
+
+static int __devinit mcu_probe(struct i2c_client *client,
+                              const struct i2c_device_id *id)
+{
+       struct mcu *mcu;
+       int ret;
+
+       mcu = kzalloc(sizeof(*mcu), GFP_KERNEL);
+       if (!mcu)
+               return -ENOMEM;
+
+       mutex_init(&mcu->lock);
+       mcu->client = client;
+       i2c_set_clientdata(client, mcu);
+
+       ret = i2c_smbus_read_byte_data(mcu->client, MCU_REG_CTRL);
+       if (ret < 0)
+               goto err;
+       mcu->reg_ctrl = ret;
+
+       ret = mcu_gpiochip_add(mcu);
+       if (ret)
+               goto err;
+
+       /* XXX: this is potentially racy, but there is no lock for ppc_md */
+       if (!ppc_md.power_off) {
+               glob_mcu = mcu;
+               ppc_md.power_off = mcu_power_off;
+               dev_info(&client->dev, "will provide power-off service\n");
+       }
+
+       return 0;
+err:
+       kfree(mcu);
+       return ret;
+}
+
+static int __devexit mcu_remove(struct i2c_client *client)
+{
+       struct mcu *mcu = i2c_get_clientdata(client);
+       int ret;
+
+       if (glob_mcu == mcu) {
+               ppc_md.power_off = NULL;
+               glob_mcu = NULL;
+       }
+
+       ret = mcu_gpiochip_remove(mcu);
+       if (ret)
+               return ret;
+       i2c_set_clientdata(client, NULL);
+       kfree(mcu);
+       return 0;
+}
+
+static const struct i2c_device_id mcu_ids[] = {
+       { "mcu-mpc8349emitx", },
+       {},
+};
+MODULE_DEVICE_TABLE(i2c, mcu_ids);
+
+static struct i2c_driver mcu_driver = {
+       .driver = {
+               .name = "mcu-mpc8349emitx",
+               .owner = THIS_MODULE,
+       },
+       .probe = mcu_probe,
+       .remove = __devexit_p(mcu_remove),
+       .id_table = mcu_ids,
+};
+
+static int __init mcu_init(void)
+{
+       return i2c_add_driver(&mcu_driver);
+}
+module_init(mcu_init);
+
+static void __exit mcu_exit(void)
+{
+       i2c_del_driver(&mcu_driver);
+}
+module_exit(mcu_exit);
+
+MODULE_DESCRIPTION("Power Management and GPIO expander driver for "
+                  "MPC8349E-mITX-compatible MCU");
+MODULE_AUTHOR("Anton Vorontsov <avorontsov@ru.mvista.com>");
+MODULE_LICENSE("GPL");
index 47fe2bea9865db32befbbdff10e8a14d2325de19..200b9cb900ea4b34280d270b51bebe773ca142b4 100644 (file)
@@ -323,4 +323,15 @@ config SIMPLE_GPIO
          chip-selects, Ethernet/USB PHY's power and various other small
          on-board peripherals.
 
+config MCU_MPC8349EMITX
+       tristate "MPC8349E-mITX MCU driver"
+       depends on I2C && PPC_83xx
+       select GENERIC_GPIO
+       select ARCH_REQUIRE_GPIOLIB
+       help
+         Say Y here to enable soft power-off functionality on the Freescale
+         boards with the MPC8349E-mITX-compatible MCU chips. This driver will
+         also register MCU GPIOs with the generic GPIO API, so you'll able
+         to use MCU pins as GPIOs.
+
 endmenu
index 59c3d23f5bdccdf54e0dbc7d5099e136ad7768e8..b9bef04b7be4fffb9eace1743621e98f7d0d47eb 100644 (file)
@@ -139,15 +139,4 @@ config SENSORS_TSL2550
          This driver can also be built as a module.  If so, the module
          will be called tsl2550.
 
-config MCU_MPC8349EMITX
-       tristate "MPC8349E-mITX MCU driver"
-       depends on I2C && PPC_83xx
-       select GENERIC_GPIO
-       select ARCH_REQUIRE_GPIOLIB
-       help
-         Say Y here to enable soft power-off functionality on the Freescale
-         boards with the MPC8349E-mITX-compatible MCU chips. This driver will
-         also register MCU GPIOs with the generic GPIO API, so you'll able
-         to use MCU pins as GPIOs.
-
 endmenu
index 83accaaf8164bfcc81397da82e363ee25e20434d..00fcb5193ac2c2fbe814193cfce09d114abe3631 100644 (file)
@@ -19,7 +19,6 @@ obj-$(CONFIG_SENSORS_PCF8574) += pcf8574.o
 obj-$(CONFIG_PCF8575)          += pcf8575.o
 obj-$(CONFIG_SENSORS_PCF8591)  += pcf8591.o
 obj-$(CONFIG_SENSORS_TSL2550)  += tsl2550.o
-obj-$(CONFIG_MCU_MPC8349EMITX) += mcu_mpc8349emitx.o
 
 ifeq ($(CONFIG_I2C_DEBUG_CHIP),y)
 EXTRA_CFLAGS += -DDEBUG
diff --git a/drivers/i2c/chips/mcu_mpc8349emitx.c b/drivers/i2c/chips/mcu_mpc8349emitx.c
deleted file mode 100644 (file)
index 82a9bcb..0000000
+++ /dev/null
@@ -1,209 +0,0 @@
-/*
- * Power Management and GPIO expander driver for MPC8349E-mITX-compatible MCU
- *
- * Copyright (c) 2008  MontaVista Software, Inc.
- *
- * Author: Anton Vorontsov <avorontsov@ru.mvista.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/init.h>
-#include <linux/kernel.h>
-#include <linux/module.h>
-#include <linux/device.h>
-#include <linux/mutex.h>
-#include <linux/i2c.h>
-#include <linux/gpio.h>
-#include <linux/of.h>
-#include <linux/of_gpio.h>
-#include <asm/prom.h>
-#include <asm/machdep.h>
-
-/*
- * I don't have specifications for the MCU firmware, I found this register
- * and bits positions by the trial&error method.
- */
-#define MCU_REG_CTRL   0x20
-#define MCU_CTRL_POFF  0x40
-
-#define MCU_NUM_GPIO   2
-
-struct mcu {
-       struct mutex lock;
-       struct device_node *np;
-       struct i2c_client *client;
-       struct of_gpio_chip of_gc;
-       u8 reg_ctrl;
-};
-
-static struct mcu *glob_mcu;
-
-static void mcu_power_off(void)
-{
-       struct mcu *mcu = glob_mcu;
-
-       pr_info("Sending power-off request to the MCU...\n");
-       mutex_lock(&mcu->lock);
-       i2c_smbus_write_byte_data(glob_mcu->client, MCU_REG_CTRL,
-                                 mcu->reg_ctrl | MCU_CTRL_POFF);
-       mutex_unlock(&mcu->lock);
-}
-
-static void mcu_gpio_set(struct gpio_chip *gc, unsigned int gpio, int val)
-{
-       struct of_gpio_chip *of_gc = to_of_gpio_chip(gc);
-       struct mcu *mcu = container_of(of_gc, struct mcu, of_gc);
-       u8 bit = 1 << (4 + gpio);
-
-       mutex_lock(&mcu->lock);
-       if (val)
-               mcu->reg_ctrl &= ~bit;
-       else
-               mcu->reg_ctrl |= bit;
-
-       i2c_smbus_write_byte_data(mcu->client, MCU_REG_CTRL, mcu->reg_ctrl);
-       mutex_unlock(&mcu->lock);
-}
-
-static int mcu_gpio_dir_out(struct gpio_chip *gc, unsigned int gpio, int val)
-{
-       mcu_gpio_set(gc, gpio, val);
-       return 0;
-}
-
-static int mcu_gpiochip_add(struct mcu *mcu)
-{
-       struct device_node *np;
-       struct of_gpio_chip *of_gc = &mcu->of_gc;
-       struct gpio_chip *gc = &of_gc->gc;
-       int ret;
-
-       np = of_find_compatible_node(NULL, NULL, "fsl,mcu-mpc8349emitx");
-       if (!np)
-               return -ENODEV;
-
-       gc->owner = THIS_MODULE;
-       gc->label = np->full_name;
-       gc->can_sleep = 1;
-       gc->ngpio = MCU_NUM_GPIO;
-       gc->base = -1;
-       gc->set = mcu_gpio_set;
-       gc->direction_output = mcu_gpio_dir_out;
-       of_gc->gpio_cells = 2;
-       of_gc->xlate = of_gpio_simple_xlate;
-
-       np->data = of_gc;
-       mcu->np = np;
-
-       /*
-        * We don't want to lose the node, its ->data and ->full_name...
-        * So, if succeeded, we don't put the node here.
-        */
-       ret = gpiochip_add(gc);
-       if (ret)
-               of_node_put(np);
-       return ret;
-}
-
-static int mcu_gpiochip_remove(struct mcu *mcu)
-{
-       int ret;
-
-       ret = gpiochip_remove(&mcu->of_gc.gc);
-       if (ret)
-               return ret;
-       of_node_put(mcu->np);
-
-       return 0;
-}
-
-static int __devinit mcu_probe(struct i2c_client *client,
-                              const struct i2c_device_id *id)
-{
-       struct mcu *mcu;
-       int ret;
-
-       mcu = kzalloc(sizeof(*mcu), GFP_KERNEL);
-       if (!mcu)
-               return -ENOMEM;
-
-       mutex_init(&mcu->lock);
-       mcu->client = client;
-       i2c_set_clientdata(client, mcu);
-
-       ret = i2c_smbus_read_byte_data(mcu->client, MCU_REG_CTRL);
-       if (ret < 0)
-               goto err;
-       mcu->reg_ctrl = ret;
-
-       ret = mcu_gpiochip_add(mcu);
-       if (ret)
-               goto err;
-
-       /* XXX: this is potentially racy, but there is no lock for ppc_md */
-       if (!ppc_md.power_off) {
-               glob_mcu = mcu;
-               ppc_md.power_off = mcu_power_off;
-               dev_info(&client->dev, "will provide power-off service\n");
-       }
-
-       return 0;
-err:
-       kfree(mcu);
-       return ret;
-}
-
-static int __devexit mcu_remove(struct i2c_client *client)
-{
-       struct mcu *mcu = i2c_get_clientdata(client);
-       int ret;
-
-       if (glob_mcu == mcu) {
-               ppc_md.power_off = NULL;
-               glob_mcu = NULL;
-       }
-
-       ret = mcu_gpiochip_remove(mcu);
-       if (ret)
-               return ret;
-       i2c_set_clientdata(client, NULL);
-       kfree(mcu);
-       return 0;
-}
-
-static const struct i2c_device_id mcu_ids[] = {
-       { "mcu-mpc8349emitx", },
-       {},
-};
-MODULE_DEVICE_TABLE(i2c, mcu_ids);
-
-static struct i2c_driver mcu_driver = {
-       .driver = {
-               .name = "mcu-mpc8349emitx",
-               .owner = THIS_MODULE,
-       },
-       .probe = mcu_probe,
-       .remove = __devexit_p(mcu_remove),
-       .id_table = mcu_ids,
-};
-
-static int __init mcu_init(void)
-{
-       return i2c_add_driver(&mcu_driver);
-}
-module_init(mcu_init);
-
-static void __exit mcu_exit(void)
-{
-       i2c_del_driver(&mcu_driver);
-}
-module_exit(mcu_exit);
-
-MODULE_DESCRIPTION("Power Management and GPIO expander driver for "
-                  "MPC8349E-mITX-compatible MCU");
-MODULE_AUTHOR("Anton Vorontsov <avorontsov@ru.mvista.com>");
-MODULE_LICENSE("GPL");