Message ID | 1496380753-15411-2-git-send-email-j-keerthy@ti.com |
---|---|
State | Superseded |
Delegated to: | Jaehoon Chung |
Headers | show |
Hi, On 1 June 2017 at 23:19, Keerthy <j-keerthy@ti.com> wrote: > Add support to bind the regulators/child nodes with the pmic. > > Signed-off-by: Keerthy <j-keerthy@ti.com> > --- > drivers/power/pmic/Kconfig | 7 ++++ > drivers/power/pmic/Makefile | 1 + > drivers/power/pmic/lp87565.c | 85 ++++++++++++++++++++++++++++++++++++++++++++ > include/power/lp87565.h | 12 +++++++ > 4 files changed, 105 insertions(+) > create mode 100644 drivers/power/pmic/lp87565.c > create mode 100644 include/power/lp87565.h > > diff --git a/drivers/power/pmic/Kconfig b/drivers/power/pmic/Kconfig > index 3f50c12..e3f9e4d 100644 > --- a/drivers/power/pmic/Kconfig > +++ b/drivers/power/pmic/Kconfig > @@ -188,6 +188,13 @@ config PMIC_LP873X > The LP873X is a PMIC containing couple of LDOs and couple of SMPS. > This driver binds the pmic children. > > +config PMIC_LP87565 > + bool "Enable driver for Texas Instruments LP87565 PMIC" > + depends on DM_PMIC > + ---help--- > + The LP87565 is a PMIC containing a bunch of SMPS. > + This driver binds the pmic children. > + > config POWER_MC34VR500 > bool "Enable driver for Freescale MC34VR500 PMIC" > ---help--- > diff --git a/drivers/power/pmic/Makefile b/drivers/power/pmic/Makefile > index f409e3a..f488799 100644 > --- a/drivers/power/pmic/Makefile > +++ b/drivers/power/pmic/Makefile > @@ -21,6 +21,7 @@ obj-$(CONFIG_PMIC_TPS65090) += tps65090.o > obj-$(CONFIG_PMIC_S5M8767) += s5m8767.o > obj-$(CONFIG_$(SPL_)PMIC_PALMAS) += palmas.o > obj-$(CONFIG_$(SPL_)PMIC_LP873X) += lp873x.o > +obj-$(CONFIG_$(SPL_)PMIC_LP87565) += lp87565.o > > obj-$(CONFIG_POWER_LTC3676) += pmic_ltc3676.o > obj-$(CONFIG_POWER_MAX77696) += pmic_max77696.o > diff --git a/drivers/power/pmic/lp87565.c b/drivers/power/pmic/lp87565.c > new file mode 100644 > index 0000000..e82bb4a > --- /dev/null > +++ b/drivers/power/pmic/lp87565.c > @@ -0,0 +1,85 @@ > +/* > + * (C) Copyright 2017 Texas Instruments Incorporated, <www.ti.com> > + * Keerthy <j-keerthy@ti.com> > + * > + * SPDX-License-Identifier: GPL-2.0+ > + */ > + > +#include <common.h> > +#include <fdtdec.h> > +#include <errno.h> > +#include <dm.h> > +#include <i2c.h> > +#include <power/pmic.h> > +#include <power/regulator.h> > +#include <power/lp87565.h> > +#include <dm/device.h> > + > +DECLARE_GLOBAL_DATA_PTR; > + > +static const struct pmic_child_info pmic_children_info[] = { > + { .prefix = "buck", .driver = LP87565_BUCK_DRIVER }, > + { }, > +}; > + > +static int lp87565_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 lp87565_read(struct udevice *dev, uint reg, uint8_t *buff, int len) > +{ > + if (dm_i2c_read(dev, reg, buff, len)) { > + error("read error from device: %p register: %#x!", dev, reg); > + return -EIO; > + } > + > + return 0; > +} > + > +static int lp87565_bind(struct udevice *dev) > +{ > + int regulators_node; > + const void *blob = gd->fdt_blob; > + int children; > + int node = dev->of_offset; > + > + regulators_node = fdt_subnode_offset(blob, node, "regulators"); Can you please update this to livetree? See rk8xx for an example. > + > + if (regulators_node <= 0) { > + printf("%s: %s reg subnode not found!", __func__, dev->name); > + return -ENXIO; > + } > + > + children = pmic_bind_children(dev, regulators_node, pmic_children_info); > + if (!children) > + printf("%s: %s - no child found\n", __func__, dev->name); > + > + /* Always return success for this device */ > + return 0; > +} > + > +static struct dm_pmic_ops lp87565_ops = { > + .read = lp87565_read, > + .write = lp87565_write, > +}; > + > +static const struct udevice_id lp87565_ids[] = { > + { .compatible = "ti,lp87565", .data = LP87565 }, > + { .compatible = "ti,lp87565-q1", .data = LP87565_Q1 }, > + { } > +}; > + > +U_BOOT_DRIVER(pmic_lp87565) = { > + .name = "lp87565_pmic", > + .id = UCLASS_PMIC, > + .of_match = lp87565_ids, > + .bind = lp87565_bind, > + .ops = &lp87565_ops, > +}; > diff --git a/include/power/lp87565.h b/include/power/lp87565.h > new file mode 100644 > index 0000000..48e1386 > --- /dev/null > +++ b/include/power/lp87565.h > @@ -0,0 +1,12 @@ > +#define LP87565 0x0 > +#define LP87565_Q1 0x1 > + > +#define LP87565_BUCK_NUM 6 > + > +/* Drivers name */ > +#define LP87565_BUCK_DRIVER "lp87565_buck" > + > +#define LP87565_BUCK_VOLT_MASK 0xFF > +#define LP87565_BUCK_VOLT_MAX_HEX 0xFF > +#define LP87565_BUCK_VOLT_MAX 3360000 > +#define LP87565_BUCK_MODE_MASK 0x80 > -- > 1.9.1 > Regards, Simon
On Wednesday 07 June 2017 02:38 AM, Simon Glass wrote: > Hi, > > On 1 June 2017 at 23:19, Keerthy <j-keerthy@ti.com> wrote: >> Add support to bind the regulators/child nodes with the pmic. >> >> Signed-off-by: Keerthy <j-keerthy@ti.com> >> --- >> drivers/power/pmic/Kconfig | 7 ++++ >> drivers/power/pmic/Makefile | 1 + >> drivers/power/pmic/lp87565.c | 85 ++++++++++++++++++++++++++++++++++++++++++++ >> include/power/lp87565.h | 12 +++++++ >> 4 files changed, 105 insertions(+) >> create mode 100644 drivers/power/pmic/lp87565.c >> create mode 100644 include/power/lp87565.h >> >> diff --git a/drivers/power/pmic/Kconfig b/drivers/power/pmic/Kconfig >> index 3f50c12..e3f9e4d 100644 >> --- a/drivers/power/pmic/Kconfig >> +++ b/drivers/power/pmic/Kconfig >> @@ -188,6 +188,13 @@ config PMIC_LP873X >> The LP873X is a PMIC containing couple of LDOs and couple of SMPS. >> This driver binds the pmic children. >> >> +config PMIC_LP87565 >> + bool "Enable driver for Texas Instruments LP87565 PMIC" >> + depends on DM_PMIC >> + ---help--- >> + The LP87565 is a PMIC containing a bunch of SMPS. >> + This driver binds the pmic children. >> + >> config POWER_MC34VR500 >> bool "Enable driver for Freescale MC34VR500 PMIC" >> ---help--- >> diff --git a/drivers/power/pmic/Makefile b/drivers/power/pmic/Makefile >> index f409e3a..f488799 100644 >> --- a/drivers/power/pmic/Makefile >> +++ b/drivers/power/pmic/Makefile >> @@ -21,6 +21,7 @@ obj-$(CONFIG_PMIC_TPS65090) += tps65090.o >> obj-$(CONFIG_PMIC_S5M8767) += s5m8767.o >> obj-$(CONFIG_$(SPL_)PMIC_PALMAS) += palmas.o >> obj-$(CONFIG_$(SPL_)PMIC_LP873X) += lp873x.o >> +obj-$(CONFIG_$(SPL_)PMIC_LP87565) += lp87565.o >> >> obj-$(CONFIG_POWER_LTC3676) += pmic_ltc3676.o >> obj-$(CONFIG_POWER_MAX77696) += pmic_max77696.o >> diff --git a/drivers/power/pmic/lp87565.c b/drivers/power/pmic/lp87565.c >> new file mode 100644 >> index 0000000..e82bb4a >> --- /dev/null >> +++ b/drivers/power/pmic/lp87565.c >> @@ -0,0 +1,85 @@ >> +/* >> + * (C) Copyright 2017 Texas Instruments Incorporated, <www.ti.com> >> + * Keerthy <j-keerthy@ti.com> >> + * >> + * SPDX-License-Identifier: GPL-2.0+ >> + */ >> + >> +#include <common.h> >> +#include <fdtdec.h> >> +#include <errno.h> >> +#include <dm.h> >> +#include <i2c.h> >> +#include <power/pmic.h> >> +#include <power/regulator.h> >> +#include <power/lp87565.h> >> +#include <dm/device.h> >> + >> +DECLARE_GLOBAL_DATA_PTR; >> + >> +static const struct pmic_child_info pmic_children_info[] = { >> + { .prefix = "buck", .driver = LP87565_BUCK_DRIVER }, >> + { }, >> +}; >> + >> +static int lp87565_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 lp87565_read(struct udevice *dev, uint reg, uint8_t *buff, int len) >> +{ >> + if (dm_i2c_read(dev, reg, buff, len)) { >> + error("read error from device: %p register: %#x!", dev, reg); >> + return -EIO; >> + } >> + >> + return 0; >> +} >> + >> +static int lp87565_bind(struct udevice *dev) >> +{ >> + int regulators_node; >> + const void *blob = gd->fdt_blob; >> + int children; >> + int node = dev->of_offset; >> + >> + regulators_node = fdt_subnode_offset(blob, node, "regulators"); > > Can you please update this to livetree? See rk8xx for an example. Okay i believe using: regulators_node = dev_read_subnode(dev, "regulators"); I will do that. > >> + >> + if (regulators_node <= 0) { >> + printf("%s: %s reg subnode not found!", __func__, dev->name); >> + return -ENXIO; >> + } >> + >> + children = pmic_bind_children(dev, regulators_node, pmic_children_info); >> + if (!children) >> + printf("%s: %s - no child found\n", __func__, dev->name); >> + >> + /* Always return success for this device */ >> + return 0; >> +} >> + >> +static struct dm_pmic_ops lp87565_ops = { >> + .read = lp87565_read, >> + .write = lp87565_write, >> +}; >> + >> +static const struct udevice_id lp87565_ids[] = { >> + { .compatible = "ti,lp87565", .data = LP87565 }, >> + { .compatible = "ti,lp87565-q1", .data = LP87565_Q1 }, >> + { } >> +}; >> + >> +U_BOOT_DRIVER(pmic_lp87565) = { >> + .name = "lp87565_pmic", >> + .id = UCLASS_PMIC, >> + .of_match = lp87565_ids, >> + .bind = lp87565_bind, >> + .ops = &lp87565_ops, >> +}; >> diff --git a/include/power/lp87565.h b/include/power/lp87565.h >> new file mode 100644 >> index 0000000..48e1386 >> --- /dev/null >> +++ b/include/power/lp87565.h >> @@ -0,0 +1,12 @@ >> +#define LP87565 0x0 >> +#define LP87565_Q1 0x1 >> + >> +#define LP87565_BUCK_NUM 6 >> + >> +/* Drivers name */ >> +#define LP87565_BUCK_DRIVER "lp87565_buck" >> + >> +#define LP87565_BUCK_VOLT_MASK 0xFF >> +#define LP87565_BUCK_VOLT_MAX_HEX 0xFF >> +#define LP87565_BUCK_VOLT_MAX 3360000 >> +#define LP87565_BUCK_MODE_MASK 0x80 >> -- >> 1.9.1 >> > > Regards, > Simon >
diff --git a/drivers/power/pmic/Kconfig b/drivers/power/pmic/Kconfig index 3f50c12..e3f9e4d 100644 --- a/drivers/power/pmic/Kconfig +++ b/drivers/power/pmic/Kconfig @@ -188,6 +188,13 @@ config PMIC_LP873X The LP873X is a PMIC containing couple of LDOs and couple of SMPS. This driver binds the pmic children. +config PMIC_LP87565 + bool "Enable driver for Texas Instruments LP87565 PMIC" + depends on DM_PMIC + ---help--- + The LP87565 is a PMIC containing a bunch of SMPS. + This driver binds the pmic children. + config POWER_MC34VR500 bool "Enable driver for Freescale MC34VR500 PMIC" ---help--- diff --git a/drivers/power/pmic/Makefile b/drivers/power/pmic/Makefile index f409e3a..f488799 100644 --- a/drivers/power/pmic/Makefile +++ b/drivers/power/pmic/Makefile @@ -21,6 +21,7 @@ obj-$(CONFIG_PMIC_TPS65090) += tps65090.o obj-$(CONFIG_PMIC_S5M8767) += s5m8767.o obj-$(CONFIG_$(SPL_)PMIC_PALMAS) += palmas.o obj-$(CONFIG_$(SPL_)PMIC_LP873X) += lp873x.o +obj-$(CONFIG_$(SPL_)PMIC_LP87565) += lp87565.o obj-$(CONFIG_POWER_LTC3676) += pmic_ltc3676.o obj-$(CONFIG_POWER_MAX77696) += pmic_max77696.o diff --git a/drivers/power/pmic/lp87565.c b/drivers/power/pmic/lp87565.c new file mode 100644 index 0000000..e82bb4a --- /dev/null +++ b/drivers/power/pmic/lp87565.c @@ -0,0 +1,85 @@ +/* + * (C) Copyright 2017 Texas Instruments Incorporated, <www.ti.com> + * Keerthy <j-keerthy@ti.com> + * + * SPDX-License-Identifier: GPL-2.0+ + */ + +#include <common.h> +#include <fdtdec.h> +#include <errno.h> +#include <dm.h> +#include <i2c.h> +#include <power/pmic.h> +#include <power/regulator.h> +#include <power/lp87565.h> +#include <dm/device.h> + +DECLARE_GLOBAL_DATA_PTR; + +static const struct pmic_child_info pmic_children_info[] = { + { .prefix = "buck", .driver = LP87565_BUCK_DRIVER }, + { }, +}; + +static int lp87565_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 lp87565_read(struct udevice *dev, uint reg, uint8_t *buff, int len) +{ + if (dm_i2c_read(dev, reg, buff, len)) { + error("read error from device: %p register: %#x!", dev, reg); + return -EIO; + } + + return 0; +} + +static int lp87565_bind(struct udevice *dev) +{ + int regulators_node; + const void *blob = gd->fdt_blob; + int children; + int node = dev->of_offset; + + regulators_node = fdt_subnode_offset(blob, node, "regulators"); + + if (regulators_node <= 0) { + printf("%s: %s reg subnode not found!", __func__, dev->name); + return -ENXIO; + } + + children = pmic_bind_children(dev, regulators_node, pmic_children_info); + if (!children) + printf("%s: %s - no child found\n", __func__, dev->name); + + /* Always return success for this device */ + return 0; +} + +static struct dm_pmic_ops lp87565_ops = { + .read = lp87565_read, + .write = lp87565_write, +}; + +static const struct udevice_id lp87565_ids[] = { + { .compatible = "ti,lp87565", .data = LP87565 }, + { .compatible = "ti,lp87565-q1", .data = LP87565_Q1 }, + { } +}; + +U_BOOT_DRIVER(pmic_lp87565) = { + .name = "lp87565_pmic", + .id = UCLASS_PMIC, + .of_match = lp87565_ids, + .bind = lp87565_bind, + .ops = &lp87565_ops, +}; diff --git a/include/power/lp87565.h b/include/power/lp87565.h new file mode 100644 index 0000000..48e1386 --- /dev/null +++ b/include/power/lp87565.h @@ -0,0 +1,12 @@ +#define LP87565 0x0 +#define LP87565_Q1 0x1 + +#define LP87565_BUCK_NUM 6 + +/* Drivers name */ +#define LP87565_BUCK_DRIVER "lp87565_buck" + +#define LP87565_BUCK_VOLT_MASK 0xFF +#define LP87565_BUCK_VOLT_MAX_HEX 0xFF +#define LP87565_BUCK_VOLT_MAX 3360000 +#define LP87565_BUCK_MODE_MASK 0x80
Add support to bind the regulators/child nodes with the pmic. Signed-off-by: Keerthy <j-keerthy@ti.com> --- drivers/power/pmic/Kconfig | 7 ++++ drivers/power/pmic/Makefile | 1 + drivers/power/pmic/lp87565.c | 85 ++++++++++++++++++++++++++++++++++++++++++++ include/power/lp87565.h | 12 +++++++ 4 files changed, 105 insertions(+) create mode 100644 drivers/power/pmic/lp87565.c create mode 100644 include/power/lp87565.h