From mboxrd@z Thu Jan  1 00:00:00 1970
Content-Type: multipart/mixed; boundary="===============8939330901871155404=="
MIME-Version: 1.0
From: Bram Bouwens <bram at bouwens.org>
Subject: Re: [Powertop] [PATCH v2] conditionally disable pci if not supported
Date: Mon, 25 Jun 2012 19:00:09 +0200
Message-ID: <4FE89919.6030803@bouwens.org>
In-Reply-To: CA+Z25wWdE=FYXpTqcyebk4hwJpjvz9BodfrH__Tn1D47jgEj1Q@mail.gmail.com
To: powertop@lists.01.org
List-ID: <powertop@lists.01.org>

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

On 06/25/2012 06:34 PM, Rajagopal Venkat wrote:
>
>
> On 25 June 2012 21:34, Arjan van de Ven <arjan(a)linux.intel.com <mailto:=
arjan(a)linux.intel.com>> wrote:
>
>     On 6/25/2012 9:00 AM, Rajagopal Venkat wrote:
>
>     <no commit message .. why? one sentence is cheap>
>
> Ok. Taken care.
>
>
>      >
>      > diff --git a/src/lib.h b/src/lib.h
>      > index 588c023..d354151 100644
>      > --- a/src/lib.h
>      > +++ b/src/lib.h
>      > @@ -30,6 +30,10 @@
>      >  #endif
>      >  #include <stdint.h>
>      >
>      > +#if HAVE_CONFIG_H
>      > +#include "config.h"
>      > +#endif
>      > +
>      >  #ifndef DISABLE_I18N
>      >  #define _(STRING)    gettext(STRING)
>      >  #else
>      >
>
>
>
>     ok this chunk I do not understand;
>     the rest of your patch looks very reasonable.
>     why is config.h inside an if ?
>
>
> config.h is auto generated by autoconf and is not applicable for Android =
build.
> Yes. it should be wrapped in ifdef.
>

You might indicate this in a comment in the code.

Other than that, I like the idea of being able to build powertop on my Beag=
leBone!

Bram

--===============8939330901871155404==--