| Message ID | 20221010065702.35190-1-luke@ljones.dev (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | 66ba7c88507344dee68ad1acbdb630473ab36114 |
| Headers | show |
| Series | Fixes bc2c23549ccd ("ALSA: hda/realtek: Add pincfg for ASUS G533Z HP jack") | expand |
On Mon, 10 Oct 2022 08:57:02 +0200, Luke D. Jones wrote: > > The initial fix for ASUS G533Z was based on faulty information. This > fixes the pincfg to values that have been verified with no existing > module options or other hacks enabled. > > Enables headphone jack, and 5.1 surround. > > Signed-off-by: Luke D. Jones <luke@ljones.dev> The changes look OK, but the subject line should be rather describing what about the patch is about. I corrected the subject and put the proper Fixes tag (as well as Cc-to-stable). Also, the indent level wasn't right, so I corrected it locally, too. thanks, Takashi
On Mon, 2022-10-10 at 14:45 +0200, Takashi Iwai wrote: > On Mon, 10 Oct 2022 08:57:02 +0200, > Luke D. Jones wrote: > > > > The initial fix for ASUS G533Z was based on faulty information. > > This > > fixes the pincfg to values that have been verified with no existing > > module options or other hacks enabled. > > > > Enables headphone jack, and 5.1 surround. > > > > Signed-off-by: Luke D. Jones <luke@ljones.dev> > > The changes look OK, but the subject line should be rather describing > what about the patch is about. I corrected the subject and put the > proper Fixes tag (as well as Cc-to-stable). > > Also, the indent level wasn't right, so I corrected it locally, too. Oh... Sorry about the hassle there, I could have sworn it was correct. Thanks for accepting regardless :) > > thanks, > > Takashi
diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c index bce82b834cec..24406f81da27 100644 --- a/sound/pci/hda/patch_realtek.c +++ b/sound/pci/hda/patch_realtek.c @@ -8449,11 +8449,13 @@ static const struct hda_fixup alc269_fixups[] = { [ALC285_FIXUP_ASUS_G533Z_PINS] = { .type = HDA_FIXUP_PINS, .v.pins = (const struct hda_pintbl[]) { - { 0x14, 0x90170120 }, - { } + { 0x14, 0x90170152 }, /* Speaker Surround Playback Switch */ + { 0x19, 0x03a19020 }, /* Mic Boost Volume */ + { 0x1a, 0x03a11c30 }, /* Mic Boost Volume */ + { 0x1e, 0x90170151 }, /* Rear jack, IN OUT EAPD Detect */ + { 0x21, 0x03211420 }, + { } }, - .chained = true, - .chain_id = ALC294_FIXUP_ASUS_G513_PINS, }, [ALC294_FIXUP_ASUS_COEF_1B] = { .type = HDA_FIXUP_VERBS,
The initial fix for ASUS G533Z was based on faulty information. This fixes the pincfg to values that have been verified with no existing module options or other hacks enabled. Enables headphone jack, and 5.1 surround. Signed-off-by: Luke D. Jones <luke@ljones.dev> --- sound/pci/hda/patch_realtek.c | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-)