From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp-out2.suse.de (smtp-out2.suse.de [195.135.223.131]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 4E61320DD5C for ; Thu, 27 Mar 2025 10:09:48 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=195.135.223.131 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1743070190; cv=none; b=KkjD7r2YvDE3rdghHZ5Ber9OcIfontx0gz18sLu22E3eHNvZ7F7xRmlcvYisQXSQXGDvgun0VOIyFmoOQAd91XonvW1jox0BNOR/rmpie5qAm1L92h05RkgL0vvamV7uiotLrX9lrdgnyQFeP2qULPOb4m764IO5zhRxC+DWki0= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1743070190; c=relaxed/simple; bh=bXzzb0Avu4HcmZF1e9Vm0rDE6sTRUkV82fEsxmMvhyI=; h=Date:Message-ID:From:To:Cc:Subject:In-Reply-To:References: MIME-Version:Content-Type; b=CUMGNzDYmM4casSah3aeR+Hsdia7msKDBciCpz9sXMVFAEdH5psCzTLz/JhXzRyHOzxMAngGdXQEJcljZHa4IU9DFsJmOuXm0lS8eXEwyKl0VLWefOY+7p4B1TPzoA7+EFfvQXFckv7i7EqoIzqUh+6aCYxCf7S0bhZaN9YNgtw= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=suse.de; spf=pass smtp.mailfrom=suse.de; dkim=pass (1024-bit key) header.d=suse.de header.i=@suse.de header.b=T6wHJtTJ; dkim=permerror (0-bit key) header.d=suse.de header.i=@suse.de header.b=uOjj+ayq; dkim=pass (1024-bit key) header.d=suse.de header.i=@suse.de header.b=T6wHJtTJ; dkim=permerror (0-bit key) header.d=suse.de header.i=@suse.de header.b=uOjj+ayq; arc=none smtp.client-ip=195.135.223.131 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=suse.de Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=suse.de Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=suse.de header.i=@suse.de header.b="T6wHJtTJ"; dkim=permerror (0-bit key) header.d=suse.de header.i=@suse.de header.b="uOjj+ayq"; dkim=pass (1024-bit key) header.d=suse.de header.i=@suse.de header.b="T6wHJtTJ"; dkim=permerror (0-bit key) header.d=suse.de header.i=@suse.de header.b="uOjj+ayq" Received: from imap1.dmz-prg2.suse.org (unknown [10.150.64.97]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by smtp-out2.suse.de (Postfix) with ESMTPS id 4644C1F451; Thu, 27 Mar 2025 10:09:46 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.de; s=susede2_rsa; t=1743070186; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc: mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=35YETAU0Mo+uDyLoadZw5UOuffpmd9hXuDvw03DJNvc=; b=T6wHJtTJ0Ez5BK3UpFvYjVd1vX+ZtuA0hcmuL7+UGYJQg1Rh068lAsl7UXyJl/TRsA4Pyk PdADuIEmxFLynHriho/FZc4J7ji470ebV4/h1b6xTb7QSq0K1SDyRY+yuA/8WeVt/4lbzH smzTcKvG6c0serf0ch6bwZPNNkBg+Bo= DKIM-Signature: v=1; a=ed25519-sha256; c=relaxed/relaxed; d=suse.de; s=susede2_ed25519; t=1743070186; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc: mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=35YETAU0Mo+uDyLoadZw5UOuffpmd9hXuDvw03DJNvc=; b=uOjj+ayqNT1BGk9nLURyQ2GyLrp0ttKGq3o1D5PmS3qvmAU8LRZVJ1T9LsSBxk5WilITUe cZcA4CeNyGxNSxBw== Authentication-Results: smtp-out2.suse.de; none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.de; s=susede2_rsa; t=1743070186; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc: mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=35YETAU0Mo+uDyLoadZw5UOuffpmd9hXuDvw03DJNvc=; b=T6wHJtTJ0Ez5BK3UpFvYjVd1vX+ZtuA0hcmuL7+UGYJQg1Rh068lAsl7UXyJl/TRsA4Pyk PdADuIEmxFLynHriho/FZc4J7ji470ebV4/h1b6xTb7QSq0K1SDyRY+yuA/8WeVt/4lbzH smzTcKvG6c0serf0ch6bwZPNNkBg+Bo= DKIM-Signature: v=1; a=ed25519-sha256; c=relaxed/relaxed; d=suse.de; s=susede2_ed25519; t=1743070186; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc: mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=35YETAU0Mo+uDyLoadZw5UOuffpmd9hXuDvw03DJNvc=; b=uOjj+ayqNT1BGk9nLURyQ2GyLrp0ttKGq3o1D5PmS3qvmAU8LRZVJ1T9LsSBxk5WilITUe cZcA4CeNyGxNSxBw== Received: from imap1.dmz-prg2.suse.org (localhost [127.0.0.1]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by imap1.dmz-prg2.suse.org (Postfix) with ESMTPS id 0A8D5139D4; Thu, 27 Mar 2025 10:09:46 +0000 (UTC) Received: from dovecot-director2.suse.de ([2a07:de40:b281:106:10:150:64:167]) by imap1.dmz-prg2.suse.org with ESMTPSA id /oBaAeoj5Wd5DwAAD6G6ig (envelope-from ); Thu, 27 Mar 2025 10:09:46 +0000 Date: Thu, 27 Mar 2025 11:09:45 +0100 Message-ID: <87a5969446.wl-tiwai@suse.de> From: Takashi Iwai To: Benjamin Berg Cc: Takashi Iwai , linux-sound@vger.kernel.org, Jaroslav Kysela , Takashi Iwai Subject: Re: [PATCH] ALSA: hda/realtek: Enable speakers on further Samsung Galaxy Book2 Pro In-Reply-To: <3766ead24f99ba9ac7490a2066d99f0ba968df27.camel@sipsolutions.net> References: <20250326201953.131948-1-benjamin@sipsolutions.net> <87jz8a9731.wl-tiwai@suse.de> <3766ead24f99ba9ac7490a2066d99f0ba968df27.camel@sipsolutions.net> User-Agent: Wanderlust/2.15.9 (Almost Unreal) Emacs/27.2 Mule/6.0 Precedence: bulk X-Mailing-List: linux-sound@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 (generated by SEMI-EPG 1.14.7 - "Harue") Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 8bit X-Spam-Score: -3.30 X-Spamd-Result: default: False [-3.30 / 50.00]; BAYES_HAM(-3.00)[100.00%]; MID_CONTAINS_FROM(1.00)[]; NEURAL_HAM_LONG(-1.00)[-1.000]; NEURAL_HAM_SHORT(-0.20)[-1.000]; MIME_GOOD(-0.10)[text/plain]; MIME_TRACE(0.00)[0:+]; RCVD_VIA_SMTP_AUTH(0.00)[]; TO_DN_SOME(0.00)[]; ARC_NA(0.00)[]; RCVD_TLS_ALL(0.00)[]; FUZZY_BLOCKED(0.00)[rspamd.com]; RCPT_COUNT_FIVE(0.00)[5]; FROM_HAS_DN(0.00)[]; DKIM_SIGNED(0.00)[suse.de:s=susede2_rsa,suse.de:s=susede2_ed25519]; FROM_EQ_ENVFROM(0.00)[]; TO_MATCH_ENVRCPT_ALL(0.00)[]; RCVD_COUNT_TWO(0.00)[2]; DBL_BLOCKED_OPENRESOLVER(0.00)[suse.de:mid] X-Spam-Flag: NO X-Spam-Level: On Thu, 27 Mar 2025 10:55:04 +0100, Benjamin Berg wrote: > > On Thu, 2025-03-27 at 10:05 +0100, Takashi Iwai wrote: > > On Wed, 26 Mar 2025 21:19:53 +0100, > > Benjamin Berg wrote: > > > Note that I didn't really manage to test this directly. Trying to > > > do a > > > reconfigure resulted in a hang and I didn't figure out how to use > > > the > > > patch= option with a firmware file. > > > However, the machine does work fine when configuring the two > > > speakers > > > from userspace, so I am confident that this patch is correct. > > > > FYI, the equivalent can be achieved with model option, too. > > e.g. passing model=144d:c870 will apply the same existing quirk as > > SSID 144d:c870.  If your target is the secondary codec, pass like > > "model=,144d:c870" instead. > > Hmm, it is probably really straight forward, but I didn't grasp the > documentation in Documentation/sound/hd-audio/notes.rst on a quick > read. It's mentioned in "Model Option" subsection of that document. ``` A new style for the model option that was introduced since 5.15 kernel is to pass the PCI or codec SSID in the form of ``model=XXXX:YYYY`` where XXXX and YYYY are the sub-vendor and sub-device IDs in hex numbers, respectively. This is a kind of aliasing to another device; when this form is given, the driver will refer to that SSID as a reference to the quirk table. It'd be useful especially when the target quirk isn't listed in the model table. For example, passing model=103c:8862 will apply the quirk for HP ProBook 445 G8 (which isn't found in the model table as of writing) as long as the device is handled equivalently by the same driver. ``` > I am not sure anymore what I did (and I don't have access to the > machine anymore). It might be I tried putting the model= line into a > firmware file instead of just passing it on the kernel command line. > How do I even know whether it is the first or second codec? Is that > what the D0 in hwC0D0 is? I meant the device probe order, so card#0 device#0 must be the very first one :) > > > --- > > >  sound/pci/hda/patch_realtek.c | 1 + > > >  1 file changed, 1 insertion(+) > > > > > > diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c > > > index c735f630ecb5..e4ea5ea5bb27 100644 > > > --- a/sound/pci/hda/patch_realtek.c > > > +++ b/sound/pci/hda/patch_realtek.c > > > @@ -10758,6 +10758,7 @@ static const struct hda_quirk alc269_fixup_tbl[] = { > > >   SND_PCI_QUIRK(0x144d, 0xca06, "Samsung Galaxy Book3 360 (NP730QFG)", ALC298_FIXUP_SAMSUNG_HEADPHONE_VERY_QUIET), > > >   SND_PCI_QUIRK(0x144d, 0xc868, "Samsung Galaxy Book2 Pro (NP930XED)", ALC298_FIXUP_SAMSUNG_AMP), > > >   SND_PCI_QUIRK(0x144d, 0xc870, "Samsung Galaxy Book2 Pro (NP950XED)", ALC298_FIXUP_SAMSUNG_AMP_V2_2_AMPS), > > > + SND_PCI_QUIRK(0x144d, 0xc01d, "Samsung Galaxy Book2 Pro (NP950QED)", ALC298_FIXUP_SAMSUNG_AMP_V2_2_AMPS), > > >   SND_PCI_QUIRK(0x144d, 0xc872, "Samsung Galaxy Book2 Pro (NP950XEE)", ALC298_FIXUP_SAMSUNG_AMP_V2_2_AMPS), > > >   SND_PCI_QUIRK(0x144d, 0xc886, "Samsung Galaxy Book3 Pro (NP964XFG)", ALC298_FIXUP_SAMSUNG_AMP_V2_4_AMPS), > > >   SND_PCI_QUIRK(0x144d, 0xc1ca, "Samsung Galaxy Book3 Pro 360 (NP960QFG)", ALC298_FIXUP_SAMSUNG_AMP_V2_4_AMPS), > > > > The table is sorted in PCI SSID order.  Could you try to put at the > > right position? > > Sorry, will fix and resend. Thanks! Takashi