On 26/02/2021 07:08, Chanwoo Choi wrote: > > > On 21. 2. 24. 오후 11:42, Daniel Lezcano wrote: >> HZ unit conversion macros are available in units.h, use them and >> remove the duplicate definition. >> >> Signed-off-by: Daniel Lezcano <[email protected]> >> Reviewed-by: Christian Eggers <[email protected]> >> Acked-by: Chanwoo Choi <[email protected]> >> --- >> drivers/devfreq/devfreq.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/drivers/devfreq/devfreq.c b/drivers/devfreq/devfreq.c >> index bf3047896e41..b6d63f02d293 100644 >> --- a/drivers/devfreq/devfreq.c >> +++ b/drivers/devfreq/devfreq.c >> @@ -26,6 +26,7 @@ >> #include <linux/hrtimer.h> >> #include <linux/of.h> >> #include <linux/pm_qos.h> >> +#include <linux/units.h> >> #include "governor.h" >> #define CREATE_TRACE_POINTS >> @@ -33,7 +34,6 @@ >> #define IS_SUPPORTED_FLAG(f, name) ((f & DEVFREQ_GOV_FLAG_##name) >> ? true : false) >> #define IS_SUPPORTED_ATTR(f, name) ((f & DEVFREQ_GOV_ATTR_##name) ? >> true : false) >> -#define HZ_PER_KHZ 1000 >> static struct class *devfreq_class; >> static struct dentry *devfreq_debugfs; >> > > I changed the patch title with 'PM /' prefix as following > in order to keep the consistent patch style if there are no any special > objection. > - PM / devfreq: Use HZ macros > > Applied it.
It should not compile. This patch and others depend on 1/9. It would make sense to merge all of them through linux-pm. -- <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook | <http://twitter.com/#!/linaroorg> Twitter | <http://www.linaro.org/linaro-blog/> Blog
