cpuidle: power_usage should be declared signed integer
authorBoris Ostrovsky <boris.ostrovsky@amd.com>
Tue, 13 Mar 2012 18:55:10 +0000 (19:55 +0100)
committerLen Brown <len.brown@intel.com>
Fri, 30 Mar 2012 07:23:30 +0000 (03:23 -0400)
power_usage is always assigned a negative value and should be declared
a signed integer

Signed-off-by: Boris Ostrovsky <boris.ostrovsky@amd.com>
Signed-off-by: Len Brown <len.brown@intel.com>
drivers/cpuidle/governors/menu.c
include/linux/cpuidle.h

index 5c17ca1..0633575 100644 (file)
@@ -236,7 +236,7 @@ static int menu_select(struct cpuidle_driver *drv, struct cpuidle_device *dev)
 {
        struct menu_device *data = &__get_cpu_var(menu_devices);
        int latency_req = pm_qos_request(PM_QOS_CPU_DMA_LATENCY);
-       unsigned int power_usage = -1;
+       int power_usage = -1;
        int i;
        int multiplier;
        struct timespec t;
index d557bcd..6c26a3d 100644 (file)
@@ -44,7 +44,7 @@ struct cpuidle_state {
 
        unsigned int    flags;
        unsigned int    exit_latency; /* in US */
-       unsigned int    power_usage; /* in mW */
+       int             power_usage; /* in mW */
        unsigned int    target_residency; /* in US */
        unsigned int    disable;