linux/drivers/devfreq/devfreq.c, branch v4.9

PM / devfreq: Skip status update on uninitialized previous_freq

2016-10-10T22:01:20Z

In case devfreq->previous_freq is still uninitialized in devfreq_update_status(), i.e. it has value '0', the lookups in that function fail, eventually leading to some error message: [ 3.041292] devfreq bus_dmc: Couldn't update frequency transition information. Just skip the statup update in this situation. Signed-off-by: Tobias Jakobi Acked-by: MyungJoo Ham Signed-off-by: Rafael J. Wysocki

PM / devfreq: Add proper locking around list_del()

2016-10-10T22:00:20Z

Use devfreq_list_lock around list_del() to prevent list corruption. Signed-off-by: Axel Lin Acked-by: MyungJoo Ham Signed-off-by: Rafael J. Wysocki

PM / devfreq: add missing of_node_put after calling of_parse_phandle

2016-07-06T04:11:24Z

of_node_put needs to be called when the device node which is got from of_parse_phandle has finished using. Cc: MyungJoo Ham Cc: Kyungmin Park Signed-off-by: Peter Chen Signed-off-by: MyungJoo Ham

PM / devfreq: make devfreq explicitly non-modular

2016-07-06T04:11:22Z

The Kconfig currently controlling compilation of this code is: menuconfig PM_DEVFREQ bool "Generic Dynamic Voltage and Frequency Scaling (DVFS) support" ...meaning that it currently is not being built as a module by anyone. Lets remove the modular code that is essentially orphaned, so that when reading the driver there is no doubt it is builtin-only. This code wasn't using module_init, so we don't need to be concerned with altering the initcall level here. We also delete the MODULE_LICENSE tag etc. since all that information is already contained at the top of the file in the comments. We don't replace module.h with init.h since the file already has that. But we do add export.h since this file does export some symbols. Signed-off-by: Paul Gortmaker Reviewed-by: Chanwoo Choi Signed-off-by: MyungJoo Ham

PM / devfreq: Send the DEVFREQ_POSTCHANGE notification when target() is failed

2016-06-23T21:15:12Z

This patch sends the DEVFREQ_POSTCHANGE notification when devfreq->profile->targer() is failed. The PRECHANGE/POSTCHANGE should be paired. Fixes: 0fe3a66410a3 (PM / devfreq: Add new DEVFREQ_TRANSITION_NOTIFIER notifier) Reported-by: Lin Huang Signed-off-by: Chanwoo Choi Signed-off-by: Rafael J. Wysocki

PM / devfreq: fix initialization of current frequency in last status

2016-06-22T04:53:00Z

Some systems need current frequency from last_status for calculation but it is zeroed during initialization. When the device starts there is no history, but we can assume that the last frequency was the same as the initial frequency (which is also used in 'previous_freq'). The log shows the result of this misinterpreted value. [ 2.042847] ... Failed to get voltage for frequency 0: -34 Signed-off-by: Lukasz Luba Reviewed-by: Javi Merino Signed-off-by: MyungJoo Ham

PM / devfreq: remove double put_device

2016-06-22T04:52:52Z

When device_register() returns with error, it has already done put_device() on the input device pointer. Signed-off-by: MyungJoo Ham

PM / devfreq: fix double call put_device

2016-06-22T04:52:43Z

1295 */ 1296 void device_unregister(struct device *dev) 1297 { 1298 pr_debug("device: '%s': %s\n", dev_name(dev), __func__); 1299 device_del(dev); 1300 put_device(dev); 1301 } 1302 EXPORT_SYMBOL_GPL(device_unregister); 1303 device_unregister is called put_device, there is no need to call put_device(&devfreq->dev) again. Signed-off-by: Cai Zhiyong Reviewed-by: Chanwoo Choi Signed-off-by: MyungJoo Ham

PM / devfreq: fix duplicated kfree on devfreq pointer

2016-06-22T04:52:40Z

device_unregister() calls kfree already. Signed-off-by: MyungJoo Ham

PM / devfreq: devm_kzalloc to have dev pointer more precisely

2016-06-22T04:51:08Z

devm_kzalloc of devfreq's statistics data structure has been using its parent device as the dev allocated for. If a device's devfreq is disabled in run-time, such allocated memory won't be freed. Desginating more precisely with the devfreq device pointer fixes the issue. Signed-off-by: MyungJoo Ham