diff options
| author | John Crispin <blogic@openwrt.org> | 2011-09-26 10:35:51 +0000 |
|---|---|---|
| committer | John Crispin <blogic@openwrt.org> | 2011-09-26 10:35:51 +0000 |
| commit | 10ad7c5a3ecde10da3176886a2ed49643625c16f (patch) | |
| tree | c7724ebcde00285a4bdb8b1ab7a1687dfef484e4 | |
| parent | 8d8f9e99d2e20223fc2a9799ef0fc032a35d5989 (diff) | |
| download | upstream-10ad7c5a3ecde10da3176886a2ed49643625c16f.tar.gz upstream-10ad7c5a3ecde10da3176886a2ed49643625c16f.tar.bz2 upstream-10ad7c5a3ecde10da3176886a2ed49643625c16f.zip | |
[lantiq-wdt]
The enable function was using the global timeout variable for local operations.
This resulted in the value of the global variable being corrupted, thus
breaking the code.
git-svn-id: svn://svn.openwrt.org/openwrt/trunk@28310 3c298f89-4303-0410-b956-a3cf2f4a3e73
| -rw-r--r-- | target/linux/lantiq/patches-3.0/991-fix-wdt-timeout.patch | 40 |
1 files changed, 40 insertions, 0 deletions
diff --git a/target/linux/lantiq/patches-3.0/991-fix-wdt-timeout.patch b/target/linux/lantiq/patches-3.0/991-fix-wdt-timeout.patch new file mode 100644 index 0000000000..32e82cfb5f --- /dev/null +++ b/target/linux/lantiq/patches-3.0/991-fix-wdt-timeout.patch @@ -0,0 +1,40 @@ +From 14ea48a5f5702ddc97425cbe520600e187e14e4a Mon Sep 17 00:00:00 2001 +From: John Crispin <blogic@openwrt.org> +Date: Thu, 11 Aug 2011 13:58:39 +0200 +Subject: [PATCH 03/16] MIPS: lantiq: fix watchdogs timeout handling + +The enable function was using the global timeout variable for local operations. +This resulted in the value of the global variable being corrupted, thus +breaking the code. + +Signed-off-by: John Crispin <blogic@openwrt.org> +Signed-off-by: Thomas Langer <thomas.langer@lantiq.com> +Cc: linux-watchdog@vger.kernel.org +Cc: linux-mips@linux-mips.org +--- + drivers/watchdog/lantiq_wdt.c | 8 ++++---- + 1 files changed, 4 insertions(+), 4 deletions(-) + +--- a/drivers/watchdog/lantiq_wdt.c ++++ b/drivers/watchdog/lantiq_wdt.c +@@ -51,16 +51,16 @@ static int ltq_wdt_ok_to_close; + static void + ltq_wdt_enable(void) + { +- ltq_wdt_timeout = ltq_wdt_timeout * ++ unsigned long int timeout = ltq_wdt_timeout * + (ltq_io_region_clk_rate / LTQ_WDT_DIVIDER) + 0x1000; +- if (ltq_wdt_timeout > LTQ_MAX_TIMEOUT) +- ltq_wdt_timeout = LTQ_MAX_TIMEOUT; ++ if (timeout > LTQ_MAX_TIMEOUT) ++ timeout = LTQ_MAX_TIMEOUT; + + /* write the first password magic */ + ltq_w32(LTQ_WDT_PW1, ltq_wdt_membase + LTQ_WDT_CR); + /* write the second magic plus the configuration and new timeout */ + ltq_w32(LTQ_WDT_SR_EN | LTQ_WDT_SR_PWD | LTQ_WDT_SR_CLKDIV | +- LTQ_WDT_PW2 | ltq_wdt_timeout, ltq_wdt_membase + LTQ_WDT_CR); ++ LTQ_WDT_PW2 | timeout, ltq_wdt_membase + LTQ_WDT_CR); + } + + static void |