From patchwork Sun Apr 5 18:34:51 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Hans de Goede X-Patchwork-Id: 193099 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-6.9 required=3.0 tests=DKIMWL_WL_HIGH, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_PATCH, MAILING_LIST_MULTI, SIGNED_OFF_BY, SPF_HELO_NONE, SPF_PASS autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id AF01EC2BA15 for ; Sun, 5 Apr 2020 18:36:57 +0000 (UTC) Received: from alsa0.perex.cz (alsa0.perex.cz [77.48.224.243]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 3631E2051A for ; Sun, 5 Apr 2020 18:36:57 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (1024-bit key) header.d=alsa-project.org header.i=@alsa-project.org header.b="C/WiUcnh"; dkim=pass (1024-bit key) header.d=redhat.com header.i=@redhat.com header.b="Dxkw6AUy" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 3631E2051A Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=redhat.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=alsa-devel-bounces@alsa-project.org Received: from alsa1.perex.cz (alsa1.perex.cz [207.180.221.201]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by alsa0.perex.cz (Postfix) with ESMTPS id 8B41F167D; Sun, 5 Apr 2020 20:36:05 +0200 (CEST) DKIM-Filter: OpenDKIM Filter v2.11.0 alsa0.perex.cz 8B41F167D DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=alsa-project.org; s=default; t=1586111815; bh=j61JYqZLvrhLMRmq0tZN/b2WXUyHXc4BAuYjSYtnjKs=; h=From:To:Subject:Date:In-Reply-To:References:Cc:List-Id: List-Unsubscribe:List-Archive:List-Post:List-Help:List-Subscribe: From; b=C/WiUcnhs4KCN76ADHniUT7WdSvqxG3w+ISIA5qGu1wLJcASkW9Ua8CALu92gICnN S3BBKbzg0ywjxBNGrFkii/a1sYdoBEeJbplyQamllLHInAfyxobqFyAQW0aduF7DAZ jXjhSi7592quuUeysn8BysOiqcTjFnT6X7pyZoyI= Received: from alsa1.perex.cz (localhost.localdomain [127.0.0.1]) by alsa1.perex.cz (Postfix) with ESMTP id 720E3F80216; Sun, 5 Apr 2020 20:35:13 +0200 (CEST) Received: by alsa1.perex.cz (Postfix, from userid 50401) id 51F65F8015A; Sun, 5 Apr 2020 20:35:08 +0200 (CEST) Received: from us-smtp-delivery-1.mimecast.com (us-smtp-1.mimecast.com [205.139.110.61]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-SHA384 (256/256 bits)) (No client certificate requested) by alsa1.perex.cz (Postfix) with ESMTPS id 5D787F80121 for ; Sun, 5 Apr 2020 20:35:02 +0200 (CEST) DKIM-Filter: OpenDKIM Filter v2.11.0 alsa1.perex.cz 5D787F80121 Authentication-Results: alsa1.perex.cz; dkim=pass (1024-bit key) header.d=redhat.com header.i=@redhat.com header.b="Dxkw6AUy" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1586111701; h=from:from:reply-to:subject:subject: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=j61JYqZLvrhLMRmq0tZN/b2WXUyHXc4BAuYjSYtnjKs=; b=Dxkw6AUyCHKY1V7+TNlyGPbWyELbr5tERCQ4CiVzmPYbff956Sd8UFCJ+1kDfgbLyTh3Gt 4zZ+HDESGacQwcMjM68KLwzL6cKqCRofmRPLygw17jgPZgUNLpiDwtFGv8WYZbWMehK6w4 COwveA6jWvlWSjtpyflPlp/Gr4gYugk= Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-368-2wq22LkCOKS6XbRln5ykxw-1; Sun, 05 Apr 2020 14:35:00 -0400 X-MC-Unique: 2wq22LkCOKS6XbRln5ykxw-1 Received: from smtp.corp.redhat.com (int-mx03.intmail.prod.int.phx2.redhat.com [10.5.11.13]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id 2BB4B800D53; Sun, 5 Apr 2020 18:34:59 +0000 (UTC) Received: from x1.localdomain.com (ovpn-112-60.ams2.redhat.com [10.36.112.60]) by smtp.corp.redhat.com (Postfix) with ESMTP id 2E0309A24B; Sun, 5 Apr 2020 18:34:58 +0000 (UTC) From: Hans de Goede To: Jaroslav Kysela Subject: [PATCH 2/5] chtrt5645: Add ASUSTeKCOMPUTERINC.-T100HAN-1.0 symlink to chtrt5645-dmic1.conf Date: Sun, 5 Apr 2020 20:34:51 +0200 Message-Id: <20200405183454.34515-2-hdegoede@redhat.com> In-Reply-To: <20200405183454.34515-1-hdegoede@redhat.com> References: <20200405183454.34515-1-hdegoede@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.13 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Cc: Hans de Goede , alsa-devel@alsa-project.org X-BeenThere: alsa-devel@alsa-project.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: "Alsa-devel mailing list for ALSA developers - http://www.alsa-project.org" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: alsa-devel-bounces@alsa-project.org Sender: "Alsa-devel" We already have a ASUSTeKCOMPUTERINC.-T100HAN-1.0-T100HAN symlink to chtrt5645-dmic1.conf, but since kernel commit 39870b0dec68 ("ASoC: DMI long name - avoid to add board name if matches with product name"), the kernel omits the (second) -T100HAN. So with newer kernels that no longer works. Arguably this is a kernel regressions, but chances are we already have some configs depending on the new behavior, so lets fix this on the UCM2 side by adding a second symlink using the new long name. Signed-off-by: Hans de Goede --- ucm2/chtrt5645/ASUSTeKCOMPUTERINC.-T100HAN-1.0.conf | 1 + 1 file changed, 1 insertion(+) create mode 120000 ucm2/chtrt5645/ASUSTeKCOMPUTERINC.-T100HAN-1.0.conf diff --git a/ucm2/chtrt5645/ASUSTeKCOMPUTERINC.-T100HAN-1.0.conf b/ucm2/chtrt5645/ASUSTeKCOMPUTERINC.-T100HAN-1.0.conf new file mode 120000 index 0000000..b4d2816 --- /dev/null +++ b/ucm2/chtrt5645/ASUSTeKCOMPUTERINC.-T100HAN-1.0.conf @@ -0,0 +1 @@ +chtrt5645-dmic1.conf \ No newline at end of file From patchwork Sun Apr 5 18:34:54 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Hans de Goede X-Patchwork-Id: 193098 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-6.9 required=3.0 tests=DKIMWL_WL_HIGH, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_PATCH, MAILING_LIST_MULTI, SIGNED_OFF_BY, SPF_HELO_NONE, SPF_PASS autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id F1C2DC2BA15 for ; Sun, 5 Apr 2020 18:38:15 +0000 (UTC) Received: from alsa0.perex.cz (alsa0.perex.cz [77.48.224.243]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 8133E2051A for ; Sun, 5 Apr 2020 18:38:15 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (1024-bit key) header.d=alsa-project.org header.i=@alsa-project.org header.b="usic+y8M"; dkim=pass (1024-bit key) header.d=redhat.com header.i=@redhat.com header.b="JARdVQgf" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 8133E2051A Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=redhat.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=alsa-devel-bounces@alsa-project.org Received: from alsa1.perex.cz (alsa1.perex.cz [207.180.221.201]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by alsa0.perex.cz (Postfix) with ESMTPS id E16811675; Sun, 5 Apr 2020 20:37:23 +0200 (CEST) DKIM-Filter: OpenDKIM Filter v2.11.0 alsa0.perex.cz E16811675 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=alsa-project.org; s=default; t=1586111894; bh=iHEkUq0IUK55nHoSxmM4+QS78Ikzk095quaQYY5zsQE=; h=From:To:Subject:Date:In-Reply-To:References:Cc:List-Id: List-Unsubscribe:List-Archive:List-Post:List-Help:List-Subscribe: From; b=usic+y8M8Z9ExzpLdrOEAeBfVmM6QQvURCeRZCP3oT5c1MKhnJNWzfm1qBx4akH0d LqZO/Rlx5YvUB+5qSjXRxeeMUienJ+m2VWvB8kTj27+giMu90bram0OIMDliLQynAc tirAEVAuqO290hLU4VIgnAAvFqvnkw8t4Q7HXLJM= Received: from alsa1.perex.cz (localhost.localdomain [127.0.0.1]) by alsa1.perex.cz (Postfix) with ESMTP id 0FB65F80290; Sun, 5 Apr 2020 20:35:18 +0200 (CEST) Received: by alsa1.perex.cz (Postfix, from userid 50401) id 92921F80216; Sun, 5 Apr 2020 20:35:11 +0200 (CEST) Received: from us-smtp-delivery-1.mimecast.com (us-smtp-1.mimecast.com [205.139.110.61]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-SHA384 (256/256 bits)) (No client certificate requested) by alsa1.perex.cz (Postfix) with ESMTPS id C5850F801DA for ; Sun, 5 Apr 2020 20:35:06 +0200 (CEST) DKIM-Filter: OpenDKIM Filter v2.11.0 alsa1.perex.cz C5850F801DA Authentication-Results: alsa1.perex.cz; dkim=pass (1024-bit key) header.d=redhat.com header.i=@redhat.com header.b="JARdVQgf" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1586111705; h=from:from:reply-to:subject:subject: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=tDVqz1CEwwlFtzdvOTwRVcni+AzFty+d+ZTbj+MDgHU=; b=JARdVQgfnUXykqr4M2nDuEZYIhh84jqL8jvKJWzLM1SgWIO9uHtNPMb3hZpdZMXIGdypiM nbBWeZP7L6HUDN8iQYyyvo+rjneHloj77rwpygCmhUJMs431EG41yvFr74xsNki3LyMdpK YegOJaUAefd2grlv8vjSXjiYusrFP6Y= Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-202-8WFJYjDpM5ujgabYTYcCAA-1; Sun, 05 Apr 2020 14:35:03 -0400 X-MC-Unique: 8WFJYjDpM5ujgabYTYcCAA-1 Received: from smtp.corp.redhat.com (int-mx03.intmail.prod.int.phx2.redhat.com [10.5.11.13]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id 7C94F107ACC7; Sun, 5 Apr 2020 18:35:02 +0000 (UTC) Received: from x1.localdomain.com (ovpn-112-60.ams2.redhat.com [10.36.112.60]) by smtp.corp.redhat.com (Postfix) with ESMTP id A858F98A51; Sun, 5 Apr 2020 18:35:01 +0000 (UTC) From: Hans de Goede To: Jaroslav Kysela Subject: [PATCH 5/5] cht-bsw-rt5672: Fix HiFi-stereo-dmic2 conf not loading / working Date: Sun, 5 Apr 2020 20:34:54 +0200 Message-Id: <20200405183454.34515-5-hdegoede@redhat.com> In-Reply-To: <20200405183454.34515-1-hdegoede@redhat.com> References: <20200405183454.34515-1-hdegoede@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.13 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Cc: Hans de Goede , alsa-devel@alsa-project.org X-BeenThere: alsa-devel@alsa-project.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: "Alsa-devel mailing list for ALSA developers - http://www.alsa-project.org" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: alsa-devel-bounces@alsa-project.org Sender: "Alsa-devel" Fix HiFi-stereo-dmic2 conf not loading because of references to non existing devices in the ConflictingDevices sections. Stop playing Rename + Remove tricks with the output (Speaker2) and input (DMIC1) devices which we do not want to be part of this config, this does not work now that we only allow devices which are actually part of our config in the ConflictingDevices sections. Instead start with only having devices which all configs have in the ConflictingDevices sections (Headphones / Headset) and add the other ones from our HiFi*.conf files after including the shared basic configs. BugLink: https://bugzilla.redhat.com/show_bug.cgi?id=1786723 Signed-off-by: Hans de Goede --- I have tested that both HiFi.conf and HiFi-stereo-dmic2.conf work properly after these changes. --- ucm2/cht-bsw-rt5672/HiFi-stereo-dmic2.conf | 6 ++---- ucm2/cht-bsw-rt5672/HiFi.conf | 8 ++++++++ ucm2/codecs/rt5672/DMIC1.conf | 1 - ucm2/codecs/rt5672/DMIC2.conf | 1 - ucm2/codecs/rt5672/HeadPhones.conf | 2 -- ucm2/codecs/rt5672/HeadsetMic.conf | 2 -- ucm2/codecs/rt5672/MonoSpeaker.conf | 1 - ucm2/codecs/rt5672/Speaker.conf | 1 - 8 files changed, 10 insertions(+), 12 deletions(-) diff --git a/ucm2/cht-bsw-rt5672/HiFi-stereo-dmic2.conf b/ucm2/cht-bsw-rt5672/HiFi-stereo-dmic2.conf index 82c976b..5e06b42 100644 --- a/ucm2/cht-bsw-rt5672/HiFi-stereo-dmic2.conf +++ b/ucm2/cht-bsw-rt5672/HiFi-stereo-dmic2.conf @@ -12,11 +12,9 @@ SectionVerb { } -RenameDevice."Speaker1" "Speaker" -RemoveDevice."Speaker2" "Speaker2" +SectionDevice."Headphones".ConflictingDevice.0 "Speaker1" -RenameDevice."Mic2" "Mic" -RemoveDevice."Mic1" "Mic1" +SectionDevice."Headset".ConflictingDevice.0 "Mic2" diff --git a/ucm2/cht-bsw-rt5672/HiFi.conf b/ucm2/cht-bsw-rt5672/HiFi.conf index 9a64cc3..2dfa479 100644 --- a/ucm2/cht-bsw-rt5672/HiFi.conf +++ b/ucm2/cht-bsw-rt5672/HiFi.conf @@ -14,7 +14,15 @@ SectionVerb { +SectionDevice."Speaker1".ConflictingDevice.1 "Speaker2" +SectionDevice."Speaker2".ConflictingDevice.1 "Speaker1" +SectionDevice."Headphones".ConflictingDevice.0 "Speaker1" +SectionDevice."Headphones".ConflictingDevice.1 "Speaker2" +SectionDevice."Mic1".ConflictingDevice.1 "Mic2" +SectionDevice."Mic2".ConflictingDevice.1 "Mic1" +SectionDevice."Headset".ConflictingDevice.0 "Mic1" +SectionDevice."Headset".ConflictingDevice.1 "Mic2" diff --git a/ucm2/codecs/rt5672/DMIC1.conf b/ucm2/codecs/rt5672/DMIC1.conf index 2018283..acc26f7 100644 --- a/ucm2/codecs/rt5672/DMIC1.conf +++ b/ucm2/codecs/rt5672/DMIC1.conf @@ -2,7 +2,6 @@ SectionDevice."Mic1" { Comment "Internal Digital Microphone on DMIC1" ConflictingDevice [ - "Mic2" "Headset" ] diff --git a/ucm2/codecs/rt5672/DMIC2.conf b/ucm2/codecs/rt5672/DMIC2.conf index 48e6170..abe139b 100644 --- a/ucm2/codecs/rt5672/DMIC2.conf +++ b/ucm2/codecs/rt5672/DMIC2.conf @@ -2,7 +2,6 @@ SectionDevice."Mic2" { Comment "Internal Digital Microphone on DMIC2" ConflictingDevice [ - "Mic1" "Headset" ] diff --git a/ucm2/codecs/rt5672/HeadPhones.conf b/ucm2/codecs/rt5672/HeadPhones.conf index 2c8d6e8..9178607 100644 --- a/ucm2/codecs/rt5672/HeadPhones.conf +++ b/ucm2/codecs/rt5672/HeadPhones.conf @@ -2,8 +2,6 @@ SectionDevice."Headphones" { Comment "Headphones" ConflictingDevice [ - "Speaker1" - "Speaker2" ] EnableSequence [ diff --git a/ucm2/codecs/rt5672/HeadsetMic.conf b/ucm2/codecs/rt5672/HeadsetMic.conf index a764d82..e04c3ce 100644 --- a/ucm2/codecs/rt5672/HeadsetMic.conf +++ b/ucm2/codecs/rt5672/HeadsetMic.conf @@ -2,8 +2,6 @@ SectionDevice."Headset" { Comment "Headset Microphone" ConflictingDevice [ - "Mic1" - "Mic2" ] EnableSequence [ diff --git a/ucm2/codecs/rt5672/MonoSpeaker.conf b/ucm2/codecs/rt5672/MonoSpeaker.conf index f08c9c6..88c39d2 100644 --- a/ucm2/codecs/rt5672/MonoSpeaker.conf +++ b/ucm2/codecs/rt5672/MonoSpeaker.conf @@ -2,7 +2,6 @@ SectionDevice."Speaker2" { Comment "Mono Speaker" ConflictingDevice [ - "Speaker1" "Headphones" ] diff --git a/ucm2/codecs/rt5672/Speaker.conf b/ucm2/codecs/rt5672/Speaker.conf index 8b7bcf8..c70ffae 100644 --- a/ucm2/codecs/rt5672/Speaker.conf +++ b/ucm2/codecs/rt5672/Speaker.conf @@ -2,7 +2,6 @@ SectionDevice."Speaker1" { Comment "Stereo Speakers" ConflictingDevice [ - "Speaker2" "Headphones" ]