[PATCH 2/2] regulator: fixed: add off-on-delay

From: Peng Fan
Date: Mon Oct 28 2019 - 21:22:50 EST


From: Peng Fan <peng.fan@xxxxxxx>

Depends on board design, the gpio controlling regulator may
connects with a big capacitance. When need off, it takes some time
to let the regulator to be truly off. If not add enough delay, the
regulator might have always been on, so introduce off-on-delay to
handle such case.

Signed-off-by: Peng Fan <peng.fan@xxxxxxx>
---

V1:
There is a checkpatch warning "
Prefer 'unsigned int' to bare use of 'unsigned' ",
so I use unsigned int for off_on_delay.

drivers/regulator/fixed.c | 2 ++
include/linux/regulator/fixed.h | 1 +
2 files changed, 3 insertions(+)

diff --git a/drivers/regulator/fixed.c b/drivers/regulator/fixed.c
index d90a6fd8cbc7..5dbdc41743f3 100644
--- a/drivers/regulator/fixed.c
+++ b/drivers/regulator/fixed.c
@@ -123,6 +123,7 @@ of_get_fixed_voltage_config(struct device *dev,
config->enabled_at_boot = true;

of_property_read_u32(np, "startup-delay-us", &config->startup_delay);
+ of_property_read_u32(np, "off-on-delay-us", &config->off_on_delay);

if (of_find_property(np, "vin-supply", NULL))
config->input_supply = "vin";
@@ -190,6 +191,7 @@ static int reg_fixed_voltage_probe(struct platform_device *pdev)
}

drvdata->desc.enable_time = config->startup_delay;
+ drvdata->desc.off_on_delay = config->off_on_delay;

if (config->input_supply) {
drvdata->desc.supply_name = devm_kstrdup(&pdev->dev,
diff --git a/include/linux/regulator/fixed.h b/include/linux/regulator/fixed.h
index d44ce5f18a56..623a5c574782 100644
--- a/include/linux/regulator/fixed.h
+++ b/include/linux/regulator/fixed.h
@@ -36,6 +36,7 @@ struct fixed_voltage_config {
const char *input_supply;
int microvolts;
unsigned startup_delay;
+ unsigned int off_on_delay;
unsigned enabled_at_boot:1;
struct regulator_init_data *init_data;
};
--
2.16.4