Re: Silencing plops / click sounds on start / stop of playing audio when snd_hda_intel.power_save=1

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Dne 15.12.2017 v 13:35 Hans de Goede napsal(a):
> Hi All,
> 
> For Fedora 28 we plan to ship the kernel with snd_hda_intel.power_save
> defaulting to 1:
> 
> https://fedoraproject.org/wiki/Changes/ImprovedLaptopBatteryLife
> 
> We've had an update for F27 in updates-testing for a while which made
> the same change for F27, but we've received several bug reports
> about plops / click sounds on start / stop of playing audio:
> 
> https://bugzilla.redhat.com/show_bug.cgi?id=1525104
> 
> I know others have probably already tried fixing this, but I'm
> wondering if there are any ideas how to fix this / things to try?
> 
> Ideas of my own:
> 
> 1) Mute the speakers / headphone output in the mixer before
> activating power-saving. Question how hard would it be to
> make this happen?
> 
> 2) Put a delay between various steps involved in turning
> things off / on.

Hi,

I am a bit skeptic about those proposals. I think that the drivers
should do this job, because some hardware variants have different
requirements for the proper power-off/on sequence.

The problem appears like an improper voltage change and the HDA driver
has already a lot of code which is trying to eliminate these pops (but
as you can see, it is not sufficient).

Honestly, I don't know if it's good to keep this "hidden" to users or
enable the power save feature by default and let users to report this,
so we can look into this problem more when the affected hardware is
identified.

					Jaroslav

--
Jaroslav Kysela <perex@xxxxxxxx>
Linux Sound Maintainer; ALSA Project; Red Hat, Inc.
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel



[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux