Commit ee181853 authored by Frans Klaver's avatar Frans Klaver Committed by Sebastian Reichel

power: reset: ltc2952: make trigger input optional

Currently the ltc2952 supports only one button sequence to initiate
powerdown. This is not always desirable, as even prolonged button
presses can happen in use.

Allow ltc2952 users to pick their own power down sequence, by making the
trigger input optional. Since this still means that the ltc2952 may
power down the platform if the power button is pressed for about 5
seconds, we still need to make sure to start the watchdog toggle to
prolong the system power for as long as we need it.

This will still allow the system to control power using the kill signal.
Signed-off-by: default avatarFrans Klaver <frans.klaver@xsens.com>
Signed-off-by: default avatarSebastian Reichel <sre@kernel.org>
parent c1ada2ff
...@@ -32,7 +32,9 @@ ...@@ -32,7 +32,9 @@
* - trigger (input) * - trigger (input)
* A level change indicates the shut-down trigger. If it's state reverts * A level change indicates the shut-down trigger. If it's state reverts
* within the time-out defined by trigger_delay, the shut down is not * within the time-out defined by trigger_delay, the shut down is not
* executed. * executed. If no pin is assigned to this input, the driver will start the
* watchdog toggle immediately. The chip will only power off the system if
* it is requested to do so through the kill line.
* *
* - watchdog (output) * - watchdog (output)
* Once a shut down is triggered, the driver will toggle this signal, * Once a shut down is triggered, the driver will toggle this signal,
...@@ -116,15 +118,10 @@ static enum hrtimer_restart ltc2952_poweroff_timer_wde(struct hrtimer *timer) ...@@ -116,15 +118,10 @@ static enum hrtimer_restart ltc2952_poweroff_timer_wde(struct hrtimer *timer)
return HRTIMER_RESTART; return HRTIMER_RESTART;
} }
static enum hrtimer_restart static void ltc2952_poweroff_start_wde(struct ltc2952_poweroff *data)
ltc2952_poweroff_timer_trigger(struct hrtimer *timer)
{ {
struct ltc2952_poweroff *data = to_ltc2952(timer, timer_trigger); if (hrtimer_start(&data->timer_wde, data->wde_interval,
int ret = hrtimer_start(&data->timer_wde, HRTIMER_MODE_REL)) {
data->wde_interval, HRTIMER_MODE_REL);
if (ret) {
dev_err(data->dev, "unable to start the timer\n");
/* /*
* The device will not toggle the watchdog reset, * The device will not toggle the watchdog reset,
* thus shut down is only safe if the PowerPath controller * thus shut down is only safe if the PowerPath controller
...@@ -133,10 +130,17 @@ ltc2952_poweroff_timer_trigger(struct hrtimer *timer) ...@@ -133,10 +130,17 @@ ltc2952_poweroff_timer_trigger(struct hrtimer *timer)
* *
* Only sending a warning as the system will power-off anyway * Only sending a warning as the system will power-off anyway
*/ */
dev_err(data->dev, "unable to start the timer\n");
} }
}
dev_info(data->dev, "executing shutdown\n"); static enum hrtimer_restart
ltc2952_poweroff_timer_trigger(struct hrtimer *timer)
{
struct ltc2952_poweroff *data = to_ltc2952(timer, timer_trigger);
ltc2952_poweroff_start_wde(data);
dev_info(data->dev, "executing shutdown\n");
orderly_poweroff(true); orderly_poweroff(true);
return HRTIMER_NORESTART; return HRTIMER_NORESTART;
...@@ -190,7 +194,7 @@ static void ltc2952_poweroff_default(struct ltc2952_poweroff *data) ...@@ -190,7 +194,7 @@ static void ltc2952_poweroff_default(struct ltc2952_poweroff *data)
static int ltc2952_poweroff_init(struct platform_device *pdev) static int ltc2952_poweroff_init(struct platform_device *pdev)
{ {
int ret, virq; int ret;
struct ltc2952_poweroff *data = platform_get_drvdata(pdev); struct ltc2952_poweroff *data = platform_get_drvdata(pdev);
ltc2952_poweroff_default(data); ltc2952_poweroff_default(data);
...@@ -210,29 +214,48 @@ static int ltc2952_poweroff_init(struct platform_device *pdev) ...@@ -210,29 +214,48 @@ static int ltc2952_poweroff_init(struct platform_device *pdev)
return ret; return ret;
} }
data->gpio_trigger = devm_gpiod_get(&pdev->dev, "trigger", data->gpio_trigger = devm_gpiod_get(&pdev->dev, "trigger", GPIOD_IN);
GPIOD_IN); if (IS_ERR(data->gpio_trigger)) {
if (IS_ERR(ltc2952_data->gpio_trigger)) { /*
ret = PTR_ERR(ltc2952_data->gpio_trigger); * It's not a problem if the trigger gpio isn't available, but
dev_err(&pdev->dev, "unable to claim gpio \"trigger\"\n"); * it is worth a warning if its use was defined in the device
return ret; * tree.
} */
if (PTR_ERR(data->gpio_trigger) != -ENOENT)
virq = gpiod_to_irq(data->gpio_trigger); dev_err(&pdev->dev,
if (virq < 0) { "unable to claim gpio \"trigger\"\n");
dev_err(&pdev->dev, "cannot map GPIO as interrupt"); data->gpio_trigger = NULL;
return ret;
} }
ret = devm_request_irq(&pdev->dev, virq, if (devm_request_irq(&pdev->dev, gpiod_to_irq(data->gpio_trigger),
ltc2952_poweroff_handler, ltc2952_poweroff_handler,
(IRQF_TRIGGER_FALLING | IRQF_TRIGGER_RISING), (IRQF_TRIGGER_FALLING | IRQF_TRIGGER_RISING),
"ltc2952-poweroff", "ltc2952-poweroff",
data); data)) {
/*
if (ret) { * Some things may have happened:
dev_err(&pdev->dev, "cannot configure an interrupt handler\n"); * - No trigger input was defined
return ret; * - Claiming the GPIO failed
* - We could not map to an IRQ
* - We couldn't register an interrupt handler
*
* None of these really are problems, but all of them
* disqualify the push button from controlling the power.
*
* It is therefore important to note that if the ltc2952
* detects a button press for long enough, it will still start
* its own powerdown window and cut the power on us if we don't
* start the watchdog trigger.
*/
if (data->gpio_trigger) {
dev_warn(&pdev->dev,
"unable to configure the trigger interrupt\n");
devm_gpiod_put(&pdev->dev, data->gpio_trigger);
data->gpio_trigger = NULL;
}
dev_info(&pdev->dev,
"power down trigger input will not be used\n");
ltc2952_poweroff_start_wde(data);
} }
return 0; return 0;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment