Linux Sound subsystem development
 help / color / mirror / Atom feed
* [PATCH 0/2] Fix beep notifications by Thinkpad's ACPI firmware
@ 2024-10-30 17:18 Jarosław Janik
  2024-10-30 17:18 ` [PATCH 1/2] Revert "ALSA: hda/conexant: Mute speakers at suspend / shutdown" Jarosław Janik
                   ` (2 more replies)
  0 siblings, 3 replies; 23+ messages in thread
From: Jarosław Janik @ 2024-10-30 17:18 UTC (permalink / raw)
  To: Takashi Iwai; +Cc: linux-sound, Jarosław Janik

In Comment 17 of the following bug report:
https://bugzilla.suse.com/show_bug.cgi?id=1228269
user tigerlily had complained about "audible blip on shutdown" on a
Thinkpad laptop, that led to a commit 4f61c8fe3520 ("ALSA: hda/conexant:
Mute speakers at suspend / shutdown"), which mutes speakers on system
shutdown or whenever HDA controller is suspended by PM. The
aforementioned "blip" is a feature, not a bug however - Thinkpad's ACPI
firmware uses short beeps / beep sequences as notifications in response
to various events (enter/leave suspend or hibernation, AC power
connect/disconnect, low battery, etc.); these can also be triggered by
writing to /proc/acpi/ibm/beep, see details here:
https://www.kernel.org/doc/html/v5.4/admin-guide/laptops/thinkpad-acpi.html#acpi-sounds-proc-acpi-ibm-beep
The firmware doesn't touch any mixer, so depending on current mixer
settings these beeps can be loud, silent or even muted completely,
whatever has been set by the user.

The patch in question interferes this badly:
- suspend/hibernate/shutdown related events are muted altogether because
  HDA controller is in suspend mode when they occur (or snd_hda_intel
  module has been closed in the event of system shutdown)
- other events work "randomly", e.g. you can hear AC plug/unplug beep
  if something is playing audio at the moment, otherwise the HDA
  controller is likely in suspend mode, so you can't hear anything
  (unless you disabled PM in snd_hda_intel module).

That said - let's revert this; this is what 1st commit does, the 2nd
commit is somewhat optional - it removes helpers introduced to implement
this muting, as they are no longer used.

PS. Don't forget to have this backported to LTS kernels, please.

Jarosław Janik (2):
  Revert "ALSA: hda/conexant: Mute speakers at suspend / shutdown"
  Revert "ALSA: hda/generic: Add a helper to mute speakers at
    suspend/shutdown"

 sound/pci/hda/hda_generic.c    | 63 ----------------------------------
 sound/pci/hda/hda_generic.h    |  1 -
 sound/pci/hda/patch_conexant.c |  2 --
 3 files changed, 66 deletions(-)

-- 
2.39.5


^ permalink raw reply	[flat|nested] 23+ messages in thread

end of thread, other threads:[~2024-11-21 15:39 UTC | newest]

Thread overview: 23+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-10-30 17:18 [PATCH 0/2] Fix beep notifications by Thinkpad's ACPI firmware Jarosław Janik
2024-10-30 17:18 ` [PATCH 1/2] Revert "ALSA: hda/conexant: Mute speakers at suspend / shutdown" Jarosław Janik
2024-10-30 17:18 ` [PATCH 2/2] Revert "ALSA: hda/generic: Add a helper to mute speakers at suspend/shutdown" Jarosław Janik
2024-10-31  9:15 ` [PATCH 0/2] Fix beep notifications by Thinkpad's ACPI firmware Takashi Iwai
2024-10-31 16:12   ` Jarosław Janik
2024-10-31 16:21     ` Takashi Iwai
2024-10-31 16:43       ` Jarosław Janik
2024-11-01  8:18         ` Takashi Iwai
2024-11-01 23:15           ` Jarosław Janik
2024-11-02  8:27             ` Takashi Iwai
2024-11-02 22:45               ` Jarosław Janik
2024-11-08 14:17                 ` Takashi Iwai
2024-11-14  0:04                   ` Jarosław Janik
2024-11-14 13:42                     ` Takashi Iwai
2024-11-15  4:34                       ` Jarosław Janik
2024-11-19 15:21                         ` Takashi Iwai
2024-11-20  1:29                           ` Jarosław Janik
2024-11-20  7:04                             ` Takashi Iwai
2024-11-20 15:00                               ` Jarosław Janik
2024-11-20 15:09                                 ` Takashi Iwai
2024-11-21  1:15                                   ` Jarosław Janik
2024-11-21 15:39                                     ` Takashi Iwai
2024-10-31 18:53   ` Thorsten Leemhuis

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox