CONFIG_SND_AC97_POWER_SAVE should be enabled in kernel 2.6.22

Bug #116679 reported by Brian Neltner
6
Affects Status Importance Assigned to Milestone
linux-source-2.6.22 (Ubuntu)
Fix Released
Low
Ben Collins

Bug Description

Binary package hint: linux-image-2.6.22-5-generic

This has probably been reported a half dozen times (although I don't see it anywhere), but running powertop suggests that (on a dell latitude D620):

Enable the CONFIG_SND_AC97_POWER_SAVE kernel configuration option.
This option will automatically power down your sound codec when not in use,
and can save approximately half a Watt of power.

Looking at /boot/config-2.6.22-5-generic shows that it is indeed, not enabled.

Simple change for the next image release!

Thanks,

Changed in linux-source-2.6.22:
assignee: nobody → ben-collins
importance: Undecided → Low
status: Unconfirmed → Fix Committed
Revision history for this message
Nicolò Chieffo (yelo3) wrote :

fix committed means that the kernel 2.6.22-6 will be compiled with this option?

Revision history for this message
Ben Collins (ben-collins) wrote :

No, next upload.

Revision history for this message
Ben Collins (ben-collins) wrote :
Download full text (19.7 KiB)

linux-source-2.6.22 (2.6.22-6.13) gutsy; urgency=low

  [Ben Collins]

  * Bump ABI
  * build/scripts: Remove all remnants of debconf from control scripts
  * build/config: Re-enable paravirt/vmi
  * build/config: Build ide-core as a module
  * i386/x86_64: Allow disabling the putstr's from compressed boot wrapper
  * PM: Do not require dev spew to get PM_DEBUG
  * RTC: Ratelimit "lost interrupts" message
  * UNUSUAL_DEV: Sync up some reported devices from Ubuntu
  * build/d-i: Include ide-core in storage-core udeb, not that it's modular
  * build/d-i: Make ide-modules depend on storage-code-modules
  * build/config: Enable CONFIG_TIMER_STATS on x86_64.
  * build/config: Disable CONFIG_RTC_DRV_CMOS
  * build/config: Enable TIMER_STATS everywhere.
  * build/config: Enable SND_AC97_POWER_SAVE
    - LP: #116679
  * kmod: Improve call_usermodehelper_pipe to handle data close
  * coredump: Convert to new call_usermodehelper_pipe symantics
  * PPC: Only set hwif stuff when ide-core is non-modular
  * PPC/MEDIABAY: Export some functions for modular ide-core/ppc-ide

  [Colin Watson]

  * Move isofs to storage-core-modules udeb from fs-core-modules.

  [Upstream Kernel Changes]

  * Input: logips2pp - add type 72 (PS/2 TrackMan Marble)
  * Input: adbhid - do not access input_dev->private directly
  * sh: Shut up compiler warnings in __do_page_fault().
  * sh: Fix up psw build rules for r7780rp.
  * sh: Kill off pmb slab cache destructor.
  * sh: landisk: rtc-rs5c313 support.
  * sh: landisk: Header cleanups.
  * input: hp680_ts compile fixes.
  * [ARM] 4375/1: sharpsl_pm: Fix compile warnings
  * [ARM] 4376/1: Selects GENERIC_GPIO for ARCH_IXP4XX in Kconfig
  * [ARM] 4378/1: KS8695: Serial driver fix
  * [ARM] Remove Integrator/CP SMP platform support
  * [ARM] 4382/1: iop13xx: fix msi support
  * [ARM] 4383/1: iop: fix usage of '__init' and 'inline' in iop files
  * [ARM] 4384/1: S3C2412/13 SPI registers offset correction
  * [ARM] Update ARM syscalls
  * [ARM] Silence OMAP kernel configuration warning
  * [ARM] gic: Fix gic cascade irq handling
  * [ARM] integrator: fix pci_v3 compile error with DEBUG_LL
  * [ARM] ARMv6: add CPU_HAS_ASID configuration
  * [CRYPTO] padlock: Make CRYPTO_DEV_PADLOCK a tristate again
  * [CRYPTO] tcrypt: Add missing error check
  * eventfd use waitqueue lock ...
  * timerfd use waitqueue lock ...
  * [IA64] Fix bogus messages about system calls not implemented.
  * [IA64] Yet another section mismatch warning
  * Fix roundup_pow_of_two(1)
  * Further update of the i386 boot documentation
  * cciss: Fix pci_driver.shutdown while device is still active
  * Linux v2.6.22-rc2
  * [CRYPTO] api: Read module pointer before freeing algorithm
  * powerpc: Fix the MODALIAS generation in modpost for of devices
  * kbuild: include limits.h in sumversion.c for PATH_MAX
  * kconfig: search harder for curses library in check-lxdialog.sh
  * kbuild: make modpost section warnings clearer
  * kbuild: make better section mismatch reports on i386, arm and mips
  * kbuild: add "Section mismatch" warning whitelist for powerpc
  * all-archs: consolidate .text section definition in asm-generic
  * all-archs: consolidate .data section definitio...

Changed in linux-source-2.6.22:
status: Fix Committed → Fix Released
Revision history for this message
Nicolò Chieffo (yelo3) wrote :

Ok now the option is enabled, but powertop suggests this:

Suggestion: enable AC97 powersave mode by executing the following command:
   echo 1 > /sys/module/snd_ac97_codec/parameters/power_save
or by passing power_save=1 as module parameter.

So the powersaving features are not enabled...

Revision history for this message
Ben Collins (ben-collins) wrote : Re: [Bug 116679] Re: CONFIG_SND_AC97_POWER_SAVE should be enabled in kernel 2.6.22

On Wed, 2007-06-06 at 21:16 +0000, Nicolò Chieffo wrote:
> Ok now the option is enabled, but powertop suggests this:
>
> Suggestion: enable AC97 powersave mode by executing the following command:
> echo 1 > /sys/module/snd_ac97_codec/parameters/power_save
> or by passing power_save=1 as module parameter.
>
> So the powersaving features are not enabled...

Get some wide spread testing on this feature being enabled, and we might
do so by default via a module-init-tools modprobe.d addition.

--
Ubuntu : http://www.ubuntu.com/
Linux1394: http://wiki.linux1394.org/

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.