[U-Boot] [PATCH 24/55] dm: power: Add support for S5M8767 regulators
Przemyslaw Marczak
p.marczak at samsung.com
Fri Jul 10 13:52:49 CEST 2015
Hello Simon,
On 07/03/2015 02:16 AM, Simon Glass wrote:
> This PMIC is used with SoCs which need a combination of BUCKs and LDOs. The
> driver supports changing voltage and enabling/disabling each regulator. It
> supports the standard device tree binding and supports driver model.
>
> Signed-off-by: Simon Glass <sjg at chromium.org>
> ---
>
> drivers/power/regulator/Kconfig | 9 ++
> drivers/power/regulator/Makefile | 1 +
> drivers/power/regulator/s5m8767.c | 269 ++++++++++++++++++++++++++++++++++++++
> 3 files changed, 279 insertions(+)
> create mode 100644 drivers/power/regulator/s5m8767.c
>
> diff --git a/drivers/power/regulator/Kconfig b/drivers/power/regulator/Kconfig
> index d090650..e85c692 100644
> --- a/drivers/power/regulator/Kconfig
> +++ b/drivers/power/regulator/Kconfig
> @@ -32,6 +32,15 @@ config DM_REGULATOR_FIXED
> features for fixed value regulators. The driver implements get/set api
> for enable and get only for voltage value.
>
> +config REGULATOR_S5M8767
> + bool "Enable support for S5M8767 regulator"
> + depends on DM_REGULATOR && PMIC_S5M8767
> + ---help---
> + This enables the regulator features of the S5M8767, allowing voltages
> + to be set, etc. The driver is not fully complete but supports most
> + common requirements, including all LDOs and BUCKs. This allows many
> + supplies to be set automatically using the device tree values.
> +
> config DM_REGULATOR_SANDBOX
> bool "Enable Driver Model for Sandbox PMIC regulator"
> depends on DM_REGULATOR && DM_PMIC_SANDBOX
> diff --git a/drivers/power/regulator/Makefile b/drivers/power/regulator/Makefile
> index 43f4d2c..08d7b0d 100644
> --- a/drivers/power/regulator/Makefile
> +++ b/drivers/power/regulator/Makefile
> @@ -8,5 +8,6 @@
> obj-$(CONFIG_DM_REGULATOR) += regulator-uclass.o
> obj-$(CONFIG_DM_REGULATOR_MAX77686) += max77686.o
> obj-$(CONFIG_DM_REGULATOR_FIXED) += fixed.o
> +obj-$(CONFIG_REGULATOR_S5M8767) += s5m8767.o
> obj-$(CONFIG_DM_REGULATOR_SANDBOX) += sandbox.o
> obj-$(CONFIG_REGULATOR_TPS65090) += tps65090_regulator.o
> diff --git a/drivers/power/regulator/s5m8767.c b/drivers/power/regulator/s5m8767.c
> new file mode 100644
> index 0000000..93a3c94
> --- /dev/null
> +++ b/drivers/power/regulator/s5m8767.c
> @@ -0,0 +1,269 @@
> +/*
> + * Copyright (C) 2015 Google, Inc
> + *
> + * 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/s5m8767.h>
> +
> +DECLARE_GLOBAL_DATA_PTR;
> +
> +static const struct sec_voltage_desc buck_v1 = {
> + .max = 2225000,
> + .min = 650000,
> + .step = 6250,
> +};
> +
> +static const struct sec_voltage_desc buck_v2 = {
> + .max = 1600000,
> + .min = 600000,
> + .step = 6250,
> +};
> +
> +static const struct sec_voltage_desc buck_v3 = {
> + .max = 3000000,
> + .min = 750000,
> + .step = 12500,
> +};
> +
> +static const struct sec_voltage_desc ldo_v1 = {
> + .max = 3950000,
> + .min = 800000,
> + .step = 50000,
> +};
> +
> +static const struct sec_voltage_desc ldo_v2 = {
> + .max = 2375000,
> + .min = 800000,
> + .step = 25000,
> +};
> +
> +static const struct s5m8767_para buck_param[] = {
> + /*
> + * | voltage ----| | enable -| voltage
> + * regnum addr bpos mask addr on desc
> + */
> + {S5M8767_BUCK1, 0x33, 0x0, 0xff, 0x32, 0x3, &buck_v1},
> + {S5M8767_BUCK2, 0x35, 0x0, 0xff, 0x34, 0x1, &buck_v2},
> + {S5M8767_BUCK3, 0x3e, 0x0, 0xff, 0x3d, 0x1, &buck_v2},
> + {S5M8767_BUCK4, 0x47, 0x0, 0xff, 0x46, 0x1, &buck_v2},
> + {S5M8767_BUCK5, 0x50, 0x0, 0xff, 0x4f, 0x3, &buck_v1},
> + {S5M8767_BUCK6, 0x55, 0x0, 0xff, 0x54, 0x3, &buck_v1},
> + {S5M8767_BUCK7, 0x57, 0x0, 0xff, 0x56, 0x3, &buck_v3},
> + {S5M8767_BUCK8, 0x59, 0x0, 0xff, 0x58, 0x3, &buck_v3},
> + {S5M8767_BUCK9, 0x5b, 0x0, 0xff, 0x5a, 0x3, &buck_v3},
> +};
> +
> +static const struct s5m8767_para ldo_param[] = {
> + {S5M8767_LDO1, 0x5c, 0x0, 0x3f, 0x5c, 0x3, &ldo_v2},
> + {S5M8767_LDO2, 0x5d, 0x0, 0x3f, 0x5d, 0x1, &ldo_v2},
> + {S5M8767_LDO3, 0x61, 0x0, 0x3f, 0x61, 0x3, &ldo_v1},
> + {S5M8767_LDO4, 0x62, 0x0, 0x3f, 0x62, 0x3, &ldo_v1},
> + {S5M8767_LDO5, 0x63, 0x0, 0x3f, 0x63, 0x3, &ldo_v1},
> + {S5M8767_LDO6, 0x64, 0x0, 0x3f, 0x64, 0x1, &ldo_v2},
> + {S5M8767_LDO7, 0x65, 0x0, 0x3f, 0x65, 0x1, &ldo_v2},
> + {S5M8767_LDO8, 0x66, 0x0, 0x3f, 0x66, 0x1, &ldo_v2},
> + {S5M8767_LDO9, 0x67, 0x0, 0x3f, 0x67, 0x3, &ldo_v1},
> + {S5M8767_LDO10, 0x68, 0x0, 0x3f, 0x68, 0x1, &ldo_v1},
> + {S5M8767_LDO11, 0x69, 0x0, 0x3f, 0x69, 0x1, &ldo_v1},
> + {S5M8767_LDO12, 0x6a, 0x0, 0x3f, 0x6a, 0x1, &ldo_v1},
> + {S5M8767_LDO13, 0x6b, 0x0, 0x3f, 0x6b, 0x3, &ldo_v1},
> + {S5M8767_LDO14, 0x6c, 0x0, 0x3f, 0x6c, 0x1, &ldo_v1},
> + {S5M8767_LDO15, 0x6d, 0x0, 0x3f, 0x6d, 0x1, &ldo_v2},
> + {S5M8767_LDO16, 0x6e, 0x0, 0x3f, 0x6e, 0x1, &ldo_v1},
> + {S5M8767_LDO17, 0x6f, 0x0, 0x3f, 0x6f, 0x3, &ldo_v1},
> + {S5M8767_LDO18, 0x70, 0x0, 0x3f, 0x70, 0x3, &ldo_v1},
> + {S5M8767_LDO19, 0x71, 0x0, 0x3f, 0x71, 0x3, &ldo_v1},
> + {S5M8767_LDO20, 0x72, 0x0, 0x3f, 0x72, 0x3, &ldo_v1},
> + {S5M8767_LDO21, 0x73, 0x0, 0x3f, 0x73, 0x3, &ldo_v1},
> + {S5M8767_LDO22, 0x74, 0x0, 0x3f, 0x74, 0x3, &ldo_v1},
> + {S5M8767_LDO23, 0x75, 0x0, 0x3f, 0x75, 0x3, &ldo_v1},
> + {S5M8767_LDO24, 0x76, 0x0, 0x3f, 0x76, 0x3, &ldo_v1},
> + {S5M8767_LDO25, 0x77, 0x0, 0x3f, 0x77, 0x3, &ldo_v1},
> + {S5M8767_LDO26, 0x78, 0x0, 0x3f, 0x78, 0x3, &ldo_v1},
> + {S5M8767_LDO27, 0x79, 0x0, 0x3f, 0x79, 0x3, &ldo_v1},
> + {S5M8767_LDO28, 0x7a, 0x0, 0x3f, 0x7a, 0x3, &ldo_v1},
> +};
> +
> +enum {
> + ENABLE_SHIFT = 6,
> + ENABLE_MASK = 3,
> +};
> +
> +static int reg_get_value(struct udevice *dev, const struct s5m8767_para *param)
> +{
> + const struct sec_voltage_desc *desc;
> + int ret, uv, val;
> +
> + ret = pmic_reg_read(dev->parent, param->vol_addr);
> + if (ret < 0)
> + return ret;
> +
> + desc = param->vol;
> + val = (ret >> param->vol_bitpos) & param->vol_bitmask;
> + uv = desc->min + val * desc->step;
> +
> + return uv;
> +}
> +
> +static int reg_set_value(struct udevice *dev, const struct s5m8767_para *param,
> + int uv)
> +{
> + const struct sec_voltage_desc *desc;
> + int ret, val;
> +
> + desc = param->vol;
> + if (uv < desc->min || uv > desc->max)
> + return -EINVAL;
> + val = (uv - desc->min) / desc->step;
> + val = (val & param->vol_bitmask) << param->vol_bitpos;
> + ret = pmic_clrsetbits(dev->parent, param->vol_addr,
> + param->vol_bitmask << param->vol_bitpos,
> + val);
> +
> + return ret;
> +}
> +
> +static int s5m8767_ldo_probe(struct udevice *dev)
> +{
> + struct dm_regulator_uclass_platdata *uc_pdata;
> +
> + uc_pdata = dev_get_uclass_platdata(dev);
> +
> + uc_pdata->type = REGULATOR_TYPE_LDO;
> + uc_pdata->mode_count = 0;
> +
> + return 0;
> +}
> +static int ldo_get_value(struct udevice *dev)
> +{
> + int ldo = dev->driver_data;
> +
> + return reg_get_value(dev, &ldo_param[ldo]);
> +}
> +
> +static int ldo_set_value(struct udevice *dev, int uv)
> +{
> + int ldo = dev->driver_data;
> +
> + return reg_set_value(dev, &ldo_param[ldo], uv);
> +}
> +
> +static int reg_get_enable(struct udevice *dev, const struct s5m8767_para *param)
> +{
> + bool enable;
> + int ret;
> +
> + ret = pmic_reg_read(dev->parent, param->reg_enaddr);
> + if (ret < 0)
> + return ret;
> +
> + enable = (ret >> ENABLE_SHIFT) & ENABLE_MASK;
> +
> + return enable;
> +}
> +
> +static int reg_set_enable(struct udevice *dev, const struct s5m8767_para *param,
> + bool enable)
> +{
> + int ret;
> +
> + ret = pmic_reg_read(dev->parent, param->reg_enaddr);
> + if (ret < 0)
> + return ret;
> +
> + ret = pmic_clrsetbits(dev->parent, param->reg_enaddr,
> + ENABLE_MASK << ENABLE_SHIFT,
> + enable ? param->reg_enbiton << ENABLE_SHIFT : 0);
> +
> + return ret;
> +}
> +
> +static bool ldo_get_enable(struct udevice *dev)
> +{
> + int ldo = dev->driver_data;
> +
> + return reg_get_enable(dev, &ldo_param[ldo]);
> +}
> +
> +static int ldo_set_enable(struct udevice *dev, bool enable)
> +{
> + int ldo = dev->driver_data;
> +
> + return reg_set_enable(dev, &ldo_param[ldo], enable);
> +}
> +
> +static int s5m8767_buck_probe(struct udevice *dev)
> +{
> + struct dm_regulator_uclass_platdata *uc_pdata;
> +
> + uc_pdata = dev_get_uclass_platdata(dev);
> +
> + uc_pdata->type = REGULATOR_TYPE_BUCK;
> + uc_pdata->mode_count = 0;
> +
> + return 0;
> +}
> +
> +static int buck_get_value(struct udevice *dev)
> +{
> + int buck = dev->driver_data;
> +
> + return reg_get_value(dev, &buck_param[buck]);
> +}
> +
> +static int buck_set_value(struct udevice *dev, int uv)
> +{
> + int buck = dev->driver_data;
> +
> + return reg_set_value(dev, &buck_param[buck], uv);
> +}
> +
> +static bool buck_get_enable(struct udevice *dev)
> +{
> + int buck = dev->driver_data;
> +
> + return reg_get_enable(dev, &buck_param[buck]);
> +}
> +
> +static int buck_set_enable(struct udevice *dev, bool enable)
> +{
> + int buck = dev->driver_data;
> +
> + return reg_set_enable(dev, &buck_param[buck], enable);
> +}
> +
> +static const struct dm_regulator_ops s5m8767_ldo_ops = {
> + .get_value = ldo_get_value,
> + .set_value = ldo_set_value,
> + .get_enable = ldo_get_enable,
> + .set_enable = ldo_set_enable,
> +};
> +
> +U_BOOT_DRIVER(s5m8767_ldo) = {
> + .name = S5M8767_LDO_DRIVER,
> + .id = UCLASS_REGULATOR,
> + .ops = &s5m8767_ldo_ops,
> + .probe = s5m8767_ldo_probe,
> +};
> +
> +static const struct dm_regulator_ops s5m8767_buck_ops = {
> + .get_value = buck_get_value,
> + .set_value = buck_set_value,
> + .get_enable = buck_get_enable,
> + .set_enable = buck_set_enable,
> +};
> +
> +U_BOOT_DRIVER(s5m8767_buck) = {
> + .name = S5M8767_BUCK_DRIVER,
> + .id = UCLASS_REGULATOR,
> + .ops = &s5m8767_buck_ops,
> + .probe = s5m8767_buck_probe,
> +};
>
Acked-by: Przemyslaw Marczak <p.marczak at samsung.com>
Regards
--
Przemyslaw Marczak
Samsung R&D Institute Poland
Samsung Electronics
p.marczak at samsung.com
More information about the U-Boot
mailing list