On 12/24/2010 1:09 AM, Török Edwin wrote:
> On 2010-12-24 10:12, Andreas Mohr wrote:
>> Hi,
>>
>> On Thu, Dec 23, 2010 at 07:07:39PM -0800, Arjan van de Ven wrote:
>>> I'd like to ask everyone who's interested to give it a spin and give
>>> feedback on how you like the new design,
>> I don't like it since I cannot even see it ;)
>>
>> cc -Wall -O2 -g -fno-omit-frame-pointer -fstack-protector -Wshadow -Wformat 
>> -DCONFIG_LIBNL20   -Wall -O2 -g -fno-omit-frame-pointer  -c -o tuning/iw.o 
>> tuning/iw.c
>> tuning/iw.c: In function âset_power_saveâ:
>> tuning/iw.c:131: error: storage size of âps_stateâ isnât known
>> tuning/iw.c:133: error: âNL80211_PS_DISABLEDâ undeclared (first use in 
>> this function)
> Yeah I got the same error.
> It looks like powertop wants the includes from a very recent kernel.
> So I added -I /home/edwin/linux-2.6/linux-2.6-stable/include to CPPFLAGS
> in the Makefile and then powertop compiled for me.
>
> Maybe powertop should provide a way for us to specify the include dir of
> the kernel we are actually using (and not use the system wide one
> installed by an old distro package).

I just added a private copy of the header to the git tree; not worth the 
hassle of dealing with the old debian case in any other way.
> BTW the make install target is missing some $, it installs stuff into a
> directory called {BINDIR}.

thanks for that report as well; just fixed this too.


_______________________________________________
Power mailing list
[email protected]
http://www.bughost.org/mailman/listinfo/power

Reply via email to