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 5c17ca112fc2404cd30e909efb57c923b1117ff8..06335756ea14269bcaaeebc14c0089939a49fff0 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 d557bcd0ada78a17b1ac5b073eb642160d2ed339..6c26a3da0e03173b29ce1535c9833537f85a6786 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;