watchdog: sp5100_tco: Fix wrong indirect I/O access for getting value of reserved...
authorTakahisa Tanaka <mc74hc00@gmail.com>
Mon, 14 Jan 2013 02:01:57 +0000 (11:01 +0900)
committerWim Van Sebroeck <wim@iguana.be>
Fri, 1 Mar 2013 11:19:26 +0000 (12:19 +0100)
commit10ab329b5db7e592a3a60b4594e4e5f40b60c45c
tree52e8c155fe20f181df604c3ac5cefa2a306c0bbb
parent6737176569d1d6356c644694d60ea2b265cb3870
watchdog: sp5100_tco: Fix wrong indirect I/O access for getting value of reserved bits

In case of SB800 or later chipset and re-programming MMIO address(*),
sp5100_tco module may read incorrect value of reserved bit, because the module
reads a value from an incorrect I/O address. However, this bug doesn't cause
a problem, because when re-programming MMIO address, by chance the module
writes zero (this is BIOS's default value) to the low three bits of register.
* In most cases, PC with SB8x0 or later chipset doesn't need to re-programming
  MMIO address, because such PC can enable AcpiMmio and can use 0xfed80b00 for
  watchdog register base address.

This patch fixes this bug.

Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=43176
Signed-off-by: Takahisa Tanaka <mc74hc00@gmail.com>
Tested-by: Paul Menzel <paulepanter@users.sourceforge.net>
Signed-off-by: Wim Van Sebroeck <wim@iguana.be>
Cc: stable <stable@vger.kernel.org>
drivers/watchdog/sp5100_tco.c