lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20151013073449.GU17172@x1>
Date:	Tue, 13 Oct 2015 08:34:49 +0100
From:	Lee Jones <lee.jones@...aro.org>
To:	"Andrew F. Davis" <afd@...com>
Cc:	Rob Herring <robh+dt@...nel.org>, Pawel Moll <pawel.moll@....com>,
	Mark Rutland <mark.rutland@....com>,
	Ian Campbell <ijc+devicetree@...lion.org.uk>,
	Kumar Gala <galak@...eaurora.org>,
	Mark Brown <broonie@...nel.org>,
	Alexandre Courbot <gnurou@...il.com>,
	Grygorii Strashko <grygorii.strashko@...com>,
	devicetree@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v4 3/5] mfd: tps65912: Add driver for the TPS65912 PMIC

On Mon, 12 Oct 2015, Andrew F. Davis wrote:

> On 10/05/2015 04:27 AM, Lee Jones wrote:
> >On Mon, 05 Oct 2015, Lee Jones wrote:
> >
> >>On Thu, 01 Oct 2015, Andrew F. Davis wrote:
> >>
> >>>This patch adds support for TPS65912 mfd device. It provides
> >>>communication through the I2C and SPI interfaces. It contains
> >>>the following components:
> >>>
> >>>  - Regulators
> >>>  - GPIO controller
> >>>
> >>>Signed-off-by: Andrew F. Davis <afd@...com>
> >>>---
> >>>  drivers/mfd/Kconfig          |  25 ++++
> >>>  drivers/mfd/Makefile         |   3 +
> >>>  drivers/mfd/tps65912-core.c  | 108 ++++++++++++++
> >>>  drivers/mfd/tps65912-i2c.c   |  82 ++++++++++
> >>>  drivers/mfd/tps65912-spi.c   |  81 ++++++++++
> >>>  include/linux/mfd/tps65912.h | 345 +++++++++++++++++++++++++++++++++++++++++++
> >>>  6 files changed, 644 insertions(+)
> >>>  create mode 100644 drivers/mfd/tps65912-core.c
> >>>  create mode 100644 drivers/mfd/tps65912-i2c.c
> >>>  create mode 100644 drivers/mfd/tps65912-spi.c
> >>>  create mode 100644 include/linux/mfd/tps65912.h
> >>
> >>Applied, thanks.
> >
> >Whoops, I forgot about the removal patch.
> >
> >For this patch:
> >   Acked-by: Lee Jones <lee.jones@...aro.org>
> >
> >Can you break up the removal patch into subsystems as well?
> >
> 
> I can if you would like, but as I responded in the other patch I
> think that will cause problems as the Kconfig symbols in the new
> will allow the old to be compiled and this will break build. All
> the removals may have to be brought in in a single patch, again
> up to you.

If the other Maintainers are happy, I'll just take the set through
MFD and provide the necessaries.  Just collect the Acks you need and
I'll do the rest.

> >>>diff --git a/drivers/mfd/Kconfig b/drivers/mfd/Kconfig
> >>>index 9a8df8e..02b321f 100644
> >>>--- a/drivers/mfd/Kconfig
> >>>+++ b/drivers/mfd/Kconfig
> >>>@@ -1167,6 +1167,31 @@ config MFD_TPS65910
> >>>  	  if you say yes here you get support for the TPS65910 series of
> >>>  	  Power Management chips.
> >>>
> >>>+config MFD_TPS65912
> >>>+	tristate
> >>>+	select REGMAP
> >>>+	select REGMAP_IRQ
> >>>+
> >>>+config MFD_TPS65912_I2C
> >>>+	tristate "TI TPS65912 Power Management chip with I2C"
> >>>+	select MFD_TPS65912
> >>>+	select REGMAP_I2C
> >>>+	depends on I2C
> >>>+	depends on OF || COMPILE_TEST
> >>>+	help
> >>>+	  If you say yes here you get support for the TPS65912 series of
> >>>+	  PM chips with I2C interface.
> >>>+
> >>>+config MFD_TPS65912_SPI
> >>>+	tristate "TI TPS65912 Power Management chip with SPI"
> >>>+	select MFD_TPS65912
> >>>+	select REGMAP_SPI
> >>>+	depends on SPI_MASTER
> >>>+	depends on OF || COMPILE_TEST
> >>>+	help
> >>>+	  If you say yes here you get support for the TPS65912 series of
> >>>+	  PM chips with SPI interface.
> >>>+
> >>>  config MFD_TPS80031
> >>>  	bool "TI TPS80031/TPS80032 Power Management chips"
> >>>  	depends on I2C=y
> >>>diff --git a/drivers/mfd/Makefile b/drivers/mfd/Makefile
> >>>index 004aa76..49c3530 100644
> >>>--- a/drivers/mfd/Makefile
> >>>+++ b/drivers/mfd/Makefile
> >>>@@ -69,6 +69,9 @@ obj-$(CONFIG_TPS6507X)		+= tps6507x.o
> >>>  obj-$(CONFIG_MFD_TPS65217)	+= tps65217.o
> >>>  obj-$(CONFIG_MFD_TPS65218)	+= tps65218.o
> >>>  obj-$(CONFIG_MFD_TPS65910)	+= tps65910.o
> >>>+obj-$(CONFIG_MFD_TPS65912)	+= tps65912-core.o
> >>>+obj-$(CONFIG_MFD_TPS65912_I2C)	+= tps65912-i2c.o
> >>>+obj-$(CONFIG_MFD_TPS65912_SPI)  += tps65912-spi.o
> >>>  obj-$(CONFIG_MFD_TPS80031)	+= tps80031.o
> >>>  obj-$(CONFIG_MENELAUS)		+= menelaus.o
> >>>
> >>>diff --git a/drivers/mfd/tps65912-core.c b/drivers/mfd/tps65912-core.c
> >>>new file mode 100644
> >>>index 0000000..e787950
> >>>--- /dev/null
> >>>+++ b/drivers/mfd/tps65912-core.c
> >>>@@ -0,0 +1,108 @@
> >>>+/*
> >>>+ * Core functions for TI TPS65912x PMIC
> >>>+ *
> >>>+ * Copyright (C) 2015 Texas Instruments Incorporated - http://www.ti.com/
> >>>+ *
> >>>+ * Author: Andrew F. Davis <afd@...com>
> >>>+ *
> >>>+ * This program is free software; you can redistribute it and/or
> >>>+ * modify it under the terms of the GNU General Public License version 2 as
> >>>+ * published by the Free Software Foundation.
> >>>+ *
> >>>+ * This program is distributed "as is" WITHOUT ANY WARRANTY of any
> >>>+ * kind, whether expressed or implied; without even the implied warranty
> >>>+ * of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> >>>+ * GNU General Public License version 2 for more details.
> >>>+ *
> >>>+ * Based on the TPS65218 driver and the previous TPS65912 driver by
> >>>+ * Margarita Olaya Cabrera <magi@...mlogic.co.uk>
> >>>+ */
> >>>+
> >>>+#include <linux/interrupt.h>
> >>>+#include <linux/module.h>
> >>>+#include <linux/of_device.h>
> >>>+
> >>>+#include <linux/mfd/tps65912.h>
> >>>+
> >>>+static const struct regmap_irq tps65912_irqs[] = {
> >>>+	/* INT_STS IRQs */
> >>>+	REGMAP_IRQ_REG(TPS65912_IRQ_PWRHOLD_F, 0, TPS65912_INT_STS_PWRHOLD_F),
> >>>+	REGMAP_IRQ_REG(TPS65912_IRQ_VMON, 0, TPS65912_INT_STS_VMON),
> >>>+	REGMAP_IRQ_REG(TPS65912_IRQ_PWRON, 0, TPS65912_INT_STS_PWRON),
> >>>+	REGMAP_IRQ_REG(TPS65912_IRQ_PWRON_LP, 0, TPS65912_INT_STS_PWRON_LP),
> >>>+	REGMAP_IRQ_REG(TPS65912_IRQ_PWRHOLD_R, 0, TPS65912_INT_STS_PWRHOLD_R),
> >>>+	REGMAP_IRQ_REG(TPS65912_IRQ_HOTDIE, 0, TPS65912_INT_STS_HOTDIE),
> >>>+	REGMAP_IRQ_REG(TPS65912_IRQ_GPIO1_R, 0, TPS65912_INT_STS_GPIO1_R),
> >>>+	REGMAP_IRQ_REG(TPS65912_IRQ_GPIO1_F, 0, TPS65912_INT_STS_GPIO1_F),
> >>>+	/* INT_STS2 IRQs */
> >>>+	REGMAP_IRQ_REG(TPS65912_IRQ_GPIO2_R, 1, TPS65912_INT_STS2_GPIO2_R),
> >>>+	REGMAP_IRQ_REG(TPS65912_IRQ_GPIO2_F, 1, TPS65912_INT_STS2_GPIO2_F),
> >>>+	REGMAP_IRQ_REG(TPS65912_IRQ_GPIO3_R, 1, TPS65912_INT_STS2_GPIO3_R),
> >>>+	REGMAP_IRQ_REG(TPS65912_IRQ_GPIO3_F, 1, TPS65912_INT_STS2_GPIO3_F),
> >>>+	REGMAP_IRQ_REG(TPS65912_IRQ_GPIO4_R, 1, TPS65912_INT_STS2_GPIO4_R),
> >>>+	REGMAP_IRQ_REG(TPS65912_IRQ_GPIO4_F, 1, TPS65912_INT_STS2_GPIO4_F),
> >>>+	REGMAP_IRQ_REG(TPS65912_IRQ_GPIO5_R, 1, TPS65912_INT_STS2_GPIO5_R),
> >>>+	REGMAP_IRQ_REG(TPS65912_IRQ_GPIO5_F, 1, TPS65912_INT_STS2_GPIO5_F),
> >>>+	/* INT_STS3 IRQs */
> >>>+	REGMAP_IRQ_REG(TPS65912_IRQ_PGOOD_DCDC1, 2, TPS65912_INT_STS3_PGOOD_DCDC1),
> >>>+	REGMAP_IRQ_REG(TPS65912_IRQ_PGOOD_DCDC2, 2, TPS65912_INT_STS3_PGOOD_DCDC2),
> >>>+	REGMAP_IRQ_REG(TPS65912_IRQ_PGOOD_DCDC3, 2, TPS65912_INT_STS3_PGOOD_DCDC3),
> >>>+	REGMAP_IRQ_REG(TPS65912_IRQ_PGOOD_DCDC4, 2, TPS65912_INT_STS3_PGOOD_DCDC4),
> >>>+	REGMAP_IRQ_REG(TPS65912_IRQ_PGOOD_LDO1, 2, TPS65912_INT_STS3_PGOOD_LDO1),
> >>>+	REGMAP_IRQ_REG(TPS65912_IRQ_PGOOD_LDO2, 2, TPS65912_INT_STS3_PGOOD_LDO2),
> >>>+	REGMAP_IRQ_REG(TPS65912_IRQ_PGOOD_LDO3, 2, TPS65912_INT_STS3_PGOOD_LDO3),
> >>>+	REGMAP_IRQ_REG(TPS65912_IRQ_PGOOD_LDO4, 2, TPS65912_INT_STS3_PGOOD_LDO4),
> >>>+	/* INT_STS4 IRQs */
> >>>+	REGMAP_IRQ_REG(TPS65912_IRQ_PGOOD_LDO5, 3, TPS65912_INT_STS4_PGOOD_LDO5),
> >>>+	REGMAP_IRQ_REG(TPS65912_IRQ_PGOOD_LDO6, 3, TPS65912_INT_STS4_PGOOD_LDO6),
> >>>+	REGMAP_IRQ_REG(TPS65912_IRQ_PGOOD_LDO7, 3, TPS65912_INT_STS4_PGOOD_LDO7),
> >>>+	REGMAP_IRQ_REG(TPS65912_IRQ_PGOOD_LDO8, 3, TPS65912_INT_STS4_PGOOD_LDO8),
> >>>+	REGMAP_IRQ_REG(TPS65912_IRQ_PGOOD_LDO9, 3, TPS65912_INT_STS4_PGOOD_LDO9),
> >>>+	REGMAP_IRQ_REG(TPS65912_IRQ_PGOOD_LDO10, 3, TPS65912_INT_STS4_PGOOD_LDO10),
> >>>+};
> >>>+
> >>>+static struct regmap_irq_chip tps65912_irq_chip = {
> >>>+	.name = "tps65912",
> >>>+	.irqs = tps65912_irqs,
> >>>+	.num_irqs = ARRAY_SIZE(tps65912_irqs),
> >>>+	.num_regs = 4,
> >>>+	.irq_reg_stride = 2,
> >>>+	.mask_base = TPS65912_INT_MSK,
> >>>+	.status_base = TPS65912_INT_STS,
> >>>+	.ack_base = TPS65912_INT_STS,
> >>>+	.init_ack_masked = true,
> >>>+};
> >>>+
> >>>+int tps65912_device_init(struct tps65912 *tps)
> >>>+{
> >>>+	int ret;
> >>>+
> >>>+	ret = regmap_add_irq_chip(tps->regmap, tps->irq, IRQF_ONESHOT, 0,
> >>>+				  &tps65912_irq_chip, &tps->irq_data);
> >>>+	if (ret < 0)
> >>>+		return ret;
> >>>+
> >>>+	ret = of_platform_populate(tps->dev->of_node, NULL, NULL, tps->dev);
> >>>+	if (ret < 0)
> >>>+		goto err_irq;
> >>>+
> >>>+	return 0;
> >>>+
> >>>+err_irq:
> >>>+	regmap_del_irq_chip(tps->irq, tps->irq_data);
> >>>+
> >>>+	return ret;
> >>>+}
> >>>+EXPORT_SYMBOL_GPL(tps65912_device_init);
> >>>+
> >>>+int tps65912_device_exit(struct tps65912 *tps)
> >>>+{
> >>>+	regmap_del_irq_chip(tps->irq, tps->irq_data);
> >>>+
> >>>+	return 0;
> >>>+}
> >>>+EXPORT_SYMBOL_GPL(tps65912_device_exit);
> >>>+
> >>>+MODULE_AUTHOR("Andrew F. Davis <afd@...com>");
> >>>+MODULE_DESCRIPTION("TPS65912x MFD Driver");
> >>>+MODULE_LICENSE("GPL v2");
> >>>diff --git a/drivers/mfd/tps65912-i2c.c b/drivers/mfd/tps65912-i2c.c
> >>>new file mode 100644
> >>>index 0000000..bdae10b01
> >>>--- /dev/null
> >>>+++ b/drivers/mfd/tps65912-i2c.c
> >>>@@ -0,0 +1,82 @@
> >>>+/*
> >>>+ * I2C access driver for TI TPS65912x PMIC
> >>>+ *
> >>>+ * Copyright (C) 2015 Texas Instruments Incorporated - http://www.ti.com/
> >>>+ *
> >>>+ * Author: Andrew F. Davis <afd@...com>
> >>>+ *
> >>>+ * This program is free software; you can redistribute it and/or
> >>>+ * modify it under the terms of the GNU General Public License version 2 as
> >>>+ * published by the Free Software Foundation.
> >>>+ *
> >>>+ * This program is distributed "as is" WITHOUT ANY WARRANTY of any
> >>>+ * kind, whether expressed or implied; without even the implied warranty
> >>>+ * of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> >>>+ * GNU General Public License version 2 for more details.
> >>>+ *
> >>>+ * Based on the TPS65218 driver and the previous TPS65912 driver by
> >>>+ * Margarita Olaya Cabrera <magi@...mlogic.co.uk>
> >>>+ */
> >>>+
> >>>+#include <linux/i2c.h>
> >>>+#include <linux/module.h>
> >>>+#include <linux/of_device.h>
> >>>+#include <linux/regmap.h>
> >>>+
> >>>+#include <linux/mfd/tps65912.h>
> >>>+
> >>>+static const struct of_device_id tps65912_i2c_of_match_table[] = {
> >>>+	{ .compatible = "ti,tps65912", },
> >>>+	{ /* sentinel */ }
> >>>+};
> >>>+
> >>>+static int tps65912_i2c_probe(struct i2c_client *client,
> >>>+			      const struct i2c_device_id *ids)
> >>>+{
> >>>+	struct tps65912 *tps;
> >>>+
> >>>+	tps = devm_kzalloc(&client->dev, sizeof(*tps), GFP_KERNEL);
> >>>+	if (!tps)
> >>>+		return -ENOMEM;
> >>>+
> >>>+	i2c_set_clientdata(client, tps);
> >>>+	tps->dev = &client->dev;
> >>>+	tps->irq = client->irq;
> >>>+
> >>>+	tps->regmap = devm_regmap_init_i2c(client, &tps65912_regmap_config);
> >>>+	if (IS_ERR(tps->regmap)) {
> >>>+		dev_err(tps->dev, "Failed to initialize register map\n");
> >>>+		return PTR_ERR(tps->regmap);
> >>>+	}
> >>>+
> >>>+	return tps65912_device_init(tps);
> >>>+}
> >>>+
> >>>+static int tps65912_i2c_remove(struct i2c_client *client)
> >>>+{
> >>>+	struct tps65912 *tps = i2c_get_clientdata(client);
> >>>+
> >>>+	return tps65912_device_exit(tps);
> >>>+}
> >>>+
> >>>+static const struct i2c_device_id tps65912_i2c_id_table[] = {
> >>>+	{ "tps65912", 0 },
> >>>+	{ /* sentinel */ },
> >>>+};
> >>>+MODULE_DEVICE_TABLE(i2c, tps65912_i2c_id_table);
> >>>+
> >>>+static struct i2c_driver tps65912_i2c_driver = {
> >>>+	.driver		= {
> >>>+		.name	= "tps65912",
> >>>+		.of_match_table = tps65912_i2c_of_match_table,
> >>>+	},
> >>>+	.probe		= tps65912_i2c_probe,
> >>>+	.remove		= tps65912_i2c_remove,
> >>>+	.id_table       = tps65912_i2c_id_table,
> >>>+};
> >>>+
> >>>+module_i2c_driver(tps65912_i2c_driver);
> >>>+
> >>>+MODULE_AUTHOR("Andrew F. Davis <afd@...com>");
> >>>+MODULE_DESCRIPTION("TPS65912x I2C Interface Driver");
> >>>+MODULE_LICENSE("GPL v2");
> >>>diff --git a/drivers/mfd/tps65912-spi.c b/drivers/mfd/tps65912-spi.c
> >>>new file mode 100644
> >>>index 0000000..24c5c97
> >>>--- /dev/null
> >>>+++ b/drivers/mfd/tps65912-spi.c
> >>>@@ -0,0 +1,81 @@
> >>>+/*
> >>>+ * SPI access driver for TI TPS65912x PMIC
> >>>+ *
> >>>+ * Copyright (C) 2015 Texas Instruments Incorporated - http://www.ti.com/
> >>>+ *
> >>>+ * Author: Andrew F. Davis <afd@...com>
> >>>+ *
> >>>+ * This program is free software; you can redistribute it and/or
> >>>+ * modify it under the terms of the GNU General Public License version 2 as
> >>>+ * published by the Free Software Foundation.
> >>>+ *
> >>>+ * This program is distributed "as is" WITHOUT ANY WARRANTY of any
> >>>+ * kind, whether expressed or implied; without even the implied warranty
> >>>+ * of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> >>>+ * GNU General Public License version 2 for more details.
> >>>+ *
> >>>+ * Based on the TPS65218 driver and the previous TPS65912 driver by
> >>>+ * Margarita Olaya Cabrera <magi@...mlogic.co.uk>
> >>>+ */
> >>>+
> >>>+#include <linux/module.h>
> >>>+#include <linux/of_device.h>
> >>>+#include <linux/regmap.h>
> >>>+#include <linux/spi/spi.h>
> >>>+
> >>>+#include <linux/mfd/tps65912.h>
> >>>+
> >>>+static const struct of_device_id tps65912_spi_of_match_table[] = {
> >>>+	{ .compatible = "ti,tps65912", },
> >>>+	{ /* sentinel */ }
> >>>+};
> >>>+
> >>>+static int tps65912_spi_probe(struct spi_device *spi)
> >>>+{
> >>>+	struct tps65912 *tps;
> >>>+
> >>>+	tps = devm_kzalloc(&spi->dev, sizeof(*tps), GFP_KERNEL);
> >>>+	if (!tps)
> >>>+		return -ENOMEM;
> >>>+
> >>>+	spi_set_drvdata(spi, tps);
> >>>+	tps->dev = &spi->dev;
> >>>+	tps->irq = spi->irq;
> >>>+
> >>>+	tps->regmap = devm_regmap_init_spi(spi, &tps65912_regmap_config);
> >>>+	if (IS_ERR(tps->regmap)) {
> >>>+		dev_err(tps->dev, "Failed to initialize register map\n");
> >>>+		return PTR_ERR(tps->regmap);
> >>>+	}
> >>>+
> >>>+	return tps65912_device_init(tps);
> >>>+}
> >>>+
> >>>+static int tps65912_spi_remove(struct spi_device *client)
> >>>+{
> >>>+	struct tps65912 *tps = spi_get_drvdata(client);
> >>>+
> >>>+	return tps65912_device_exit(tps);
> >>>+}
> >>>+
> >>>+static const struct spi_device_id tps65912_spi_id_table[] = {
> >>>+	{ "tps65912", 0 },
> >>>+	{ /* sentinel */ },
> >>>+};
> >>>+MODULE_DEVICE_TABLE(spi, tps65912_spi_id_table);
> >>>+
> >>>+static struct spi_driver tps65912_spi_driver = {
> >>>+	.driver		= {
> >>>+		.name	= "tps65912",
> >>>+		.of_match_table = tps65912_spi_of_match_table,
> >>>+	},
> >>>+	.probe		= tps65912_spi_probe,
> >>>+	.remove		= tps65912_spi_remove,
> >>>+	.id_table       = tps65912_spi_id_table,
> >>>+};
> >>>+
> >>>+module_spi_driver(tps65912_spi_driver);
> >>>+
> >>>+MODULE_AUTHOR("Andrew F. Davis <afd@...com>");
> >>>+MODULE_DESCRIPTION("TPS65912x SPI Interface Driver");
> >>>+MODULE_LICENSE("GPL v2");
> >>>diff --git a/include/linux/mfd/tps65912.h b/include/linux/mfd/tps65912.h
> >>>new file mode 100644
> >>>index 0000000..8843c0d
> >>>--- /dev/null
> >>>+++ b/include/linux/mfd/tps65912.h
> >>>@@ -0,0 +1,345 @@
> >>>+/*
> >>>+ * TI TPS65912x
> >>>+ *
> >>>+ * Copyright (C) 2015 Texas Instruments Incorporated - http://www.ti.com/
> >>>+ *
> >>>+ * Author: Andrew F. Davis <afd@...com>
> >>>+ *
> >>>+ * This program is free software; you can redistribute it and/or
> >>>+ * modify it under the terms of the GNU General Public License version 2 as
> >>>+ * published by the Free Software Foundation.
> >>>+ *
> >>>+ * This program is distributed "as is" WITHOUT ANY WARRANTY of any
> >>>+ * kind, whether expressed or implied; without even the implied warranty
> >>>+ * of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> >>>+ * GNU General Public License version 2 for more details.
> >>>+ *
> >>>+ * Based on the TPS65218 driver and the previous TPS65912 driver by
> >>>+ * Margarita Olaya Cabrera <magi@...mlogic.co.uk>
> >>>+ */
> >>>+
> >>>+#ifndef __LINUX_MFD_TPS65912_H
> >>>+#define __LINUX_MFD_TPS65912_H
> >>>+
> >>>+#include <linux/device.h>
> >>>+#include <linux/regmap.h>
> >>>+#include <linux/regulator/driver.h>
> >>>+
> >>>+/* List of registers for TPS65912 */
> >>>+#define TPS65912_DCDC1_CTRL		0x00
> >>>+#define TPS65912_DCDC2_CTRL		0x01
> >>>+#define TPS65912_DCDC3_CTRL		0x02
> >>>+#define TPS65912_DCDC4_CTRL		0x03
> >>>+#define TPS65912_DCDC1_OP		0x04
> >>>+#define TPS65912_DCDC1_AVS		0x05
> >>>+#define TPS65912_DCDC1_LIMIT		0x06
> >>>+#define TPS65912_DCDC2_OP		0x07
> >>>+#define TPS65912_DCDC2_AVS		0x08
> >>>+#define TPS65912_DCDC2_LIMIT		0x09
> >>>+#define TPS65912_DCDC3_OP		0x0A
> >>>+#define TPS65912_DCDC3_AVS		0x0B
> >>>+#define TPS65912_DCDC3_LIMIT		0x0C
> >>>+#define TPS65912_DCDC4_OP		0x0D
> >>>+#define TPS65912_DCDC4_AVS		0x0E
> >>>+#define TPS65912_DCDC4_LIMIT		0x0F
> >>>+#define TPS65912_LDO1_OP		0x10
> >>>+#define TPS65912_LDO1_AVS		0x11
> >>>+#define TPS65912_LDO1_LIMIT		0x12
> >>>+#define TPS65912_LDO2_OP		0x13
> >>>+#define TPS65912_LDO2_AVS		0x14
> >>>+#define TPS65912_LDO2_LIMIT		0x15
> >>>+#define TPS65912_LDO3_OP		0x16
> >>>+#define TPS65912_LDO3_AVS		0x17
> >>>+#define TPS65912_LDO3_LIMIT		0x18
> >>>+#define TPS65912_LDO4_OP		0x19
> >>>+#define TPS65912_LDO4_AVS		0x1A
> >>>+#define TPS65912_LDO4_LIMIT		0x1B
> >>>+#define TPS65912_LDO5			0x1C
> >>>+#define TPS65912_LDO6			0x1D
> >>>+#define TPS65912_LDO7			0x1E
> >>>+#define TPS65912_LDO8			0x1F
> >>>+#define TPS65912_LDO9			0x20
> >>>+#define TPS65912_LDO10			0x21
> >>>+#define TPS65912_THRM			0x22
> >>>+#define TPS65912_CLK32OUT		0x23
> >>>+#define TPS65912_DEVCTRL		0x24
> >>>+#define TPS65912_DEVCTRL2		0x25
> >>>+#define TPS65912_I2C_SPI_CFG		0x26
> >>>+#define TPS65912_KEEP_ON		0x27
> >>>+#define TPS65912_KEEP_ON2		0x28
> >>>+#define TPS65912_SET_OFF1		0x29
> >>>+#define TPS65912_SET_OFF2		0x2A
> >>>+#define TPS65912_DEF_VOLT		0x2B
> >>>+#define TPS65912_DEF_VOLT_MAPPING	0x2C
> >>>+#define TPS65912_DISCHARGE		0x2D
> >>>+#define TPS65912_DISCHARGE2		0x2E
> >>>+#define TPS65912_EN1_SET1		0x2F
> >>>+#define TPS65912_EN1_SET2		0x30
> >>>+#define TPS65912_EN2_SET1		0x31
> >>>+#define TPS65912_EN2_SET2		0x32
> >>>+#define TPS65912_EN3_SET1		0x33
> >>>+#define TPS65912_EN3_SET2		0x34
> >>>+#define TPS65912_EN4_SET1		0x35
> >>>+#define TPS65912_EN4_SET2		0x36
> >>>+#define TPS65912_PGOOD			0x37
> >>>+#define TPS65912_PGOOD2			0x38
> >>>+#define TPS65912_INT_STS		0x39
> >>>+#define TPS65912_INT_MSK		0x3A
> >>>+#define TPS65912_INT_STS2		0x3B
> >>>+#define TPS65912_INT_MSK2		0x3C
> >>>+#define TPS65912_INT_STS3		0x3D
> >>>+#define TPS65912_INT_MSK3		0x3E
> >>>+#define TPS65912_INT_STS4		0x3F
> >>>+#define TPS65912_INT_MSK4		0x40
> >>>+#define TPS65912_GPIO1			0x41
> >>>+#define TPS65912_GPIO2			0x42
> >>>+#define TPS65912_GPIO3			0x43
> >>>+#define TPS65912_GPIO4			0x44
> >>>+#define TPS65912_GPIO5			0x45
> >>>+#define TPS65912_VMON			0x46
> >>>+#define TPS65912_LEDA_CTRL1		0x47
> >>>+#define TPS65912_LEDA_CTRL2		0x48
> >>>+#define TPS65912_LEDA_CTRL3		0x49
> >>>+#define TPS65912_LEDA_CTRL4		0x4A
> >>>+#define TPS65912_LEDA_CTRL5		0x4B
> >>>+#define TPS65912_LEDA_CTRL6		0x4C
> >>>+#define TPS65912_LEDA_CTRL7		0x4D
> >>>+#define TPS65912_LEDA_CTRL8		0x4E
> >>>+#define TPS65912_LEDB_CTRL1		0x4F
> >>>+#define TPS65912_LEDB_CTRL2		0x50
> >>>+#define TPS65912_LEDB_CTRL3		0x51
> >>>+#define TPS65912_LEDB_CTRL4		0x52
> >>>+#define TPS65912_LEDB_CTRL5		0x53
> >>>+#define TPS65912_LEDB_CTRL6		0x54
> >>>+#define TPS65912_LEDB_CTRL7		0x55
> >>>+#define TPS65912_LEDB_CTRL8		0x56
> >>>+#define TPS65912_LEDC_CTRL1		0x57
> >>>+#define TPS65912_LEDC_CTRL2		0x58
> >>>+#define TPS65912_LEDC_CTRL3		0x59
> >>>+#define TPS65912_LEDC_CTRL4		0x5A
> >>>+#define TPS65912_LEDC_CTRL5		0x5B
> >>>+#define TPS65912_LEDC_CTRL6		0x5C
> >>>+#define TPS65912_LEDC_CTRL7		0x5D
> >>>+#define TPS65912_LEDC_CTRL8		0x5E
> >>>+#define TPS65912_LED_RAMP_UP_TIME	0x5F
> >>>+#define TPS65912_LED_RAMP_DOWN_TIME	0x60
> >>>+#define TPS65912_LED_SEQ_EN		0x61
> >>>+#define TPS65912_LOADSWITCH		0x62
> >>>+#define TPS65912_SPARE			0x63
> >>>+#define TPS65912_VERNUM			0x64
> >>>+#define TPS6591X_MAX_REGISTER		0x64
> >>>+
> >>>+/* INT_STS Register field definitions */
> >>>+#define TPS65912_INT_STS_PWRHOLD_F	BIT(0)
> >>>+#define TPS65912_INT_STS_VMON		BIT(1)
> >>>+#define TPS65912_INT_STS_PWRON		BIT(2)
> >>>+#define TPS65912_INT_STS_PWRON_LP	BIT(3)
> >>>+#define TPS65912_INT_STS_PWRHOLD_R	BIT(4)
> >>>+#define TPS65912_INT_STS_HOTDIE		BIT(5)
> >>>+#define TPS65912_INT_STS_GPIO1_R	BIT(6)
> >>>+#define TPS65912_INT_STS_GPIO1_F	BIT(7)
> >>>+
> >>>+/* INT_STS Register field definitions */
> >>>+#define TPS65912_INT_STS2_GPIO2_R	BIT(0)
> >>>+#define TPS65912_INT_STS2_GPIO2_F	BIT(1)
> >>>+#define TPS65912_INT_STS2_GPIO3_R	BIT(2)
> >>>+#define TPS65912_INT_STS2_GPIO3_F	BIT(3)
> >>>+#define TPS65912_INT_STS2_GPIO4_R	BIT(4)
> >>>+#define TPS65912_INT_STS2_GPIO4_F	BIT(5)
> >>>+#define TPS65912_INT_STS2_GPIO5_R	BIT(6)
> >>>+#define TPS65912_INT_STS2_GPIO5_F	BIT(7)
> >>>+
> >>>+/* INT_STS Register field definitions */
> >>>+#define TPS65912_INT_STS3_PGOOD_DCDC1	BIT(0)
> >>>+#define TPS65912_INT_STS3_PGOOD_DCDC2	BIT(1)
> >>>+#define TPS65912_INT_STS3_PGOOD_DCDC3	BIT(2)
> >>>+#define TPS65912_INT_STS3_PGOOD_DCDC4	BIT(3)
> >>>+#define TPS65912_INT_STS3_PGOOD_LDO1	BIT(4)
> >>>+#define TPS65912_INT_STS3_PGOOD_LDO2	BIT(5)
> >>>+#define TPS65912_INT_STS3_PGOOD_LDO3	BIT(6)
> >>>+#define TPS65912_INT_STS3_PGOOD_LDO4	BIT(7)
> >>>+
> >>>+/* INT_STS Register field definitions */
> >>>+#define TPS65912_INT_STS4_PGOOD_LDO5	BIT(0)
> >>>+#define TPS65912_INT_STS4_PGOOD_LDO6	BIT(1)
> >>>+#define TPS65912_INT_STS4_PGOOD_LDO7	BIT(2)
> >>>+#define TPS65912_INT_STS4_PGOOD_LDO8	BIT(3)
> >>>+#define TPS65912_INT_STS4_PGOOD_LDO9	BIT(4)
> >>>+#define TPS65912_INT_STS4_PGOOD_LDO10	BIT(5)
> >>>+
> >>>+/* GPIO 1 and 2 Register field definitions */
> >>>+#define GPIO_SLEEP_MASK			0x80
> >>>+#define GPIO_SLEEP_SHIFT		7
> >>>+#define GPIO_DEB_MASK			0x10
> >>>+#define GPIO_DEB_SHIFT			4
> >>>+#define GPIO_CFG_MASK			0x04
> >>>+#define GPIO_CFG_SHIFT			2
> >>>+#define GPIO_STS_MASK			0x02
> >>>+#define GPIO_STS_SHIFT			1
> >>>+#define GPIO_SET_MASK			0x01
> >>>+#define GPIO_SET_SHIFT			0
> >>>+
> >>>+/* GPIO 3 Register field definitions */
> >>>+#define GPIO3_SLEEP_MASK		0x80
> >>>+#define GPIO3_SLEEP_SHIFT		7
> >>>+#define GPIO3_SEL_MASK			0x40
> >>>+#define GPIO3_SEL_SHIFT			6
> >>>+#define GPIO3_ODEN_MASK			0x20
> >>>+#define GPIO3_ODEN_SHIFT		5
> >>>+#define GPIO3_DEB_MASK			0x10
> >>>+#define GPIO3_DEB_SHIFT			4
> >>>+#define GPIO3_PDEN_MASK			0x08
> >>>+#define GPIO3_PDEN_SHIFT		3
> >>>+#define GPIO3_CFG_MASK			0x04
> >>>+#define GPIO3_CFG_SHIFT			2
> >>>+#define GPIO3_STS_MASK			0x02
> >>>+#define GPIO3_STS_SHIFT			1
> >>>+#define GPIO3_SET_MASK			0x01
> >>>+#define GPIO3_SET_SHIFT			0
> >>>+
> >>>+/* GPIO 4 Register field definitions */
> >>>+#define GPIO4_SLEEP_MASK		0x80
> >>>+#define GPIO4_SLEEP_SHIFT		7
> >>>+#define GPIO4_SEL_MASK			0x40
> >>>+#define GPIO4_SEL_SHIFT			6
> >>>+#define GPIO4_ODEN_MASK			0x20
> >>>+#define GPIO4_ODEN_SHIFT		5
> >>>+#define GPIO4_DEB_MASK			0x10
> >>>+#define GPIO4_DEB_SHIFT			4
> >>>+#define GPIO4_PDEN_MASK			0x08
> >>>+#define GPIO4_PDEN_SHIFT		3
> >>>+#define GPIO4_CFG_MASK			0x04
> >>>+#define GPIO4_CFG_SHIFT			2
> >>>+#define GPIO4_STS_MASK			0x02
> >>>+#define GPIO4_STS_SHIFT			1
> >>>+#define GPIO4_SET_MASK			0x01
> >>>+#define GPIO4_SET_SHIFT			0
> >>>+
> >>>+/* Register THERM  (0x80) register.RegisterDescription */
> >>>+#define THERM_THERM_HD_MASK		0x20
> >>>+#define THERM_THERM_HD_SHIFT		5
> >>>+#define THERM_THERM_TS_MASK		0x10
> >>>+#define THERM_THERM_TS_SHIFT		4
> >>>+#define THERM_THERM_HDSEL_MASK		0x0C
> >>>+#define THERM_THERM_HDSEL_SHIFT		2
> >>>+#define THERM_RSVD1_MASK		0x02
> >>>+#define THERM_RSVD1_SHIFT		1
> >>>+#define THERM_THERM_STATE_MASK		0x01
> >>>+#define THERM_THERM_STATE_SHIFT		0
> >>>+
> >>>+/* Register DCDCCTRL1 register.RegisterDescription */
> >>>+#define DCDCCTRL_VCON_ENABLE_MASK	0x80
> >>>+#define DCDCCTRL_VCON_ENABLE_SHIFT	7
> >>>+#define DCDCCTRL_VCON_RANGE1_MASK	0x40
> >>>+#define DCDCCTRL_VCON_RANGE1_SHIFT	6
> >>>+#define DCDCCTRL_VCON_RANGE0_MASK	0x20
> >>>+#define DCDCCTRL_VCON_RANGE0_SHIFT	5
> >>>+#define DCDCCTRL_TSTEP2_MASK		0x10
> >>>+#define DCDCCTRL_TSTEP2_SHIFT		4
> >>>+#define DCDCCTRL_TSTEP1_MASK		0x08
> >>>+#define DCDCCTRL_TSTEP1_SHIFT		3
> >>>+#define DCDCCTRL_TSTEP0_MASK		0x04
> >>>+#define DCDCCTRL_TSTEP0_SHIFT		2
> >>>+#define DCDCCTRL_DCDC1_MODE_MASK	0x02
> >>>+#define DCDCCTRL_DCDC1_MODE_SHIFT	1
> >>>+
> >>>+/* Register DCDCCTRL2 and DCDCCTRL3 register.RegisterDescription */
> >>>+#define DCDCCTRL_TSTEP2_MASK		0x10
> >>>+#define DCDCCTRL_TSTEP2_SHIFT		4
> >>>+#define DCDCCTRL_TSTEP1_MASK		0x08
> >>>+#define DCDCCTRL_TSTEP1_SHIFT		3
> >>>+#define DCDCCTRL_TSTEP0_MASK		0x04
> >>>+#define DCDCCTRL_TSTEP0_SHIFT		2
> >>>+#define DCDCCTRL_DCDC_MODE_MASK		0x02
> >>>+#define DCDCCTRL_DCDC_MODE_SHIFT	1
> >>>+#define DCDCCTRL_RSVD0_MASK		0x01
> >>>+#define DCDCCTRL_RSVD0_SHIFT		0
> >>>+
> >>>+/* Register DCDCCTRL4 register.RegisterDescription */
> >>>+#define DCDCCTRL_RAMP_TIME_MASK		0x01
> >>>+#define DCDCCTRL_RAMP_TIME_SHIFT	0
> >>>+
> >>>+/* Register DCDCx_AVS */
> >>>+#define DCDC_AVS_ENABLE_MASK		0x80
> >>>+#define DCDC_AVS_ENABLE_SHIFT		7
> >>>+#define DCDC_AVS_ECO_MASK		0x40
> >>>+#define DCDC_AVS_ECO_SHIFT		6
> >>>+
> >>>+/* Register DCDCx_LIMIT */
> >>>+#define DCDC_LIMIT_RANGE_MASK		0xC0
> >>>+#define DCDC_LIMIT_RANGE_SHIFT		6
> >>>+#define DCDC_LIMIT_MAX_SEL_MASK		0x3F
> >>>+#define DCDC_LIMIT_MAX_SEL_SHIFT	0
> >>>+
> >>>+/* Define the TPS65912 IRQ numbers */
> >>>+enum tps65912_irqs {
> >>>+	/* INT_STS registers */
> >>>+	TPS65912_IRQ_PWRHOLD_F,
> >>>+	TPS65912_IRQ_VMON,
> >>>+	TPS65912_IRQ_PWRON,
> >>>+	TPS65912_IRQ_PWRON_LP,
> >>>+	TPS65912_IRQ_PWRHOLD_R,
> >>>+	TPS65912_IRQ_HOTDIE,
> >>>+	TPS65912_IRQ_GPIO1_R,
> >>>+	TPS65912_IRQ_GPIO1_F,
> >>>+	/* INT_STS2 registers */
> >>>+	TPS65912_IRQ_GPIO2_R,
> >>>+	TPS65912_IRQ_GPIO2_F,
> >>>+	TPS65912_IRQ_GPIO3_R,
> >>>+	TPS65912_IRQ_GPIO3_F,
> >>>+	TPS65912_IRQ_GPIO4_R,
> >>>+	TPS65912_IRQ_GPIO4_F,
> >>>+	TPS65912_IRQ_GPIO5_R,
> >>>+	TPS65912_IRQ_GPIO5_F,
> >>>+	/* INT_STS3 registers */
> >>>+	TPS65912_IRQ_PGOOD_DCDC1,
> >>>+	TPS65912_IRQ_PGOOD_DCDC2,
> >>>+	TPS65912_IRQ_PGOOD_DCDC3,
> >>>+	TPS65912_IRQ_PGOOD_DCDC4,
> >>>+	TPS65912_IRQ_PGOOD_LDO1,
> >>>+	TPS65912_IRQ_PGOOD_LDO2,
> >>>+	TPS65912_IRQ_PGOOD_LDO3,
> >>>+	TPS65912_IRQ_PGOOD_LDO4,
> >>>+	/* INT_STS4 registers */
> >>>+	TPS65912_IRQ_PGOOD_LDO5,
> >>>+	TPS65912_IRQ_PGOOD_LDO6,
> >>>+	TPS65912_IRQ_PGOOD_LDO7,
> >>>+	TPS65912_IRQ_PGOOD_LDO8,
> >>>+	TPS65912_IRQ_PGOOD_LDO9,
> >>>+	TPS65912_IRQ_PGOOD_LDO10,
> >>>+};
> >>>+
> >>>+/*
> >>>+ * struct tps65912 - state holder for the tps65912 driver
> >>>+ *
> >>>+ * Device data may be used to access the TPS65912 chip
> >>>+ */
> >>>+struct tps65912 {
> >>>+	struct device *dev;
> >>>+	struct regmap *regmap;
> >>>+
> >>>+	/* IRQ Data */
> >>>+	int irq;
> >>>+	struct regmap_irq_chip_data *irq_data;
> >>>+};
> >>>+
> >>>+static const struct regmap_range tps65912_yes_ranges[] = {
> >>>+	regmap_reg_range(TPS65912_INT_STS, TPS65912_GPIO5),
> >>>+};
> >>>+
> >>>+static const struct regmap_access_table tps65912_volatile_table = {
> >>>+	.yes_ranges = tps65912_yes_ranges,
> >>>+	.n_yes_ranges = ARRAY_SIZE(tps65912_yes_ranges),
> >>>+};
> >>>+
> >>>+static const struct regmap_config tps65912_regmap_config = {
> >>>+	.reg_bits = 8,
> >>>+	.val_bits = 8,
> >>>+	.cache_type = REGCACHE_RBTREE,
> >>>+	.volatile_table = &tps65912_volatile_table,
> >>>+};
> >>>+
> >>>+int tps65912_device_init(struct tps65912 *tps);
> >>>+int tps65912_device_exit(struct tps65912 *tps);
> >>>+
> >>>+#endif /*  __LINUX_MFD_TPS65912_H */
> >>
> >
> 

-- 
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ