From nobody Tue Feb 10 00:58:23 2026 Received: from sender4-op-o15.zoho.com (sender4-op-o15.zoho.com [136.143.188.15]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 86959289374; Sat, 17 Jan 2026 21:01:24 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=pass smtp.client-ip=136.143.188.15 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1768683685; cv=pass; b=XzYEN6xH99nSBJsKs4bTYlbjfScrRoI/4pB4o4/StOjcnhQ9lnxEu/u7xZES0zr3k5ZCw4os9nwSUiw2YQ0Zzkrz5mwCW1zm+gaSKvZrw1LW8qjUWiWodoz4/MFbR3BumJN4LXraiHD7QVoTNjvLcENDo4JBqq7kxHVwOvjcuKc= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1768683685; c=relaxed/simple; bh=bJxx+BBuu5wOpM8I7XOoLpP/fg4e7MsHw458WiZKB8U=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=s/Q0Sg//vR6V9Ib4HyemETPpaWAtgVlfP9Jz5K9tINk4FPjCEmvzoolF+2Nh8kJSAeMxeGTnp5QNvXXdMza669no4+mPzJ+2rVSM26mWjoj6f2hxiGE96vm6FrOQXT3BErH55scba1jCR2ChXD2E6W9x4uDoMPE14StMO5S+fAQ= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=rong.moe; spf=pass smtp.mailfrom=rong.moe; dkim=pass (1024-bit key) header.d=rong.moe header.i=i@rong.moe header.b=097a69c0; arc=pass smtp.client-ip=136.143.188.15 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=rong.moe Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=rong.moe Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=rong.moe header.i=i@rong.moe header.b="097a69c0" ARC-Seal: i=1; a=rsa-sha256; t=1768683663; cv=none; d=zohomail.com; s=zohoarc; b=m/jEvTnWhMkOWIQWsdGqRrPq5tzLIA9OSIiIC82fDbVqLiBpjSvlaPUQOkYQ59roCFwlPBFXxv2R4o/pcp7hsdLL4TJ11O3Ck+uUOSqmRiCndkfws4IGBHzR6FCeCSvO0pUd20fmQFCpv0clgceB5p1bHe3XlG8jEftp2AfBLeA= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1768683663; h=Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:In-Reply-To:MIME-Version:Message-ID:References:Subject:Subject:To:To:Message-Id:Reply-To; bh=ONW/Kjo4Iy8zIk9zApmrQ4M3TAi6EzH5b9Jv+iGr0Ys=; b=Pyp0M+A5x42Y5tUT/2usNdd2k50JCKhfpziMu/08Zb3ePvPXc4Lq5jaw/IbBg9s9oKV3PYEgWsEAA9/0GudypamipNUaXMwtS8LFjc0ExRDFdNQx8rQzL99lApJGt6ggBbjXNewvlIYBbAbZl7zFPOcDjOeqOju/2rYrIErHurE= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass header.i=rong.moe; spf=pass smtp.mailfrom=i@rong.moe; dmarc=pass header.from= DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; t=1768683663; s=zmail; d=rong.moe; i=i@rong.moe; h=From:From:To:To:Cc:Cc:Subject:Subject:Date:Date:Message-ID:In-Reply-To:References:MIME-Version:Content-Transfer-Encoding:Message-Id:Reply-To; bh=ONW/Kjo4Iy8zIk9zApmrQ4M3TAi6EzH5b9Jv+iGr0Ys=; b=097a69c0ubrsp4U2QPZ6YTr1LJUKDgG3U8KvF5lYgrjUCM11zCJrAjj2/afd7Cbr sB8xBHJbGNbcz9Hv/vbzS2wB+2EwS02CYg+zJUpfO8hGRVKWBTnffoaxDDaSPMU++YZ j0IEVRLNXZUoOPUrxa+0tUMKO8rMOTu1PFak59BQ= Received: by mx.zohomail.com with SMTPS id 1768683662700355.7582306571446; Sat, 17 Jan 2026 13:01:02 -0800 (PST) From: Rong Zhang To: Mark Pearson , "Derek J. Clark" , Armin Wolf , Hans de Goede , =?UTF-8?q?Ilpo=20J=C3=A4rvinen?= Cc: Rong Zhang , Guenter Roeck , Kurt Borja , platform-driver-x86@vger.kernel.org, linux-kernel@vger.kernel.org, linux-hwmon@vger.kernel.org Subject: [PATCH v10 1/7] platform/x86: lenovo-wmi-helpers: Convert returned buffer into u32 Date: Sun, 18 Jan 2026 05:00:42 +0800 Message-ID: <20260117210051.108988-2-i@rong.moe> X-Mailer: git-send-email 2.51.0 In-Reply-To: <20260117210051.108988-1-i@rong.moe> References: <20260117210051.108988-1-i@rong.moe> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-ZohoMailClient: External Content-Type: text/plain; charset="utf-8" The Windows WMI-ACPI driver converts all ACPI objects into a common buffer format, so returning a buffer with four bytes will look like an integer for WMI consumers under Windows. Therefore, some devices may simply implement the corresponding ACPI methods to always return a buffer. While lwmi_dev_evaluate_int() expects an integer (u32), convert returned >=3D4B buffer into u32 to support these devices. Suggested-by: Armin Wolf Link: https://lore.kernel.org/r/f1787927-b655-4321-b9d9-bc12353c72db@gmx.de/ Signed-off-by: Rong Zhang Reviewed-by: Derek J. Clark Tested-by: Derek J. Clark Reviewed-by: Armin Wolf --- Changes in v7: - Fix mistakenly inverted boundary check Changes in v4: - Accept oversized buffer (thanks Armin Wolf) - Use get_unaligned_le32() (ditto) Changes in v2: - New patch (thanks Armin Wolf) --- drivers/platform/x86/lenovo/wmi-helpers.c | 21 ++++++++++++++++++--- 1 file changed, 18 insertions(+), 3 deletions(-) diff --git a/drivers/platform/x86/lenovo/wmi-helpers.c b/drivers/platform/x= 86/lenovo/wmi-helpers.c index f6fef6296251e..7379defac5002 100644 --- a/drivers/platform/x86/lenovo/wmi-helpers.c +++ b/drivers/platform/x86/lenovo/wmi-helpers.c @@ -21,6 +21,7 @@ #include #include #include +#include #include =20 #include "wmi-helpers.h" @@ -59,10 +60,24 @@ int lwmi_dev_evaluate_int(struct wmi_device *wdev, u8 i= nstance, u32 method_id, if (!ret_obj) return -ENODATA; =20 - if (ret_obj->type !=3D ACPI_TYPE_INTEGER) - return -ENXIO; + switch (ret_obj->type) { + /* + * The ACPI method may simply return a buffer when a u32 + * is expected. This is valid on Windows as its WMI-ACPI + * driver converts everything to a common buffer. + */ + case ACPI_TYPE_BUFFER: + if (ret_obj->buffer.length < sizeof(u32)) + return -ENXIO; =20 - *retval =3D (u32)ret_obj->integer.value; + *retval =3D get_unaligned_le32(ret_obj->buffer.pointer); + return 0; + case ACPI_TYPE_INTEGER: + *retval =3D (u32)ret_obj->integer.value; + return 0; + default: + return -ENXIO; + } } =20 return 0; --=20 2.51.0