| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Date: Fri, 12 Apr 2013 06:32:21 -0700
From: Guenter Roeck <linux@...ck-us.net>
To: Anthony Olech <anthony.olech.opensource@...semi.com>
Cc: Mark Brown <broonie@...nsource.wolfsonmicro.com>,
Liam Girdwood <lrg@...com>, Jean Delvare <khali@...ux-fr.org>,
Randy Dunlap <rdunlap@...otime.net>,
LKML <linux-kernel@...r.kernel.org>,
David Dajun Chen <david.chen@...semi.com>
Subject: Re: [NEW DRIVER V4 7/7] DA9058 REGULATOR driver
On Fri, Apr 12, 2013 at 02:05:28PM +0100, Anthony Olech wrote:
> This is the REGULATOR component driver of the Dialog DA9058 PMIC.
> This driver is just one component of the whole DA9058 PMIC driver.
> It depends on the CORE component driver of the DA9058 MFD.
>
> There are 6 warnings from scripts/checkpatch.pl, but since it seems to
> be complaining about variable names such as min_uV are in CamelCase,
> when it is obvious that they are not in CamelCase I have ignored them.
>
??? min_uV _is_ CamelCase ???
Ok, maybe it is camelcasE, but you are splitting hairs here.
Guenter
> Signed-off-by: Anthony Olech <anthony.olech.opensource@...semi.com>
> Signed-off-by: David Dajun Chen <david.chen@...semi.com>
> ---
> drivers/regulator/Kconfig | 11 ++
> drivers/regulator/Makefile | 1 +
> drivers/regulator/da9058-regulator.c | 228 ++++++++++++++++++++++++++++++++++
> 3 files changed, 240 insertions(+)
> create mode 100644 drivers/regulator/da9058-regulator.c
>
> diff --git a/drivers/regulator/Kconfig b/drivers/regulator/Kconfig
> index a5d97ea..3b7b098 100644
> --- a/drivers/regulator/Kconfig
> +++ b/drivers/regulator/Kconfig
> @@ -64,6 +64,17 @@ config REGULATOR_USERSPACE_CONSUMER
>
> If unsure, say no.
>
> +config REGULATOR_DA9058
> + tristate "Support regulators on Dialog Semiconductor DA9058 PMIC"
> + depends on MFD_DA9058
> + help
> + Say y here to support the BUCKs and LDOs regulators found on
> + Dialog Semiconductor DA9058 PMIC.
> +
> + This driver can also be built as a module. If so, the module
> + will be called da9058-regulator.
> +
> +
> config REGULATOR_GPIO
> tristate "GPIO regulator support"
> depends on GENERIC_GPIO
> diff --git a/drivers/regulator/Makefile b/drivers/regulator/Makefile
> index 6e82503..f8e1784 100644
> --- a/drivers/regulator/Makefile
> +++ b/drivers/regulator/Makefile
> @@ -25,6 +25,7 @@ obj-$(CONFIG_REGULATOR_DB8500_PRCMU) += db8500-prcmu.o
> obj-$(CONFIG_REGULATOR_FAN53555) += fan53555.o
> obj-$(CONFIG_REGULATOR_GPIO) += gpio-regulator.o
> obj-$(CONFIG_REGULATOR_ISL6271A) += isl6271a-regulator.o
> +obj-$(CONFIG_REGULATOR_DA9058) += da9058-regulator.o
> obj-$(CONFIG_REGULATOR_LP3971) += lp3971.o
> obj-$(CONFIG_REGULATOR_LP3972) += lp3972.o
> obj-$(CONFIG_REGULATOR_LP872X) += lp872x.o
> diff --git a/drivers/regulator/da9058-regulator.c b/drivers/regulator/da9058-regulator.c
> new file mode 100644
> index 0000000..a033364
> --- /dev/null
> +++ b/drivers/regulator/da9058-regulator.c
> @@ -0,0 +1,228 @@
> +/*
> + * Copyright (C) 2012 Dialog Semiconductor Ltd.
> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License as published by
> + * the Free Software Foundation; either version 2 of the License, or
> + * (at your option) any later version.
> + *
> + */
> +
> +#include <linux/module.h>
> +#include <linux/err.h>
> +#include <linux/regulator/machine.h>
> +#include <linux/regulator/driver.h>
> +#include <linux/regmap.h>
> +#include <linux/mfd/core.h>
> +
> +#include <linux/mfd/da9058/version.h>
> +#include <linux/mfd/da9058/registers.h>
> +#include <linux/mfd/da9058/core.h>
> +#include <linux/mfd/da9058/regulator.h>
> +
> +struct da9058_regulator {
> + struct da9058 *da9058;
> + int ramp_register;
> + int ramp_enable_mask;
> + struct platform_device *pdev;
> + struct regulator_dev *reg_dev;
> + struct regulator_desc desc;
> + struct regulator_init_data init;
> +};
> +
> +static int da9058_buck_ramp_voltage(struct regulator_dev *rdev,
> + unsigned int old_selector,
> + unsigned int new_selector)
> +{
> + struct da9058_regulator *regulator = rdev_get_drvdata(rdev);
> + struct da9058 *da9058 = regulator->da9058;
> + int ret;
> +
> + if (regulator->ramp_register == 0)
> + return -EINVAL;
> +
> + if (regulator->ramp_enable_mask == 0)
> + return -EINVAL;
> +
> + ret = da9058_set_bits(da9058, regulator->ramp_register,
> + regulator->ramp_enable_mask);
> +
> + if (ret)
> + return ret;
> +
> + return 2200; /* micro Seconds needed to ramp to new voltage*/
> +}
> +
> +static struct regulator_ops da9058_buck_regulator_ops = {
> + .map_voltage = regulator_map_voltage_linear,
> + .list_voltage = regulator_list_voltage_linear,
> + .set_voltage_time_sel = da9058_buck_ramp_voltage,
> + .get_voltage_sel = regulator_get_voltage_sel_regmap,
> + .set_voltage_sel = regulator_set_voltage_sel_regmap,
> + .enable = regulator_enable_regmap,
> + .disable = regulator_disable_regmap,
> + .is_enabled = regulator_is_enabled_regmap,
> +};
> +
> +static struct regulator_ops da9058_ldo_regulator_ops = {
> + .map_voltage = regulator_map_voltage_linear,
> + .list_voltage = regulator_list_voltage_linear,
> + .get_voltage_sel = regulator_get_voltage_sel_regmap,
> + .set_voltage_sel = regulator_set_voltage_sel_regmap,
> + .enable = regulator_enable_regmap,
> + .disable = regulator_disable_regmap,
> + .is_enabled = regulator_is_enabled_regmap,
> +};
> +
> +static int da9058_regulator_probe(struct platform_device *pdev)
> +{
> + struct da9058 *da9058 = dev_get_drvdata(pdev->dev.parent);
> + const struct mfd_cell *cell = mfd_get_cell(pdev);
> + struct da9058_regulator_pdata *rpdata;
> + struct da9058_regulator *reg;
> + struct regulator_dev *rdev;
> + struct regulator_config config = { };
> + int ret;
> + unsigned int val;
> +
> + if (cell == NULL) {
> + ret = -ENODEV;
> + goto exit;
> + }
> +
> + rpdata = cell->platform_data;
> +
> + if (rpdata == NULL) {
> + ret = -EINVAL;
> + goto exit;
> + }
> +
> + if (rpdata->control_register == 0) {
> + ret = -EINVAL;
> + goto exit;
> + }
> +
> + if (rpdata->control_enable_mask == 0) {
> + ret = -EINVAL;
> + goto exit;
> + }
> +
> + reg = devm_kzalloc(&pdev->dev, sizeof(struct da9058_regulator),
> + GFP_KERNEL);
> + if (!reg) {
> + ret = -ENOMEM;
> + goto exit;
> + }
> +
> + platform_set_drvdata(pdev, reg);
> +
> + reg->da9058 = da9058;
> + reg->pdev = pdev;
> + reg->ramp_register = rpdata->ramp_register;
> + reg->ramp_enable_mask = rpdata->ramp_enable_mask;
> +
> + reg->desc.name = rpdata->regulator_name;
> + reg->desc.id = rpdata->regulator_id;
> + reg->desc.type = REGULATOR_VOLTAGE;
> + reg->desc.n_voltages = 1;
> +
> + if (rpdata->control_voltage_step > 0)
> + reg->desc.n_voltages += (rpdata->max_uV - rpdata->min_uV) /
> + rpdata->control_voltage_step;
> +
> + reg->desc.owner = THIS_MODULE;
> + reg->desc.enable_reg = rpdata->control_register;
> + reg->desc.enable_mask = rpdata->control_enable_mask;
> +
> + reg->desc.vsel_reg = rpdata->control_register;
> + reg->desc.vsel_mask = rpdata->control_step_mask;
> + reg->desc.min_uV = rpdata->min_uV;
> + reg->desc.uV_step = rpdata->control_voltage_step;
> +
> + if (reg->ramp_register)
> + reg->desc.ops = &da9058_buck_regulator_ops;
> + else
> + reg->desc.ops = &da9058_ldo_regulator_ops;
> +
> + reg->init.constraints.name = rpdata->regulator_name;
> + reg->init.constraints.min_uV = rpdata->min_uV;
> + reg->init.constraints.max_uV = rpdata->max_uV;
> + reg->init.constraints.valid_ops_mask = rpdata->valid_ops_mask;
> + reg->init.constraints.valid_modes_mask = rpdata->valid_modes_mask;
> + reg->init.constraints.boot_on = rpdata->boot_on;
> + reg->init.constraints.always_on = rpdata->always_on;
> + reg->init.num_consumer_supplies = rpdata->num_consumer_supplies;
> + reg->init.consumer_supplies = rpdata->consumer_supplies;
> +
> + config.dev = pdev->dev.parent;
> + config.init_data = ®->init;
> + config.driver_data = reg;
> + config.regmap = da9058->regmap;
> +
> + rdev = regulator_register(®->desc, &config);
> +
> + if (IS_ERR(rdev)) {
> + dev_err(&pdev->dev, "failed to register %s\n",
> + rpdata->regulator_name);
> + ret = PTR_ERR(rdev);
> + goto failed_to_register;
> + }
> + reg->reg_dev = rdev;
> +
> + /* before we do anything check the lock bit */
> + ret = da9058_reg_read(da9058, DA9058_SUPPLY_REG, &val);
> + if (ret)
> + goto unlock_failed;
> +
> + if (val & DA9058_SUPPLY_VLOCK)
> + ret = da9058_clear_bits(da9058, DA9058_SUPPLY_REG,
> + DA9058_SUPPLY_VLOCK);
> + if (ret)
> + goto unlock_failed;
> +
> + goto exit;
> +
> +unlock_failed:
> + regulator_unregister(rdev);
> +failed_to_register:
> + platform_set_drvdata(pdev, NULL);
> +exit:
> + return ret;
> +}
> +
> +static int da9058_regulator_remove(struct platform_device *pdev)
> +{
> + struct regulator_dev *rdev = platform_get_drvdata(pdev);
> +
> + regulator_unregister(rdev);
> +
> + return 0;
> +}
> +
> +static struct platform_driver da9058_regulator_driver = {
> + .probe = da9058_regulator_probe,
> + .remove = da9058_regulator_remove,
> + .driver = {
> + .name = "da9058-regulator",
> + .owner = THIS_MODULE,
> + },
> +};
> +
> +static int __init da9058_regulator_init(void)
> +{
> + return platform_driver_register(&da9058_regulator_driver);
> +}
> +
> +subsys_initcall(da9058_regulator_init);
> +
> +static void __exit da9058_regulator_exit(void)
> +{
> + platform_driver_unregister(&da9058_regulator_driver);
> +}
> +
> +module_exit(da9058_regulator_exit);
> +
> +MODULE_DESCRIPTION("Dialog DA9058 PMIC voltage and current regulator");
> +MODULE_AUTHOR("Anthony Olech <Anthony.Olech@...semi.com>");
> +MODULE_LICENSE("GPL v2");
> +MODULE_ALIAS("platform:da9058-regulator");
> --
> end-of-patch for NEW DRIVER V4
>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists