thermal: use integers rather than strings for thermal values

The thermal API currently uses strings to pass values to userspace. This
makes it difficult to use from within the kernel. Change the interface
to use integers and fix up the consumers.

Signed-off-by: Matthew Garrett <mjg@redhat.com>
Acked-by: Zhang Rui <rui.zhang@intel.com>
Acked-by: Thomas Renninger <trenn@suse.de>
Signed-off-by: Len Brown <len.brown@intel.com>

1 files changed, 11 insertions, 9 deletions

diff --git a/drivers/acpi/processor_thermal.c b/drivers/acpi/processor_thermal.c
index b1eb376fae45..0e47e299a9ac 100644
--- a/drivers/acpi/processor_thermal.c
+++ b/drivers/acpi/processor_thermal.c
@@ -373,7 +373,8 @@ static int acpi_processor_max_state(struct acpi_processor *pr)
         return max_state;
 }
 static int
-processor_get_max_state(struct thermal_cooling_device *cdev, char *buf)
+processor_get_max_state(struct thermal_cooling_device *cdev,
+                        unsigned long *state)
 {
         struct acpi_device *device = cdev->devdata;
         struct acpi_processor *pr = acpi_driver_data(device);
@@ -381,28 +382,29 @@ processor_get_max_state(struct thermal_cooling_device *cdev, char *buf)
         if (!device || !pr)
                 return -EINVAL;
 
-        return sprintf(buf, "%d\n", acpi_processor_max_state(pr));
+        *state = acpi_processor_max_state(pr);
+        return 0;
 }
 
 static int
-processor_get_cur_state(struct thermal_cooling_device *cdev, char *buf)
+processor_get_cur_state(struct thermal_cooling_device *cdev,
+                        unsigned long *cur_state)
 {
         struct acpi_device *device = cdev->devdata;
         struct acpi_processor *pr = acpi_driver_data(device);
-        int cur_state;
 
         if (!device || !pr)
                 return -EINVAL;
 
-        cur_state = cpufreq_get_cur_state(pr->id);
+        *cur_state = cpufreq_get_cur_state(pr->id);
         if (pr->flags.throttling)
-                cur_state += pr->throttling.state;
-
-        return sprintf(buf, "%d\n", cur_state);
+                *cur_state += pr->throttling.state;
+        return 0;
 }
 
 static int
-processor_set_cur_state(struct thermal_cooling_device *cdev, unsigned int state)
+processor_set_cur_state(struct thermal_cooling_device *cdev,
+                        unsigned long state)
 {
         struct acpi_device *device = cdev->devdata;
         struct acpi_processor *pr = acpi_driver_data(device);