Re: [U-Boot] [PATCH v1 3/3] wdt: Unify option of timeout value

2017-07-06 Thread Simon Glass
On 5 July 2017 at 11:44, Andy Shevchenko wrote: > There is no need to duplicate same option with different name. > > Kill HW_WATCHDOG_TIMEOUT_MS in favor of WATCHDOG_TIMEOUT_MSECS. > > Signed-off-by: Andy Shevchenko > --- > drivers/watchdog/designware_wdt.c | 2 +- > include/configs/socfpga_comm

Re: [U-Boot] [PATCH v1 3/3] wdt: Unify option of timeout value

2017-07-05 Thread Heiko Schocher
Hello Andy, Am 05.07.2017 um 19:44 schrieb Andy Shevchenko: There is no need to duplicate same option with different name. Kill HW_WATCHDOG_TIMEOUT_MS in favor of WATCHDOG_TIMEOUT_MSECS. Signed-off-by: Andy Shevchenko --- drivers/watchdog/designware_wdt.c | 2 +- include/configs/socfpga_co

Re: [U-Boot] [PATCH v1 3/3] wdt: Unify option of timeout value

2017-07-05 Thread Bin Meng
On Thu, Jul 6, 2017 at 1:44 AM, Andy Shevchenko wrote: > There is no need to duplicate same option with different name. > > Kill HW_WATCHDOG_TIMEOUT_MS in favor of WATCHDOG_TIMEOUT_MSECS. > > Signed-off-by: Andy Shevchenko > --- > drivers/watchdog/designware_wdt.c | 2 +- > include/configs/socfp

Re: [U-Boot] [PATCH v1 3/3] wdt: Unify option of timeout value

2017-07-05 Thread Andy Shevchenko
On Wed, 2017-07-05 at 20:44 +0300, Andy Shevchenko wrote: > There is no need to duplicate same option with different name. > > Kill HW_WATCHDOG_TIMEOUT_MS in favor of WATCHDOG_TIMEOUT_MSECS. > Yes, I know about WATCHDOG_TIMEOUT, I spent too much time already on this crap. Feel free to make a fol