Skip to content

Commit

Permalink
WATCHDOG: sb_wdog: Fix 32 bit build failure
Browse files Browse the repository at this point in the history
Fixes the following linking problem:
drivers/watchdog/sb_wdog.c:211: undefined reference to `__udivdi3'

This results from reading a 64 bit register, then dividing the value by
1000000.  For 32 bit kernels gcc will use the helper function __udivdi3
from libgcc which the kernel intentionally doesn't provide.

In the read registerbits 23..63 are always zero and only bits 0..22 are
signficant.  So a simple cast to truncate the read value to 32 bits
fixes the issue.

Reported and initial patch by Markos Chandras <markos.chandras@imgtec.com>.

Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
Reported-by: Markos Chandras <markos.chandras@imgtec.com>
  • Loading branch information
Ralf Baechle committed Jun 21, 2013
1 parent 3188835 commit cd6cbde
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion drivers/watchdog/sb_wdog.c
Original file line number Diff line number Diff line change
Expand Up @@ -208,7 +208,7 @@ static long sbwdog_ioctl(struct file *file, unsigned int cmd,
* get the remaining count from the ... count register
* which is 1*8 before the config register
*/
ret = put_user(__raw_readq(user_dog - 8) / 1000000, p);
ret = put_user((u32)__raw_readq(user_dog - 8) / 1000000, p);
break;
}
return ret;
Expand Down

0 comments on commit cd6cbde

Please sign in to comment.