summaryrefslogtreecommitdiffstats
path: root/tools/power
diff options
context:
space:
mode:
authorSrinivas Pandruvada <srinivas.pandruvada@linux.intel.com>2019-10-10 13:29:36 -0700
committerAndy Shevchenko <andriy.shevchenko@linux.intel.com>2019-10-15 11:02:28 +0300
commitabd120e3bdf3dd72ba1ed9ac077a861e0e3dc43a (patch)
tree98f2f9892178f18b8889745593166dfc2aa24598 /tools/power
parentfd13c8622a5ad4f7317b64de4f6aa2de1962220e (diff)
downloadlinux-abd120e3bdf3dd72ba1ed9ac077a861e0e3dc43a.tar.gz
linux-abd120e3bdf3dd72ba1ed9ac077a861e0e3dc43a.tar.bz2
linux-abd120e3bdf3dd72ba1ed9ac077a861e0e3dc43a.zip
tools/power/x86/intel-speed-select: Remove warning for unused result
Fix warning for: isst-config.c: In function ‘set_cpu_online_offline’: isst-config.c:221:3: warning: ignoring return value of ‘write’, declared with attribute warn_unused_result [-Wunused-result] write(fd, "1\n", 2); Signed-off-by: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com> Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Diffstat (limited to 'tools/power')
-rw-r--r--tools/power/x86/intel-speed-select/isst-config.c9
1 files changed, 6 insertions, 3 deletions
diff --git a/tools/power/x86/intel-speed-select/isst-config.c b/tools/power/x86/intel-speed-select/isst-config.c
index 2a9890c8395a..21fcfe621d3a 100644
--- a/tools/power/x86/intel-speed-select/isst-config.c
+++ b/tools/power/x86/intel-speed-select/isst-config.c
@@ -169,7 +169,7 @@ int get_topo_max_cpus(void)
static void set_cpu_online_offline(int cpu, int state)
{
char buffer[128];
- int fd;
+ int fd, ret;
snprintf(buffer, sizeof(buffer),
"/sys/devices/system/cpu/cpu%d/online", cpu);
@@ -179,9 +179,12 @@ static void set_cpu_online_offline(int cpu, int state)
err(-1, "%s open failed", buffer);
if (state)
- write(fd, "1\n", 2);
+ ret = write(fd, "1\n", 2);
else
- write(fd, "0\n", 2);
+ ret = write(fd, "0\n", 2);
+
+ if (ret == -1)
+ perror("Online/Offline: Operation failed\n");
close(fd);
}