From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-ej1-f48.google.com (mail-ej1-f48.google.com [209.85.218.48]) (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 4AB942FE04E for ; Wed, 18 Feb 2026 22:45:31 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.218.48 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1771454732; cv=none; b=px4iMqxqv9G4YNFaHOqmRxuKZLiQEatc520dpmutkd/pX8pi61aFn4742gAgHi4vXWYtFhrqEHuGP0fmCU0eYVDu0eah7mcnHHPZ6cuuGjBRgQfLJLp9FMyuXwW+ViZgd1H6TTT2LXsw+1ZyFFihRiS9lFxQlKw2ES50yX8p+08= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1771454732; c=relaxed/simple; bh=AnM104S1r6iwHGiyNexvZNE4VCfd99SLmovq/8pbc0k=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=DIXX5BVSHcg/hJGeTjQ+UEMMiqho6H4X2YkiT0Gi8l1tOFP8DJDYdUhCdq/lRI7YhDsYpV21mMOYWPq8tZ246EM1JdVn5YlDkOge6mS0vmKEqO/8L2+7j68LQ5ohYBagWiiGygzi2sjMmcY3VpiQCYaKCH44T/a0VN/4+4gWFco= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=DMcA9b5d; arc=none smtp.client-ip=209.85.218.48 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="DMcA9b5d" Received: by mail-ej1-f48.google.com with SMTP id a640c23a62f3a-b904e1cd038so46041566b.1 for ; Wed, 18 Feb 2026 14:45:31 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1771454730; x=1772059530; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=Omq69PVtzUakJe73el69HdkBb1Z6AdKJRH0nDsT2Rn8=; b=DMcA9b5dikF2slQNiXwwF1rIIFHdVad8Z4wIT9Ox+0RpoiQ03IVqVeCeqYUoCoLlK5 MG72szisxwTDDcxjn9mrtEEy5nKgdnKwnIdtRtaH+2sf3tUgfaHQB91ocmWA+oQpoLm6 mfpjtdD1RfymOIx9THuf+AnBVJJNHK77x9eU/lq9ijdueQIPZ+U6U+amU9CUsp59pUcu vzHZ+guL1mmDxQUlpL7RFmLeKa+PtKKDXcxH2yqJT9eUtKgoy616dTMOr/OAw1snVRMF ni4CI4UZzwG8T7u+vQBf41gctZyU6QLQKzCQwdKH1VtvI9TW+x20GjfoVC7/JusmUlAv YzTQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1771454730; x=1772059530; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-gg:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=Omq69PVtzUakJe73el69HdkBb1Z6AdKJRH0nDsT2Rn8=; b=odar5TIqcwFngxHc7Ksvry3nbbNeaCoNliC1a91dsKoyUKumVsgPYjkDC8V/2W7rFO cON+1jpDXl0neQhHlwpVb0Jjnm1xlALozJg9XsYZ9fqz2n34Jrgk9rGo6Yihu7Wvf6aS qcYzSgxkbPo7cHnOyugAruZsfX8Q7I5s1g1GNXDW3jB6mNzxL9eawPqrk7uode05uGSH E3AJxNKhsSVXO33HhYH/ook33fDMeIX6RVn8wc4BBrlH/IsJukiEK2v94e8Rw7tmjll6 IBeNcwBQ5VfKKKsU6ociDLwdtWZaMdaLT3kj7Lj8azETCbSzMY/v2rR+F1sRukgui9nC 6Beg== X-Forwarded-Encrypted: i=1; AJvYcCVts8AfCmA6ct/wZkIX78XYM7x+kAKVWd45aEV6c1/JF9o60ipjtATavJJWJCB0I7w100Hwf/UOxNKF8D0=@vger.kernel.org X-Gm-Message-State: AOJu0YyrO+QZ1RxTExEuItxfcrSQpycndxrlbCyXZO+dKgxnbR3LgG0v Skf7TNn67aa3gJfXNW/k8JvuHXFVDbblvtNwVZucCjPv4cNDZzDcZ+Cz X-Gm-Gg: AZuq6aJOoef1gsuxFVnFuQLhSJCvm/5be9z72vdHpF8MPIRRkYCdMjZd0FOLjjOtNBM i3okHIvDhSHdlMw9zh7byKZtuSfatSukIkTyyEOKgt+WvgO3a5342MgnsSCCiAQLteMWpLU9+WG xvQgDzbuvtRgnggg08UP58ChXd0+Q0uwInuhC3uRP/FwMad5/xC0hlXI9Q7IgWolvDrWRIhUoWh n0CxlwgzqCl0NVNYDRLKPqF3wFUm9CO/EwFeFTLRKbSzYz11GuSV3HP5RdI4Z7mbRMeyFOMwpy0 Izqja3IdjR74iZkfavGZDUpZn0o0/wL7NGc/wM3OQYfLqx93rTIeu7xPn0GjOay02wikvKg744Z 0eYLFAshGQUHvIhq0BitN7tqwdDWVpKAQUerZmJEkAhe228/N4ymohi/YkuihjAisYTrqV6Caan utKRAioMEE2uUO9w== X-Received: by 2002:a17:907:2d21:b0:b73:2b08:ac70 with SMTP id a640c23a62f3a-b8fc3ceea42mr873970466b.49.1771454729485; Wed, 18 Feb 2026 14:45:29 -0800 (PST) Received: from jekhomev ([46.251.53.180]) by smtp.gmail.com with ESMTPSA id a640c23a62f3a-b8fc769111fsm503674966b.57.2026.02.18.14.45.28 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 18 Feb 2026 14:45:29 -0800 (PST) Date: Thu, 19 Feb 2026 00:45:28 +0200 From: Yauhen Kharuzhy To: Pierre-Louis Bossart Cc: linux-sound@vger.kernel.org, Cezary Rojewski , Liam Girdwood , Mark Brown , linux-kernel@vger.kernel.org, Hans de Goede Subject: Re: [PATCH v1 2/2] ASoC: Intel: soc-acpi-cht: Add Lenovo Yoga Book entries Message-ID: References: <20260217231324.1319392-1-jekhor@gmail.com> <20260217231324.1319392-3-jekhor@gmail.com> <3ff76f35-be88-4281-be92-ec8f1e160651@linux.dev> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <3ff76f35-be88-4281-be92-ec8f1e160651@linux.dev> On Wed, Feb 18, 2026 at 11:35:12AM +0100, Pierre-Louis Bossart wrote: > On 2/18/26 00:13, Yauhen Kharuzhy wrote: > > Lenovo Yoga Book YB1-X91 device uses a Cherry Trail SoC and has a valid > > ACPI DSDT entry for the RT5677 codec. This entry has some non-standard > > resource definitions, such as jack detection chip information, and > > hardware has some additional GPIO controls so use 'cht-yogabook' > > for the driver name instead of some default (like 'cht-bsw-rt5677'). > > > > Lenovo Yoga Book YB1-X90 device (Android version of the tablet) has the > > same hardware configuration but lacks a valid ACPI DSDT entry for the > > codec. To match this device, extend the existing quirk designated for the > > Lenovo Yoga Tab 3 Pro YT3-X90 and make it more generic: select a matched > > machine using the dmi_check_system() quirk mechanism. > > > > Signed-off-by: Yauhen Kharuzhy > > Looks ok to me, but since this is touching 10-yr old stuff maybe it's worth having a patch that only merged the existing quirks together. > Then you can add a new set of entries. Easier to review/test/bisect. Agree, that sounds reasonable. What about the driver naming? In this patch, I used 'cht-bsw-rt5677' as the default driver name, but this driver doesn't exist (and I doubt it ever will in the future). Should we use this 'template' approach, or add a 'cht-yogabook' driver entry only, or rename 'cht-yogabook' to the generic 'cht-bsw-rt5677'? -- Yauhen Kharuzhy