On 12 November 2016 at 03:46, Rafael J. Wysocki <raf...@kernel.org> wrote:
>> +static void sugov_work(struct kthread_work *work)
>> {
>> - struct sugov_policy *sg_policy = container_of(work, struct
>> sugov_policy, work);
>> + struct sugov_policy *sg_policy =
>> + container_of(work, struct sugov_policy, work);
>
> Why this change?
Mistake ..
>> static struct sugov_policy *sugov_policy_alloc(struct cpufreq_policy
>> *policy)
>> {
>> struct sugov_policy *sg_policy;
>> + struct task_struct *thread;
>> + struct sched_param param = { .sched_priority = 50 };
>
> I'd define a symbol for the 50. It's just one extra line of code ...
Sure.
As I asked in the cover letter, will you be fine if I send the same patch
for ondemand/conservative governors ?
