watchdog: tangier: Replace unused constant with a comment
authorAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Fri, 21 Jun 2019 10:28:07 +0000 (13:28 +0300)
committerBin Meng <bmeng.cn@gmail.com>
Sat, 22 Jun 2019 14:26:22 +0000 (22:26 +0800)
The default timeout value had been left in order to leave some traces
about default setup of watchdog done by firmware.

For better understanding and compiler burden, replace it with a comment.

Suggested-by: Stefan Roese <sr@denx.de>
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Reviewed-by: Bin Meng <bmeng.cn@gmail.com>
drivers/watchdog/tangier_wdt.c

index be4a8f467ac9da474b91c985cc65bcc9983fab6c..b2f40ef0574453b51733ba7db5e5fd3c79e5adb0 100644 (file)
 #define WDT_PRETIMEOUT         15
 #define WDT_TIMEOUT_MIN                (1 + WDT_PRETIMEOUT)
 #define WDT_TIMEOUT_MAX                170
-#define WDT_DEFAULT_TIMEOUT    90
+
+/*
+ * Note, firmware chooses 90 seconds as a default timeout for watchdog on
+ * Intel Tangier SoC. It means that without handling it in the running code
+ * the reboot will happen.
+ */
 
 #ifndef CONFIG_WATCHDOG_TIMEOUT_MSECS
 #define WATCHDOG_HEARTBEAT 60000