From mboxrd@z Thu Jan 1 00:00:00 1970 From: Matti Vaittinen Date: Mon, 8 Apr 2019 13:30:53 +0300 Subject: [U-Boot] [PATCH v1 2/2] regulator: bd718x7: support ROHM BD71837 and BD71847 PMICs In-Reply-To: References: Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: quoted-printable To: u-boot@lists.denx.de BD71837 and BD71847 is PMIC intended for powering single-core, dual-core, and quad-core SoC=E2=80=99s such as NXP-i.MX 8M. BD71847 is used for example on NXP imx8mm EVK. Add regulator driver for ROHM BD71837 and BD71847 PMICs. BD71837 contains 8 bucks and 7 LDOS. BD71847 is reduced version containing 6 bucks and 6 LDOs. Voltages for DVS bucks (1-4 on BD71837, 1 and 2 on BD71847) can be adjusted when regulators are enabled. For other bucks and LDOs we may have over- or undershooting if voltage is adjusted when regulator is enabled. Thus this is prevented by default. BD718x7 has a quirk which may leave power output disabled after reset if enable/disable state was controlled by SW. Thus the SW control is only allowed for BD71837 bucks 3 and 4 by default. The impact of this limitation must be evaluated board-by board and restrictions may need to be modified. (Linux driver get's these limitations from DT and we may want to implement same on u-Boot driver). Signed-off-by: Matti Vaittinen --- Changelog v1: This version is created based on the RFC v1. https://lists.denx.de/pipermail/u-boot/2019-March/363077.html Additionally - Support BD71847. - Unlock the PMIC protection register drivers/power/pmic/bd71837.c | 42 ++- drivers/power/regulator/Kconfig | 15 + drivers/power/regulator/Makefile | 1 + drivers/power/regulator/bd71837.c | 457 ++++++++++++++++++++++++++++++ include/power/bd71837.h | 147 ++++++---- 5 files changed, 604 insertions(+), 58 deletions(-) create mode 100644 drivers/power/regulator/bd71837.c diff --git a/drivers/power/pmic/bd71837.c b/drivers/power/pmic/bd71837.c index eadf373a18..babddcba60 100644 --- a/drivers/power/pmic/bd71837.c +++ b/drivers/power/pmic/bd71837.c @@ -2,6 +2,8 @@ // // Copyright 2018 NXP * =20 +#define DEBUG + #include #include #include @@ -15,15 +17,15 @@ DECLARE_GLOBAL_DATA_PTR; =20 static const struct pmic_child_info pmic_children_info[] =3D { /* buck */ - { .prefix =3D "b", .driver =3D BD71837_REGULATOR_DRIVER}, + { .prefix =3D "b", .driver =3D BD718XX_REGULATOR_DRIVER}, /* ldo */ - { .prefix =3D "l", .driver =3D BD71837_REGULATOR_DRIVER}, + { .prefix =3D "l", .driver =3D BD718XX_REGULATOR_DRIVER}, { }, }; =20 static int bd71837_reg_count(struct udevice *dev) { - return BD71837_REG_NUM; + return BD718XX_MAX_REGISTER - 1; } =20 static int bd71837_write(struct udevice *dev, uint reg, const uint8_t *buf= f, @@ -54,7 +56,7 @@ static int bd71837_bind(struct udevice *dev) =20 regulators_node =3D dev_read_subnode(dev, "regulators"); if (!ofnode_valid(regulators_node)) { - debug("%s: %s regulators subnode not found!", __func__, + debug("%s: %s regulators subnode not found!\n", __func__, dev->name); return -ENXIO; } @@ -69,6 +71,34 @@ static int bd71837_bind(struct udevice *dev) return 0; } =20 +static int bd718x7_probe(struct udevice *dev) +{ + int ret; + u8 unlock; + + /* Unlock the PMIC regulator control before probing the children */ + ret =3D pmic_reg_read(dev, BD718XX_REGLOCK); + if (ret < 0) { + debug("%s: %s Failed to read lock register, error %d\n", + __func__, dev->name, ret); + return ret; + } + + unlock =3D ret; + unlock &=3D ~(BD718XX_REGLOCK_PWRSEQ | BD718XX_REGLOCK_VREG); + + ret =3D pmic_reg_write(dev, BD718XX_REGLOCK, unlock); + if (ret) { + debug("%s: %s Failed to unlock regulator control\n", __func__, + dev->name); + return ret; + } + debug("%s: '%s' - BD718x7 PMIC register unlocked\n", __func__, + dev->name); + + return 0; +} + static struct dm_pmic_ops bd71837_ops =3D { .reg_count =3D bd71837_reg_count, .read =3D bd71837_read, @@ -76,7 +106,8 @@ static struct dm_pmic_ops bd71837_ops =3D { }; =20 static const struct udevice_id bd71837_ids[] =3D { - { .compatible =3D "rohm,bd71837", .data =3D 0x4b, }, + { .compatible =3D "rohm,bd71837", .data =3D ROHM_CHIP_TYPE_BD71837, }, + { .compatible =3D "rohm,bd71847", .data =3D ROHM_CHIP_TYPE_BD71847, }, { } }; =20 @@ -85,5 +116,6 @@ U_BOOT_DRIVER(pmic_bd71837) =3D { .id =3D UCLASS_PMIC, .of_match =3D bd71837_ids, .bind =3D bd71837_bind, + .probe =3D bd718x7_probe, .ops =3D &bd71837_ops, }; diff --git a/drivers/power/regulator/Kconfig b/drivers/power/regulator/Kcon= fig index 3ed0dd2264..323516587c 100644 --- a/drivers/power/regulator/Kconfig +++ b/drivers/power/regulator/Kconfig @@ -43,6 +43,21 @@ config REGULATOR_AS3722 but does not yet support change voltages. Currently this must be done using direct register writes to the PMIC. =20 +config DM_REGULATOR_BD71837 + bool "Enable Driver Model for REGULATOR BD71837" + depends on DM_REGULATOR && DM_PMIC_BD71837 + help + This config enables implementation of driver-model regulator uclass + features for REGULATOR BD71837. The driver implements get/set api for: + value and enable. + +config SPL_DM_REGULATOR_BD71837 + bool "Enable Driver Model for REGULATOR BD71837 in SPL" + depends on DM_REGULATOR_BD71837 + help + This config enables implementation of driver-model regulator uclass + features for REGULATOR BD71837 in SPL. + config DM_REGULATOR_PFUZE100 bool "Enable Driver Model for REGULATOR PFUZE100" depends on DM_REGULATOR && DM_PMIC_PFUZE100 diff --git a/drivers/power/regulator/Makefile b/drivers/power/regulator/Mak= efile index f617ce723a..898ed5f084 100644 --- a/drivers/power/regulator/Makefile +++ b/drivers/power/regulator/Makefile @@ -9,6 +9,7 @@ obj-$(CONFIG_REGULATOR_ACT8846) +=3D act8846.o obj-$(CONFIG_REGULATOR_AS3722) +=3D as3722_regulator.o obj-$(CONFIG_DM_REGULATOR_MAX77686) +=3D max77686.o obj-$(CONFIG_$(SPL_)DM_PMIC_PFUZE100) +=3D pfuze100.o +obj-$(CONFIG_$(SPL_)DM_REGULATOR_BD71837) +=3D bd71837.o obj-$(CONFIG_$(SPL_)REGULATOR_PWM) +=3D pwm_regulator.o obj-$(CONFIG_$(SPL_)DM_REGULATOR_FAN53555) +=3D fan53555.o obj-$(CONFIG_$(SPL_)DM_REGULATOR_FIXED) +=3D fixed.o diff --git a/drivers/power/regulator/bd71837.c b/drivers/power/regulator/bd= 71837.c new file mode 100644 index 0000000000..c320a1dc4e --- /dev/null +++ b/drivers/power/regulator/bd71837.c @@ -0,0 +1,457 @@ +// SPDX-License-Identifier: GPL-2.0-or-later +// +// Copyright (C) 2019 ROHM Semiconductors +// +// ROHM BD71837 regulator driver + +#include +#include +#include +#include +#include +#include +#include + +#define HW_STATE_CONTROL 0 +#define DEBUG + +struct bd71837_vrange { + unsigned int min_volt; + unsigned int step; + u8 min_sel; + u8 max_sel; + u8 rangeval; +}; + +struct bd71837_data { + const char *name; + u8 enable_reg; + u8 enablemask; + u8 volt_reg; + u8 volt_mask; + struct bd71837_vrange *ranges; + unsigned int numranges; + u8 rangemask; + u8 sel_mask; + bool dvs; +}; + +#define BD_RANGE(_min, _vstep, _sel_low, _sel_hi, _range_sel) \ +{ \ + .min_volt =3D (_min), .step =3D (_vstep), .min_sel =3D (_sel_low), \ + .max_sel =3D (_sel_hi), .rangeval =3D (_range_sel) \ +} + +#define BD_DATA(_name, enreg, enmask, vreg, vmask, _range, rmask, _dvs, se= l) \ +{ \ + .name =3D (_name), .enable_reg =3D (enreg), .enablemask =3D (enmask), \ + .volt_reg =3D (vreg), .volt_mask =3D (vmask), .ranges =3D (_range), \ + .numranges =3D ARRAY_SIZE(_range), .rangemask =3D (rmask), .dvs =3D (_dvs= ), \ + .sel_mask =3D (sel) \ +} + +static struct bd71837_vrange dvs_buck_vranges[] =3D { + BD_RANGE(700000, 10000, 0, 0x3C, 0), + BD_RANGE(1300000, 0, 0x3D, 0x3F, 0), +}; + +static struct bd71837_vrange bd71847_buck3_vranges[] =3D { + BD_RANGE(700000, 100000, 0x00, 0x03, 0), + BD_RANGE(1050000, 50000, 0x04, 0x05, 0), + BD_RANGE(1200000, 150000, 0x06, 0x07, 0), + BD_RANGE(550000, 50000, 0x0, 0x7, 0x40), + BD_RANGE(675000, 100000, 0x0, 0x3, 0x80), + BD_RANGE(1025000, 50000, 0x4, 0x5, 0x80), + BD_RANGE(1175000, 150000, 0x6, 0x7, 0x80), +}; + +static struct bd71837_vrange bd71847_buck4_vranges[] =3D { + BD_RANGE(3000000, 100000, 0x00, 0x03, 0), + BD_RANGE(2600000, 100000, 0x00, 0x03, 40), +}; + +static struct bd71837_vrange bd71837_buck5_vranges[] =3D { + BD_RANGE(700000, 100000, 0, 0x3, 0), + BD_RANGE(1050000, 50000, 0x04, 0x05, 0), + BD_RANGE(1200000, 150000, 0x06, 0x07, 0), + BD_RANGE(675000, 100000, 0x0, 0x3, 0x80), + BD_RANGE(1025000, 50000, 0x04, 0x05, 0x80), + BD_RANGE(1175000, 150000, 0x06, 0x07, 0x80), +}; + +static struct bd71837_vrange bd71837_buck6_vranges[] =3D { + BD_RANGE(3000000, 100000, 0x00, 0x03, 0), +}; + +static struct bd71837_vrange nodvs_buck3_vranges[] =3D { + BD_RANGE(1605000, 90000, 0, 1, 0), + BD_RANGE(1755000, 45000, 2, 4, 0), + BD_RANGE(1905000, 45000, 5, 7, 0), +}; + +static struct bd71837_vrange nodvs_buck4_vranges[] =3D { + BD_RANGE(800000, 10000, 0x00, 0x3C, 0), +}; + +static struct bd71837_vrange ldo1_vranges[] =3D { + BD_RANGE(3000000, 100000, 0x00, 0x03, 0), + BD_RANGE(1600000, 100000, 0x00, 0x03, 0x20), +}; + +static struct bd71837_vrange ldo2_vranges[] =3D { + BD_RANGE(900000, 0, 0, 0, 0), + BD_RANGE(800000, 0, 1, 1, 0), +}; + +static struct bd71837_vrange ldo3_vranges[] =3D { + BD_RANGE(1800000, 100000, 0x00, 0x0F, 0), +}; + +static struct bd71837_vrange ldo4_vranges[] =3D { + BD_RANGE(900000, 100000, 0x00, 0x09, 0), +}; + +static struct bd71837_vrange bd71837_ldo5_vranges[] =3D { + BD_RANGE(1800000, 100000, 0x00, 0x0F, 0), +}; + +static struct bd71837_vrange bd71847_ldo5_vranges[] =3D { + BD_RANGE(1800000, 100000, 0x00, 0x0F, 0), + BD_RANGE(800000, 100000, 0x00, 0x0F, 0x20), +}; + +static struct bd71837_vrange ldo6_vranges[] =3D { + BD_RANGE(900000, 100000, 0x00, 0x09, 0), +}; + +static struct bd71837_vrange ldo7_vranges[] =3D { + BD_RANGE(1800000, 100000, 0x00, 0x0F, 0), +}; + +/* We use enable mask 'HW_STATE_CONTROL' to indicate that this regulator + * must not be enabled or disabled by SW. The typical use-case for BD71837 + * is powering NXP i.MX8. In this use-case we (for now) only allow control + * for BUCK3 and BUCK4 which are not boot critical. + */ +static struct bd71837_data bd71837_reg_data[] =3D { +/* Bucks 1-4 which support dynamic voltage scaling */ + BD_DATA("BUCK1", BD718XX_BUCK1_CTRL, HW_STATE_CONTROL, + BD718XX_BUCK1_VOLT_RUN, DVS_BUCK_RUN_MASK, dvs_buck_vranges, 0, + true, BD718XX_BUCK_SEL), + BD_DATA("BUCK2", BD718XX_BUCK2_CTRL, HW_STATE_CONTROL, + BD718XX_BUCK2_VOLT_RUN, DVS_BUCK_RUN_MASK, dvs_buck_vranges, 0, + true, BD718XX_BUCK_SEL), + BD_DATA("BUCK3", BD71837_BUCK3_CTRL, BD718XX_BUCK_EN, + BD71837_BUCK3_VOLT_RUN, DVS_BUCK_RUN_MASK, dvs_buck_vranges, 0, + true, BD718XX_BUCK_SEL), + BD_DATA("BUCK4", BD71837_BUCK4_CTRL, BD718XX_BUCK_EN, + BD71837_BUCK4_VOLT_RUN, DVS_BUCK_RUN_MASK, dvs_buck_vranges, 0, + true, BD718XX_BUCK_SEL), +/* Bucks 5-8 which do not support dynamic voltage scaling */ + BD_DATA("BUCK5", BD718XX_1ST_NODVS_BUCK_CTRL, HW_STATE_CONTROL, + BD718XX_1ST_NODVS_BUCK_VOLT, BD718XX_1ST_NODVS_BUCK_MASK, + bd71837_buck5_vranges, 0x80, false, BD718XX_BUCK_SEL), + BD_DATA("BUCK6", BD718XX_2ND_NODVS_BUCK_CTRL, HW_STATE_CONTROL, + BD718XX_2ND_NODVS_BUCK_VOLT, BD71837_BUCK6_MASK, + bd71837_buck6_vranges, 0, false, BD718XX_BUCK_SEL), + BD_DATA("BUCK7", BD718XX_3RD_NODVS_BUCK_CTRL, HW_STATE_CONTROL, + BD718XX_3RD_NODVS_BUCK_VOLT, BD718XX_3RD_NODVS_BUCK_MASK, + nodvs_buck3_vranges, 0, false, BD718XX_BUCK_SEL), + BD_DATA("BUCK8", BD718XX_4TH_NODVS_BUCK_CTRL, HW_STATE_CONTROL, + BD718XX_4TH_NODVS_BUCK_VOLT, BD718XX_4TH_NODVS_BUCK_MASK, + nodvs_buck4_vranges, 0, false, BD718XX_BUCK_SEL), +/* LDOs */ + BD_DATA("LDO1", BD718XX_LDO1_VOLT, HW_STATE_CONTROL, BD718XX_LDO1_VOLT, + BD718XX_LDO1_MASK, ldo1_vranges, 0x20, false, BD718XX_LDO_SEL), + BD_DATA("LDO2", BD718XX_LDO2_VOLT, HW_STATE_CONTROL, BD718XX_LDO2_VOLT, + BD718XX_LDO2_MASK, ldo2_vranges, 0, false, BD718XX_LDO_SEL), + BD_DATA("LDO3", BD718XX_LDO3_VOLT, HW_STATE_CONTROL, BD718XX_LDO3_VOLT, + BD718XX_LDO3_MASK, ldo3_vranges, 0, false, BD718XX_LDO_SEL), + BD_DATA("LDO4", BD718XX_LDO4_VOLT, HW_STATE_CONTROL, BD718XX_LDO4_VOLT, + BD718XX_LDO4_MASK, ldo4_vranges, 0, false, BD718XX_LDO_SEL), + BD_DATA("LDO5", BD718XX_LDO5_VOLT, HW_STATE_CONTROL, BD718XX_LDO5_VOLT, + BD71837_LDO5_MASK, bd71837_ldo5_vranges, 0, false, + BD718XX_LDO_SEL), + BD_DATA("LDO6", BD718XX_LDO6_VOLT, HW_STATE_CONTROL, BD718XX_LDO6_VOLT, + BD718XX_LDO6_MASK, ldo6_vranges, 0, false, BD718XX_LDO_SEL), + BD_DATA("LDO7", BD71837_LDO7_VOLT, HW_STATE_CONTROL, BD71837_LDO7_VOLT, + BD71837_LDO7_MASK, ldo7_vranges, 0, false, BD718XX_LDO_SEL), +}; + +static struct bd71837_data bd71847_reg_data[] =3D { +/* Bucks 1 and 2 which support dynamic voltage scaling */ + BD_DATA("BUCK1", BD718XX_BUCK1_CTRL, HW_STATE_CONTROL, + BD718XX_BUCK1_VOLT_RUN, DVS_BUCK_RUN_MASK, dvs_buck_vranges, 0, + true, BD718XX_BUCK_SEL), + BD_DATA("BUCK2", BD718XX_BUCK2_CTRL, HW_STATE_CONTROL, + BD718XX_BUCK2_VOLT_RUN, DVS_BUCK_RUN_MASK, dvs_buck_vranges, 0, + true, BD718XX_BUCK_SEL), +/* Bucks 3-6 which do not support dynamic voltage scaling */ + BD_DATA("BUCK3", BD718XX_1ST_NODVS_BUCK_CTRL, HW_STATE_CONTROL, + BD718XX_1ST_NODVS_BUCK_VOLT, BD718XX_1ST_NODVS_BUCK_MASK, + bd71847_buck3_vranges, 0xC0, false, BD718XX_BUCK_SEL), + BD_DATA("BUCK4", BD718XX_2ND_NODVS_BUCK_CTRL, HW_STATE_CONTROL, + BD718XX_2ND_NODVS_BUCK_VOLT, BD71837_BUCK6_MASK, + bd71847_buck4_vranges, 0x40, false, BD718XX_BUCK_SEL), + BD_DATA("BUCK5", BD718XX_3RD_NODVS_BUCK_CTRL, HW_STATE_CONTROL, + BD718XX_3RD_NODVS_BUCK_VOLT, BD718XX_3RD_NODVS_BUCK_MASK, + nodvs_buck3_vranges, 0, false, BD718XX_BUCK_SEL), + BD_DATA("BUCK6", BD718XX_4TH_NODVS_BUCK_CTRL, HW_STATE_CONTROL, + BD718XX_4TH_NODVS_BUCK_VOLT, BD718XX_4TH_NODVS_BUCK_MASK, + nodvs_buck4_vranges, 0, false, BD718XX_BUCK_SEL), +/* LDOs */ + BD_DATA("LDO1", BD718XX_LDO1_VOLT, HW_STATE_CONTROL, BD718XX_LDO1_VOLT, + BD718XX_LDO1_MASK, ldo1_vranges, 0x20, false, BD718XX_LDO_SEL), + BD_DATA("LDO2", BD718XX_LDO2_VOLT, HW_STATE_CONTROL, BD718XX_LDO2_VOLT, + BD718XX_LDO2_MASK, ldo2_vranges, 0, false, BD718XX_LDO_SEL), + BD_DATA("LDO3", BD718XX_LDO3_VOLT, HW_STATE_CONTROL, BD718XX_LDO3_VOLT, + BD718XX_LDO3_MASK, ldo3_vranges, 0, false, BD718XX_LDO_SEL), + BD_DATA("LDO4", BD718XX_LDO4_VOLT, HW_STATE_CONTROL, BD718XX_LDO4_VOLT, + BD718XX_LDO4_MASK, ldo4_vranges, 0, false, BD718XX_LDO_SEL), + BD_DATA("LDO5", BD718XX_LDO5_VOLT, HW_STATE_CONTROL, BD718XX_LDO5_VOLT, + BD71847_LDO5_MASK, bd71847_ldo5_vranges, 0x20, false, + BD718XX_LDO_SEL), + BD_DATA("LDO6", BD718XX_LDO6_VOLT, HW_STATE_CONTROL, BD718XX_LDO6_VOLT, + BD718XX_LDO6_MASK, ldo6_vranges, 0, false, BD718XX_LDO_SEL), +}; + +static int vrange_find_value(struct bd71837_vrange *r, u8 sel, + unsigned int *val) +{ + if (!val || sel < r->min_sel || sel > r->max_sel) + return -EINVAL; + + *val =3D r->min_volt + r->step * (sel - r->min_sel); + return 0; +} + +static int vrange_find_selector(struct bd71837_vrange *r, int val, u8 *sel) +{ + int ret =3D -EINVAL; + int num_vals =3D r->max_sel - r->min_sel + 1; + + if (val >=3D r->min_volt && + val <=3D r->min_volt + r->step * (num_vals - 1)) { + if (r->step) { + *sel =3D r->min_sel + ((val - r->min_volt) / r->step); + ret =3D 0; + } else { + *sel =3D r->min_sel; + ret =3D 0; + } + } + return ret; +} + +static int bd71837_get_enable(struct udevice *dev) +{ + int val; + struct bd71837_data *d =3D dev_get_platdata(dev); + + /* boot critical regulators on bd71837 must not be controlled by sw + * due to the 'feature' which leaves power rails down if bd71837 is + * reseted to snvs state. hence we can't get the state here. + * + * if we are alive it means we probably are on run state and + * if the regulator can't be controlled we can assume it is + * enabled. + */ + if (d->enablemask =3D=3D HW_STATE_CONTROL) + return 1; + + val =3D pmic_reg_read(dev->parent, d->enable_reg); + + if (val < 0) + return val; + + return (val & d->enablemask); +} + +static int bd71837_set_enable(struct udevice *dev, bool enable) +{ + int val; + struct bd71837_data *d =3D dev_get_platdata(dev); + + /* boot critical regulators on bd71837 must not be controlled by sw + * due to the 'feature' which leaves power rails down if bd71837 is + * reseted to snvs state. Hence we can't set the state here. + */ + if (d->enablemask =3D=3D HW_STATE_CONTROL) + return -EINVAL; + + val =3D pmic_reg_read(dev->parent, d->enable_reg); + + if (val < 0) + return val; + + if (enable) + val |=3D d->enablemask; + else + val &=3D ~d->enablemask; + + return pmic_reg_write(dev->parent, d->enable_reg, val); +} + +static int bd71837_set_value(struct udevice *dev, int uvolt) +{ + u8 sel, reg; + u8 range; + int i; + int not_found =3D 1; + struct bd71837_data *d =3D dev_get_platdata(dev); + + /* An under/overshooting may occur if voltage is changed for other + * regulators but buck 1,2,3 or 4 when regulator is enabled. Prevent + * change to protect the HW + */ + if (!d->dvs) + if (bd71837_get_enable(dev)) { + pr_err("Only DVS bucks can be changed when enabled\n"); + return -EINVAL; + } + + for (i =3D 0; i < d->numranges; i++) { + struct bd71837_vrange *r =3D &d->ranges[i]; + + not_found =3D vrange_find_selector(r, uvolt, &sel); + if (!not_found) { + unsigned int tmp; + + /* We require exactly the requested value to be + * supported - this can be changed later if needed + */ + range =3D r->rangeval; + not_found =3D vrange_find_value(r, sel, &tmp); + if (!not_found && tmp =3D=3D uvolt) + break; + not_found =3D 1; + } + } + + if (not_found) + return -EINVAL; + + sel <<=3D ffs(d->volt_mask) - 1; + + reg =3D pmic_reg_read(dev->parent, d->volt_reg); + if (reg < 0) + return reg; + + reg &=3D ~d->volt_mask; + reg |=3D sel; + if (d->rangemask) { + reg &=3D ~d->rangemask; + reg |=3D range; + } + + return pmic_reg_write(dev->parent, d->volt_reg, reg); +} + +static int bd71837_get_value(struct udevice *dev) +{ + u8 reg, range; + unsigned int tmp; + struct bd71837_data *d =3D dev_get_platdata(dev); + int i; + + reg =3D pmic_reg_read(dev->parent, d->volt_reg); + if (reg < 0) + return reg; + + range =3D reg & d->rangemask; + + reg &=3D d->volt_mask; + reg >>=3D ffs(d->volt_mask) - 1; + + for (i =3D 0; i < d->numranges; i++) { + struct bd71837_vrange *r =3D &d->ranges[i]; + + if (d->rangemask && ((d->rangemask & range) !=3D r->rangeval)) + continue; + + if (!vrange_find_value(r, reg, &tmp)) + return tmp; + } + + pr_err("Unknown voltage value read from pmic\n"); + + return -EINVAL; +} + +static int bd71837_regulator_probe(struct udevice *dev) +{ + struct bd71837_data *d =3D dev_get_platdata(dev); + int i, ret; + struct dm_regulator_uclass_platdata *uc_pdata; + int type; + struct bd71837_data *init_data; + int data_amnt; + + type =3D dev_get_driver_data(dev_get_parent(dev)); + + switch (type) { + case ROHM_CHIP_TYPE_BD71837: + init_data =3D bd71837_reg_data; + data_amnt =3D ARRAY_SIZE(bd71837_reg_data); + break; + case ROHM_CHIP_TYPE_BD71847: + init_data =3D bd71847_reg_data; + data_amnt =3D ARRAY_SIZE(bd71847_reg_data); + break; + default: + debug("Unknown PMIC type\n"); + init_data =3D NULL; + data_amnt =3D 0; + break; + } + + for (i =3D 0; i < data_amnt; i++) { + if (!strcmp(dev->name, init_data[i].name)) { + *d =3D init_data[i]; + if (d->enablemask !=3D HW_STATE_CONTROL) { + u8 ctrl; + + /* Take the regulator under SW control. Ensure + * the initial state matches dt flags and then + * write the SEL bit + */ + uc_pdata =3D dev_get_uclass_platdata(dev); + ret =3D bd71837_set_enable(dev, + !!(uc_pdata->boot_on || + uc_pdata->always_on)); + if (ret) + return ret; + + ret =3D pmic_reg_read(dev->parent, + d->enable_reg); + if (ret < 0) + return ret; + + ctrl =3D ret; + ctrl |=3D d->sel_mask; + return pmic_reg_write(dev->parent, + d->enable_reg, ctrl); + } + return 0; + } + } + + pr_err("Unknown regulator '%s'\n", dev->name); + + return -EINVAL; +} + +static const struct dm_regulator_ops bd71837_regulator_ops =3D { + .get_value =3D bd71837_get_value, + .set_value =3D bd71837_set_value, + .get_enable =3D bd71837_get_enable, + .set_enable =3D bd71837_set_enable, +}; + +U_BOOT_DRIVER(bd71837_regulator) =3D { + .name =3D BD718XX_REGULATOR_DRIVER, + .id =3D UCLASS_REGULATOR, + .ops =3D &bd71837_regulator_ops, + .probe =3D bd71837_regulator_probe, + .platdata_auto_alloc_size =3D sizeof(struct bd71837_data), +}; diff --git a/include/power/bd71837.h b/include/power/bd71837.h index 9c74f6fc61..cdb9955860 100644 --- a/include/power/bd71837.h +++ b/include/power/bd71837.h @@ -1,64 +1,105 @@ /* SPDX-License-Identifier: GPL-2.0-or-later */ /* Copyright (C) 2018 ROHM Semiconductors */ =20 -#ifndef BD71837_H_ -#define BD71837_H_ +#ifndef BD718XX_H_ +#define BD718XX_H_ =20 -#define BD71837_REGULATOR_DRIVER "bd71837_regulator" +#define BD718XX_REGULATOR_DRIVER "bd718x7_regulator" =20 enum { - BD71837_REV =3D 0x00, - BD71837_SWRESET =3D 0x01, - BD71837_I2C_DEV =3D 0x02, - BD71837_PWRCTRL0 =3D 0x03, - BD71837_PWRCTRL1 =3D 0x04, - BD71837_BUCK1_CTRL =3D 0x05, - BD71837_BUCK2_CTRL =3D 0x06, - BD71837_BUCK3_CTRL =3D 0x07, - BD71837_BUCK4_CTRL =3D 0x08, - BD71837_BUCK5_CTRL =3D 0x09, - BD71837_BUCK6_CTRL =3D 0x0A, - BD71837_BUCK7_CTRL =3D 0x0B, - BD71837_BUCK8_CTRL =3D 0x0C, - BD71837_BUCK1_VOLT_RUN =3D 0x0D, - BD71837_BUCK1_VOLT_IDLE =3D 0x0E, - BD71837_BUCK1_VOLT_SUSP =3D 0x0F, - BD71837_BUCK2_VOLT_RUN =3D 0x10, - BD71837_BUCK2_VOLT_IDLE =3D 0x11, - BD71837_BUCK3_VOLT_RUN =3D 0x12, - BD71837_BUCK4_VOLT_RUN =3D 0x13, - BD71837_BUCK5_VOLT =3D 0x14, - BD71837_BUCK6_VOLT =3D 0x15, - BD71837_BUCK7_VOLT =3D 0x16, - BD71837_BUCK8_VOLT =3D 0x17, - BD71837_LDO1_VOLT =3D 0x18, - BD71837_LDO2_VOLT =3D 0x19, - BD71837_LDO3_VOLT =3D 0x1A, - BD71837_LDO4_VOLT =3D 0x1B, - BD71837_LDO5_VOLT =3D 0x1C, - BD71837_LDO6_VOLT =3D 0x1D, - BD71837_LDO7_VOLT =3D 0x1E, - BD71837_TRANS_COND0 =3D 0x1F, - BD71837_TRANS_COND1 =3D 0x20, - BD71837_VRFAULTEN =3D 0x21, - BD71837_MVRFLTMASK0 =3D 0x22, - BD71837_MVRFLTMASK1 =3D 0x23, - BD71837_MVRFLTMASK2 =3D 0x24, - BD71837_RCVCFG =3D 0x25, - BD71837_RCVNUM =3D 0x26, - BD71837_PWRONCONFIG0 =3D 0x27, - BD71837_PWRONCONFIG1 =3D 0x28, - BD71837_RESETSRC =3D 0x29, - BD71837_MIRQ =3D 0x2A, - BD71837_IRQ =3D 0x2B, - BD71837_IN_MON =3D 0x2C, - BD71837_POW_STATE =3D 0x2D, - BD71837_OUT32K =3D 0x2E, - BD71837_REGLOCK =3D 0x2F, - BD71837_MUXSW_EN =3D 0x30, - BD71837_REG_NUM, + ROHM_CHIP_TYPE_BD71837 =3D 0, + ROHM_CHIP_TYPE_BD71847, + ROHM_CHIP_TYPE_BD70528, + ROHM_CHIP_TYPE_AMOUNT }; =20 +enum { + BD718XX_REV =3D 0x00, + BD718XX_SWRESET =3D 0x01, + BD718XX_I2C_DEV =3D 0x02, + BD718XX_PWRCTRL0 =3D 0x03, + BD718XX_PWRCTRL1 =3D 0x04, + BD718XX_BUCK1_CTRL =3D 0x05, + BD718XX_BUCK2_CTRL =3D 0x06, + BD71837_BUCK3_CTRL =3D 0x07, + BD71837_BUCK4_CTRL =3D 0x08, + BD718XX_1ST_NODVS_BUCK_CTRL =3D 0x09, + BD718XX_2ND_NODVS_BUCK_CTRL =3D 0x0A, + BD718XX_3RD_NODVS_BUCK_CTRL =3D 0x0B, + BD718XX_4TH_NODVS_BUCK_CTRL =3D 0x0C, + BD718XX_BUCK1_VOLT_RUN =3D 0x0D, + BD718XX_BUCK1_VOLT_IDLE =3D 0x0E, + BD718XX_BUCK1_VOLT_SUSP =3D 0x0F, + BD718XX_BUCK2_VOLT_RUN =3D 0x10, + BD718XX_BUCK2_VOLT_IDLE =3D 0x11, + BD71837_BUCK3_VOLT_RUN =3D 0x12, + BD71837_BUCK4_VOLT_RUN =3D 0x13, + BD718XX_1ST_NODVS_BUCK_VOLT =3D 0x14, + BD718XX_2ND_NODVS_BUCK_VOLT =3D 0x15, + BD718XX_3RD_NODVS_BUCK_VOLT =3D 0x16, + BD718XX_4TH_NODVS_BUCK_VOLT =3D 0x17, + BD718XX_LDO1_VOLT =3D 0x18, + BD718XX_LDO2_VOLT =3D 0x19, + BD718XX_LDO3_VOLT =3D 0x1A, + BD718XX_LDO4_VOLT =3D 0x1B, + BD718XX_LDO5_VOLT =3D 0x1C, + BD718XX_LDO6_VOLT =3D 0x1D, + BD71837_LDO7_VOLT =3D 0x1E, + BD718XX_TRANS_COND0 =3D 0x1F, + BD718XX_TRANS_COND1 =3D 0x20, + BD718XX_VRFAULTEN =3D 0x21, + BD718XX_MVRFLTMASK0 =3D 0x22, + BD718XX_MVRFLTMASK1 =3D 0x23, + BD718XX_MVRFLTMASK2 =3D 0x24, + BD718XX_RCVCFG =3D 0x25, + BD718XX_RCVNUM =3D 0x26, + BD718XX_PWRONCONFIG0 =3D 0x27, + BD718XX_PWRONCONFIG1 =3D 0x28, + BD718XX_RESETSRC =3D 0x29, + BD718XX_MIRQ =3D 0x2A, + BD718XX_IRQ =3D 0x2B, + BD718XX_IN_MON =3D 0x2C, + BD718XX_POW_STATE =3D 0x2D, + BD718XX_OUT32K =3D 0x2E, + BD718XX_REGLOCK =3D 0x2F, + BD718XX_MUXSW_EN =3D 0x30, + BD718XX_REG_OTPVER =3D 0xFF, + BD718XX_MAX_REGISTER =3D 0x100, +}; + +#define BD718XX_REGLOCK_PWRSEQ 0x1 +#define BD718XX_REGLOCK_VREG 0x10 + +#define BD718XX_BUCK_EN 0x01 +#define BD718XX_LDO_EN 0x40 +#define BD718XX_BUCK_SEL 0x02 +#define BD718XX_LDO_SEL 0x80 + +#define DVS_BUCK_RUN_MASK 0x3F +#define BD718XX_1ST_NODVS_BUCK_MASK 0x07 +#define BD718XX_3RD_NODVS_BUCK_MASK 0x07 +#define BD718XX_4TH_NODVS_BUCK_MASK 0x3F + +#define BD71847_BUCK3_MASK 0x07 +#define BD71847_BUCK3_RANGE_MASK 0xC0 +#define BD71847_BUCK4_MASK 0x03 +#define BD71847_BUCK4_RANGE_MASK 0x40 + +#define BD71837_BUCK5_RANGE_MASK 0x80 +#define BD71837_BUCK6_MASK 0x03 + +#define BD718XX_LDO1_MASK 0x03 +#define BD718XX_LDO1_RANGE_MASK 0x20 +#define BD718XX_LDO2_MASK 0x20 +#define BD718XX_LDO3_MASK 0x0F +#define BD718XX_LDO4_MASK 0x0F +#define BD718XX_LDO6_MASK 0x0F + +#define BD71837_LDO5_MASK 0x0F +#define BD71847_LDO5_MASK 0x0F +#define BD71847_LDO5_RANGE_MASK 0x20 +#define BD71837_LDO7_MASK 0x0F + int power_bd71837_init(unsigned char bus); =20 #endif --=20 2.17.2 --=20 Matti Vaittinen, Linux device drivers ROHM Semiconductors, Finland SWDC Kiviharjunlenkki 1E 90220 OULU FINLAND ~~~ "I don't think so," said Rene Descartes. Just then he vanished ~~~