Re: [PATCH] cros_ec_spi: Even though we're RT priority, don't bump cpu freq

From: Quentin Perret
Date: Thu Jun 11 2020 - 07:03:17 EST


Hi Doug,

On Wednesday 10 Jun 2020 at 15:18:43 (-0700), Douglas Anderson wrote:
> The cros_ec_spi driver is realtime priority so that it doesn't get
> preempted by other taks while it's talking to the EC but overall it
> really doesn't need lots of compute power. Unfortunately, by default,
> the kernel assumes that all realtime tasks should cause the cpufreq to
> jump to max and burn through power to get things done as quickly as
> possible. That's just not the correct behavior for cros_ec_spi.

Is this specific to this driver, or something you would want applied
more globally to all RT tasks in ChromeOS (which is what we'd like to
have in Android for instance)?

IOW, how do you feel about 20200511154053.7822-1-qais.yousef@xxxxxxx ?

Otherwise, the patch looks good to me.

Thanks,
Quentin