Re: [Powertop] Suspecting incorrect if check during powertop init

--===============5997265425209800920==
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: quoted-printable

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)=
 =3D=3D 0 ||
>             access("/data/local/powertop/saved_parameters.powertop", R_OK=
) =3D=3D 0) {
>                 global_fixed_parameters =3D 1;
>                 global_power_override =3D 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 lo=
ck 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


--===============5997265425209800920==--