Regulator: LP3972 PMIC regulator driver
[linux-2.6.git] / drivers / regulator / lp3972.c
1 /*
2  * Regulator driver for National Semiconductors LP3972 PMIC chip
3  *
4  * Based on lp3971.c
5  *
6  * This program is free software; you can redistribute it and/or modify
7  * it under the terms of the GNU General Public License version 2 as
8  * published by the Free Software Foundation.
9  *
10  */
11
12 #include <linux/bug.h>
13 #include <linux/err.h>
14 #include <linux/i2c.h>
15 #include <linux/kernel.h>
16 #include <linux/regulator/driver.h>
17 #include <linux/regulator/lp3972.h>
18 #include <linux/slab.h>
19
20 struct lp3972 {
21         struct device *dev;
22         struct mutex io_lock;
23         struct i2c_client *i2c;
24         int num_regulators;
25         struct regulator_dev **rdev;
26 };
27
28 /* LP3972 Control Registers */
29 #define LP3972_SCR_REG          0x07
30 #define LP3972_OVER1_REG        0x10
31 #define LP3972_OVSR1_REG        0x11
32 #define LP3972_OVER2_REG        0x12
33 #define LP3972_OVSR2_REG        0x13
34 #define LP3972_VCC1_REG         0x20
35 #define LP3972_ADTV1_REG        0x23
36 #define LP3972_ADTV2_REG        0x24
37 #define LP3972_AVRC_REG         0x25
38 #define LP3972_CDTC1_REG        0x26
39 #define LP3972_CDTC2_REG        0x27
40 #define LP3972_SDTV1_REG        0x29
41 #define LP3972_SDTV2_REG        0x2A
42 #define LP3972_MDTV1_REG        0x32
43 #define LP3972_MDTV2_REG        0x33
44 #define LP3972_L2VCR_REG        0x39
45 #define LP3972_L34VCR_REG       0x3A
46 #define LP3972_SCR1_REG         0x80
47 #define LP3972_SCR2_REG         0x81
48 #define LP3972_OEN3_REG         0x82
49 #define LP3972_OSR3_REG         0x83
50 #define LP3972_LOER4_REG        0x84
51 #define LP3972_B2TV_REG         0x85
52 #define LP3972_B3TV_REG         0x86
53 #define LP3972_B32RC_REG        0x87
54 #define LP3972_ISRA_REG         0x88
55 #define LP3972_BCCR_REG         0x89
56 #define LP3972_II1RR_REG        0x8E
57 #define LP3972_II2RR_REG        0x8F
58
59 #define LP3972_SYS_CONTROL1_REG         LP3972_SCR1_REG
60 /* System control register 1 initial value,
61  * bits 5, 6 and 7 are EPROM programmable */
62 #define SYS_CONTROL1_INIT_VAL           0x02
63 #define SYS_CONTROL1_INIT_MASK          0x1F
64
65 #define LP3972_VOL_CHANGE_REG           LP3972_VCC1_REG
66 #define LP3972_VOL_CHANGE_FLAG_GO       0x01
67 #define LP3972_VOL_CHANGE_FLAG_MASK     0x03
68
69 /* LDO output enable mask */
70 #define LP3972_OEN3_L1EN        BIT(0)
71 #define LP3972_OVER2_LDO2_EN    BIT(2)
72 #define LP3972_OVER2_LDO3_EN    BIT(3)
73 #define LP3972_OVER2_LDO4_EN    BIT(4)
74 #define LP3972_OVER1_S_EN       BIT(2)
75
76 static const int ldo1_voltage_map[] = {
77         1700, 1725, 1750, 1775, 1800, 1825, 1850, 1875,
78         1900, 1925, 1950, 1975, 2000,
79 };
80
81 static const int ldo23_voltage_map[] = {
82         1800, 1900, 2000, 2100, 2200, 2300, 2400, 2500,
83         2600, 2700, 2800, 2900, 3000, 3100, 3200, 3300,
84 };
85
86 static const int ldo4_voltage_map[] = {
87         1000, 1050, 1100, 1150, 1200, 1250, 1300, 1350,
88         1400, 1500, 1800, 1900, 2500, 2800, 3000, 3300,
89 };
90
91 static const int ldo5_voltage_map[] = {
92            0,    0,    0,    0,    0,  850,  875,  900,
93          925,  950,  975, 1000, 1025, 1050, 1075, 1100,
94         1125, 1150, 1175, 1200, 1225, 1250, 1275, 1300,
95         1325, 1350, 1375, 1400, 1425, 1450, 1475, 1500,
96 };
97
98 static const int buck1_voltage_map[] = {
99          725,  750,  775,  800,  825,  850,  875,  900,
100          925,  950,  975, 1000, 1025, 1050, 1075, 1100,
101         1125, 1150, 1175, 1200, 1225, 1250, 1275, 1300,
102         1325, 1350, 1375, 1400, 1425, 1450, 1475, 1500,
103 };
104
105 static const int buck23_voltage_map[] = {
106            0,  800,  850,  900,  950, 1000, 1050, 1100,
107         1150, 1200, 1250, 1300, 1350, 1400, 1450, 1500,
108         1550, 1600, 1650, 1700, 1800, 1900, 2500, 2800,
109         3000, 3300,
110 };
111
112 static const int *ldo_voltage_map[] = {
113         ldo1_voltage_map,
114         ldo23_voltage_map,
115         ldo23_voltage_map,
116         ldo4_voltage_map,
117         ldo5_voltage_map,
118 };
119
120 static const int *buck_voltage_map[] = {
121         buck1_voltage_map,
122         buck23_voltage_map,
123         buck23_voltage_map,
124 };
125
126 static const int ldo_output_enable_mask[] = {
127         LP3972_OEN3_L1EN,
128         LP3972_OVER2_LDO2_EN,
129         LP3972_OVER2_LDO3_EN,
130         LP3972_OVER2_LDO4_EN,
131         LP3972_OVER1_S_EN,
132 };
133
134 static const int ldo_output_enable_addr[] = {
135         LP3972_OEN3_REG,
136         LP3972_OVER2_REG,
137         LP3972_OVER2_REG,
138         LP3972_OVER2_REG,
139         LP3972_OVER1_REG,
140 };
141
142 static const int ldo_vol_ctl_addr[] = {
143         LP3972_MDTV1_REG,
144         LP3972_L2VCR_REG,
145         LP3972_L34VCR_REG,
146         LP3972_L34VCR_REG,
147         LP3972_SDTV1_REG,
148 };
149
150 static const int buck_vol_enable_addr[] = {
151         LP3972_OVER1_REG,
152         LP3972_OEN3_REG,
153         LP3972_OEN3_REG,
154 };
155
156 static const int buck_base_addr[] = {
157         LP3972_ADTV1_REG,
158         LP3972_B2TV_REG,
159         LP3972_B3TV_REG,
160 };
161
162 #define LP3972_LDO_VOL_VALUE_MAP(x) (ldo_voltage_map[x])
163 #define LP3972_LDO_OUTPUT_ENABLE_MASK(x) (ldo_output_enable_mask[x])
164 #define LP3972_LDO_OUTPUT_ENABLE_REG(x) (ldo_output_enable_addr[x])
165
166 /*      LDO voltage control registers shift:
167         LP3972_LDO1 -> 0, LP3972_LDO2 -> 4
168         LP3972_LDO3 -> 0, LP3972_LDO4 -> 4
169         LP3972_LDO5 -> 0
170 */
171 #define LP3972_LDO_VOL_CONTR_SHIFT(x) (((x) & 1) << 2)
172 #define LP3972_LDO_VOL_CONTR_REG(x) (ldo_vol_ctl_addr[x])
173 #define LP3972_LDO_VOL_CHANGE_SHIFT(x) ((x) ? 4 : 6)
174
175 #define LP3972_LDO_VOL_MASK(x) (((x) % 4) ? 0x0f : 0x1f)
176 #define LP3972_LDO_VOL_MIN_IDX(x) (((x) == 4) ? 0x05 : 0x00)
177 #define LP3972_LDO_VOL_MAX_IDX(x) ((x) ? (((x) == 4) ? 0x1f : 0x0f) : 0x0c)
178
179 #define LP3972_BUCK_VOL_VALUE_MAP(x) (buck_voltage_map[x])
180 #define LP3972_BUCK_VOL_ENABLE_REG(x) (buck_vol_enable_addr[x])
181 #define LP3972_BUCK_VOL1_REG(x) (buck_base_addr[x])
182 #define LP3972_BUCK_VOL_MASK 0x1f
183 #define LP3972_BUCK_VOL_MIN_IDX(x) ((x) ? 0x01 : 0x00)
184 #define LP3972_BUCK_VOL_MAX_IDX(x) ((x) ? 0x19 : 0x1f)
185
186 static int lp3972_i2c_read(struct i2c_client *i2c, char reg, int count,
187         u16 *dest)
188 {
189         int ret;
190
191         if (count != 1)
192                 return -EIO;
193         ret = i2c_smbus_read_byte_data(i2c, reg);
194         if (ret < 0)
195                 return -EIO;
196
197         *dest = ret;
198         return 0;
199 }
200
201 static int lp3972_i2c_write(struct i2c_client *i2c, char reg, int count,
202         const u16 *src)
203 {
204         if (count != 1)
205                 return -EIO;
206         return i2c_smbus_write_byte_data(i2c, reg, *src);
207 }
208
209 static u8 lp3972_reg_read(struct lp3972 *lp3972, u8 reg)
210 {
211         u16 val = 0;
212
213         mutex_lock(&lp3972->io_lock);
214
215         lp3972_i2c_read(lp3972->i2c, reg, 1, &val);
216
217         dev_dbg(lp3972->dev, "reg read 0x%02x -> 0x%02x\n", (int)reg,
218                 (unsigned)val&0xff);
219
220         mutex_unlock(&lp3972->io_lock);
221
222         return val & 0xff;
223 }
224
225 static int lp3972_set_bits(struct lp3972 *lp3972, u8 reg, u16 mask, u16 val)
226 {
227         u16 tmp;
228         int ret;
229
230         mutex_lock(&lp3972->io_lock);
231
232         ret = lp3972_i2c_read(lp3972->i2c, reg, 1, &tmp);
233         tmp = (tmp & ~mask) | val;
234         if (ret == 0) {
235                 ret = lp3972_i2c_write(lp3972->i2c, reg, 1, &tmp);
236                 dev_dbg(lp3972->dev, "reg write 0x%02x -> 0x%02x\n", (int)reg,
237                         (unsigned)val&0xff);
238         }
239         mutex_unlock(&lp3972->io_lock);
240
241         return ret;
242 }
243
244 static int lp3972_ldo_list_voltage(struct regulator_dev *dev, unsigned index)
245 {
246         int ldo = rdev_get_id(dev) - LP3972_LDO1;
247         return 1000 * LP3972_LDO_VOL_VALUE_MAP(ldo)[index];
248 }
249
250 static int lp3972_ldo_is_enabled(struct regulator_dev *dev)
251 {
252         struct lp3972 *lp3972 = rdev_get_drvdata(dev);
253         int ldo = rdev_get_id(dev) - LP3972_LDO1;
254         u16 mask = LP3972_LDO_OUTPUT_ENABLE_MASK(ldo);
255         u16 val;
256
257         val = lp3972_reg_read(lp3972, LP3972_LDO_OUTPUT_ENABLE_REG(ldo));
258         return !!(val & mask);
259 }
260
261 static int lp3972_ldo_enable(struct regulator_dev *dev)
262 {
263         struct lp3972 *lp3972 = rdev_get_drvdata(dev);
264         int ldo = rdev_get_id(dev) - LP3972_LDO1;
265         u16 mask = LP3972_LDO_OUTPUT_ENABLE_MASK(ldo);
266
267         return lp3972_set_bits(lp3972, LP3972_LDO_OUTPUT_ENABLE_REG(ldo),
268                                 mask, mask);
269 }
270
271 static int lp3972_ldo_disable(struct regulator_dev *dev)
272 {
273         struct lp3972 *lp3972 = rdev_get_drvdata(dev);
274         int ldo = rdev_get_id(dev) - LP3972_LDO1;
275         u16 mask = LP3972_LDO_OUTPUT_ENABLE_MASK(ldo);
276
277         return lp3972_set_bits(lp3972, LP3972_LDO_OUTPUT_ENABLE_REG(ldo),
278                                 mask, 0);
279 }
280
281 static int lp3972_ldo_get_voltage(struct regulator_dev *dev)
282 {
283         struct lp3972 *lp3972 = rdev_get_drvdata(dev);
284         int ldo = rdev_get_id(dev) - LP3972_LDO1;
285         u16 mask = LP3972_LDO_VOL_MASK(ldo);
286         u16 val, reg;
287
288         reg = lp3972_reg_read(lp3972, LP3972_LDO_VOL_CONTR_REG(ldo));
289         val = (reg >> LP3972_LDO_VOL_CONTR_SHIFT(ldo)) & mask;
290
291         return 1000 * LP3972_LDO_VOL_VALUE_MAP(ldo)[val];
292 }
293
294 static int lp3972_ldo_set_voltage(struct regulator_dev *dev,
295                                   int min_uV, int max_uV)
296 {
297         struct lp3972 *lp3972 = rdev_get_drvdata(dev);
298         int ldo = rdev_get_id(dev) - LP3972_LDO1;
299         int min_vol = min_uV / 1000, max_vol = max_uV / 1000;
300         const int *vol_map = LP3972_LDO_VOL_VALUE_MAP(ldo);
301         u16 val;
302         int shift, ret;
303
304         if (min_vol < vol_map[LP3972_LDO_VOL_MIN_IDX(ldo)] ||
305             min_vol > vol_map[LP3972_LDO_VOL_MAX_IDX(ldo)])
306                 return -EINVAL;
307
308         for (val = LP3972_LDO_VOL_MIN_IDX(ldo);
309                 val <= LP3972_LDO_VOL_MAX_IDX(ldo); val++)
310                 if (vol_map[val] >= min_vol)
311                         break;
312
313         if (val > LP3972_LDO_VOL_MAX_IDX(ldo) || vol_map[val] > max_vol)
314                 return -EINVAL;
315
316         shift = LP3972_LDO_VOL_CONTR_SHIFT(ldo);
317         ret = lp3972_set_bits(lp3972, LP3972_LDO_VOL_CONTR_REG(ldo),
318                 LP3972_LDO_VOL_MASK(ldo) << shift, val << shift);
319
320         if (ret)
321                 return ret;
322
323         switch (ldo) {
324         case LP3972_LDO1:
325         case LP3972_LDO5:
326                 shift = LP3972_LDO_VOL_CHANGE_SHIFT(ldo);
327                 ret = lp3972_set_bits(lp3972, LP3972_VOL_CHANGE_REG,
328                         LP3972_VOL_CHANGE_FLAG_MASK << shift,
329                         LP3972_VOL_CHANGE_FLAG_GO << shift);
330                 if (ret)
331                         return ret;
332
333                 ret = lp3972_set_bits(lp3972, LP3972_VOL_CHANGE_REG,
334                         LP3972_VOL_CHANGE_FLAG_MASK << shift, 0);
335                 break;
336         }
337
338         return ret;
339 }
340
341 static struct regulator_ops lp3972_ldo_ops = {
342         .list_voltage = lp3972_ldo_list_voltage,
343         .is_enabled = lp3972_ldo_is_enabled,
344         .enable = lp3972_ldo_enable,
345         .disable = lp3972_ldo_disable,
346         .get_voltage = lp3972_ldo_get_voltage,
347         .set_voltage = lp3972_ldo_set_voltage,
348 };
349
350 static int lp3972_dcdc_list_voltage(struct regulator_dev *dev, unsigned index)
351 {
352         int buck = rdev_get_id(dev) - LP3972_DCDC1;
353         return 1000 * buck_voltage_map[buck][index];
354 }
355
356 static int lp3972_dcdc_is_enabled(struct regulator_dev *dev)
357 {
358         struct lp3972 *lp3972 = rdev_get_drvdata(dev);
359         int buck = rdev_get_id(dev) - LP3972_DCDC1;
360         u16 mask = 1 << (buck * 2);
361         u16 val;
362
363         val = lp3972_reg_read(lp3972, LP3972_BUCK_VOL_ENABLE_REG(buck));
364         return !!(val & mask);
365 }
366
367 static int lp3972_dcdc_enable(struct regulator_dev *dev)
368 {
369         struct lp3972 *lp3972 = rdev_get_drvdata(dev);
370         int buck = rdev_get_id(dev) - LP3972_DCDC1;
371         u16 mask = 1 << (buck * 2);
372         u16 val;
373
374         val = lp3972_set_bits(lp3972, LP3972_BUCK_VOL_ENABLE_REG(buck),
375                                 mask, mask);
376         return val;
377 }
378
379 static int lp3972_dcdc_disable(struct regulator_dev *dev)
380 {
381         struct lp3972 *lp3972 = rdev_get_drvdata(dev);
382         int buck = rdev_get_id(dev) - LP3972_DCDC1;
383         u16 mask = 1 << (buck * 2);
384         u16 val;
385
386         val = lp3972_set_bits(lp3972, LP3972_BUCK_VOL_ENABLE_REG(buck),
387                                 mask, 0);
388         return val;
389 }
390
391 static int lp3972_dcdc_get_voltage(struct regulator_dev *dev)
392 {
393         struct lp3972 *lp3972 = rdev_get_drvdata(dev);
394         int buck = rdev_get_id(dev) - LP3972_DCDC1;
395         u16 reg;
396         int val;
397
398         reg = lp3972_reg_read(lp3972, LP3972_BUCK_VOL1_REG(buck));
399         reg &= LP3972_BUCK_VOL_MASK;
400         if (reg <= LP3972_BUCK_VOL_MAX_IDX(buck))
401                 val = 1000 * buck_voltage_map[buck][reg];
402         else {
403                 val = 0;
404                 dev_warn(&dev->dev, "chip reported incorrect voltage value.\n");
405         }
406
407         return val;
408 }
409
410 static int lp3972_dcdc_set_voltage(struct regulator_dev *dev,
411                                   int min_uV, int max_uV)
412 {
413         struct lp3972 *lp3972 = rdev_get_drvdata(dev);
414         int buck = rdev_get_id(dev) - LP3972_DCDC1;
415         int min_vol = min_uV / 1000, max_vol = max_uV / 1000;
416         const int *vol_map = buck_voltage_map[buck];
417         u16 val;
418         int ret;
419
420         if (min_vol < vol_map[LP3972_BUCK_VOL_MIN_IDX(buck)] ||
421             min_vol > vol_map[LP3972_BUCK_VOL_MAX_IDX(buck)])
422                 return -EINVAL;
423
424         for (val = LP3972_BUCK_VOL_MIN_IDX(buck);
425                 val <= LP3972_BUCK_VOL_MAX_IDX(buck); val++)
426                 if (vol_map[val] >= min_vol)
427                         break;
428
429         if (val > LP3972_BUCK_VOL_MAX_IDX(buck) ||
430             vol_map[val] > max_vol)
431                 return -EINVAL;
432
433         ret = lp3972_set_bits(lp3972, LP3972_BUCK_VOL1_REG(buck),
434                                 LP3972_BUCK_VOL_MASK, val);
435         if (ret)
436                 return ret;
437
438         if (buck != 0)
439                 return ret;
440
441         ret = lp3972_set_bits(lp3972, LP3972_VOL_CHANGE_REG,
442                 LP3972_VOL_CHANGE_FLAG_MASK, LP3972_VOL_CHANGE_FLAG_GO);
443         if (ret)
444                 return ret;
445
446         return lp3972_set_bits(lp3972, LP3972_VOL_CHANGE_REG,
447                                 LP3972_VOL_CHANGE_FLAG_MASK, 0);
448 }
449
450 static struct regulator_ops lp3972_dcdc_ops = {
451         .list_voltage = lp3972_dcdc_list_voltage,
452         .is_enabled = lp3972_dcdc_is_enabled,
453         .enable = lp3972_dcdc_enable,
454         .disable = lp3972_dcdc_disable,
455         .get_voltage = lp3972_dcdc_get_voltage,
456         .set_voltage = lp3972_dcdc_set_voltage,
457 };
458
459 static struct regulator_desc regulators[] = {
460         {
461                 .name = "LDO1",
462                 .id = LP3972_LDO1,
463                 .ops = &lp3972_ldo_ops,
464                 .n_voltages = ARRAY_SIZE(ldo1_voltage_map),
465                 .type = REGULATOR_VOLTAGE,
466                 .owner = THIS_MODULE,
467         },
468         {
469                 .name = "LDO2",
470                 .id = LP3972_LDO2,
471                 .ops = &lp3972_ldo_ops,
472                 .n_voltages = ARRAY_SIZE(ldo23_voltage_map),
473                 .type = REGULATOR_VOLTAGE,
474                 .owner = THIS_MODULE,
475         },
476         {
477                 .name = "LDO3",
478                 .id = LP3972_LDO3,
479                 .ops = &lp3972_ldo_ops,
480                 .n_voltages = ARRAY_SIZE(ldo23_voltage_map),
481                 .type = REGULATOR_VOLTAGE,
482                 .owner = THIS_MODULE,
483         },
484         {
485                 .name = "LDO4",
486                 .id = LP3972_LDO4,
487                 .ops = &lp3972_ldo_ops,
488                 .n_voltages = ARRAY_SIZE(ldo4_voltage_map),
489                 .type = REGULATOR_VOLTAGE,
490                 .owner = THIS_MODULE,
491         },
492         {
493                 .name = "LDO5",
494                 .id = LP3972_LDO5,
495                 .ops = &lp3972_ldo_ops,
496                 .n_voltages = ARRAY_SIZE(ldo5_voltage_map),
497                 .type = REGULATOR_VOLTAGE,
498                 .owner = THIS_MODULE,
499         },
500         {
501                 .name = "DCDC1",
502                 .id = LP3972_DCDC1,
503                 .ops = &lp3972_dcdc_ops,
504                 .n_voltages = ARRAY_SIZE(buck1_voltage_map),
505                 .type = REGULATOR_VOLTAGE,
506                 .owner = THIS_MODULE,
507         },
508         {
509                 .name = "DCDC2",
510                 .id = LP3972_DCDC2,
511                 .ops = &lp3972_dcdc_ops,
512                 .n_voltages = ARRAY_SIZE(buck23_voltage_map),
513                 .type = REGULATOR_VOLTAGE,
514                 .owner = THIS_MODULE,
515         },
516         {
517                 .name = "DCDC3",
518                 .id = LP3972_DCDC3,
519                 .ops = &lp3972_dcdc_ops,
520                 .n_voltages = ARRAY_SIZE(buck23_voltage_map),
521                 .type = REGULATOR_VOLTAGE,
522                 .owner = THIS_MODULE,
523         },
524 };
525
526 static int setup_regulators(struct lp3972 *lp3972,
527         struct lp3972_platform_data *pdata)
528 {
529         int i, err;
530
531         lp3972->num_regulators = pdata->num_regulators;
532         lp3972->rdev = kcalloc(pdata->num_regulators,
533                                 sizeof(struct regulator_dev *), GFP_KERNEL);
534         if (!lp3972->rdev) {
535                 err = -ENOMEM;
536                 goto err_nomem;
537         }
538
539         /* Instantiate the regulators */
540         for (i = 0; i < pdata->num_regulators; i++) {
541                 struct lp3972_regulator_subdev *reg = &pdata->regulators[i];
542                 lp3972->rdev[i] = regulator_register(&regulators[reg->id],
543                                         lp3972->dev, reg->initdata, lp3972);
544
545                 if (IS_ERR(lp3972->rdev[i])) {
546                         err = PTR_ERR(lp3972->rdev[i]);
547                         dev_err(lp3972->dev, "regulator init failed: %d\n",
548                                 err);
549                         goto error;
550                 }
551         }
552
553         return 0;
554 error:
555         while (--i >= 0)
556                 regulator_unregister(lp3972->rdev[i]);
557         kfree(lp3972->rdev);
558         lp3972->rdev = NULL;
559 err_nomem:
560         return err;
561 }
562
563 static int __devinit lp3972_i2c_probe(struct i2c_client *i2c,
564                             const struct i2c_device_id *id)
565 {
566         struct lp3972 *lp3972;
567         struct lp3972_platform_data *pdata = i2c->dev.platform_data;
568         int ret;
569         u16 val;
570
571         if (!pdata) {
572                 dev_dbg(&i2c->dev, "No platform init data supplied\n");
573                 return -ENODEV;
574         }
575
576         lp3972 = kzalloc(sizeof(struct lp3972), GFP_KERNEL);
577         if (!lp3972)
578                 return -ENOMEM;
579
580         lp3972->i2c = i2c;
581         lp3972->dev = &i2c->dev;
582
583         mutex_init(&lp3972->io_lock);
584
585         /* Detect LP3972 */
586         ret = lp3972_i2c_read(i2c, LP3972_SYS_CONTROL1_REG, 1, &val);
587         if (ret == 0 && (val & SYS_CONTROL1_INIT_MASK) != SYS_CONTROL1_INIT_VAL)
588                 ret = -ENODEV;
589         if (ret < 0) {
590                 dev_err(&i2c->dev, "failed to detect device\n");
591                 goto err_detect;
592         }
593
594         ret = setup_regulators(lp3972, pdata);
595         if (ret < 0)
596                 goto err_detect;
597
598         i2c_set_clientdata(i2c, lp3972);
599         return 0;
600
601 err_detect:
602         kfree(lp3972);
603         return ret;
604 }
605
606 static int __devexit lp3972_i2c_remove(struct i2c_client *i2c)
607 {
608         struct lp3972 *lp3972 = i2c_get_clientdata(i2c);
609         int i;
610
611         for (i = 0; i < lp3972->num_regulators; i++)
612                 regulator_unregister(lp3972->rdev[i]);
613         kfree(lp3972->rdev);
614         kfree(lp3972);
615
616         return 0;
617 }
618
619 static const struct i2c_device_id lp3972_i2c_id[] = {
620         { "lp3972", 0 },
621         { }
622 };
623 MODULE_DEVICE_TABLE(i2c, lp3972_i2c_id);
624
625 static struct i2c_driver lp3972_i2c_driver = {
626         .driver = {
627                 .name = "lp3972",
628                 .owner = THIS_MODULE,
629         },
630         .probe    = lp3972_i2c_probe,
631         .remove   = __devexit_p(lp3972_i2c_remove),
632         .id_table = lp3972_i2c_id,
633 };
634
635 static int __init lp3972_module_init(void)
636 {
637         return i2c_add_driver(&lp3972_i2c_driver);
638 }
639 subsys_initcall(lp3972_module_init);
640
641 static void __exit lp3972_module_exit(void)
642 {
643         i2c_del_driver(&lp3972_i2c_driver);
644 }
645 module_exit(lp3972_module_exit);
646
647 MODULE_LICENSE("GPL");
648 MODULE_AUTHOR("Axel Lin <axel.lin@gmail.com>");
649 MODULE_DESCRIPTION("LP3972 PMIC driver");