From mboxrd@z Thu Jan 1 00:00:00 1970 From: Przemyslaw Marczak Date: Fri, 10 Jul 2015 13:52:10 +0200 Subject: [U-Boot] [PATCH 21/55] dm: power: Add a new driver for the TPS65090 PMIC In-Reply-To: <1435882592-487-22-git-send-email-sjg@chromium.org> References: <1435882592-487-1-git-send-email-sjg@chromium.org> <1435882592-487-22-git-send-email-sjg@chromium.org> Message-ID: <559FB1EA.2090909@samsung.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hello Simon, On 07/03/2015 02:15 AM, Simon Glass wrote: > The existing TPS65090 driver does not support driver model. Add a new one > that does. This can be used as a base for a regulator driver also. It uses > the standard device tree binding. > > Signed-off-by: Simon Glass > --- > > drivers/power/pmic/Kconfig | 9 +++++ > drivers/power/pmic/Makefile | 1 + > drivers/power/pmic/tps65090.c | 94 +++++++++++++++++++++++++++++++++++++++++++ > include/power/tps65090.h | 56 ++++++++++++++++++++++++++ > 4 files changed, 160 insertions(+) > create mode 100644 drivers/power/pmic/tps65090.c > create mode 100644 include/power/tps65090.h > > diff --git a/drivers/power/pmic/Kconfig b/drivers/power/pmic/Kconfig > index 164f421..fd8af81 100644 > --- a/drivers/power/pmic/Kconfig > +++ b/drivers/power/pmic/Kconfig > @@ -41,3 +41,12 @@ config DM_PMIC_SANDBOX > - set by i2c emul driver's probe() (defaults in header) > > Driver binding info: doc/device-tree-bindings/pmic/sandbox.txt > + > +config PMIC_TPS65090 > + bool "Enable driver for Texas Instruments TPS65090 PMIC" > + depends on DM_PMIC > + ---help--- > + The TPS65090 is a PMIC containing several LDOs, DC to DC convertors, > + FETs and a battery charger. This driver provides register access > + only, and you can enable the regulator/charger drivers separately if > + required. > diff --git a/drivers/power/pmic/Makefile b/drivers/power/pmic/Makefile > index ae86f04..2316e3b 100644 > --- a/drivers/power/pmic/Makefile > +++ b/drivers/power/pmic/Makefile > @@ -8,6 +8,7 @@ > obj-$(CONFIG_DM_PMIC) += pmic-uclass.o > obj-$(CONFIG_DM_PMIC_MAX77686) += max77686.o > obj-$(CONFIG_DM_PMIC_SANDBOX) += sandbox.o i2c_pmic_emul.o > +obj-$(CONFIG_PMIC_TPS65090) += tps65090.o > obj-$(CONFIG_POWER_LTC3676) += pmic_ltc3676.o > obj-$(CONFIG_POWER_MAX8998) += pmic_max8998.o > obj-$(CONFIG_POWER_MAX8997) += pmic_max8997.o > diff --git a/drivers/power/pmic/tps65090.c b/drivers/power/pmic/tps65090.c > new file mode 100644 > index 0000000..4797f32 > --- /dev/null > +++ b/drivers/power/pmic/tps65090.c > @@ -0,0 +1,94 @@ > +/* > + * Copyright (c) 2015 Google, Inc > + * Written by Simon Glass > + * > + * SPDX-License-Identifier: GPL-2.0+ > + */ > + > +#include > +#include > +#include > +#include > +#include > +#include > +#include > + > +DECLARE_GLOBAL_DATA_PTR; > + > +static const struct pmic_child_info pmic_children_info[] = { > + { .prefix = "fet", .driver = TPS65090_FET_DRIVER }, > + { }, > +}; > + > +static int tps65090_reg_count(struct udevice *dev) > +{ > + return TPS65090_NUM_REGS; > +} > + > +static int tps65090_write(struct udevice *dev, uint reg, const uint8_t *buff, > + int len) > +{ > + if (dm_i2c_write(dev, reg, buff, len)) { > + error("write error to device: %p register: %#x!", dev, reg); > + return -EIO; > + } > + > + return 0; > +} > + > +static int tps65090_read(struct udevice *dev, uint reg, uint8_t *buff, int len) > +{ > + int ret; > + > + ret = dm_i2c_read(dev, reg, buff, len); > + if (ret) { > + error("read error %d from device: %p register: %#x!", ret, dev, > + reg); > + return -EIO; > + } > + > + return 0; > +} > + > +static int tps65090_bind(struct udevice *dev) > +{ > + int regulators_node; > + const void *blob = gd->fdt_blob; > + int children; > + > + regulators_node = fdt_subnode_offset(blob, dev->of_offset, > + "regulators"); > + if (regulators_node <= 0) { > + debug("%s: %s regulators subnode not found!", __func__, > + dev->name); > + return -ENXIO; > + } > + > + debug("%s: '%s' - found regulators subnode\n", __func__, dev->name); > + > + children = pmic_bind_children(dev, regulators_node, pmic_children_info); > + if (!children) > + debug("%s: %s - no child found\n", __func__, dev->name); > + > + /* Always return success for this device */ > + return 0; > +} > + > +static struct dm_pmic_ops tps65090_ops = { > + .reg_count = tps65090_reg_count, > + .read = tps65090_read, > + .write = tps65090_write, > +}; > + > +static const struct udevice_id tps65090_ids[] = { > + { .compatible = "ti,tps65090" }, > + { } > +}; > + > +U_BOOT_DRIVER(pmic_tps65090) = { > + .name = "tps65090 pmic", > + .id = UCLASS_PMIC, > + .of_match = tps65090_ids, > + .bind = tps65090_bind, > + .ops = &tps65090_ops, > +}; > diff --git a/include/power/tps65090.h b/include/power/tps65090.h > new file mode 100644 > index 0000000..3a0690b > --- /dev/null > +++ b/include/power/tps65090.h > @@ -0,0 +1,56 @@ > +/* > + * Copyright (c) 2015 Google, Inc > + * Written by Simon Glass > + * > + * SPDX-License-Identifier: GPL-2.0+ > + */ > + > +#ifndef __TPS65090_PMIC_H_ > +#define __TPS65090_PMIC_H_ > + > +/* I2C device address for TPS65090 PMU */ > +#define TPS65090_I2C_ADDR 0x48 > + > +/* TPS65090 register addresses */ > +enum { > + REG_IRQ1 = 0, > + REG_CG_CTRL0 = 4, > + REG_CG_STATUS1 = 0xa, > + REG_FET_BASE = 0xe, /* Not a real register, FETs count from here */ > + REG_FET1_CTRL, > + REG_FET2_CTRL, > + REG_FET3_CTRL, > + REG_FET4_CTRL, > + REG_FET5_CTRL, > + REG_FET6_CTRL, > + REG_FET7_CTRL, > + TPS65090_NUM_REGS, > +}; > + > +enum { > + IRQ1_VBATG = 1 << 3, > + CG_CTRL0_ENC_MASK = 0x01, > + > + MAX_FET_NUM = 7, > + MAX_CTRL_READ_TRIES = 5, > + > + /* TPS65090 FET_CTRL register values */ > + FET_CTRL_TOFET = 1 << 7, /* Timeout, startup, overload */ > + FET_CTRL_PGFET = 1 << 4, /* Power good for FET status */ > + FET_CTRL_WAIT = 3 << 2, /* Overcurrent timeout max */ > + FET_CTRL_ADENFET = 1 << 1, /* Enable output auto discharge */ > + FET_CTRL_ENFET = 1 << 0, /* Enable FET */ > +}; > + > +enum { > + /* Status register fields */ > + TPS65090_ST1_OTC = 1 << 0, > + TPS65090_ST1_OCC = 1 << 1, > + TPS65090_ST1_STATE_SHIFT = 4, > + TPS65090_ST1_STATE_MASK = 0xf << TPS65090_ST1_STATE_SHIFT, > +}; > + > +/* Drivers name */ > +#define TPS65090_FET_DRIVER "tps65090_fet" > + > +#endif /* __TPS65090_PMIC_H_ */ > Acked-by: Przemyslaw Marczak Regards -- Przemyslaw Marczak Samsung R&D Institute Poland Samsung Electronics p.marczak at samsung.com