On 2/22/22 17:03, Lukasz Majewski wrote:
The tps65217 PMIC driver is used with am335x SoC based designs.
It is used in the SPL (MLO) as well, so the DM conversion only is for u-boot proper.
This driver only allows simple reading/writing/dumping of the content of its registers and requires the DM_I2C for proper operation.
Moreover, new CONFIG_PMIC_TPS65217 has been introduced in Kconfig to be used with boards, which both support DM_PMIC and DM_I2C.
Signed-off-by: Lukasz Majewski lukma@denx.de
Reviewed-by: Jaehoon Chung jh80.chung@samsung.com
Best Regards, Jaehoon Chung
drivers/power/pmic/Kconfig | 8 +++ drivers/power/pmic/pmic_tps65217.c | 82 ++++++++++++++++++++++++++++++ 2 files changed, 90 insertions(+)
diff --git a/drivers/power/pmic/Kconfig b/drivers/power/pmic/Kconfig index ce0adb18a4..c7739228da 100644 --- a/drivers/power/pmic/Kconfig +++ b/drivers/power/pmic/Kconfig @@ -306,6 +306,14 @@ config PMIC_TPS65090 only, and you can enable the regulator/charger drivers separately if required.
+config PMIC_TPS65217
- bool "Enable driver for Texas Instruments TPS65217 PMIC"
- ---help---
- The TPS65217 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.
config PMIC_PALMAS bool "Enable driver for Texas Instruments PALMAS PMIC" ---help--- diff --git a/drivers/power/pmic/pmic_tps65217.c b/drivers/power/pmic/pmic_tps65217.c index c7f532df4d..ccbf223593 100644 --- a/drivers/power/pmic/pmic_tps65217.c +++ b/drivers/power/pmic/pmic_tps65217.c @@ -6,8 +6,13 @@
#include <common.h> #include <i2c.h> +#include <dm.h> +#include <errno.h> +#include <fdtdec.h> +#include <power/pmic.h> #include <power/tps65217.h>
+#if !CONFIG_IS_ENABLED(DM_PMIC) struct udevice *tps65217_dev __section(".data") = NULL;
/** @@ -148,3 +153,80 @@ int power_tps65217_init(unsigned char bus) #endif return 0; } +#else /* CONFIG_IS_ENABLED(DM_PMIC) */ +static const struct pmic_child_info pmic_children_info[] = {
- { .prefix = "ldo", .driver = "tps65217_ldo" },
- { },
+};
+static int tps65217_reg_count(struct udevice *dev) +{
- return TPS65217_PMIC_NUM_OF_REGS;
+}
+static int tps65217_write(struct udevice *dev, uint reg, const uint8_t *buff,
int len)+{
- if (dm_i2c_write(dev, reg, buff, len)) {
pr_err("write error to device: %p register: %#x!\n", dev, reg);return -EIO;- }
- return 0;
+}
+static int tps65217_read(struct udevice *dev, uint reg, uint8_t *buff, int len) +{
- int ret;
- ret = dm_i2c_read(dev, reg, buff, len);
- if (ret) {
pr_err("read error %d from device: %p register: %#x!\n", ret,dev, reg);return -EIO;- }
- return 0;
+}
+static int tps65217_bind(struct udevice *dev) +{
- ofnode regulators_node;
- int children;
- regulators_node = dev_read_subnode(dev, "regulators");
- if (!ofnode_valid(regulators_node)) {
debug("%s: %s regulators subnode not found!\n", __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 tps65217_ops = {
- .reg_count = tps65217_reg_count,
- .read = tps65217_read,
- .write = tps65217_write,
+};
+static const struct udevice_id tps65217_ids[] = {
- { .compatible = "ti,tps65217" },
- { }
+};
+U_BOOT_DRIVER(pmic_tps65217) = {
- .name = "tps65217 pmic",
- .id = UCLASS_PMIC,
- .of_match = tps65217_ids,
- .bind = tps65217_bind,
- .ops = &tps65217_ops,
+}; +#endif