Re: [PATCH v4 7/8] PM / devfreq: Define the constant governor name

[MyungJoo Ham]

On Fri, Oct 13, 2017 at 4:48 PM, Chanwoo Choi <cw00.choi@xxxxxxxxxxx> wrote:
> Prior to that, the devfreq device uses the governor name when adding
> the itself. In order to prevent the mistake used the wrong governor name,
> this patch defines the governor name as a constant and then uses them
> instead of using the string directly.
>
> Signed-off-by: Chanwoo Choi <cw00.choi@xxxxxxxxxxx>
> Cc: Kukjin Kim <kgene@xxxxxxxxxx>
> Cc: Krzysztof Kozlowski <krzk@xxxxxxxxxx>
> Cc: linux-samsung-soc@xxxxxxxxxxxxxxx
> Cc: linux-arm-kernel@xxxxxxxxxxxxxxxxxxx
> Cc: linux-kernel@xxxxxxxxxxxxxxx

Acked-by: MyungJoo Ham <myungjoo.ham@xxxxxxxxxxx>

> ---
>  drivers/devfreq/exynos-bus.c              | 5 +++--
>  drivers/devfreq/governor_passive.c        | 2 +-
>  drivers/devfreq/governor_performance.c    | 2 +-
>  drivers/devfreq/governor_powersave.c      | 2 +-
>  drivers/devfreq/governor_simpleondemand.c | 2 +-
>  drivers/devfreq/governor_userspace.c      | 2 +-
>  drivers/devfreq/rk3399_dmc.c              | 2 +-
>  include/linux/devfreq.h                   | 7 +++++++
>  8 files changed, 16 insertions(+), 8 deletions(-)
>
[]


-- 
MyungJoo Ham, Ph.D.
S/W Center, Samsung Electronics