x86: tsc: Remove the fail handling in try_msr_calibrate_tsc()

If either ratio or freq is zero, the return value is zero. There
is no need to create a fail branch and return zero there.

This keeps in sync with Linux kernel commit:
  14bb4e3: x86/tsc_msr: Remove debugging messages

Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
Reviewed-by: Simon Glass <sjg@chromium.org>
master
Bin Meng 7 years ago
parent 0b992e4932
commit fde1801eaa
  1. 9
      drivers/timer/tsc_timer.c

@ -103,9 +103,6 @@ static unsigned long __maybe_unused try_msr_calibrate_tsc(void)
} }
debug("Maximum core-clock to bus-clock ratio: 0x%x\n", ratio); debug("Maximum core-clock to bus-clock ratio: 0x%x\n", ratio);
if (!ratio)
goto fail;
if (freq_desc_tables[cpu_index].msr_plat == 2) { if (freq_desc_tables[cpu_index].msr_plat == 2) {
/* TODO: Figure out how best to deal with this */ /* TODO: Figure out how best to deal with this */
freq = FREQ_100; freq = FREQ_100;
@ -118,18 +115,12 @@ static unsigned long __maybe_unused try_msr_calibrate_tsc(void)
debug("Resolved frequency ID: %u, frequency: %u KHz\n", debug("Resolved frequency ID: %u, frequency: %u KHz\n",
freq_id, freq); freq_id, freq);
} }
if (!freq)
goto fail;
/* TSC frequency = maximum resolved freq * maximum resolved bus ratio */ /* TSC frequency = maximum resolved freq * maximum resolved bus ratio */
res = freq * ratio / 1000; res = freq * ratio / 1000;
debug("TSC runs at %lu MHz\n", res); debug("TSC runs at %lu MHz\n", res);
return res; return res;
fail:
debug("Fast TSC calibration using MSR failed\n");
return 0;
} }
/* /*

Loading…
Cancel
Save