From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f49.google.com (mail-wm1-f49.google.com [209.85.128.49]) (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 15B16280035 for ; Thu, 25 Dec 2025 17:13:58 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.49 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1766682843; cv=none; b=d18WxsCjdr0sOtPxzUvMLMMGV9e5Itp2pB3A9SYEALPo1Ig0EMIE4EOmu+z+mo9cqlzsnvcSb3+gAwj4QZS5o0oIbdZcB+AA9AWiZVo4CDeX3rGr+ABB76+D+gktJ0qC8WXyLRsbsWiWdd6CrxtvZ+HpKrPsGVO7bgzk9fzr/7E= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1766682843; c=relaxed/simple; bh=JyqiU+nSIoi30u5aBUuzbePbJQaUZjsH/VU6JqQh40w=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=ZcZA/ouaHQ9S/vw+35/Gxp24Poyf9BeBAaYtCjqcnDLWRkpN6NqdYfSCKt0n0EgVRDOrzADU7IGRovCHsSyoR5rO1RPlLAq8oTNb6wjF5F7xd7TnP4nitSgOHM8EHX8ogepjMxMtUhi84FJy3LW8Y6owOEJf9XuNPLIgZahs91I= 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=PaWpZv3y; arc=none smtp.client-ip=209.85.128.49 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="PaWpZv3y" Received: by mail-wm1-f49.google.com with SMTP id 5b1f17b1804b1-477632b0621so42428685e9.2 for ; Thu, 25 Dec 2025 09:13:58 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1766682837; x=1767287637; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=J02DSARA50fLgHgJBNysiJn5LkYB29lpMUN/ojmuv5g=; b=PaWpZv3yRMgtW0qlKQIJb9cbf3nRt9oNK1EUFRncFnajMn98U2VzE3LPDW7dgclWcJ t9l/qbGPLvtIjqih/Y+4KLesjH/QwJ8XAvdNVG44asV6OTUq8T9gohtSGXiOCBOMrMba k6g3NXYDegX05UIjKpBOkRxRuC3oo5XOEmyA1M7lrD2BhzmrCDfon24ZAF55zMrmj3N9 c4h2+/G2RBkT6HfX6WoDOcLz3npidmXlx9ZX1NJHRX54bz+CF1LOs8pShqIe4uuSKjxd qro0h8GFyTSgAYdS9PO9iylF4Wju8Kq1UGEXeo+NZc98Pwi3yUHwl2vkJ+Hf98RxEbtC evYg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1766682837; x=1767287637; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-gg:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=J02DSARA50fLgHgJBNysiJn5LkYB29lpMUN/ojmuv5g=; b=DVE20c8I77OFmf6f67Thp7GHNKCE2MSCeP8uYby/OF37qac1yu91xio9zjnYGVsk6V aSmYDHXSh2Ya8Rf2UaOYQgLec5dz8f8armvACNb45pztLKltPy5SedEd0JeEAIo6tJva pEDgAnHoR2rkZhb6b3CYLrKQYZGQWln90oJc68J9yOxy5AM/WiQatbpBudFwfC4N7dl/ 6R+dbNH4ITCXYnSfCD3UbVS1+b12e4OLC6V/XxDzUS+XI1+sVAvXxgIpUHL9j9VW1vZu 8zF5uKuvuba8i7LFObqPEtllRZH4XOm3IorooHzni0WMOtGZqiFOJjK8ujkOkPekthYS nu0Q== X-Forwarded-Encrypted: i=1; AJvYcCXCSKpHgtY+GC3e19t2idTkjDIL2gCwyo5rNFM+OslwbS+cU5qtm+9ahNLgp822zDo8h3FNUZ5c5edKiwE=@vger.kernel.org X-Gm-Message-State: AOJu0YzKNgdmN9BUzuVq9Pgfxo9cTJYXuwLCqPM4iZv8C+hTO0dI/60Q X+9iqvSSUWadyYpgtw5+B+wiQyVZz10OPeTeN1Riq5bCunFLYvsQquQK X-Gm-Gg: AY/fxX5dgH2s1awnzrKWnloaajxbOs2ul1OZqGp6Y5s4s/jryQDxMTY0AKVU3X9sTIm QRF5nc7yQzF9wJZdID7bLmKjyRDkSAAPuFRipJwakdl0eHZsWG9Zv5sxxjYgjzWkEhgCBI8KPYC iUQ1eTxjp8iGRlgcEXlIYOhikbrIZUQVtdAN/mESb98kF/IcmDcl/n3SvnaM1g+MUU76iysL5e1 GLmNKj7f83YLcHY5oPpKVBraTvHtphT69WM89eACXqMJJxQWCJjc07BA4yYBgJp6tkhKJD5tO1f ui2c4r7b/5F6ukQsW235CCMBy2hR6mwMRA8vxfxgoII04geltU8AxV+DdHEiCw8g8dU3yxsfk3f kN7CRikh/t0oloeWTz7BV4i/GLV3ZN/3zxh3wC0/roskz0u+hTpag0dSXKO9hn0XjfE3bvVX1Kw DB6n+kRWxagHyN8HJSgUJRxQU= X-Google-Smtp-Source: AGHT+IHoZhq8SVAipdNH7xE9wyhQnfuq3z56aGjMvtz7PZ/zOPLpecF3r2gK41lOG6sx9jcWcULjGw== X-Received: by 2002:a05:600c:1d1d:b0:477:abea:9023 with SMTP id 5b1f17b1804b1-47d19577fb8mr217809585e9.9.1766682836550; Thu, 25 Dec 2025 09:13:56 -0800 (PST) Received: from [192.168.1.121] ([176.206.93.222]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-47be2723d19sm434751195e9.2.2025.12.25.09.13.55 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Thu, 25 Dec 2025 09:13:56 -0800 (PST) Message-ID: Date: Thu, 25 Dec 2025 18:13:55 +0100 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH] platform/x86: asus-nb-wmi: add quirk_asus_ignore_fan for UX3405MA To: Tim Wassink Cc: Corentin Chary , "Luke D. Jones" , Hans de Goede , =?UTF-8?Q?Ilpo_J=C3=A4rvinen?= , platform-driver-x86@vger.kernel.org, linux-kernel@vger.kernel.org References: <20251221210218.12274-1-timwassink.dev@gmail.com> <74f1a53d-3556-4e67-8dc2-2a86d6c52365@gmail.com> Content-Language: en-US, it-IT, en-US-large From: Denis Benato In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit On 12/25/25 00:43, Tim Wassink wrote: > Hi, >   > yes the only problem is the error in dmesg, the rest seems to work. Then I think we can leave this as-is, unless there is a particular reason to ensure no errors are being reported in this case. This is not the only module that does things this way, so I don't think it's going to be a problem. Thank you for your time and looking into the matter :) > > On Tue, Dec 23, 2025 at 2:50 PM Denis Benato wrote: >> >> On 12/22/25 11:44, Tim Wassink wrote: >>> On Sun, Dec 21, 2025 at 10:01 PM Denis Benato wrote: >>>> On 12/21/25 22:01, Tim Wassink wrote: >>>>> The ASUS Zenbook 14 (UX3405MA) uses a newer WMI interface for thermal >>>>> management that does not support the legacy WMI fan control methods. >>>>> Currently, this results in ENODEV (-19) errors in dmesg when the driver >>>>> attempts to fetch factory fan curve defaults. >>>>> >>>>> Add a quirk to use quirk_asus_ignore_fan to silence these errors and >>>>> signal that legacy fan control is intentionally unsupported, as thermal >>>>> policies are handled through the platform_profile interface. >>>> It is my understanding that this patch suppresses the error, >>>> while maintaining the current behavior in every other aspect, >>>> correct? >>> I am actually not sure anymore if this patch is ok, as it introduces a >>> trade-off regarding telemetry. >>> >>> While the patch successfully silences the ENODEV (-19) errors, I have >>> verified locally that it also inhibits the registration of the fan >>> telemetry in hwmon. This means the read-only 'fan1_input' (RPM) is no >>> longer available to the user. >>> >>> I initially followed the precedent of commit 82cc5c6c624c ("platform/x86: >>> asus-wmi: Ignore fan on E410MA"), which can be found here: >>> https://lore.kernel.org/all/20221221-asus-fan-v1-2-e07f3949725b@weissschuh.net/ >>> >>> Now I realize that case was specifically for a fanless system. The >>> UX3405MA does have a physical fan >>> which remains functional via the platform_profile (AIPT) interface >>> after this patch, as confirmed by stress testing. >>> >>> I thought this quirk was the standard way to handle it, but I am not >>> so sure anymore. >> Hi, >> >> I think what's happening is that asus-nb-wmi is binding multiple devices: for those >> that are unsupported the driver returns -ENODEV and the kernel, as a result, >> probes successive drivers for those devices that can't be bound. >> >> It's very likely that using that quirk prevents asus-nb-wmi to bind all devices, >> and this can very well result in what you are seeing... >> >> The only runtime problem is that an error appears in dmesg, right? The rest is >> working from what I can understand from your messages. >>> Best regards, >>> Tim >>> >>> On Mon, Dec 22, 2025 at 3:54 AM Denis Benato wrote: >>>> On 12/21/25 22:01, Tim Wassink wrote: >>>>> The ASUS Zenbook 14 (UX3405MA) uses a newer WMI interface for thermal >>>>> management that does not support the legacy WMI fan control methods. >>>>> Currently, this results in ENODEV (-19) errors in dmesg when the driver >>>>> attempts to fetch factory fan curve defaults. >>>>> >>>>> Add a quirk to use quirk_asus_ignore_fan to silence these errors and >>>>> signal that legacy fan control is intentionally unsupported, as thermal >>>>> policies are handled through the platform_profile interface. >>>> It is my understanding that this patch suppresses the error, >>>> while maintaining the current behavior in every other aspect, >>>> correct? >>>>> Signed-off-by: Tim Wassink >>>>> --- >>>>> drivers/platform/x86/asus-nb-wmi.c | 9 +++++++++ >>>>> 1 file changed, 9 insertions(+) >>>>> >>>>> diff --git a/drivers/platform/x86/asus-nb-wmi.c b/drivers/platform/x86/asus-nb-wmi.c >>>>> index 6a62bc5b02fd..eaa8abe506cb 100644 >>>>> --- a/drivers/platform/x86/asus-nb-wmi.c >>>>> +++ b/drivers/platform/x86/asus-nb-wmi.c >>>>> @@ -544,6 +544,15 @@ static const struct dmi_system_id asus_quirks[] = { >>>>> }, >>>>> .driver_data = &quirk_asus_zenbook_duo_kbd, >>>>> }, >>>>> + { >>>>> + .callback = dmi_matched, >>>>> + .ident = "ASUS Zenbook 14 UX3405MA", >>>>> + .matches = { >>>>> + DMI_MATCH(DMI_SYS_VENDOR, "ASUSTeK COMPUTER INC."), >>>>> + DMI_MATCH(DMI_PRODUCT_NAME, "UX3405MA"), >>>>> + }, >>>>> + .driver_data = &quirk_asus_ignore_fan, >>>>> + }, >>>>> { >>>>> .callback = dmi_matched, >>>>> .ident = "ASUS ROG Z13",