PM / devfreq: Fix incorrect type issue.
authorXiaolong Ye <yexl@marvell.com>
Fri, 11 Sep 2015 03:05:23 +0000 (11:05 +0800)
committerMyungJoo Ham <myungjoo.ham@samsung.com>
Fri, 11 Sep 2015 05:23:30 +0000 (14:23 +0900)
time_in_state in struct devfreq is defined as unsigned long, so
devm_kzalloc should use sizeof(unsigned long) as argument instead
of sizeof(unsigned int), otherwise it will cause unexpected result
in 64bit system.

Signed-off-by: Xiaolong Ye <yexl@marvell.com>
Signed-off-by: Kevin Liu <kliu5@marvell.com>
Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>
drivers/devfreq/devfreq.c

index 1f3e5ea5439fc2560e858fc6e1dc073a17a2138f..3927ed9fdbd51f16d765aede8fef14418994ab55 100644 (file)
@@ -482,7 +482,7 @@ struct devfreq *devfreq_add_device(struct device *dev,
                                                devfreq->profile->max_state *
                                                devfreq->profile->max_state,
                                                GFP_KERNEL);
-       devfreq->time_in_state = devm_kzalloc(dev, sizeof(unsigned int) *
+       devfreq->time_in_state = devm_kzalloc(dev, sizeof(unsigned long) *
                                                devfreq->profile->max_state,
                                                GFP_KERNEL);
        devfreq->last_stat_updated = jiffies;