diff options
author | Bjorn Andersson <bjorn.andersson@sonymobile.com> | 2015-11-10 01:20:37 -0500 |
---|---|---|
committer | Mark Brown <broonie@kernel.org> | 2015-11-17 13:54:07 -0500 |
commit | 3ff3f518a135fa4592fe2817e9ac2cce1fa23dc2 (patch) | |
tree | 442835eec00be4e89cbca34a48c1c40b0cde629b | |
parent | 8005c49d9aea74d382f474ce11afbbc7d7130bec (diff) |
regulator: Make bulk API support optional supplies
Make it possible to use the bulk API with optional supplies, by allowing
the consumer to marking supplies as optional in the regulator_bulk_data.
Signed-off-by: Bjorn Andersson <bjorn.andersson@sonymobile.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
-rw-r--r-- | drivers/regulator/core.c | 6 | ||||
-rw-r--r-- | drivers/regulator/devres.c | 7 | ||||
-rw-r--r-- | include/linux/regulator/consumer.h | 3 |
3 files changed, 12 insertions, 4 deletions
diff --git a/drivers/regulator/core.c b/drivers/regulator/core.c index 73b7683355cd..4cf1390784e5 100644 --- a/drivers/regulator/core.c +++ b/drivers/regulator/core.c | |||
@@ -3451,8 +3451,10 @@ int regulator_bulk_get(struct device *dev, int num_consumers, | |||
3451 | consumers[i].consumer = NULL; | 3451 | consumers[i].consumer = NULL; |
3452 | 3452 | ||
3453 | for (i = 0; i < num_consumers; i++) { | 3453 | for (i = 0; i < num_consumers; i++) { |
3454 | consumers[i].consumer = regulator_get(dev, | 3454 | consumers[i].consumer = _regulator_get(dev, |
3455 | consumers[i].supply); | 3455 | consumers[i].supply, |
3456 | false, | ||
3457 | !consumers[i].optional); | ||
3456 | if (IS_ERR(consumers[i].consumer)) { | 3458 | if (IS_ERR(consumers[i].consumer)) { |
3457 | ret = PTR_ERR(consumers[i].consumer); | 3459 | ret = PTR_ERR(consumers[i].consumer); |
3458 | dev_err(dev, "Failed to get supply '%s': %d\n", | 3460 | dev_err(dev, "Failed to get supply '%s': %d\n", |
diff --git a/drivers/regulator/devres.c b/drivers/regulator/devres.c index 6ec1d400adae..6ad8ab4c578d 100644 --- a/drivers/regulator/devres.c +++ b/drivers/regulator/devres.c | |||
@@ -164,8 +164,11 @@ int devm_regulator_bulk_get(struct device *dev, int num_consumers, | |||
164 | consumers[i].consumer = NULL; | 164 | consumers[i].consumer = NULL; |
165 | 165 | ||
166 | for (i = 0; i < num_consumers; i++) { | 166 | for (i = 0; i < num_consumers; i++) { |
167 | consumers[i].consumer = devm_regulator_get(dev, | 167 | consumers[i].consumer = _devm_regulator_get(dev, |
168 | consumers[i].supply); | 168 | consumers[i].supply, |
169 | consumers[i].optional ? | ||
170 | OPTIONAL_GET : | ||
171 | NORMAL_GET); | ||
169 | if (IS_ERR(consumers[i].consumer)) { | 172 | if (IS_ERR(consumers[i].consumer)) { |
170 | ret = PTR_ERR(consumers[i].consumer); | 173 | ret = PTR_ERR(consumers[i].consumer); |
171 | dev_err(dev, "Failed to get supply '%s': %d\n", | 174 | dev_err(dev, "Failed to get supply '%s': %d\n", |
diff --git a/include/linux/regulator/consumer.h b/include/linux/regulator/consumer.h index 9e0e76992be0..48603506f8de 100644 --- a/include/linux/regulator/consumer.h +++ b/include/linux/regulator/consumer.h | |||
@@ -140,6 +140,8 @@ struct regulator; | |||
140 | * | 140 | * |
141 | * @supply: The name of the supply. Initialised by the user before | 141 | * @supply: The name of the supply. Initialised by the user before |
142 | * using the bulk regulator APIs. | 142 | * using the bulk regulator APIs. |
143 | * @optional: The supply should be considered optional. Initialised by the user | ||
144 | * before using the bulk regulator APIs. | ||
143 | * @consumer: The regulator consumer for the supply. This will be managed | 145 | * @consumer: The regulator consumer for the supply. This will be managed |
144 | * by the bulk API. | 146 | * by the bulk API. |
145 | * | 147 | * |
@@ -149,6 +151,7 @@ struct regulator; | |||
149 | */ | 151 | */ |
150 | struct regulator_bulk_data { | 152 | struct regulator_bulk_data { |
151 | const char *supply; | 153 | const char *supply; |
154 | bool optional; | ||
152 | struct regulator *consumer; | 155 | struct regulator *consumer; |
153 | 156 | ||
154 | /* private: Internal use */ | 157 | /* private: Internal use */ |