[PATCH] support PMIC mc13892

Yong Shen yong.shen at linaro.org
Mon Dec 6 21:41:29 EST 2010


Hi Uwe,

2010/12/6 Uwe Kleine-König <u.kleine-koenig at pengutronix.de>:
> Hello,
>
> On Mon, Dec 06, 2010 at 02:25:35PM +0800, yong.shen at freescale.com wrote:
>> From: Yong Shen <yong.shen at linaro.org>
>>
>> add support for mc13892, tested on mx51 babbage board
>>
>> Signed-off-by: Yong Shen <yong.shen at linaro.org>
>> ---
>>  drivers/mfd/Kconfig                   |    4 +
>>  drivers/regulator/Kconfig             |    8 +
>>  drivers/regulator/Makefile            |    1 +
>>  drivers/regulator/mc13892-regulator.c |  647 +++++++++++++++++++++++++++++++++
>>  include/linux/mfd/mc13892.h           |   38 ++
>>  5 files changed, 698 insertions(+), 0 deletions(-)
>>  create mode 100644 drivers/regulator/mc13892-regulator.c
>>  create mode 100644 include/linux/mfd/mc13892.h
>>
>> diff --git a/drivers/mfd/Kconfig b/drivers/mfd/Kconfig
>> index 3a1493b..7659af9 100644
>> --- a/drivers/mfd/Kconfig
>> +++ b/drivers/mfd/Kconfig
>> @@ -427,11 +427,15 @@ config MFD_PCF50633
>>  config MFD_MC13783
>>       tristate
>>
>> +config MFD_MC13892
>> +     tristate
>> +
> This is not needed, MFD_MC13783 only exists because when MFD_MC13XXX was
> introduced some drivers still depended on MFD_MC13783.  As mc13xxx.[ch]
> provided enough compatibility cruft MFD_MC13XXX just selected MFD_MC13783.
My original thought was to keep align with previous structure. In the
case you mentioned, I will remove it.
>
>>  config MFD_MC13XXX
>>       tristate "Support Freescale MC13783 and MC13892"
>>       depends on SPI_MASTER
>>       select MFD_CORE
>>       select MFD_MC13783
>> +     select MFD_MC13892
>>       help
>>         Support for the Freescale (Atlas) PMIC and audio CODECs
>>         MC13783 and MC13892.
>> diff --git a/drivers/regulator/Kconfig b/drivers/regulator/Kconfig
>> index 6e54253..7f7bc22 100644
>> --- a/drivers/regulator/Kconfig
>> +++ b/drivers/regulator/Kconfig
>> @@ -197,6 +197,14 @@ config REGULATOR_MC13783
>>         Say y here to support the regulators found on the Freescale MC13783
>>         PMIC.
>>
>> +config REGULATOR_MC13892
>> +     tristate "Support regulators on Freescale MC13892 PMIC"
>> +     depends on MFD_MC13892
>> +     select REGULATOR_MC13XXX_CORE
>> +     help
>> +       Say y here to support the regulators found on the Freescale MC13892
>> +       PMIC.
>> +
>>  config REGULATOR_AB3100
>>       tristate "ST-Ericsson AB3100 Regulator functions"
>>       depends on AB3100_CORE
>> diff --git a/drivers/regulator/Makefile b/drivers/regulator/Makefile
>> index 11876be..3107480 100644
>> --- a/drivers/regulator/Makefile
>> +++ b/drivers/regulator/Makefile
>> @@ -30,6 +30,7 @@ obj-$(CONFIG_REGULATOR_DA903X)      += da903x.o
>>  obj-$(CONFIG_REGULATOR_PCF50633) += pcf50633-regulator.o
>>  obj-$(CONFIG_REGULATOR_PCAP) += pcap-regulator.o
>>  obj-$(CONFIG_REGULATOR_MC13783) += mc13783-regulator.o
>> +obj-$(CONFIG_REGULATOR_MC13892) += mc13892-regulator.o
>>  obj-$(CONFIG_REGULATOR_MC13XXX_CORE) +=  mc13xxx-regulator-core.o
>>  obj-$(CONFIG_REGULATOR_AB3100) += ab3100.o
>>
>> diff --git a/drivers/regulator/mc13892-regulator.c b/drivers/regulator/mc13892-regulator.c
>> new file mode 100644
>> index 0000000..6e613d4
>> --- /dev/null
>> +++ b/drivers/regulator/mc13892-regulator.c
>> @@ -0,0 +1,647 @@
>> +/*
>> + * Regulator Driver for Freescale MC13892 PMIC
>> + *
>> + * Copyright 2010 Yong Shen <yong.shen at linaro.org>
>> + *
>> + * Based on draft driver from Arnaud Patard <arnaud.patard at rtp-net.org>
>> + *
>> + * This program is free software; you can redistribute it and/or modify
>> + * it under the terms of the GNU General Public License version 2 as
>> + * published by the Free Software Foundation.
>> + */
>> +
>> +#include <linux/mfd/mc13892.h>
>> +#include <linux/regulator/machine.h>
>> +#include <linux/regulator/driver.h>
>> +#include <linux/platform_device.h>
>> +#include <linux/kernel.h>
>> +#include <linux/slab.h>
>> +#include <linux/init.h>
>> +#include <linux/err.h>
>> +#include "mc13xxx.h"
> shouldn't that be
>
>        #include <linux/mfd/mc13xxx.h>
>
> ?
>> +
>> +#define MC13892_REVISION                             7
>> +
>> +#define MC13892_POWERCTL0                            13
>> +#define MC13892_POWERCTL0_USEROFFSPI         3
>> +#define MC13892_POWERCTL0_VCOINCELLVSEL              20
>> +#define MC13892_POWERCTL0_VCOINCELLVSEL_M            (7<<20)
>> +#define MC13892_POWERCTL0_VCOINCELLEN                (1<<23)
>> +
>> +#define MC13892_SWITCHERS0_SWxHI                     (1<<23)
>> +
>> +#define MC13892_SWITCHERS0                           24
>> +#define MC13892_SWITCHERS0_SW1VSEL                   0
>> +#define MC13892_SWITCHERS0_SW1VSEL_M         (0x1f<<0)
>> +#define MC13892_SWITCHERS0_SW1HI                     (1<<23)
>> +#define MC13892_SWITCHERS0_SW1EN             0
>> +
>> +#define MC13892_SWITCHERS1                           25
>> +#define MC13892_SWITCHERS1_SW2VSEL                   0
>> +#define MC13892_SWITCHERS1_SW2VSEL_M         (0x1f<<0)
>> +#define MC13892_SWITCHERS1_SW2HI                     (1<<23)
>> +#define MC13892_SWITCHERS1_SW2EN             0
>> +
>> +#define MC13892_SWITCHERS2                           26
>> +#define MC13892_SWITCHERS2_SW3VSEL                   0
>> +#define MC13892_SWITCHERS2_SW3VSEL_M         (0x1f<<0)
>> +#define MC13892_SWITCHERS2_SW3HI                     (1<<23)
>> +#define MC13892_SWITCHERS2_SW3EN             0
>> +
>> +#define MC13892_SWITCHERS3                           27
>> +#define MC13892_SWITCHERS3_SW4VSEL                   0
>> +#define MC13892_SWITCHERS3_SW4VSEL_M         (0x1f<<0)
>> +#define MC13892_SWITCHERS3_SW4HI                     (1<<23)
>> +#define MC13892_SWITCHERS3_SW4EN             0
>> +
>> +#define MC13892_SWITCHERS4                           28
>> +#define MC13892_SWITCHERS4_SW1MODE                   0
>> +#define MC13892_SWITCHERS4_SW1MODE_AUTO              (8<<0)
>> +#define MC13892_SWITCHERS4_SW1MODE_M         (0xf<<0)
>> +#define MC13892_SWITCHERS4_SW2MODE                   10
>> +#define MC13892_SWITCHERS4_SW2MODE_AUTO              (8<<10)
>> +#define MC13892_SWITCHERS4_SW2MODE_M         (0xf<<10)
>> +
>> +#define MC13892_SWITCHERS5                           29
>> +#define MC13892_SWITCHERS5_SW3MODE                   0
>> +#define MC13892_SWITCHERS5_SW3MODE_AUTO              (8<<0)
>> +#define MC13892_SWITCHERS5_SW3MODE_M         (0xf<<0)
>> +#define MC13892_SWITCHERS5_SW4MODE                   8
>> +#define MC13892_SWITCHERS5_SW4MODE_AUTO              (8<<8)
>> +#define MC13892_SWITCHERS5_SW4MODE_M         (0xf<<8)
>> +#define MC13892_SWITCHERS5_SWBSTEN                   (1<<20)
>> +
>> +
>> +#define MC13892_REGULATORSETTING0                    30
>> +#define MC13892_REGULATORSETTING0_VGEN1VSEL          0
>> +#define MC13892_REGULATORSETTING0_VDIGVSEL           4
>> +#define MC13892_REGULATORSETTING0_VGEN2VSEL          6
>> +#define MC13892_REGULATORSETTING0_VPLLVSEL           9
>> +#define MC13892_REGULATORSETTING0_VUSB2VSEL          11
>> +#define MC13892_REGULATORSETTING0_VGEN3VSEL          14
>> +#define MC13892_REGULATORSETTING0_VCAMVSEL           16
>> +
>> +#define MC13892_REGULATORSETTING0_VGEN1VSEL_M        (3<<0)
>> +#define MC13892_REGULATORSETTING0_VDIGVSEL_M (3<<4)
>> +#define MC13892_REGULATORSETTING0_VGEN2VSEL_M        (7<<6)
>> +#define MC13892_REGULATORSETTING0_VPLLVSEL_M (3<<9)
>> +#define MC13892_REGULATORSETTING0_VUSB2VSEL_M        (3<<11)
>> +#define MC13892_REGULATORSETTING0_VGEN3VSEL_M        (1<<14)
>> +#define MC13892_REGULATORSETTING0_VCAMVSEL_M (3<<16)
>> +
>> +#define MC13892_REGULATORSETTING1                    31
>> +#define MC13892_REGULATORSETTING1_VVIDEOVSEL 2
>> +#define MC13892_REGULATORSETTING1_VAUDIOVSEL 4
>> +#define MC13892_REGULATORSETTING1_VSDVSEL            6
>> +
>> +#define MC13892_REGULATORSETTING1_VVIDEOVSEL_M       (3<<2)
>> +#define MC13892_REGULATORSETTING1_VAUDIOVSEL_M       (3<<4)
>> +#define MC13892_REGULATORSETTING1_VSDVSEL_M          (7<<6)
>> +
>> +#define MC13892_REGULATORMODE0                       32
>> +#define MC13892_REGULATORMODE0_VGEN1EN               (1<<0)
>> +#define MC13892_REGULATORMODE0_VGEN1STDBY            (1<<1)
>> +#define MC13892_REGULATORMODE0_VGEN1MODE             (1<<2)
>> +#define MC13892_REGULATORMODE0_VIOHIEN               (1<<3)
>> +#define MC13892_REGULATORMODE0_VIOHISTDBY            (1<<4)
>> +#define MC13892_REGULATORMODE0_VIOHIMODE             (1<<5)
>> +#define MC13892_REGULATORMODE0_VDIGEN                (1<<9)
>> +#define MC13892_REGULATORMODE0_VDIGSTDBY             (1<<10)
>> +#define MC13892_REGULATORMODE0_VDIGMODE              (1<<11)
>> +#define MC13892_REGULATORMODE0_VGEN2EN               (1<<12)
>> +#define MC13892_REGULATORMODE0_VGEN2STDBY            (1<<13)
>> +#define MC13892_REGULATORMODE0_VGEN2MODE             (1<<14)
>> +#define MC13892_REGULATORMODE0_VPLLEN                (1<<15)
>> +#define MC13892_REGULATORMODE0_VPLLSTDBY             (1<<16)
>> +#define MC13892_REGULATORMODE0_VPLLMODE              (1<<17)
>> +#define MC13892_REGULATORMODE0_VUSB2EN               (1<<18)
>> +#define MC13892_REGULATORMODE0_VUSB2STDBY            (1<<19)
>> +#define MC13892_REGULATORMODE0_VUSB2MODE             (1<<20)
>> +
>> +#define MC13892_REGULATORMODE1                       33
>> +#define MC13892_REGULATORMODE1_VGEN3EN               (1<<0)
>> +#define MC13892_REGULATORMODE1_VGEN3STDBY            (1<<1)
>> +#define MC13892_REGULATORMODE1_VGEN3MODE             (1<<2)
>> +#define MC13892_REGULATORMODE1_VCAMEN                (1<<6)
>> +#define MC13892_REGULATORMODE1_VCAMSTDBY             (1<<7)
>> +#define MC13892_REGULATORMODE1_VCAMMODE              (1<<8)
>> +#define MC13892_REGULATORMODE1_VCAMCONFIGEN          (1<<9)
>> +#define MC13892_REGULATORMODE1_VVIDEOEN              (1<<12)
>> +#define MC13892_REGULATORMODE1_VVIDEOSTDBY           (1<<13)
>> +#define MC13892_REGULATORMODE1_VVIDEOMODE            (1<<14)
>> +#define MC13892_REGULATORMODE1_VAUDIOEN              (1<<15)
>> +#define MC13892_REGULATORMODE1_VAUDIOSTDBY           (1<<16)
>> +#define MC13892_REGULATORMODE1_VAUDIOMODE            (1<<17)
>> +#define MC13892_REGULATORMODE1_VSDEN         (1<<18)
>> +#define MC13892_REGULATORMODE1_VSDSTDBY              (1<<19)
>> +#define MC13892_REGULATORMODE1_VSDMODE               (1<<20)
>> +
>> +#define MC13892_POWERMISC                            34
>> +#define MC13892_POWERMISC_GPO1EN                     (1<<6)
>> +#define MC13892_POWERMISC_GPO2EN                     (1<<8)
>> +#define MC13892_POWERMISC_GPO3EN                     (1<<10)
>> +#define MC13892_POWERMISC_GPO4EN                     (1<<12)
>> +#define MC13892_POWERMISC_PWGT1SPIEN         (1<<15)
>> +#define MC13892_POWERMISC_PWGT2SPIEN         (1<<16)
>> +#define MC13892_POWERMISC_GPO4ADINEN         (1<<21)
>> +
>> +#define MC13892_POWERMISC_PWGTSPI_M                  (3 << 15)
>> +
>> +#define MC13892_USB1                         50
>> +#define MC13892_USB1_VUSBEN                          (1<<3)
>> +
>> +static const int mc13892_vcoincell[] = {
>> +     2500000, 2700000, 2800000, 2900000, 3000000, 3100000,
>> +     3200000, 3300000,
>> +};
>> +
>> +static const int mc13892_sw1[] = {
>> +     600000,   625000,  650000,  675000,  700000,  725000,
>> +     750000,   775000,  800000,  825000,  850000,  875000,
>> +     900000,   925000,  950000,  975000, 1000000, 1025000,
>> +     1050000, 1075000, 1100000, 1125000, 1150000, 1175000,
>> +     1200000, 1225000, 1250000, 1275000, 1300000, 1325000,
>> +     1350000, 1375000
>> +};
>> +
>> +static const int mc13892_sw[] = {
>> +     600000,   625000,  650000,  675000,  700000,  725000,
>> +     750000,   775000,  800000,  825000,  850000,  875000,
>> +     900000,   925000,  950000,  975000, 1000000, 1025000,
>> +     1050000, 1075000, 1100000, 1125000, 1150000, 1175000,
>> +     1200000, 1225000, 1250000, 1275000, 1300000, 1325000,
>> +     1350000, 1375000, 1400000, 1425000, 1450000, 1475000,
>> +     1500000, 1525000, 1550000, 1575000, 1600000, 1625000,
>> +     1650000, 1675000, 1700000, 1725000, 1750000, 1775000,
>> +     1800000, 1825000, 1850000, 1875000
>> +};
>> +
>> +static const int mc13892_swbst[] = {
>> +     5000000,
>> +};
>> +
>> +static const int mc13892_viohi[] = {
>> +     2775000,
>> +};
>> +
>> +static const int mc13892_vpll[] = {
>> +     1050000, 1250000, 1650000, 1800000,
>> +};
>> +
>> +static const int mc13892_vdig[] = {
>> +     1050000, 1250000, 1650000, 1800000,
>> +};
>> +
>> +static const int mc13892_vsd[] = {
>> +     1800000, 2000000, 2600000, 2700000,
>> +     2800000, 2900000, 3000000, 3150000,
>> +};
>> +
>> +static const int mc13892_vusb2[] = {
>> +     2400000, 2600000, 2700000, 2775000,
>> +};
>> +
>> +static const int mc13892_vvideo[] = {
>> +     2500000, 2600000, 270000, 2775000,
>> +};
>> +
>> +static const int mc13892_vaudio[] = {
>> +     2300000, 2500000, 2775000, 3000000,
>> +};
>> +
>> +static const int mc13892_vcam[] = {
>> +     2500000, 2600000, 2750000, 3000000,
>> +};
>> +
>> +static const int mc13892_vgen1[] = {
>> +     1200000, 1500000, 2775000, 3150000,
>> +};
>> +
>> +static const int mc13892_vgen2[] = {
>> +     1200000, 1500000, 1600000, 1800000,
>> +     2700000, 2800000, 3000000, 3150000,
>> +};
>> +
>> +static const int mc13892_vgen3[] = {
>> +     1800000, 2900000,
>> +};
>> +
>> +static const int mc13892_vusb[] = {
>> +     3300000,
>> +};
>> +
>> +static const int mc13892_gpo[] = {
>> +     2750000,
>> +};
>> +
>> +static const int mc13892_pwgtdrv[] = {
>> +     5000000,
>> +};
>> +
>> +static struct regulator_ops mc13892_gpo_regulator_ops;
>> +/* sw regulators need special care due to the "hi bit" */
>> +static struct regulator_ops mc13892_sw_regulator_ops;
>> +
>> +
>> +#define MC13892_FIXED_DEFINE(name, reg, voltages)\
>> +     MC13xxx_FIXED_DEFINE(MC13892_, name, reg, voltages, \
>> +                     mc13xxx_fixed_regulator_ops)
>> +
>> +#define MC13892_GPO_DEFINE(name, reg, voltages)              \
>> +     MC13xxx_GPO_DEFINE(MC13892_, name, reg, voltages, \
>> +                     mc13892_gpo_regulator_ops)
>> +
>> +#define MC13892_SW_DEFINE(name, reg, vsel_reg, voltages)             \
>> +     MC13xxx_DEFINE(MC13892_, name, reg, vsel_reg, voltages, \
>> +                     mc13892_sw_regulator_ops)
>> +
>> +#define MC13892_DEFINE_REGU(name, reg, vsel_reg, voltages)           \
>> +     MC13xxx_DEFINE(MC13892_, name, reg, vsel_reg, voltages, \
>> +                     mc13xxx_regulator_ops)
>> +
>> +static struct mc13xxx_regulator mc13892_regulators[] = {
>> +     MC13892_DEFINE_REGU(VCOINCELL, POWERCTL0, POWERCTL0,            \
>> +                         mc13892_vcoincell),
>> +     MC13892_SW_DEFINE(SW1, SWITCHERS0, SWITCHERS0, mc13892_sw1),
>> +     MC13892_SW_DEFINE(SW2, SWITCHERS1, SWITCHERS1, mc13892_sw),
>> +     MC13892_SW_DEFINE(SW3, SWITCHERS2, SWITCHERS2, mc13892_sw),
>> +     MC13892_SW_DEFINE(SW4, SWITCHERS3, SWITCHERS3, mc13892_sw),
>> +     MC13892_FIXED_DEFINE(SWBST, SWITCHERS5, mc13892_swbst),
>> +     MC13892_FIXED_DEFINE(VIOHI, REGULATORMODE0, mc13892_viohi),
>> +     MC13892_DEFINE_REGU(VPLL, REGULATORMODE0, REGULATORSETTING0,    \
>> +                         mc13892_vpll),
>> +     MC13892_DEFINE_REGU(VDIG, REGULATORMODE0, REGULATORSETTING0,    \
>> +                         mc13892_vdig),
>> +     MC13892_DEFINE_REGU(VSD, REGULATORMODE1, REGULATORSETTING1,     \
>> +                         mc13892_vsd),
>> +     MC13892_DEFINE_REGU(VUSB2, REGULATORMODE0, REGULATORSETTING0,   \
>> +                         mc13892_vusb2),
>> +     MC13892_DEFINE_REGU(VVIDEO, REGULATORMODE1, REGULATORSETTING1,  \
>> +                         mc13892_vvideo),
>> +     MC13892_DEFINE_REGU(VAUDIO, REGULATORMODE1, REGULATORSETTING1,  \
>> +                         mc13892_vaudio),
>> +     MC13892_DEFINE_REGU(VCAM, REGULATORMODE1, REGULATORSETTING0, \
>> +                         mc13892_vcam),
>> +     MC13892_DEFINE_REGU(VGEN1, REGULATORMODE0, REGULATORSETTING0,   \
>> +                         mc13892_vgen1),
>> +     MC13892_DEFINE_REGU(VGEN2, REGULATORMODE0, REGULATORSETTING0,   \
>> +                         mc13892_vgen2),
>> +     MC13892_DEFINE_REGU(VGEN3, REGULATORMODE1, REGULATORSETTING0,   \
>> +                         mc13892_vgen3),
>> +     MC13892_FIXED_DEFINE(VUSB, USB1, mc13892_vusb),
>> +     MC13892_GPO_DEFINE(GPO1, POWERMISC, mc13892_gpo),
>> +     MC13892_GPO_DEFINE(GPO2, POWERMISC, mc13892_gpo),
>> +     MC13892_GPO_DEFINE(GPO3, POWERMISC, mc13892_gpo),
>> +     MC13892_GPO_DEFINE(GPO4, POWERMISC, mc13892_gpo),
>> +     MC13892_GPO_DEFINE(PWGT1SPI, POWERMISC, mc13892_pwgtdrv),
>> +     MC13892_GPO_DEFINE(PWGT2SPI, POWERMISC, mc13892_pwgtdrv),
>> +};
>> +
>> +int mc13892_powermisc_rmw(struct mc13xxx_regulator_priv *priv, u32 mask,
>> +                                                                     u32 val)
>> +{
>> +     struct mc13xxx *mc13892 = priv->mc13xxx;
>> +     int ret;
>> +     u32 valread;
>> +
>> +     BUG_ON(val & ~mask);
>> +
>> +     ret = mc13xxx_reg_read(mc13892, MC13892_POWERMISC, &valread);
>> +     if (ret)
>> +             return ret;
>> +
>> +     /* Update the stored state for Power Gates. */
>> +     priv->powermisc_pwgt_state =
>> +                             (priv->powermisc_pwgt_state & ~mask) | val;
>> +     priv->powermisc_pwgt_state &= MC13892_POWERMISC_PWGTSPI_M;
>> +
>> +     /* Construct the new register value */
>> +     valread = (valread & ~mask) | val;
>> +     /* Overwrite the PWGTxEN with the stored version */
>> +     valread = (valread & ~MC13892_POWERMISC_PWGTSPI_M) |
>> +                                             priv->powermisc_pwgt_state;
>> +
>> +     return mc13xxx_reg_write(mc13892, MC13892_POWERMISC, valread);
>> +}
>> +
>> +static int mc13892_gpo_regulator_enable(struct regulator_dev *rdev)
>> +{
>> +     struct mc13xxx_regulator_priv *priv = rdev_get_drvdata(rdev);
>> +     int id = rdev_get_id(rdev);
>> +     int ret;
>> +     u32 en_val = mc13892_regulators[id].enable_bit;
>> +     u32 mask = mc13892_regulators[id].enable_bit;
>> +
>> +     dev_dbg(rdev_get_dev(rdev), "%s id: %d\n", __func__, id);
>> +
>> +     /* Power Gate enable value is 0 */
>> +     if (id == MC13892_PWGT1SPI ||
>> +         id == MC13892_PWGT2SPI)
>> +             en_val = 0;
>> +
>> +     if (id == MC13892_GPO4)
>> +             mask |= MC13892_POWERMISC_GPO4ADINEN;
>> +
>> +     mc13xxx_lock(priv->mc13xxx);
>> +     ret = mc13892_powermisc_rmw(priv, mask, en_val);
>> +     mc13xxx_unlock(priv->mc13xxx);
>> +
>> +     return ret;
>> +}
>> +
>> +static int mc13892_gpo_regulator_disable(struct regulator_dev *rdev)
>> +{
>> +     struct mc13xxx_regulator_priv *priv = rdev_get_drvdata(rdev);
>> +     int id = rdev_get_id(rdev);
>> +     int ret;
>> +     u32 dis_val = 0;
>> +
>> +     dev_dbg(rdev_get_dev(rdev), "%s id: %d\n", __func__, id);
>> +
>> +     /* Power Gate disable value is 1 */
>> +     if (id == MC13892_PWGT1SPI ||
>> +         id == MC13892_PWGT2SPI)
>> +             dis_val = mc13892_regulators[id].enable_bit;
>> +
>> +     mc13xxx_lock(priv->mc13xxx);
>> +     ret = mc13892_powermisc_rmw(priv, mc13892_regulators[id].enable_bit,
>> +                                     dis_val);
>> +     mc13xxx_unlock(priv->mc13xxx);
>> +
>> +     return ret;
>> +}
>> +
>> +static int mc13892_gpo_regulator_is_enabled(struct regulator_dev *rdev)
>> +{
>> +     struct mc13xxx_regulator_priv *priv = rdev_get_drvdata(rdev);
>> +     int ret, id = rdev_get_id(rdev);
>> +     unsigned int val;
>> +
>> +     mc13xxx_lock(priv->mc13xxx);
>> +     ret = mc13xxx_reg_read(priv->mc13xxx, mc13892_regulators[id].reg, &val);
>> +     mc13xxx_unlock(priv->mc13xxx);
>> +
>> +     if (ret)
>> +             return ret;
>> +
>> +     /* Power Gates state is stored in powermisc_pwgt_state
>> +      * where the meaning of bits is negated */
>> +     val = (val & ~MC13892_POWERMISC_PWGTSPI_M) |
>> +           (priv->powermisc_pwgt_state ^ MC13892_POWERMISC_PWGTSPI_M);
>> +
>> +     return (val & mc13892_regulators[id].enable_bit) != 0;
>> +}
>> +
>> +
>> +static struct regulator_ops mc13892_gpo_regulator_ops = {
>> +     .enable = mc13892_gpo_regulator_enable,
>> +     .disable = mc13892_gpo_regulator_disable,
>> +     .is_enabled = mc13892_gpo_regulator_is_enabled,
>> +     .list_voltage = mc13xxx_regulator_list_voltage,
>> +     .set_voltage = mc13xxx_fixed_regulator_set_voltage,
>> +     .get_voltage = mc13xxx_fixed_regulator_get_voltage,
>> +};
>> +
>> +static int mc13892_sw_regulator_get_voltage(struct regulator_dev *rdev)
>> +{
>> +     struct mc13xxx_regulator_priv *priv = rdev_get_drvdata(rdev);
>> +     int ret, id = rdev_get_id(rdev);
>> +     unsigned int val, hi;
>> +
>> +     dev_dbg(rdev_get_dev(rdev), "%s id: %d\n", __func__, id);
>> +
>> +     mc13xxx_lock(priv->mc13xxx);
>> +     ret = mc13xxx_reg_read(priv->mc13xxx,
>> +                             mc13892_regulators[id].vsel_reg, &val);
>> +     mc13xxx_unlock(priv->mc13xxx);
>> +     if (ret)
>> +             return ret;
>> +
>> +     hi  = val & MC13892_SWITCHERS0_SWxHI;
>> +     val = (val & mc13892_regulators[id].vsel_mask)
>> +             >> mc13892_regulators[id].vsel_shift;
>> +
>> +     dev_dbg(rdev_get_dev(rdev), "%s id: %d val: %d\n", __func__, id, val);
>> +
>> +     if (hi)
>> +             val = (25000 * val) + 1100000;
>> +     else
>> +             val = (25000 * val) + 600000;
>> +
>> +     return val;
>> +}
>> +
>> +static int mc13892_sw_regulator_set_voltage(struct regulator_dev *rdev,
>> +                                             int min_uV, int max_uV)
>> +{
>> +     struct mc13xxx_regulator_priv *priv = rdev_get_drvdata(rdev);
>> +     int hi, value, val, mask, id = rdev_get_id(rdev);
>> +     int ret;
>> +
>> +     dev_dbg(rdev_get_dev(rdev), "%s id: %d min_uV: %d max_uV: %d\n",
>> +             __func__, id, min_uV, max_uV);
>> +
>> +     /* Find the best index */
>> +     value = mc13xxx_get_best_voltage_index(rdev, min_uV, max_uV);
>> +     dev_dbg(rdev_get_dev(rdev), "%s best value: %d\n", __func__, value);
>> +     if (value < 0)
>> +             return value;
>> +
>> +     value = mc13892_regulators[id].voltages[value];
>> +
>> +     mc13xxx_lock(priv->mc13xxx);
>> +     ret = mc13xxx_reg_read(priv->mc13xxx,
>> +                             mc13892_regulators[id].vsel_reg, &val);
>> +     mc13xxx_unlock(priv->mc13xxx);
>> +     if (ret)
>> +             return ret;
>> +
>> +     hi  = val & MC13892_SWITCHERS0_SWxHI;
>> +     if (value > 1375)
>> +             hi = 1;
>> +     if (value < 1100)
>> +             hi = 0;
>> +
>> +     if (hi) {
>> +             value = (value - 1100000) / 25000;
>> +             value |= MC13892_SWITCHERS0_SWxHI;
>> +     } else
>> +             value = (value - 600000) / 25000;
>> +
>> +     mask = mc13892_regulators[id].vsel_mask | MC13892_SWITCHERS0_SWxHI;
>> +     mc13xxx_lock(priv->mc13xxx);
>> +     ret = mc13xxx_reg_rmw(priv->mc13xxx, mc13892_regulators[id].vsel_reg,
>> +                     mask,
>> +                     value << mc13892_regulators[id].vsel_shift);
>> +     mc13xxx_unlock(priv->mc13xxx);
>> +
>> +     return ret;
>> +}
>> +
>> +static struct regulator_ops mc13892_sw_regulator_ops = {
>> +     .is_enabled = mc13xxx_sw_regulator_is_enabled,
>> +     .list_voltage = mc13xxx_regulator_list_voltage,
>> +     .set_voltage = mc13892_sw_regulator_set_voltage,
>> +     .get_voltage = mc13892_sw_regulator_get_voltage,
>> +};
>> +
>> +static int mc13892_vcam_set_mode(struct regulator_dev *rdev, unsigned int mode)
>> +{
>> +     unsigned int en_val = 0;
>> +     struct mc13xxx_regulator_priv *priv = rdev_get_drvdata(rdev);
>> +     int ret, id = rdev_get_id(rdev);
>> +
>> +     if (mode == REGULATOR_MODE_FAST)
>> +             en_val = MC13892_REGULATORMODE1_VCAMCONFIGEN;
>> +
>> +     mc13xxx_lock(priv->mc13xxx);
>> +     ret = mc13xxx_reg_rmw(priv->mc13xxx, mc13892_regulators[id].reg,
>> +                     MC13892_REGULATORMODE1_VCAMCONFIGEN,
>> +                     en_val);
>> +     mc13xxx_unlock(priv->mc13xxx);
>> +
>> +     return ret;
>> +}
>> +
>> +unsigned int mc13892_vcam_get_mode(struct regulator_dev *rdev)
>> +{
>> +     struct mc13xxx_regulator_priv *priv = rdev_get_drvdata(rdev);
>> +     int ret, id = rdev_get_id(rdev);
>> +     unsigned int val;
>> +
>> +     mc13xxx_lock(priv->mc13xxx);
>> +     ret = mc13xxx_reg_read(priv->mc13xxx, mc13892_regulators[id].reg, &val);
>> +     mc13xxx_unlock(priv->mc13xxx);
>> +
>> +     if (ret)
>> +             return ret;
>> +
>> +     if (val & MC13892_REGULATORMODE1_VCAMCONFIGEN)
>> +             return REGULATOR_MODE_FAST;
>> +
>> +     return REGULATOR_MODE_NORMAL;
>> +}
>> +
>> +
>> +static int __devinit mc13892_regulator_probe(struct platform_device *pdev)
>> +{
>> +     struct mc13xxx_regulator_priv *priv;
>> +     struct mc13xxx *mc13892 = dev_get_drvdata(pdev->dev.parent);
>> +     struct mc13xxx_regulator_platform_data *pdata =
>> +             dev_get_platdata(&pdev->dev);
>> +     struct mc13xxx_regulator_init_data *init_data;
>> +     int i, ret;
>> +     u32 val;
>> +
>> +     priv = kzalloc(sizeof(*priv) +
>> +                     pdata->num_regulators * sizeof(priv->regulators[0]),
>> +                     GFP_KERNEL);
>> +     if (!priv)
>> +             return -ENOMEM;
>> +
>> +     priv->mc13xxx_regulators = mc13892_regulators;
>> +     priv->mc13xxx = mc13892;
>> +
>> +     mc13xxx_lock(mc13892);
>> +     ret = mc13xxx_reg_read(mc13892, MC13892_REVISION, &val);
>> +     mc13xxx_unlock(mc13892);
>> +     if (ret)
>> +             goto err_alloc;
> err_alloc looks wrong
>
>> +
>> +     /* enable switch auto mode - ENGR00120510, ENGR00121057 */
>> +     /* no doc with regs description means no way to know if */
>> +     /* there's a better way to do it                        */
> This is an unusual commenting style.  Please use
True. In fact, I am going to remove ENGR*** stuff, cause they are only
meaningful inside freescale.
>
>        /*
>         * enable switch auto mode ...
>         */
>
>> +     if ((val & 0x0000FFFF) == 0x45d0) {
>> +             mc13xxx_lock(mc13892);
>> +             ret = mc13xxx_reg_rmw(mc13892, MC13892_SWITCHERS4,
>> +                     MC13892_SWITCHERS4_SW1MODE_M |
>> +                     MC13892_SWITCHERS4_SW2MODE_M,
>> +                     MC13892_SWITCHERS4_SW1MODE_AUTO |
>> +                     MC13892_SWITCHERS4_SW2MODE_AUTO);
>> +             mc13xxx_unlock(mc13892);
> It should be possible to hold the lock for longer that a single mc13xxx
> API call.  Your error handler just needs to unlock for that.
Agreed.
>
>> +             if (ret)
>> +                     goto err_alloc;
>> +
>> +             mc13xxx_lock(mc13892);
>> +             mc13xxx_reg_rmw(mc13892, MC13892_SWITCHERS5,
>> +                     MC13892_SWITCHERS5_SW3MODE_M |
>> +                     MC13892_SWITCHERS5_SW4MODE_M,
>> +                     MC13892_SWITCHERS5_SW3MODE_AUTO |
>> +                     MC13892_SWITCHERS5_SW4MODE_AUTO);
>> +             mc13xxx_unlock(mc13892);
>> +             if (ret)
>> +                     goto err_alloc;
>> +     }
>> +
>> +     mc13892_regulators[MC13892_VCAM].desc.ops->set_mode
>> +             = mc13892_vcam_set_mode;
>> +     mc13892_regulators[MC13892_VCAM].desc.ops->get_mode
>> +             = mc13892_vcam_get_mode;
>> +     for (i = 0; i < pdata->num_regulators; i++) {
>> +             init_data = &pdata->regulators[i];
>> +             priv->regulators[i] = regulator_register(
>> +                             &mc13892_regulators[init_data->id].desc,
>> +                             &pdev->dev, init_data->init_data, priv);
>> +
>> +             if (IS_ERR(priv->regulators[i])) {
>> +                     dev_err(&pdev->dev, "failed to register regulator %s\n",
>> +                             mc13892_regulators[i].desc.name);
>> +                     ret = PTR_ERR(priv->regulators[i]);
>> +                     goto err;
>> +             }
>> +     }
>> +
>> +     platform_set_drvdata(pdev, priv);
>> +
>> +     return 0;
>> +err:
>> +     while (--i >= 0)
>> +             regulator_unregister(priv->regulators[i]);
>> +
>> +err_alloc:
>> +     kfree(priv);
>> +
>> +     return ret;
>> +}
>> +
>> +static int __devexit mc13892_regulator_remove(struct platform_device *pdev)
>> +{
>> +     struct mc13xxx_regulator_priv *priv = platform_get_drvdata(pdev);
>> +     struct mc13xxx_regulator_platform_data *pdata =
>> +             dev_get_platdata(&pdev->dev);
>> +     int i;
>> +
>> +     platform_set_drvdata(pdev, NULL);
>> +
>> +     for (i = 0; i < pdata->num_regulators; i++)
>> +             regulator_unregister(priv->regulators[i]);
>> +
>> +     kfree(priv);
>> +     return 0;
>> +}
>> +
>> +static struct platform_driver mc13892_regulator_driver = {
>> +     .driver = {
>> +             .name   = "mc13892-regulator",
>> +             .owner  = THIS_MODULE,
>> +     },
>> +     .remove         = __devexit_p(mc13892_regulator_remove),
>> +     .probe          = mc13892_regulator_probe,
> If you ask me just use a single space before =, but there is no common
> style visible in the kernel, so do as you prefer.
> (The advantage of using a single space is that if later
> .averylongmembername needs to be initialized you either have to fix
> lines that are technically unrelated or the indention isn't uniform.)
Good suggestion. What I do is use a tab in front of '=', for better looking :).
>
> Best regards
> Uwe
>
>> +};
>> +
>> +static int __init mc13892_regulator_init(void)
>> +{
>> +     return platform_driver_register(&mc13892_regulator_driver);
>> +}
>> +subsys_initcall(mc13892_regulator_init);
>> +
>> +static void __exit mc13892_regulator_exit(void)
>> +{
>> +     platform_driver_unregister(&mc13892_regulator_driver);
>> +}
>> +module_exit(mc13892_regulator_exit);
>> +
>> +MODULE_LICENSE("GPL v2");
>> +MODULE_AUTHOR("Yong Shen <yong.shen at linaro.org>");
>> +MODULE_DESCRIPTION("Regulator Driver for Freescale MC13892 PMIC");
>> +MODULE_ALIAS("platform:mc13892-regulator");
>> diff --git a/include/linux/mfd/mc13892.h b/include/linux/mfd/mc13892.h
>> new file mode 100644
>> index 0000000..7df13e8
>> --- /dev/null
>> +++ b/include/linux/mfd/mc13892.h
>> @@ -0,0 +1,38 @@
>> +/*
>> + * Copyright 2010 Yong Shen <yong.shen at linaro.org>
>> + *
>> + * This program is free software; you can redistribute it and/or modify it under
>> + * the terms of the GNU General Public License version 2 as published by the
>> + * Free Software Foundation.
>> + */
>> +
>> +#ifndef __LINUX_MFD_MC13892_H
>> +#define __LINUX_MFD_MC13892_H
>> +#include <linux/mfd/mc13xxx.h>
>> +
>> +#define MC13892_SW1          0
>> +#define MC13892_SW2          1
>> +#define MC13892_SW3          2
>> +#define MC13892_SW4          3
>> +#define MC13892_SWBST        4
>> +#define MC13892_VIOHI        5
>> +#define MC13892_VPLL 6
>> +#define MC13892_VDIG 7
>> +#define MC13892_VSD  8
>> +#define MC13892_VUSB2        9
>> +#define MC13892_VVIDEO       10
>> +#define MC13892_VAUDIO       11
>> +#define MC13892_VCAM 12
>> +#define MC13892_VGEN1        13
>> +#define MC13892_VGEN2        14
>> +#define MC13892_VGEN3        15
>> +#define MC13892_VUSB 16
>> +#define MC13892_GPO1 17
>> +#define MC13892_GPO2 18
>> +#define MC13892_GPO3 19
>> +#define MC13892_GPO4 20
>> +#define MC13892_PWGT1SPI     21
>> +#define MC13892_PWGT2SPI     22
>> +#define MC13892_VCOINCELL    23
>> +
>> +#endif
>> --
>> 1.7.0.4
>
> --
> Pengutronix e.K.                           | Uwe Kleine-König            |
> Industrial Linux Solutions                 | http://www.pengutronix.de/  |
>
Thanks
Yong



More information about the linux-arm-kernel mailing list