On 8/7/2012 3:54 AM, Rajagopal Venkat wrote: > Hi, > > Recently a patch[1] is applied to remove boardname support from powertop. > This commit adds few lines of code to powertop_init(), to initialize two > global variables. I suspect, if condition is not correct and is suppose > to be as follows. Is that intentional? > > if (access("/var/cache/powertop/saved_parameters.powertop", R_OK) == 0 || > access("/data/local/powertop/saved_parameters.powertop", R_OK) == 0) { > global_fixed_parameters = 1; > global_power_override = 1; > } > both are wrong the whole snippet that assigns these two globals needs to go. the logic used to be that you could do a boardname override, which would lock system parameters in place and stop the learning logic (say, if provided by a manufacturer) with the boardname stuff gone, those parameters shouldn't be set like this _______________________________________________ PowerTop mailing list PowerTop(a)lists.01.org https://lists.01.org/mailman/listinfo/powertop