[U-Boot] [PATCH 05/12 v2] watchdog: at91sam9_wdt: Fix WDT setup in at91_wdt_start()
Heiko Schocher
hs at denx.de
Fri Mar 29 10:22:59 UTC 2019
Hello Stefan,
Am 26.03.2019 um 13:16 schrieb Stefan Roese:
> This patch fixes the timer register setup in at91_wdt_start() to
> correctly configure the register again. The input timeout value is
> now in milli-seconds instead of seconds with the new watchdog API.
> Make sure to take this into account and only use a max timeout
> value of 16 seconds as appropriate for this SoC.
>
> Also the check against a lower timeout value than 0 is removed. This
> check makes no sense, as the timeout value is unsigned.
>
> Signed-off-by: Stefan Roese <sr at denx.de>
> Reported-by: Heiko Schocher <hs at denx.de>
> Cc: Heiko Schocher <hs at denx.de>
> Cc: Andreas Bießmann <andreas at biessmann.org>
> Cc: Eugen Hristev <eugen.hristev at microchip.com>
> ---
> v2:
> - No change
>
> drivers/watchdog/at91sam9_wdt.c | 26 ++++++++++++++------------
> 1 file changed, 14 insertions(+), 12 deletions(-)
Thanks!
Reviewed-by: Heiko Schocher <hs at denx.de>
Tested on the taurus board:
Tested-by: Heiko Schocher <hs at denx.de>
bye,
Heiko
--
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-52 Fax: +49-8142-66989-80 Email: hs at denx.de
More information about the U-Boot
mailing list