2008-09-11 14:12:01 +04:00
|
|
|
/*
|
|
|
|
* Regulator support for WM8400
|
|
|
|
*
|
|
|
|
* Copyright 2008 Wolfson Microelectronics PLC.
|
|
|
|
*
|
|
|
|
* Author: Mark Brown <broonie@opensource.wolfsonmicro.com>
|
|
|
|
*
|
|
|
|
* 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/bug.h>
|
|
|
|
#include <linux/err.h>
|
|
|
|
#include <linux/kernel.h>
|
2011-07-18 00:28:23 +04:00
|
|
|
#include <linux/module.h>
|
2008-09-11 14:12:01 +04:00
|
|
|
#include <linux/regulator/driver.h>
|
|
|
|
#include <linux/mfd/wm8400-private.h>
|
|
|
|
|
2013-07-03 02:26:36 +04:00
|
|
|
static const struct regulator_linear_range wm8400_ldo_ranges[] = {
|
2013-10-11 05:32:18 +04:00
|
|
|
REGULATOR_LINEAR_RANGE(900000, 0, 14, 50000),
|
|
|
|
REGULATOR_LINEAR_RANGE(1700000, 15, 31, 100000),
|
2013-07-03 02:26:36 +04:00
|
|
|
};
|
2008-09-11 14:12:01 +04:00
|
|
|
|
|
|
|
static struct regulator_ops wm8400_ldo_ops = {
|
2012-05-10 03:12:09 +04:00
|
|
|
.is_enabled = regulator_is_enabled_regmap,
|
|
|
|
.enable = regulator_enable_regmap,
|
|
|
|
.disable = regulator_disable_regmap,
|
2013-07-03 02:26:36 +04:00
|
|
|
.list_voltage = regulator_list_voltage_linear_range,
|
2012-05-10 03:12:09 +04:00
|
|
|
.get_voltage_sel = regulator_get_voltage_sel_regmap,
|
|
|
|
.set_voltage_sel = regulator_set_voltage_sel_regmap,
|
2013-07-03 02:26:36 +04:00
|
|
|
.map_voltage = regulator_map_voltage_linear_range,
|
2008-09-11 14:12:01 +04:00
|
|
|
};
|
|
|
|
|
|
|
|
static unsigned int wm8400_dcdc_get_mode(struct regulator_dev *dev)
|
|
|
|
{
|
|
|
|
struct wm8400 *wm8400 = rdev_get_drvdata(dev);
|
|
|
|
int offset = (rdev_get_id(dev) - WM8400_DCDC1) * 2;
|
|
|
|
u16 data[2];
|
|
|
|
int ret;
|
|
|
|
|
|
|
|
ret = wm8400_block_read(wm8400, WM8400_DCDC1_CONTROL_1 + offset, 2,
|
|
|
|
data);
|
|
|
|
if (ret != 0)
|
|
|
|
return 0;
|
|
|
|
|
|
|
|
/* Datasheet: hibernate */
|
|
|
|
if (data[0] & WM8400_DC1_SLEEP)
|
|
|
|
return REGULATOR_MODE_STANDBY;
|
|
|
|
|
|
|
|
/* Datasheet: standby */
|
|
|
|
if (!(data[0] & WM8400_DC1_ACTIVE))
|
|
|
|
return REGULATOR_MODE_IDLE;
|
|
|
|
|
|
|
|
/* Datasheet: active with or without force PWM */
|
|
|
|
if (data[1] & WM8400_DC1_FRC_PWM)
|
|
|
|
return REGULATOR_MODE_FAST;
|
|
|
|
else
|
|
|
|
return REGULATOR_MODE_NORMAL;
|
|
|
|
}
|
|
|
|
|
|
|
|
static int wm8400_dcdc_set_mode(struct regulator_dev *dev, unsigned int mode)
|
|
|
|
{
|
|
|
|
struct wm8400 *wm8400 = rdev_get_drvdata(dev);
|
|
|
|
int offset = (rdev_get_id(dev) - WM8400_DCDC1) * 2;
|
|
|
|
int ret;
|
|
|
|
|
|
|
|
switch (mode) {
|
|
|
|
case REGULATOR_MODE_FAST:
|
|
|
|
/* Datasheet: active with force PWM */
|
|
|
|
ret = wm8400_set_bits(wm8400, WM8400_DCDC1_CONTROL_2 + offset,
|
|
|
|
WM8400_DC1_FRC_PWM, WM8400_DC1_FRC_PWM);
|
|
|
|
if (ret != 0)
|
|
|
|
return ret;
|
|
|
|
|
|
|
|
return wm8400_set_bits(wm8400, WM8400_DCDC1_CONTROL_1 + offset,
|
|
|
|
WM8400_DC1_ACTIVE | WM8400_DC1_SLEEP,
|
|
|
|
WM8400_DC1_ACTIVE);
|
|
|
|
|
|
|
|
case REGULATOR_MODE_NORMAL:
|
|
|
|
/* Datasheet: active */
|
|
|
|
ret = wm8400_set_bits(wm8400, WM8400_DCDC1_CONTROL_2 + offset,
|
|
|
|
WM8400_DC1_FRC_PWM, 0);
|
|
|
|
if (ret != 0)
|
|
|
|
return ret;
|
|
|
|
|
|
|
|
return wm8400_set_bits(wm8400, WM8400_DCDC1_CONTROL_1 + offset,
|
|
|
|
WM8400_DC1_ACTIVE | WM8400_DC1_SLEEP,
|
|
|
|
WM8400_DC1_ACTIVE);
|
|
|
|
|
|
|
|
case REGULATOR_MODE_IDLE:
|
|
|
|
/* Datasheet: standby */
|
|
|
|
return wm8400_set_bits(wm8400, WM8400_DCDC1_CONTROL_1 + offset,
|
2012-08-01 05:15:11 +04:00
|
|
|
WM8400_DC1_ACTIVE | WM8400_DC1_SLEEP, 0);
|
2008-09-11 14:12:01 +04:00
|
|
|
default:
|
|
|
|
return -EINVAL;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
static unsigned int wm8400_dcdc_get_optimum_mode(struct regulator_dev *dev,
|
|
|
|
int input_uV, int output_uV,
|
|
|
|
int load_uA)
|
|
|
|
{
|
|
|
|
return REGULATOR_MODE_NORMAL;
|
|
|
|
}
|
|
|
|
|
|
|
|
static struct regulator_ops wm8400_dcdc_ops = {
|
2012-05-10 03:12:09 +04:00
|
|
|
.is_enabled = regulator_is_enabled_regmap,
|
|
|
|
.enable = regulator_enable_regmap,
|
|
|
|
.disable = regulator_disable_regmap,
|
|
|
|
.list_voltage = regulator_list_voltage_linear,
|
2012-05-31 13:40:22 +04:00
|
|
|
.map_voltage = regulator_map_voltage_linear,
|
2012-05-10 03:12:09 +04:00
|
|
|
.get_voltage_sel = regulator_get_voltage_sel_regmap,
|
|
|
|
.set_voltage_sel = regulator_set_voltage_sel_regmap,
|
2008-09-11 14:12:01 +04:00
|
|
|
.get_mode = wm8400_dcdc_get_mode,
|
|
|
|
.set_mode = wm8400_dcdc_set_mode,
|
|
|
|
.get_optimum_mode = wm8400_dcdc_get_optimum_mode,
|
|
|
|
};
|
|
|
|
|
|
|
|
static struct regulator_desc regulators[] = {
|
|
|
|
{
|
|
|
|
.name = "LDO1",
|
|
|
|
.id = WM8400_LDO1,
|
|
|
|
.ops = &wm8400_ldo_ops,
|
2012-05-10 03:12:09 +04:00
|
|
|
.enable_reg = WM8400_LDO1_CONTROL,
|
|
|
|
.enable_mask = WM8400_LDO1_ENA,
|
2009-03-10 19:28:35 +03:00
|
|
|
.n_voltages = WM8400_LDO1_VSEL_MASK + 1,
|
2013-07-03 02:26:36 +04:00
|
|
|
.linear_ranges = wm8400_ldo_ranges,
|
|
|
|
.n_linear_ranges = ARRAY_SIZE(wm8400_ldo_ranges),
|
2012-05-10 03:12:09 +04:00
|
|
|
.vsel_reg = WM8400_LDO1_CONTROL,
|
|
|
|
.vsel_mask = WM8400_LDO1_VSEL_MASK,
|
2008-09-11 14:12:01 +04:00
|
|
|
.type = REGULATOR_VOLTAGE,
|
|
|
|
.owner = THIS_MODULE,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
.name = "LDO2",
|
|
|
|
.id = WM8400_LDO2,
|
|
|
|
.ops = &wm8400_ldo_ops,
|
2012-05-10 03:12:09 +04:00
|
|
|
.enable_reg = WM8400_LDO2_CONTROL,
|
|
|
|
.enable_mask = WM8400_LDO2_ENA,
|
2009-03-10 19:28:35 +03:00
|
|
|
.n_voltages = WM8400_LDO2_VSEL_MASK + 1,
|
2013-07-03 02:26:36 +04:00
|
|
|
.linear_ranges = wm8400_ldo_ranges,
|
|
|
|
.n_linear_ranges = ARRAY_SIZE(wm8400_ldo_ranges),
|
2008-09-11 14:12:01 +04:00
|
|
|
.type = REGULATOR_VOLTAGE,
|
2012-05-10 03:12:09 +04:00
|
|
|
.vsel_reg = WM8400_LDO2_CONTROL,
|
|
|
|
.vsel_mask = WM8400_LDO2_VSEL_MASK,
|
2008-09-11 14:12:01 +04:00
|
|
|
.owner = THIS_MODULE,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
.name = "LDO3",
|
|
|
|
.id = WM8400_LDO3,
|
|
|
|
.ops = &wm8400_ldo_ops,
|
2012-05-10 03:12:09 +04:00
|
|
|
.enable_reg = WM8400_LDO3_CONTROL,
|
|
|
|
.enable_mask = WM8400_LDO3_ENA,
|
2009-03-10 19:28:35 +03:00
|
|
|
.n_voltages = WM8400_LDO3_VSEL_MASK + 1,
|
2013-07-03 02:26:36 +04:00
|
|
|
.linear_ranges = wm8400_ldo_ranges,
|
|
|
|
.n_linear_ranges = ARRAY_SIZE(wm8400_ldo_ranges),
|
2012-05-10 03:12:09 +04:00
|
|
|
.vsel_reg = WM8400_LDO3_CONTROL,
|
|
|
|
.vsel_mask = WM8400_LDO3_VSEL_MASK,
|
2008-09-11 14:12:01 +04:00
|
|
|
.type = REGULATOR_VOLTAGE,
|
|
|
|
.owner = THIS_MODULE,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
.name = "LDO4",
|
|
|
|
.id = WM8400_LDO4,
|
|
|
|
.ops = &wm8400_ldo_ops,
|
2012-05-10 03:12:09 +04:00
|
|
|
.enable_reg = WM8400_LDO4_CONTROL,
|
|
|
|
.enable_mask = WM8400_LDO4_ENA,
|
2009-03-10 19:28:35 +03:00
|
|
|
.n_voltages = WM8400_LDO4_VSEL_MASK + 1,
|
2013-07-03 02:26:36 +04:00
|
|
|
.linear_ranges = wm8400_ldo_ranges,
|
|
|
|
.n_linear_ranges = ARRAY_SIZE(wm8400_ldo_ranges),
|
2012-05-10 03:12:09 +04:00
|
|
|
.vsel_reg = WM8400_LDO4_CONTROL,
|
|
|
|
.vsel_mask = WM8400_LDO4_VSEL_MASK,
|
2008-09-11 14:12:01 +04:00
|
|
|
.type = REGULATOR_VOLTAGE,
|
|
|
|
.owner = THIS_MODULE,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
.name = "DCDC1",
|
|
|
|
.id = WM8400_DCDC1,
|
|
|
|
.ops = &wm8400_dcdc_ops,
|
2012-05-10 03:12:09 +04:00
|
|
|
.enable_reg = WM8400_DCDC1_CONTROL_1,
|
|
|
|
.enable_mask = WM8400_DC1_ENA_MASK,
|
2009-03-10 19:28:35 +03:00
|
|
|
.n_voltages = WM8400_DC1_VSEL_MASK + 1,
|
2012-05-10 03:12:09 +04:00
|
|
|
.vsel_reg = WM8400_DCDC1_CONTROL_1,
|
|
|
|
.vsel_mask = WM8400_DC1_VSEL_MASK,
|
|
|
|
.min_uV = 850000,
|
|
|
|
.uV_step = 25000,
|
2008-09-11 14:12:01 +04:00
|
|
|
.type = REGULATOR_VOLTAGE,
|
|
|
|
.owner = THIS_MODULE,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
.name = "DCDC2",
|
|
|
|
.id = WM8400_DCDC2,
|
|
|
|
.ops = &wm8400_dcdc_ops,
|
2012-05-10 03:12:09 +04:00
|
|
|
.enable_reg = WM8400_DCDC2_CONTROL_1,
|
|
|
|
.enable_mask = WM8400_DC1_ENA_MASK,
|
2009-03-10 19:28:35 +03:00
|
|
|
.n_voltages = WM8400_DC2_VSEL_MASK + 1,
|
2012-05-10 03:12:09 +04:00
|
|
|
.vsel_reg = WM8400_DCDC2_CONTROL_1,
|
|
|
|
.vsel_mask = WM8400_DC2_VSEL_MASK,
|
|
|
|
.min_uV = 850000,
|
|
|
|
.uV_step = 25000,
|
2008-09-11 14:12:01 +04:00
|
|
|
.type = REGULATOR_VOLTAGE,
|
|
|
|
.owner = THIS_MODULE,
|
|
|
|
},
|
|
|
|
};
|
|
|
|
|
2012-11-19 22:22:22 +04:00
|
|
|
static int wm8400_regulator_probe(struct platform_device *pdev)
|
2008-09-11 14:12:01 +04:00
|
|
|
{
|
Regulators: wm8400 - cleanup platform driver data handling
On Wed, Feb 24, 2010 at 08:40:56PM +0000, Mark Brown wrote:
> On Wed, Feb 24, 2010 at 11:21:26AM -0800, Dmitry Torokhov wrote:
> > On Wed, Feb 24, 2010 at 07:14:03PM +0000, Mark Brown wrote:
>
> > > This doesn't help unless you also provide a way for users to obtain a
> > > struct wm8400.
>
> > Why would they need it? Only code that creates instances of wm8400 needs
> > to know the definition of the sturcture, the rest can simply pass the
> > pointer around.
>
> > I guess there is disconnect between us and I do not see any users of
> > wm8400_register_regulator() in linux-next... Is there another tree I
> > could peek at?
>
> There are no users in mainline. This would be called by board specific
> code from the init callback of the wm8400 - you'd need to pass that
> callback the struct wm8400.
>
> In any case, this is clearly an unrelated change to whatever else you
> were doing to the driver so should be split off into a separate patch,
> but if this is being changed at all then it'd be much more sensible to
> change it to use a more modern pattern which completely removes the
> wm8400_register_regulator() function and just uses platform data.
Fair enough, I removed the offending part, updated patch below.
--
Dmitry
regulator: wm8400 - cleanup platform driver data handling
Driver data set by platform_set_drvdata() is for private use of
the driver currently bound to teh device and not for use by parent,
subsystem and anyone else.
Signed-off-by: Dmitry Torokhov <dtor@mail.ru>
Acked-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
Signed-off-by: Liam Girdwood <lrg@slimlogic.co.uk>
2010-02-25 12:55:37 +03:00
|
|
|
struct wm8400 *wm8400 = container_of(pdev, struct wm8400, regulators[pdev->id]);
|
2012-04-04 03:50:22 +04:00
|
|
|
struct regulator_config config = { };
|
2008-09-11 14:12:01 +04:00
|
|
|
struct regulator_dev *rdev;
|
|
|
|
|
2012-04-04 03:50:22 +04:00
|
|
|
config.dev = &pdev->dev;
|
2013-07-30 12:20:47 +04:00
|
|
|
config.init_data = dev_get_platdata(&pdev->dev);
|
2012-04-04 03:50:22 +04:00
|
|
|
config.driver_data = wm8400;
|
2012-05-10 03:12:09 +04:00
|
|
|
config.regmap = wm8400->regmap;
|
2008-09-11 14:12:01 +04:00
|
|
|
|
2013-08-31 15:00:57 +04:00
|
|
|
rdev = devm_regulator_register(&pdev->dev, ®ulators[pdev->id],
|
|
|
|
&config);
|
2008-09-11 14:12:01 +04:00
|
|
|
if (IS_ERR(rdev))
|
|
|
|
return PTR_ERR(rdev);
|
|
|
|
|
Regulators: wm8400 - cleanup platform driver data handling
On Wed, Feb 24, 2010 at 08:40:56PM +0000, Mark Brown wrote:
> On Wed, Feb 24, 2010 at 11:21:26AM -0800, Dmitry Torokhov wrote:
> > On Wed, Feb 24, 2010 at 07:14:03PM +0000, Mark Brown wrote:
>
> > > This doesn't help unless you also provide a way for users to obtain a
> > > struct wm8400.
>
> > Why would they need it? Only code that creates instances of wm8400 needs
> > to know the definition of the sturcture, the rest can simply pass the
> > pointer around.
>
> > I guess there is disconnect between us and I do not see any users of
> > wm8400_register_regulator() in linux-next... Is there another tree I
> > could peek at?
>
> There are no users in mainline. This would be called by board specific
> code from the init callback of the wm8400 - you'd need to pass that
> callback the struct wm8400.
>
> In any case, this is clearly an unrelated change to whatever else you
> were doing to the driver so should be split off into a separate patch,
> but if this is being changed at all then it'd be much more sensible to
> change it to use a more modern pattern which completely removes the
> wm8400_register_regulator() function and just uses platform data.
Fair enough, I removed the offending part, updated patch below.
--
Dmitry
regulator: wm8400 - cleanup platform driver data handling
Driver data set by platform_set_drvdata() is for private use of
the driver currently bound to teh device and not for use by parent,
subsystem and anyone else.
Signed-off-by: Dmitry Torokhov <dtor@mail.ru>
Acked-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
Signed-off-by: Liam Girdwood <lrg@slimlogic.co.uk>
2010-02-25 12:55:37 +03:00
|
|
|
platform_set_drvdata(pdev, rdev);
|
|
|
|
|
2008-09-11 14:12:01 +04:00
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
|
|
|
static struct platform_driver wm8400_regulator_driver = {
|
|
|
|
.driver = {
|
|
|
|
.name = "wm8400-regulator",
|
|
|
|
},
|
|
|
|
.probe = wm8400_regulator_probe,
|
|
|
|
};
|
|
|
|
|
|
|
|
/**
|
|
|
|
* wm8400_register_regulator - enable software control of a WM8400 regulator
|
|
|
|
*
|
|
|
|
* This function enables software control of a WM8400 regulator via
|
|
|
|
* the regulator API. It is intended to be called from the
|
|
|
|
* platform_init() callback of the WM8400 MFD driver.
|
|
|
|
*
|
|
|
|
* @param dev The WM8400 device to operate on.
|
|
|
|
* @param reg The regulator to control.
|
|
|
|
* @param initdata Regulator initdata for the regulator.
|
|
|
|
*/
|
|
|
|
int wm8400_register_regulator(struct device *dev, int reg,
|
|
|
|
struct regulator_init_data *initdata)
|
|
|
|
{
|
2009-05-01 02:21:37 +04:00
|
|
|
struct wm8400 *wm8400 = dev_get_drvdata(dev);
|
2008-09-11 14:12:01 +04:00
|
|
|
|
|
|
|
if (wm8400->regulators[reg].name)
|
|
|
|
return -EBUSY;
|
|
|
|
|
|
|
|
initdata->driver_data = wm8400;
|
|
|
|
|
|
|
|
wm8400->regulators[reg].name = "wm8400-regulator";
|
|
|
|
wm8400->regulators[reg].id = reg;
|
|
|
|
wm8400->regulators[reg].dev.parent = dev;
|
|
|
|
wm8400->regulators[reg].dev.platform_data = initdata;
|
|
|
|
|
|
|
|
return platform_device_register(&wm8400->regulators[reg]);
|
|
|
|
}
|
|
|
|
EXPORT_SYMBOL_GPL(wm8400_register_regulator);
|
|
|
|
|
|
|
|
static int __init wm8400_regulator_init(void)
|
|
|
|
{
|
|
|
|
return platform_driver_register(&wm8400_regulator_driver);
|
|
|
|
}
|
2009-04-27 21:21:18 +04:00
|
|
|
subsys_initcall(wm8400_regulator_init);
|
2008-09-11 14:12:01 +04:00
|
|
|
|
|
|
|
static void __exit wm8400_regulator_exit(void)
|
|
|
|
{
|
|
|
|
platform_driver_unregister(&wm8400_regulator_driver);
|
|
|
|
}
|
|
|
|
module_exit(wm8400_regulator_exit);
|
|
|
|
|
|
|
|
MODULE_AUTHOR("Mark Brown <broonie@opensource.wolfsonmicro.com>");
|
|
|
|
MODULE_DESCRIPTION("WM8400 regulator driver");
|
|
|
|
MODULE_LICENSE("GPL");
|
|
|
|
MODULE_ALIAS("platform:wm8400-regulator");
|