[PATCH v2] watchdog: cpwd: add of_node_put()

[PATCH v2] watchdog: cpwd: add of_node_put()

[Yangtao Li]

use of_node_put() to release the refcount.

Signed-off-by: Yangtao Li <tiny.windzz@gmail.com>
---
Changes in v2:
-do that right after its last use to also cover
 error handling.
---
 drivers/watchdog/cpwd.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/drivers/watchdog/cpwd.c b/drivers/watchdog/cpwd.c
index aee0b25cf10d..32156e199c51 100644
--- a/drivers/watchdog/cpwd.c
+++ b/drivers/watchdog/cpwd.c
@@ -570,6 +570,8 @@ static int cpwd_probe(struct platform_device *op)
 	if (str_prop)
 		p->timeout = simple_strtoul(str_prop, NULL, 10);
 
+	of_node_put(options);
+
 	/* CP1400s seem to have broken PLD implementations-- the
 	 * interrupt_mask register cannot be written, so no timer
 	 * interrupts can be masked within the PLD.
-- 
2.17.0

Re: [PATCH v2] watchdog: cpwd: add of_node_put()

[Guenter Roeck]

On 11/20/18 6:54 AM, Yangtao Li wrote:
> use of_node_put() to release the refcount.
> 
> Signed-off-by: Yangtao Li <tiny.windzz@gmail.com>

Reviewed-by: Guenter Roeck <linux@roeck-us.net>

> ---
> Changes in v2:
> -do that right after its last use to also cover
>   error handling.
> ---
>   drivers/watchdog/cpwd.c | 2 ++
>   1 file changed, 2 insertions(+)
> 
> diff --git a/drivers/watchdog/cpwd.c b/drivers/watchdog/cpwd.c
> index aee0b25cf10d..32156e199c51 100644
> --- a/drivers/watchdog/cpwd.c
> +++ b/drivers/watchdog/cpwd.c
> @@ -570,6 +570,8 @@ static int cpwd_probe(struct platform_device *op)
>   	if (str_prop)
>   		p->timeout = simple_strtoul(str_prop, NULL, 10);
>   
> +	of_node_put(options);
> +
>   	/* CP1400s seem to have broken PLD implementations-- the
>   	 * interrupt_mask register cannot be written, so no timer
>   	 * interrupts can be masked within the PLD.
>