[Powertop] Fwd: [BUG] powertop does not correctly checks for audio codec power management

[Powertop] Fwd: [BUG] powertop does not correctly checks for audio codec power management

[@ 2013-11-27 13:41 UTC (permalink / raw)]

[-- Attachment #1: Type: text/plain, Size: 634 bytes --]

Hi.

I noticed that powertop 2.4 does not correctly checks if the audio
codec power management is already enabled.

The module responsible for audio in my sistem is "snd_hda_intel", and
powertop requires that
"/sys/module/snd_hda_intel/parameters/power_save" is set to 1 to
consider that power save is enabled.

Since that parameter is the power save timeout in seconds (0 disables
power save), it should check if it is greater than 0.

In my setup, "power_save" is set to 10 seconds and
"power_save_controller" is set to "Y", but powertop still thinks that
power save is not enabled for my audio codec.

Thank you.

Re: [Powertop] Fwd: [BUG] powertop does not correctly checks for audio codec power management

[Sergey Senozhatsky]

[-- Attachment #1: Type: text/plain, Size: 950 bytes --]

On (11/27/13 11:41), Otávio Cipriani wrote:
> Hi.
> 
> I noticed that powertop 2.4 does not correctly checks if the audio
> codec power management is already enabled.
> 
> The module responsible for audio in my sistem is "snd_hda_intel", and
> powertop requires that
> "/sys/module/snd_hda_intel/parameters/power_save" is set to 1 to
> consider that power save is enabled.
> 
> Since that parameter is the power save timeout in seconds (0 disables
> power save), it should check if it is greater than 0.
> 
> In my setup, "power_save" is set to 10 seconds and
> "power_save_controller" is set to "Y", but powertop still thinks that
> power save is not enabled for my audio codec.

Hello,

good catch, thank you.
need to think how to handle this.

	-ss

> Thank you.
> _______________________________________________
> PowerTop mailing list
> PowerTop(a)lists.01.org
> https://lists.01.org/mailman/listinfo/powertop
>