diff options
author | NeilBrown <neilb@suse.de> | 2012-05-08 17:40:40 -0400 |
---|---|---|
committer | Anton Vorontsov <cbouatmailru@gmail.com> | 2012-06-19 22:13:27 -0400 |
commit | ab37813f4093a5f59cb8e083cde277289dc72ed3 (patch) | |
tree | 2b91baca976e3d1d2c92a8eb3ede3a8a693b71e8 | |
parent | 210d4bc8a3128e3e61ac3bf4657114f8e6450e2a (diff) |
twl4030_charger: Allow charger to control the regulator that feeds it
The charger needs usb3v1 to be running, so add a new consumer to
keep it running.
This allows the charger to draw current even when the USB driver has
powered down.
Signed-off-by: NeilBrown <neilb@suse.de>
Acked-by: Tero Kristo <t-kristo@ti.com>
Acked-by: Samuel Ortiz <sameo@linux.intel.com>
Signed-off-by: Anton Vorontsov <cbouatmailru@gmail.com>
-rw-r--r-- | drivers/mfd/twl-core.c | 9 | ||||
-rw-r--r-- | drivers/power/twl4030_charger.c | 15 |
2 files changed, 20 insertions, 4 deletions
diff --git a/drivers/mfd/twl-core.c b/drivers/mfd/twl-core.c index 6fc90befa79e..269b296a0975 100644 --- a/drivers/mfd/twl-core.c +++ b/drivers/mfd/twl-core.c | |||
@@ -716,8 +716,9 @@ add_children(struct twl4030_platform_data *pdata, unsigned irq_base, | |||
716 | static struct regulator_consumer_supply usb1v8 = { | 716 | static struct regulator_consumer_supply usb1v8 = { |
717 | .supply = "usb1v8", | 717 | .supply = "usb1v8", |
718 | }; | 718 | }; |
719 | static struct regulator_consumer_supply usb3v1 = { | 719 | static struct regulator_consumer_supply usb3v1[] = { |
720 | .supply = "usb3v1", | 720 | { .supply = "usb3v1" }, |
721 | { .supply = "bci3v1" }, | ||
721 | }; | 722 | }; |
722 | 723 | ||
723 | /* First add the regulators so that they can be used by transceiver */ | 724 | /* First add the regulators so that they can be used by transceiver */ |
@@ -745,7 +746,7 @@ add_children(struct twl4030_platform_data *pdata, unsigned irq_base, | |||
745 | return PTR_ERR(child); | 746 | return PTR_ERR(child); |
746 | 747 | ||
747 | child = add_regulator_linked(TWL4030_REG_VUSB3V1, | 748 | child = add_regulator_linked(TWL4030_REG_VUSB3V1, |
748 | &usb_fixed, &usb3v1, 1, | 749 | &usb_fixed, usb3v1, 2, |
749 | features); | 750 | features); |
750 | if (IS_ERR(child)) | 751 | if (IS_ERR(child)) |
751 | return PTR_ERR(child); | 752 | return PTR_ERR(child); |
@@ -766,7 +767,7 @@ add_children(struct twl4030_platform_data *pdata, unsigned irq_base, | |||
766 | if (twl_has_regulator() && child) { | 767 | if (twl_has_regulator() && child) { |
767 | usb1v5.dev_name = dev_name(child); | 768 | usb1v5.dev_name = dev_name(child); |
768 | usb1v8.dev_name = dev_name(child); | 769 | usb1v8.dev_name = dev_name(child); |
769 | usb3v1.dev_name = dev_name(child); | 770 | usb3v1[0].dev_name = dev_name(child); |
770 | } | 771 | } |
771 | } | 772 | } |
772 | if (twl_has_usb() && pdata->usb && twl_class_is_6030()) { | 773 | if (twl_has_usb() && pdata->usb && twl_class_is_6030()) { |
diff --git a/drivers/power/twl4030_charger.c b/drivers/power/twl4030_charger.c index 0511610b235f..bbda08377a42 100644 --- a/drivers/power/twl4030_charger.c +++ b/drivers/power/twl4030_charger.c | |||
@@ -21,6 +21,7 @@ | |||
21 | #include <linux/power_supply.h> | 21 | #include <linux/power_supply.h> |
22 | #include <linux/notifier.h> | 22 | #include <linux/notifier.h> |
23 | #include <linux/usb/otg.h> | 23 | #include <linux/usb/otg.h> |
24 | #include <linux/regulator/machine.h> | ||
24 | 25 | ||
25 | #define TWL4030_BCIMSTATEC 0x02 | 26 | #define TWL4030_BCIMSTATEC 0x02 |
26 | #define TWL4030_BCIICHG 0x08 | 27 | #define TWL4030_BCIICHG 0x08 |
@@ -86,6 +87,8 @@ struct twl4030_bci { | |||
86 | struct work_struct work; | 87 | struct work_struct work; |
87 | int irq_chg; | 88 | int irq_chg; |
88 | int irq_bci; | 89 | int irq_bci; |
90 | struct regulator *usb_reg; | ||
91 | int usb_enabled; | ||
89 | 92 | ||
90 | unsigned long event; | 93 | unsigned long event; |
91 | }; | 94 | }; |
@@ -183,6 +186,12 @@ static int twl4030_charger_enable_usb(struct twl4030_bci *bci, bool enable) | |||
183 | return -EACCES; | 186 | return -EACCES; |
184 | } | 187 | } |
185 | 188 | ||
189 | /* Need to keep regulator on */ | ||
190 | if (!bci->usb_enabled) { | ||
191 | regulator_enable(bci->usb_reg); | ||
192 | bci->usb_enabled = 1; | ||
193 | } | ||
194 | |||
186 | /* forcing the field BCIAUTOUSB (BOOT_BCI[1]) to 1 */ | 195 | /* forcing the field BCIAUTOUSB (BOOT_BCI[1]) to 1 */ |
187 | ret = twl4030_clear_set_boot_bci(0, TWL4030_BCIAUTOUSB); | 196 | ret = twl4030_clear_set_boot_bci(0, TWL4030_BCIAUTOUSB); |
188 | if (ret < 0) | 197 | if (ret < 0) |
@@ -193,6 +202,10 @@ static int twl4030_charger_enable_usb(struct twl4030_bci *bci, bool enable) | |||
193 | TWL4030_USBFASTMCHG, TWL4030_BCIMFSTS4); | 202 | TWL4030_USBFASTMCHG, TWL4030_BCIMFSTS4); |
194 | } else { | 203 | } else { |
195 | ret = twl4030_clear_set_boot_bci(TWL4030_BCIAUTOUSB, 0); | 204 | ret = twl4030_clear_set_boot_bci(TWL4030_BCIAUTOUSB, 0); |
205 | if (bci->usb_enabled) { | ||
206 | regulator_disable(bci->usb_reg); | ||
207 | bci->usb_enabled = 0; | ||
208 | } | ||
196 | } | 209 | } |
197 | 210 | ||
198 | return ret; | 211 | return ret; |
@@ -511,6 +524,8 @@ static int __init twl4030_bci_probe(struct platform_device *pdev) | |||
511 | bci->usb.num_properties = ARRAY_SIZE(twl4030_charger_props); | 524 | bci->usb.num_properties = ARRAY_SIZE(twl4030_charger_props); |
512 | bci->usb.get_property = twl4030_bci_get_property; | 525 | bci->usb.get_property = twl4030_bci_get_property; |
513 | 526 | ||
527 | bci->usb_reg = regulator_get(bci->dev, "bci3v1"); | ||
528 | |||
514 | ret = power_supply_register(&pdev->dev, &bci->usb); | 529 | ret = power_supply_register(&pdev->dev, &bci->usb); |
515 | if (ret) { | 530 | if (ret) { |
516 | dev_err(&pdev->dev, "failed to register usb: %d\n", ret); | 531 | dev_err(&pdev->dev, "failed to register usb: %d\n", ret); |