Messages in this thread | | | Date | Fri, 26 Jun 2020 08:23:46 +0530 | From | Viresh Kumar <> | Subject | Re: [PATCH v2 2/2] cpufreq: Specify default governor on command line |
| |
On 23-06-20, 15:21, Quentin Perret wrote: > @@ -2789,7 +2796,13 @@ static int __init cpufreq_core_init(void) > cpufreq_global_kobject = kobject_create_and_add("cpufreq", &cpu_subsys.dev_root->kobj); > BUG_ON(!cpufreq_global_kobject); > > + mutex_lock(&cpufreq_governor_mutex); > + if (!default_governor)
Also is this check really required ? The pointer will always be NULL at this point, isn't it ?
> + default_governor = cpufreq_default_governor(); > + mutex_unlock(&cpufreq_governor_mutex); > + > return 0; > } > module_param(off, int, 0444); > +module_param_string(default_governor, cpufreq_param_governor, CPUFREQ_NAME_LEN, 0444); > core_initcall(cpufreq_core_init); > -- > 2.27.0.111.gc72c7da667-goog
-- viresh
| |