summaryrefslogtreecommitdiffstats
path: root/tools
diff options
context:
space:
mode:
authorHimangi Saraogi <himangi774@gmail.com>2014-07-29 18:12:18 +0200
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>2014-07-30 01:57:12 +0200
commit97fa1c5ca680bdee2c650e3aadf2a839b92f3f0e (patch)
tree4efcc07f960082dd85cf24dcb4894dde73558326 /tools
parent103f1790715e9a45f069ae1028331be0be1e3bce (diff)
downloadop-kernel-dev-97fa1c5ca680bdee2c650e3aadf2a839b92f3f0e.zip
op-kernel-dev-97fa1c5ca680bdee2c650e3aadf2a839b92f3f0e.tar.gz
cpupower: mperf monitor: Correct use of ! and &
In commit ae91d60ba88ef0bdb1b5e9b2363bd52fc45d2af7, a bug was fixed that involved converting !x & y to !(x & y). The code below shows the same pattern, and thus should perhaps be fixed in the same way. The Coccinelle semantic patch that makes this change is as follows: // <smpl> @@ expression E1,E2; @@ ( !E1 & !E2 | - !E1 & E2 + !(E1 & E2) ) // </smpl> Signed-off-by: Himangi Saraogi <himangi774@gmail.com> Signed-off-by: Thomas Renninger <trenn@suse.de> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Diffstat (limited to 'tools')
-rw-r--r--tools/power/cpupower/utils/idle_monitor/mperf_monitor.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/tools/power/cpupower/utils/idle_monitor/mperf_monitor.c b/tools/power/cpupower/utils/idle_monitor/mperf_monitor.c
index 5650ab5..90a8c4f 100644
--- a/tools/power/cpupower/utils/idle_monitor/mperf_monitor.c
+++ b/tools/power/cpupower/utils/idle_monitor/mperf_monitor.c
@@ -237,7 +237,7 @@ static int init_maxfreq_mode(void)
unsigned long long hwcr;
unsigned long min;
- if (!cpupower_cpu_info.caps & CPUPOWER_CAP_INV_TSC)
+ if (!(cpupower_cpu_info.caps & CPUPOWER_CAP_INV_TSC))
goto use_sysfs;
if (cpupower_cpu_info.vendor == X86_VENDOR_AMD) {
OpenPOWER on IntegriCloud