* [PATCH v3 1/2] dt-bindings: power: reset: gpio-poweroff: Add 'timeout-ms' property @ 2018-02-22 23:17 Moritz Fischer 2018-02-22 23:17 ` [PATCH v3 2/2] power: reset: gpio-poweroff: Support for timeout from device property Moritz Fischer 2018-03-01 23:26 ` [PATCH v3 1/2] dt-bindings: power: reset: gpio-poweroff: Add 'timeout-ms' property Rob Herring 0 siblings, 2 replies; 4+ messages in thread From: Moritz Fischer @ 2018-02-22 23:17 UTC (permalink / raw) To: linux-kernel Cc: devicetree, linux-pm, sre, robh+dt, andy.shevchenko, Moritz Fischer Add 'timeout-ms' property to support boards where the 3s timeout that the current driver defaults to is too short. Signed-off-by: Moritz Fischer <mdf@kernel.org> --- Changes from v2: - Addressed Rob's feedback (timeout_ms -> timeout-ms) Changes from v1: - Addressed Rob's feedback (timeout -> timeout_ms) - Added to old example rather than creating separate one --- Documentation/devicetree/bindings/power/reset/gpio-poweroff.txt | 3 +++ 1 file changed, 3 insertions(+) diff --git a/Documentation/devicetree/bindings/power/reset/gpio-poweroff.txt b/Documentation/devicetree/bindings/power/reset/gpio-poweroff.txt index e62d53d844cc..6d8980c18c34 100644 --- a/Documentation/devicetree/bindings/power/reset/gpio-poweroff.txt +++ b/Documentation/devicetree/bindings/power/reset/gpio-poweroff.txt @@ -27,10 +27,13 @@ Optional properties: it to an output when the power-off handler is called. If this optional property is not specified, the GPIO is initialized as an output in its inactive state. +- timeout-ms: Time to wait before asserting a WARN_ON(1). If nothing is + specified, 3000 ms is used. Examples: gpio-poweroff { compatible = "gpio-poweroff"; gpios = <&gpio 4 0>; + timeout-ms = <3000>; }; -- 2.16.1 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* [PATCH v3 2/2] power: reset: gpio-poweroff: Support for timeout from device property 2018-02-22 23:17 [PATCH v3 1/2] dt-bindings: power: reset: gpio-poweroff: Add 'timeout-ms' property Moritz Fischer @ 2018-02-22 23:17 ` Moritz Fischer 2018-03-09 16:19 ` Sebastian Reichel 2018-03-01 23:26 ` [PATCH v3 1/2] dt-bindings: power: reset: gpio-poweroff: Add 'timeout-ms' property Rob Herring 1 sibling, 1 reply; 4+ messages in thread From: Moritz Fischer @ 2018-02-22 23:17 UTC (permalink / raw) To: linux-kernel Cc: devicetree, linux-pm, sre, robh+dt, andy.shevchenko, Moritz Fischer Add support for reading a timeout value from device property. Fall back to previous default of 3s if nothing is specified. Signed-off-by: Moritz Fischer <mdf@kernel.org> --- Changes from v2: - use device_property_read_{bool,u32} instead of of equivalent Changes from v1: - of_property_read_u32() overwrites the output only if it succeeds, simplify error handling --- drivers/power/reset/gpio-poweroff.c | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/drivers/power/reset/gpio-poweroff.c b/drivers/power/reset/gpio-poweroff.c index be3d81ff51cc..6273ad3b411d 100644 --- a/drivers/power/reset/gpio-poweroff.c +++ b/drivers/power/reset/gpio-poweroff.c @@ -19,11 +19,13 @@ #include <linux/of_platform.h> #include <linux/module.h> +#define DEFAULT_TIMEOUT_MS 3000 /* * Hold configuration here, cannot be more than one instance of the driver * since pm_power_off itself is global. */ static struct gpio_desc *reset_gpio; +static u32 timeout = DEFAULT_TIMEOUT_MS; static void gpio_poweroff_do_poweroff(void) { @@ -40,7 +42,7 @@ static void gpio_poweroff_do_poweroff(void) gpiod_set_value(reset_gpio, 1); /* give it some time */ - mdelay(3000); + mdelay(timeout); WARN_ON(1); } @@ -58,12 +60,14 @@ static int gpio_poweroff_probe(struct platform_device *pdev) return -EBUSY; } - input = of_property_read_bool(pdev->dev.of_node, "input"); + input = device_property_read_bool(&pdev->dev, "input"); if (input) flags = GPIOD_IN; else flags = GPIOD_OUT_LOW; + device_property_read_u32(&pdev->dev, "timeout-ms", &timeout); + reset_gpio = devm_gpiod_get(&pdev->dev, NULL, flags); if (IS_ERR(reset_gpio)) return PTR_ERR(reset_gpio); -- 2.16.1 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH v3 2/2] power: reset: gpio-poweroff: Support for timeout from device property 2018-02-22 23:17 ` [PATCH v3 2/2] power: reset: gpio-poweroff: Support for timeout from device property Moritz Fischer @ 2018-03-09 16:19 ` Sebastian Reichel 0 siblings, 0 replies; 4+ messages in thread From: Sebastian Reichel @ 2018-03-09 16:19 UTC (permalink / raw) To: Moritz Fischer Cc: linux-kernel, devicetree, linux-pm, robh+dt, andy.shevchenko [-- Attachment #1: Type: text/plain, Size: 2015 bytes --] Hi, On Thu, Feb 22, 2018 at 03:17:14PM -0800, Moritz Fischer wrote: > Add support for reading a timeout value from device property. > Fall back to previous default of 3s if nothing is specified. > > Signed-off-by: Moritz Fischer <mdf@kernel.org> > --- Thanks, both queued. -- Sebastian > Changes from v2: > - use device_property_read_{bool,u32} instead of of equivalent > > Changes from v1: > - of_property_read_u32() overwrites the output only if it succeeds, > simplify error handling > --- > > drivers/power/reset/gpio-poweroff.c | 8 ++++++-- > 1 file changed, 6 insertions(+), 2 deletions(-) > > diff --git a/drivers/power/reset/gpio-poweroff.c b/drivers/power/reset/gpio-poweroff.c > index be3d81ff51cc..6273ad3b411d 100644 > --- a/drivers/power/reset/gpio-poweroff.c > +++ b/drivers/power/reset/gpio-poweroff.c > @@ -19,11 +19,13 @@ > #include <linux/of_platform.h> > #include <linux/module.h> > > +#define DEFAULT_TIMEOUT_MS 3000 > /* > * Hold configuration here, cannot be more than one instance of the driver > * since pm_power_off itself is global. > */ > static struct gpio_desc *reset_gpio; > +static u32 timeout = DEFAULT_TIMEOUT_MS; > > static void gpio_poweroff_do_poweroff(void) > { > @@ -40,7 +42,7 @@ static void gpio_poweroff_do_poweroff(void) > gpiod_set_value(reset_gpio, 1); > > /* give it some time */ > - mdelay(3000); > + mdelay(timeout); > > WARN_ON(1); > } > @@ -58,12 +60,14 @@ static int gpio_poweroff_probe(struct platform_device *pdev) > return -EBUSY; > } > > - input = of_property_read_bool(pdev->dev.of_node, "input"); > + input = device_property_read_bool(&pdev->dev, "input"); > if (input) > flags = GPIOD_IN; > else > flags = GPIOD_OUT_LOW; > > + device_property_read_u32(&pdev->dev, "timeout-ms", &timeout); > + > reset_gpio = devm_gpiod_get(&pdev->dev, NULL, flags); > if (IS_ERR(reset_gpio)) > return PTR_ERR(reset_gpio); > -- > 2.16.1 > [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v3 1/2] dt-bindings: power: reset: gpio-poweroff: Add 'timeout-ms' property 2018-02-22 23:17 [PATCH v3 1/2] dt-bindings: power: reset: gpio-poweroff: Add 'timeout-ms' property Moritz Fischer 2018-02-22 23:17 ` [PATCH v3 2/2] power: reset: gpio-poweroff: Support for timeout from device property Moritz Fischer @ 2018-03-01 23:26 ` Rob Herring 1 sibling, 0 replies; 4+ messages in thread From: Rob Herring @ 2018-03-01 23:26 UTC (permalink / raw) To: Moritz Fischer; +Cc: linux-kernel, devicetree, linux-pm, sre, andy.shevchenko On Thu, Feb 22, 2018 at 03:17:13PM -0800, Moritz Fischer wrote: > Add 'timeout-ms' property to support boards where the 3s timeout that the > current driver defaults to is too short. > > Signed-off-by: Moritz Fischer <mdf@kernel.org> > --- > > Changes from v2: > - Addressed Rob's feedback (timeout_ms -> timeout-ms) > > Changes from v1: > - Addressed Rob's feedback (timeout -> timeout_ms) > - Added to old example rather than creating separate one > > --- > > Documentation/devicetree/bindings/power/reset/gpio-poweroff.txt | 3 +++ > 1 file changed, 3 insertions(+) Reviewed-by: Rob Herring <robh@kernel.org> ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2018-03-09 16:19 UTC | newest] Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2018-02-22 23:17 [PATCH v3 1/2] dt-bindings: power: reset: gpio-poweroff: Add 'timeout-ms' property Moritz Fischer 2018-02-22 23:17 ` [PATCH v3 2/2] power: reset: gpio-poweroff: Support for timeout from device property Moritz Fischer 2018-03-09 16:19 ` Sebastian Reichel 2018-03-01 23:26 ` [PATCH v3 1/2] dt-bindings: power: reset: gpio-poweroff: Add 'timeout-ms' property Rob Herring
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.