ALSA: hda/realtek – Set default power save node to 0 [Linux 5.2]

ALSA: hda/realtek – Set default power save node to 0 [Linux 5.2]

This Linux kernel change "ALSA: hda/realtek – Set default power save node to 0" is included in the Linux 5.2 release. This change is authored by Kailang Yang <kailang [at] realtek.com> on Thu May 23 14:43:04 2019 +0800. The commit for this change in Linux stable tree is 317d931 (patch).

ALSA: hda/realtek - Set default power save node to 0

I measured power consumption between power_save_node=1 and power_save_node=0.
It's almost the same.
Codec will enter to runtime suspend and suspend.
That pin also will enter to D3. Don't need to enter to D3 by single pin.
So, Disable power_save_node as default. It will avoid more issues.
Windows Driver also has not this option at runtime PM.

Signed-off-by: Kailang Yang <kailang@realtek.com>
Cc: <stable@vger.kernel.org>
Signed-off-by: Takashi Iwai <tiwai@suse.de>

There are 2 lines of Linux source code added/deleted in this change. Code changes to Linux kernel are as follows.

 sound/pci/hda/patch_realtek.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c
index b984bd1..1ca2a83 100644
--- a/sound/pci/hda/patch_realtek.c
+++ b/sound/pci/hda/patch_realtek.c
@@ -7711,7 +7711,7 @@ static int patch_alc269(struct hda_codec *codec)

    spec = codec->spec;
    spec->gen.shared_mic_vref_pin = 0x18;
-   codec->power_save_node = 1;
+   codec->power_save_node = 0;

 #ifdef CONFIG_PM
    codec->patch_ops.suspend = alc269_suspend;

Leave a Reply

Your email address will not be published. Required fields are marked *