diff options
| author | Lee Leahy <leroy.p.leahy@intel.com> | 2017-03-09 10:45:02 -0800 |
|---|---|---|
| committer | Martin Roth <martinroth@google.com> | 2017-03-13 17:48:31 +0100 |
| commit | a4447535968549136668185dac6854e95beb9930 (patch) | |
| tree | 1eacee3bb6e6b4eb5eb3b1f626b8d3e041445a0e /src/soc/intel/apollolake/tsc_freq.c | |
| parent | 2d154e8213d9b956acfd6638e8988e261980802e (diff) | |
soc/intel/apollolake: Fix issues detected by checkpatch
Fix the following errors and warnings detected by checkpatch.pl:
ERROR: switch and case should be at the same indent
ERROR: do not use assignment in if condition
WARNING: Statements terminations use 1 semicolon
WARNING: unnecessary whitespace before a quoted newline
WARNING: else is not generally useful after a break or return
TEST=Build for reef
Change-Id: I5486936dbf19b066c76179d929660affa1da5f16
Signed-off-by: Lee Leahy <leroy.p.leahy@intel.com>
Reviewed-on: https://review.coreboot.org/18727
Tested-by: build bot (Jenkins)
Reviewed-by: Martin Roth <martinroth@google.com>
Diffstat (limited to 'src/soc/intel/apollolake/tsc_freq.c')
| -rw-r--r-- | src/soc/intel/apollolake/tsc_freq.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/soc/intel/apollolake/tsc_freq.c b/src/soc/intel/apollolake/tsc_freq.c index 3bd709c03d..f91a047d08 100644 --- a/src/soc/intel/apollolake/tsc_freq.c +++ b/src/soc/intel/apollolake/tsc_freq.c @@ -37,7 +37,8 @@ void set_max_freq(void) eax = cpuid_eax(CPUID_LEAF_PM); msr = rdmsr(MSR_IA32_MISC_ENABLES); - if (!(eax &= 0x2) && ((msr.hi & APL_BURST_MODE_DISABLE) == 0)) { + eax &= 0x2; + if ((!eax) && ((msr.hi & APL_BURST_MODE_DISABLE) == 0)) { /* Burst Mode has been factory configured as disabled * and is not available in this physical processor * package. |