[bug#68309] [PATCH] gnu: Add powertop-service-type.

[Ian Eure <ian@retrospec.tv>]

Mathieu Othacehe <othacehe@gnu.org> writes:

> Hello,
>
> Looks good,
>
>> diff --git a/gnu/packages/linux.scm b/gnu/packages/linux.scm
>> index 0fe64bacf3..c53d8f82bd 100644
>> --- a/gnu/packages/linux.scm
>> +++ b/gnu/packages/linux.scm
>> @@ -3634,7 +3634,7 @@ (define-public powertop
>>      (inputs
>>       (list kmod libnl ncurses pciutils zlib))
>>      (home-page "https://01.org/powertop/")
>> -    (synopsis "Analyze power consumption on Intel-based 
>> laptops")
>> +    (synopsis "Analyze power consumption on x86-based 
>> laptops")
>
> This should be part of a separate commit.
>

Okay, I’ll send a patch series in a bit.  I was hoping it could be 
one commit, because it’s related, and sending a patch series is 
very annoying.


>> +                 (zero? (system* #$(file-append powertop 
>> "/sbin/powertop")
>> +                                 "--auto-tune"))))))))
>
> If the laptop booted with power cord plugged, but then goes on 
> battery,
> does this need to be run again?
>

No, the settings persist across power plug/unplug events.