| Subject | RE: [PATCH 04/11] PM / devfreq: Remove redundant frequency adjustment from governors | From | MyungJoo Ham <> | Date | Mon, 28 May 2018 13:57:44 +0900 |
| |
> The userspace and simpleondemand governor determine a target frequency and > then adjust it according to the df->min/max_freq limits that might have > been set by user space. This adjustment is redundant, it is done in > update_devfreq() for any governor, right after returning from > governor->get_target_freq(). > > Signed-off-by: Matthias Kaehlcke <mka@chromium.org> > --- > drivers/devfreq/governor_simpleondemand.c | 5 ----- > drivers/devfreq/governor_userspace.c | 16 ++++------------ > 2 files changed, 4 insertions(+), 17 deletions(-) >
Yes, indeed. Governors are no longer required to be aware of min/max freq.
Acked-by: MyungJoo Ham <myungjoo.ham@samsung.com>
|