On 07.12.2012 18:11, Stefan Monnier wrote:
>>> - always set max_performance.
>>> - always set min_power.
>> What would be the point of that? If you don't want pm-utils to change
>> the alpm state, set it outside of pm-utils and use SATA_ALPM_ENABLE=false.
> 
> When SATA_ALPM_ENABLE=false you get "always set max_performance" because
> sata_alpm does:

Ok, got it.

Maybe we should use something like this:
case $1 in
    true) [ "$SATA_ALPM_ENABLE" = true ] && set_sata_alpm min_power;;
    false) [ "$SATA_ALPM_ENABLE" = true ] && set_sata_alpm max_performance;;


This somehow seems more logical to me and would be more consistent with
/usr/lib/pm-utils/power.d/intel-audio-powersave,
/usr/lib/pm-utils/power.d/pci_devices and
/usr/lib/pm-utils/power.d/usb_bluetooth


Michael
-- 
Why is it that all of the instruments seeking intelligent life in the
universe are pointed away from Earth?

Attachment: signature.asc
Description: OpenPGP digital signature

Reply via email to