lkml.org 
[lkml]   [2019]   [Aug]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 4.19 16/85] ALSA: hda - Add a generic reboot_notify
    Date
    From: Hui Wang <hui.wang@canonical.com>

    commit 871b9066027702e6e6589da0e1edd3b7dede7205 upstream.

    Make codec enter D3 before rebooting or poweroff can fix the noise
    issue on some laptops. And in theory it is harmless for all codecs
    to enter D3 before rebooting or poweroff, let us add a generic
    reboot_notify, then realtek and conexant drivers can call this
    function.

    Cc: stable@vger.kernel.org
    Signed-off-by: Hui Wang <hui.wang@canonical.com>
    Signed-off-by: Takashi Iwai <tiwai@suse.de>
    Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

    ---
    sound/pci/hda/hda_generic.c | 19 +++++++++++++++++++
    sound/pci/hda/hda_generic.h | 1 +
    sound/pci/hda/patch_conexant.c | 6 +-----
    sound/pci/hda/patch_realtek.c | 11 +----------
    4 files changed, 22 insertions(+), 15 deletions(-)

    --- a/sound/pci/hda/hda_generic.c
    +++ b/sound/pci/hda/hda_generic.c
    @@ -6033,6 +6033,24 @@ void snd_hda_gen_free(struct hda_codec *
    }
    EXPORT_SYMBOL_GPL(snd_hda_gen_free);

    +/**
    + * snd_hda_gen_reboot_notify - Make codec enter D3 before rebooting
    + * @codec: the HDA codec
    + *
    + * This can be put as patch_ops reboot_notify function.
    + */
    +void snd_hda_gen_reboot_notify(struct hda_codec *codec)
    +{
    + /* Make the codec enter D3 to avoid spurious noises from the internal
    + * speaker during (and after) reboot
    + */
    + snd_hda_codec_set_power_to_all(codec, codec->core.afg, AC_PWRST_D3);
    + snd_hda_codec_write(codec, codec->core.afg, 0,
    + AC_VERB_SET_POWER_STATE, AC_PWRST_D3);
    + msleep(10);
    +}
    +EXPORT_SYMBOL_GPL(snd_hda_gen_reboot_notify);
    +
    #ifdef CONFIG_PM
    /**
    * snd_hda_gen_check_power_status - check the loopback power save state
    @@ -6060,6 +6078,7 @@ static const struct hda_codec_ops generi
    .init = snd_hda_gen_init,
    .free = snd_hda_gen_free,
    .unsol_event = snd_hda_jack_unsol_event,
    + .reboot_notify = snd_hda_gen_reboot_notify,
    #ifdef CONFIG_PM
    .check_power_status = snd_hda_gen_check_power_status,
    #endif
    --- a/sound/pci/hda/hda_generic.h
    +++ b/sound/pci/hda/hda_generic.h
    @@ -336,6 +336,7 @@ int snd_hda_gen_parse_auto_config(struct
    struct auto_pin_cfg *cfg);
    int snd_hda_gen_build_controls(struct hda_codec *codec);
    int snd_hda_gen_build_pcms(struct hda_codec *codec);
    +void snd_hda_gen_reboot_notify(struct hda_codec *codec);

    /* standard jack event callbacks */
    void snd_hda_gen_hp_automute(struct hda_codec *codec,
    --- a/sound/pci/hda/patch_conexant.c
    +++ b/sound/pci/hda/patch_conexant.c
    @@ -188,11 +188,7 @@ static void cx_auto_reboot_notify(struct
    /* Turn the problematic codec into D3 to avoid spurious noises
    from the internal speaker during (and after) reboot */
    cx_auto_turn_eapd(codec, spec->num_eapds, spec->eapds, false);
    -
    - snd_hda_codec_set_power_to_all(codec, codec->core.afg, AC_PWRST_D3);
    - snd_hda_codec_write(codec, codec->core.afg, 0,
    - AC_VERB_SET_POWER_STATE, AC_PWRST_D3);
    - msleep(10);
    + snd_hda_gen_reboot_notify(codec);
    }

    static void cx_auto_free(struct hda_codec *codec)
    --- a/sound/pci/hda/patch_realtek.c
    +++ b/sound/pci/hda/patch_realtek.c
    @@ -868,15 +868,6 @@ static void alc_reboot_notify(struct hda
    alc_shutup(codec);
    }

    -/* power down codec to D3 at reboot/shutdown; set as reboot_notify ops */
    -static void alc_d3_at_reboot(struct hda_codec *codec)
    -{
    - snd_hda_codec_set_power_to_all(codec, codec->core.afg, AC_PWRST_D3);
    - snd_hda_codec_write(codec, codec->core.afg, 0,
    - AC_VERB_SET_POWER_STATE, AC_PWRST_D3);
    - msleep(10);
    -}
    -
    #define alc_free snd_hda_gen_free

    #ifdef CONFIG_PM
    @@ -5111,7 +5102,7 @@ static void alc_fixup_tpt440_dock(struct
    struct alc_spec *spec = codec->spec;

    if (action == HDA_FIXUP_ACT_PRE_PROBE) {
    - spec->reboot_notify = alc_d3_at_reboot; /* reduce noise */
    + spec->reboot_notify = snd_hda_gen_reboot_notify; /* reduce noise */
    spec->parse_flags = HDA_PINCFG_NO_HP_FIXUP;
    codec->power_save_node = 0; /* avoid click noises */
    snd_hda_apply_pincfgs(codec, pincfgs);

    \
     
     \ /
      Last update: 2019-08-22 19:56    [W:4.299 / U:0.240 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site