Commit 42a5bf50 authored May 17, 2013 by Andy Shevchenko Committed by Zhang Rui May 28, 2013

thermal: cut the spaces when user sets policy



Setting policy results in invalid value error.
	% echo "step_wise" > policy
	% echo: write error: Invalid argument

Need clean up of the buffer which "echo" may add based on the arguments, before
comparing aganist list of governor names.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Reported-by: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
Acked-by: Eduardo Valentin <eduardo.valentin@ti.com>
Tested-by: Eduardo Valentin <eduardo.valentin@ti.com>
Signed-off-by: Zhang Rui <rui.zhang@intel.com>

parent 3db46c93

drivers/thermal/thermal_core.c

+5 −1

Original line number	Diff line number	Diff line
		@@ -33,6 +33,7 @@
		#include <linux/idr.h>
		#include <linux/thermal.h>
		#include <linux/reboot.h>
		#include <linux/string.h>
		#include <net/netlink.h>
		#include <net/genetlink.h>

		@@ -713,10 +714,13 @@ policy_store(struct device dev, struct device_attribute attr,
		int ret = -EINVAL;
		struct thermal_zone_device *tz = to_thermal_zone(dev);
		struct thermal_governor *gov;
		char name[THERMAL_NAME_LENGTH];

		snprintf(name, sizeof(name), "%s", buf);

		mutex_lock(&thermal_governor_lock);

		gov = __find_governor(buf);
		gov = __find_governor(strim(name));
		if (!gov)
		goto exit;