Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit eadc4fe1 authored by Shubhrajyoti Datta's avatar Shubhrajyoti Datta Committed by Wim Van Sebroeck
Browse files

watchdog: cadence_wdt: Fix the suspend resume 



Currently even if no users are there the suspend tries to
stop the watchdog and resume starts it.

so after resume the watchdog starts and resets the board.
Fix the same by adding a check for users.

Reviewed-by: default avatarGuenter Roeck <linux@roeck-us.net>
Signed-off-by: default avatarShubhrajyoti Datta <shubhrajyoti.datta@xilinx.com>
Signed-off-by: default avatarGuenter Roeck <linux@roeck-us.net>
Signed-off-by: default avatarWim Van Sebroeck <wim@iguana.be>
parent bfb1f46f

drivers/watchdog/cadence_wdt.c

+11 −7
Original line number Diff line number Diff line
@@ -424,8 +424,10 @@ static int __maybe_unused cdns_wdt_suspend(struct device *dev) 
	struct platform_device *pdev = to_platform_device(dev);

	struct cdns_wdt *wdt = platform_get_drvdata(pdev);



	if (watchdog_active(&wdt->cdns_wdt_device)) {

		cdns_wdt_stop(&wdt->cdns_wdt_device);

		clk_disable_unprepare(wdt->clk);

	}



	return 0;

}
@@ -442,12 +444,14 @@ static int __maybe_unused cdns_wdt_resume(struct device *dev) 
	struct platform_device *pdev = to_platform_device(dev);

	struct cdns_wdt *wdt = platform_get_drvdata(pdev);



	if (watchdog_active(&wdt->cdns_wdt_device)) {

		ret = clk_prepare_enable(wdt->clk);

		if (ret) {

			dev_err(dev, "unable to enable clock\n");

			return ret;

		}

		cdns_wdt_start(&wdt->cdns_wdt_device);

	}



	return 0;

}