diff options
| author | Bryan O'Donoghue <[email protected]> | 2016-09-26 18:01:23 +0100 |
|---|---|---|
| committer | Greg Kroah-Hartman <[email protected]> | 2016-09-27 12:59:51 +0200 |
| commit | 659849cdc065c3620e49f9a7dbc384669f4592ab (patch) | |
| tree | c8703817a3522f0e15db3b28b39b61e48625e68c | |
| parent | 3d7f3588553a9ee392aa1abb5a3e8fd4b1ba8f5e (diff) | |
staging: greybus/timesync: avoid divide by zero on X86 Qemu
A system configured without CONFIG_CPUFREQ will return 0 for cpufreq_get().
greybus-timesync can subsequently then do a divide-by-zero as result. This
patch fixes by checking for a zero return value from cpufreq_get() and
setting to a default value of 19.2MHz.
Reported-by: Rui Miguel Silva <[email protected]>
Signed-off-by: Bryan O'Donoghue <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
| -rw-r--r-- | drivers/staging/greybus/timesync_platform.c | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/drivers/staging/greybus/timesync_platform.c b/drivers/staging/greybus/timesync_platform.c index 50e8883f932f..113f3d6c4b3a 100644 --- a/drivers/staging/greybus/timesync_platform.c +++ b/drivers/staging/greybus/timesync_platform.c @@ -20,6 +20,8 @@ #include "greybus.h" #include "arche_platform.h" +#define DEFAULT_FRAMETIME_CLOCK_HZ 19200000 + static u32 gb_timesync_clock_frequency; int (*arche_platform_change_state_cb)(enum arche_platform_state state, struct gb_timesync_svc *pdata); @@ -32,8 +34,11 @@ u64 gb_timesync_platform_get_counter(void) u32 gb_timesync_platform_get_clock_rate(void) { - if (unlikely(!gb_timesync_clock_frequency)) - return cpufreq_get(0); + if (unlikely(!gb_timesync_clock_frequency)) { + gb_timesync_clock_frequency = cpufreq_get(0); + if (!gb_timesync_clock_frequency) + gb_timesync_clock_frequency = DEFAULT_FRAMETIME_CLOCK_HZ; + } return gb_timesync_clock_frequency; } |