lkml.org 
[lkml]   [2021]   [Nov]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 5.10 051/575] ALSA: hda/realtek: Fixes HP Spectre x360 15-eb1xxx speakers
    Date
    From: Davide Baldo <davide@baldo.me>

    [ Upstream commit d94befbb5ae379f6dfd4fa6d460eacc09fa7b9c3 ]

    In laptop 'HP Spectre x360 Convertible 15-eb1xxx/8811' both front and
    rear speakers are silent, this patch fixes that by overriding the pin
    layout and by initializing the amplifier which needs a GPIO pin to be
    set to 1 then 0, similar to the existing HP Spectre x360 14 model.

    In order to have volume control, both front and rear speakers were
    forced to use the DAC1.

    This patch also correctly map the mute LED but since there is no
    microphone on/off switch exposed by the alsa subsystem it never turns
    on by itself.

    There are still known audio issues in this laptop: headset microphone
    doesn't work, the button to mute/unmute microphone is not yet mapped,
    the LED of the mute/unmute speakers doesn't seems to be exposed via
    GPIO and never turns on.

    BugLink: https://bugzilla.kernel.org/show_bug.cgi?id=213953
    Signed-off-by: Davide Baldo <davide@baldo.me>
    Link: https://lore.kernel.org/r/20211015072121.5287-1-davide@baldo.me
    Signed-off-by: Takashi Iwai <tiwai@suse.de>
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    sound/pci/hda/patch_realtek.c | 46 +++++++++++++++++++++++++++++++++++
    1 file changed, 46 insertions(+)

    diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c
    index 5b28275147057..2eb06351de1fb 100644
    --- a/sound/pci/hda/patch_realtek.c
    +++ b/sound/pci/hda/patch_realtek.c
    @@ -6363,6 +6363,44 @@ static void alc_fixup_no_int_mic(struct hda_codec *codec,
    }
    }

    +/* GPIO1 = amplifier on/off
    + * GPIO3 = mic mute LED
    + */
    +static void alc285_fixup_hp_spectre_x360_eb1(struct hda_codec *codec,
    + const struct hda_fixup *fix, int action)
    +{
    + static const hda_nid_t conn[] = { 0x02 };
    +
    + struct alc_spec *spec = codec->spec;
    + static const struct hda_pintbl pincfgs[] = {
    + { 0x14, 0x90170110 }, /* front/high speakers */
    + { 0x17, 0x90170130 }, /* back/bass speakers */
    + { }
    + };
    +
    + //enable micmute led
    + alc_fixup_hp_gpio_led(codec, action, 0x00, 0x04);
    +
    + switch (action) {
    + case HDA_FIXUP_ACT_PRE_PROBE:
    + spec->micmute_led_polarity = 1;
    + /* needed for amp of back speakers */
    + spec->gpio_mask |= 0x01;
    + spec->gpio_dir |= 0x01;
    + snd_hda_apply_pincfgs(codec, pincfgs);
    + /* share DAC to have unified volume control */
    + snd_hda_override_conn_list(codec, 0x14, ARRAY_SIZE(conn), conn);
    + snd_hda_override_conn_list(codec, 0x17, ARRAY_SIZE(conn), conn);
    + break;
    + case HDA_FIXUP_ACT_INIT:
    + /* need to toggle GPIO to enable the amp of back speakers */
    + alc_update_gpio_data(codec, 0x01, true);
    + msleep(100);
    + alc_update_gpio_data(codec, 0x01, false);
    + break;
    + }
    +}
    +
    static void alc285_fixup_hp_spectre_x360(struct hda_codec *codec,
    const struct hda_fixup *fix, int action)
    {
    @@ -6515,6 +6553,7 @@ enum {
    ALC269_FIXUP_HP_DOCK_GPIO_MIC1_LED,
    ALC280_FIXUP_HP_9480M,
    ALC245_FIXUP_HP_X360_AMP,
    + ALC285_FIXUP_HP_SPECTRE_X360_EB1,
    ALC288_FIXUP_DELL_HEADSET_MODE,
    ALC288_FIXUP_DELL1_MIC_NO_PRESENCE,
    ALC288_FIXUP_DELL_XPS_13,
    @@ -8205,6 +8244,10 @@ static const struct hda_fixup alc269_fixups[] = {
    .type = HDA_FIXUP_FUNC,
    .v.func = alc285_fixup_hp_spectre_x360,
    },
    + [ALC285_FIXUP_HP_SPECTRE_X360_EB1] = {
    + .type = HDA_FIXUP_FUNC,
    + .v.func = alc285_fixup_hp_spectre_x360_eb1
    + },
    [ALC287_FIXUP_IDEAPAD_BASS_SPK_AMP] = {
    .type = HDA_FIXUP_FUNC,
    .v.func = alc285_fixup_ideapad_s740_coef,
    @@ -8555,6 +8598,8 @@ static const struct snd_pci_quirk alc269_fixup_tbl[] = {
    SND_PCI_QUIRK(0x103c, 0x87f7, "HP Spectre x360 14", ALC245_FIXUP_HP_X360_AMP),
    SND_PCI_QUIRK(0x103c, 0x8805, "HP ProBook 650 G8 Notebook PC", ALC236_FIXUP_HP_GPIO_LED),
    SND_PCI_QUIRK(0x103c, 0x880d, "HP EliteBook 830 G8 Notebook PC", ALC285_FIXUP_HP_GPIO_LED),
    + SND_PCI_QUIRK(0x103c, 0x8811, "HP Spectre x360 15-eb1xxx", ALC285_FIXUP_HP_SPECTRE_X360_EB1),
    + SND_PCI_QUIRK(0x103c, 0x8812, "HP Spectre x360 15-eb1xxx", ALC285_FIXUP_HP_SPECTRE_X360_EB1),
    SND_PCI_QUIRK(0x103c, 0x8846, "HP EliteBook 850 G8 Notebook PC", ALC285_FIXUP_HP_GPIO_LED),
    SND_PCI_QUIRK(0x103c, 0x8847, "HP EliteBook x360 830 G8 Notebook PC", ALC285_FIXUP_HP_GPIO_LED),
    SND_PCI_QUIRK(0x103c, 0x884b, "HP EliteBook 840 Aero G8 Notebook PC", ALC285_FIXUP_HP_GPIO_LED),
    @@ -8979,6 +9024,7 @@ static const struct hda_model_fixup alc269_fixup_models[] = {
    {.id = ALC245_FIXUP_HP_X360_AMP, .name = "alc245-hp-x360-amp"},
    {.id = ALC295_FIXUP_HP_OMEN, .name = "alc295-hp-omen"},
    {.id = ALC285_FIXUP_HP_SPECTRE_X360, .name = "alc285-hp-spectre-x360"},
    + {.id = ALC285_FIXUP_HP_SPECTRE_X360_EB1, .name = "alc285-hp-spectre-x360-eb1"},
    {.id = ALC287_FIXUP_IDEAPAD_BASS_SPK_AMP, .name = "alc287-ideapad-bass-spk-amp"},
    {.id = ALC623_FIXUP_LENOVO_THINKSTATION_P340, .name = "alc623-lenovo-thinkstation-p340"},
    {.id = ALC255_FIXUP_ACER_HEADPHONE_AND_MIC, .name = "alc255-acer-headphone-and-mic"},
    --
    2.33.0


    \
     
     \ /
      Last update: 2021-11-15 19:59    [W:4.100 / U:0.028 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site