diff options
author | Roland Dreier <rdreier@cisco.com> | 2008-02-28 18:38:44 +0100 |
---|---|---|
committer | Wim Van Sebroeck <wim@iguana.be> | 2008-03-06 12:10:35 +0100 |
commit | 103018aca2e4ba0d0e230efa864231c59228f419 (patch) | |
tree | bc31aec77db886f23f26fca97ef341cc15d19a72 /drivers/watchdog/hpwdt.c | |
parent | [WATCHDOG] it8712f_wdt support for 16-bit timeout values, WDIOC_GETSTATUS (diff) | |
download | linux-103018aca2e4ba0d0e230efa864231c59228f419.tar.xz linux-103018aca2e4ba0d0e230efa864231c59228f419.zip |
[WATCHDOG] Fix declaration of struct smbios_entry_point in hpwdt
On my HP DL380 G5 system running a 64-bit kernel, loading the hpwdt
driver causes a crash because the driver attempts to ioremap an
invalid physical address. This is because the driver has an incorrect
definition of the SMBIOS table entry point structure: the table
address is only a 32-bit quantity, and making it a u64 means that the
high-order 32 bits end up containing garbage.
Correcting the structure definition fixes the driver so that it loads
without any problems on my system.
Signed-off-by: Roland Dreier <rolandd@cisco.com>
Acked-by: Thomas Mingarelli <Thomas.Mingarelli@hp.com>
Signed-off-by: Wim Van Sebroeck <wim@iguana.be>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Diffstat (limited to 'drivers/watchdog/hpwdt.c')
-rw-r--r-- | drivers/watchdog/hpwdt.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/watchdog/hpwdt.c b/drivers/watchdog/hpwdt.c index a2e174b09fe7..cd1cc2dacee7 100644 --- a/drivers/watchdog/hpwdt.c +++ b/drivers/watchdog/hpwdt.c @@ -88,7 +88,7 @@ struct smbios_entry_point { u8 intermediate_anchor[5]; u8 intermediate_checksum; u16 table_length; - u64 table_address; + u32 table_address; u16 table_num_structs; u8 bcd_revision; }; |