From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-lf1-f54.google.com (mail-lf1-f54.google.com [209.85.167.54]) (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 73A6A43A7FB for ; Thu, 13 Aug 2026 09:14:21 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.167.54 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786612463; cv=none; b=iF9wxyWZ9pgylCntXK+4zb1a+ohZ7kWQ6WyQe38EjdOdpXdg2KNB7w8W88Rc5eazi14Bor1iuSDLtDCy9D/WouCELrJjeFAtNsc7al7+Oc+Bn8g4J6M2FOCPu8KG3LVyT0fBnYyFQXKpARgxKtm7YmjJZuIohqsCMyd/AwAT/Lg= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786612463; c=relaxed/simple; bh=eTxYr7/2XsqiLDOP+IV1sr/jtIbiCrXnNgl17wlllwM=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=tyPnnXsOTbFRr/KEuhfpkAZBYMpWydQfYfEYZ5N65Ish8UecvrjMLKqn7FDXGQU7g1NtXERvdhbR20Z5nmUNRGU9T9rPQyc4eE0tRH8JaxKAiERmqUEIw8SUldoocUoa6yNFm0ZGu440SyTofNsISGXDGgSNsfoyGlhQVj3gWNA= 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=mVweQqsl; arc=none smtp.client-ip=209.85.167.54 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="mVweQqsl" Received: by mail-lf1-f54.google.com with SMTP id 2adb3069b0e04-5b01910b499so696424e87.0 for ; Thu, 13 Aug 2026 02:14:21 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1786612459; x=1787217259; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to:content-type; bh=kibDlYaW2eoOH72N4rPk6CUa5B6hNZkx+E4GjRXceJA=; b=mVweQqslVaLzBZ5X8jYsU9/58k95j5O2zGOd8d4fzx99Tahwt6tnNQazupfj6woIaK P3ncWzavUQAXPkzUquVNGTjf/dmS/ilkq++dFvdlMgoOigKvYOPzBlAIIRCbxtxSTqM7 b7Szd5lwRNZFPtggjG8Jw/oDsGWD0kNdQtrWLgc9CUzWNA5BTIXQvOtnG1d8jQACFCeO IKX84PBYjbAbtQAakXylTzSJvVcuINUC/lmxi5IymCU6i7iSJP3C+552G8QJTf0fqdLl bhhAGhAI3LvTl2bBEPDU3Mmk5g4xjT1AI3adxLOKs3FgkiL6zIDGrxnhc6gx4sdNTGz9 N/AQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1786612459; x=1787217259; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to:content-type; bh=kibDlYaW2eoOH72N4rPk6CUa5B6hNZkx+E4GjRXceJA=; b=re2SjwYG0cP8mpiqiN0TVL8zh7kZ5E50+5XdUqyqgNkzRqqafSA6eK+TbjVlJqequI /n4j0l2YMa1w99kMO2KuaCpPYAV4JlD1wSDUi83Po05M75ra9CZKdF9dB+YshcLtuEbm LMPQHaEn/92/CSj1lIslLkoZU+VpN0+PzpMtMxTW0KC0odHBQCAj7j3r8X36dKnNutpZ 17vkFtFG9XHOJk2LWG/9/VkxAjUpJgFh9MTa8omUsQl8U7+WIBzKaCBqQ7b6UFVHzVih iEoGBNSt7eZ55iRePtjrNAnj/LdoZRqOfbHUr6yjJkf1r3ghhb96V/6OfOAM2/Of4wOF 9UTQ== X-Forwarded-Encrypted: i=1; AHgh+Ron0RRf/URV0Tt8LCcy+5lIvsYJwkXuitHpHNDXMHeMZQC1ZK4W6+wL0SpF1jWdMDiIYe++uaNLFOg1Q84=@vger.kernel.org X-Gm-Message-State: AOJu0Yz/+swYaB+j4WMT6rZ5I1nlAgPSwR8sxq+9zSZNpmq7gSOB68u1 QFhUqCwb1mQbscJyqh3nC9SyGF4NXBdZOVdhlbkHZyJTldaR875GWc67 X-Gm-Gg: AR+sD11tbrCc4qLFQImkScuDMA1PYhgjUx+e4tAe+uMRqjw/rxDeeMTXw2X1XUam9he oqYZtIUXlLQDcbQU7Buuj065nR22SrHbMy2ZnVYoHs1Ue1c3spA2l3hRWhYK42KN/HnW3PyjeVv UTrw47Io79iiMrpXJ95vuukQJ2iZn9Y9hfUSutPz2E+NPIVpueP4VWqZ+I9E7rIDdsEq5n0MvaD 50OtN41w+p6r1lMIscLhztsaRlkXAIGTYxksliU0iPuvDtpbAlYjARqAwYB+O9vkGSncaFwIbd3 OKAWIB+flNcTiHuzZ3U4m7/Xbx+9gb3CZIbYWUWvPZ/DidHdHP9ymZdThK3WbcFAcoW1B4vm4Om dSt0LQXT4u5/1tFuQa0wvVWk6x9DSdSnxAR3cqaf+Ah5pN1jbnJoAirLHYnOergf316Pl+Ig3jg 8mMjiaV0Ea0oySkK4hWgIXUfaJERwXYw3UVsaucjjS2xi9TW3aFtCQgD9Qm74AuiSxyZ+weEbtd 7Mt51Xp X-Received: by 2002:ac2:5458:0:b0:5aa:62df:cd0f with SMTP id 2adb3069b0e04-5b453f474dfmr590881e87.2.1786612458867; Thu, 13 Aug 2026 02:14:18 -0700 (PDT) Received: from jaennil-honor.localdomain ([173.233.147.68]) by smtp.gmail.com with ESMTPSA id 2adb3069b0e04-5b453a548besm342679e87.42.2026.08.13.02.14.14 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 13 Aug 2026 02:14:17 -0700 (PDT) From: Nikita Dubrovskih To: Guenter Roeck , linux-hwmon@vger.kernel.org Cc: Hans de Goede , =?UTF-8?q?Ilpo=20J=C3=A4rvinen?= , platform-driver-x86@vger.kernel.org, linux-kernel@vger.kernel.org, Nikita Dubrovskih Subject: [RFC PATCH v2] hwmon: Add fan monitoring support for HONOR FMI-XX Date: Thu, 13 Aug 2026 12:14:11 +0300 Message-ID: <20260813091411.1186193-1-testname142@gmail.com> X-Mailer: git-send-email 2.55.0 In-Reply-To: <20260812192242.859307-1-testname142@gmail.com> References: <20260812192242.859307-1-testname142@gmail.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit The HONOR FMI-XX firmware exposes a serialized \\GFNS ACPI method. It returns a status byte and a 16-bit fan speed in RPM for either of two firmware channels. Add a DMI-restricted, read-only hwmon driver using that firmware interface. The driver deliberately exposes no fan control or direct Embedded Controller access. The interface was validated on firmware 1.09 with fan channel 0 reporting approximately 2500-2800 RPM. Channel 1 is readable and remained at 0 RPM during idle and a short CPU load. Signed-off-by: Nikita Dubrovskih --- Changes in v2: - Drop the redundant mutex and callback argument validation. - Remove unused platform drvdata and simplify the init error path. - Align the macro definition and order the MAINTAINERS entry. Documentation/hwmon/honor-fmi.rst | 32 ++++++ Documentation/hwmon/index.rst | 1 + MAINTAINERS | 7 ++ drivers/hwmon/Kconfig | 10 ++ drivers/hwmon/Makefile | 1 + drivers/hwmon/honor-fmi.c | 178 ++++++++++++++++++++++++++++++ 6 files changed, 229 insertions(+) create mode 100644 Documentation/hwmon/honor-fmi.rst create mode 100644 drivers/hwmon/honor-fmi.c diff --git a/Documentation/hwmon/honor-fmi.rst b/Documentation/hwmon/honor-fmi.rst new file mode 100644 index 0000000..a42a1dd --- /dev/null +++ b/Documentation/hwmon/honor-fmi.rst @@ -0,0 +1,32 @@ +.. SPDX-License-Identifier: GPL-2.0-only + +Kernel driver honor-fmi +======================= + +Supported systems: + + * HONOR FMI-XX + +Author: Nikita Dubrovskih + +Description +----------- + +The driver provides read-only monitoring of the fan speed on the HONOR FMI-XX. +The system firmware implements a ``GFNS`` ACPI method which returns the speed +of one of two firmware fan channels in RPM. Embedded Controller access and +serialization are handled by the firmware method. + +The driver does not expose fan control or direct Embedded Controller access. + +Sysfs entries +------------- + +The following attributes are supported: + +======================= ======= ============================================= +Name Perm Description +======================= ======= ============================================= +``fan1_input`` RO Fan channel 0 speed in RPM +``fan2_input`` RO Fan channel 1 speed in RPM +======================= ======= ============================================= diff --git a/Documentation/hwmon/index.rst b/Documentation/hwmon/index.rst index 29130df..91052fa 100644 --- a/Documentation/hwmon/index.rst +++ b/Documentation/hwmon/index.rst @@ -90,6 +90,7 @@ Hardware Monitoring Kernel Drivers gxp-fan-ctrl hac300s hih6130 + honor-fmi hp-wmi-sensors hs3001 htu31 diff --git a/MAINTAINERS b/MAINTAINERS index 8014b9f..7f2e3f8 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -11876,6 +11876,13 @@ S: Maintained F: Documentation/devicetree/bindings/iio/pressure/honeywell,mprls0025pa.yaml F: drivers/iio/pressure/mprls0025pa* +HONOR FMI-XX HARDWARE MONITOR DRIVER +M: Nikita Dubrovskih +L: linux-hwmon@vger.kernel.org +S: Maintained +F: Documentation/hwmon/honor-fmi.rst +F: drivers/hwmon/honor-fmi.c + HP BIOSCFG DRIVER M: Jorge Lopez L: platform-driver-x86@vger.kernel.org diff --git a/drivers/hwmon/Kconfig b/drivers/hwmon/Kconfig index 2bfbcc0..8a11a30 100644 --- a/drivers/hwmon/Kconfig +++ b/drivers/hwmon/Kconfig @@ -2785,6 +2785,16 @@ config SENSORS_ASUS_EC This driver can also be built as a module. If so, the module will be called asus_ec_sensors. +config SENSORS_HONOR_FMI + tristate "HONOR FMI-XX fan monitor" + depends on X86 + help + If you say yes here, you get support for fan speed monitoring on + the HONOR FMI-XX laptop through its firmware ACPI method. + + This driver can also be built as a module. If so, the module + will be called honor-fmi. + config SENSORS_HP_WMI tristate "HP WMI Sensors" depends on ACPI_WMI diff --git a/drivers/hwmon/Makefile b/drivers/hwmon/Makefile index 63effc0..e098793 100644 --- a/drivers/hwmon/Makefile +++ b/drivers/hwmon/Makefile @@ -11,6 +11,7 @@ obj-$(CONFIG_SENSORS_ACPI_POWER) += acpi_power_meter.o obj-$(CONFIG_SENSORS_ATK0110) += asus_atk0110.o obj-$(CONFIG_SENSORS_ASUS_EC) += asus-ec-sensors.o obj-$(CONFIG_SENSORS_ASUS_WMI) += asus_wmi_sensors.o +obj-$(CONFIG_SENSORS_HONOR_FMI) += honor-fmi.o obj-$(CONFIG_SENSORS_HP_WMI) += hp-wmi-sensors.o # Native drivers diff --git a/drivers/hwmon/honor-fmi.c b/drivers/hwmon/honor-fmi.c new file mode 100644 index 0000000..3421e40 --- /dev/null +++ b/drivers/hwmon/honor-fmi.c @@ -0,0 +1,178 @@ +// SPDX-License-Identifier: GPL-2.0-only +/* + * Read-only fan monitoring for the HONOR FMI-XX. + * + * The firmware-provided \GFNS ACPI method accepts a three-byte buffer. + * Byte 2 selects fan 0 or 1. It returns a status byte followed by a + * little-endian 16-bit fan speed in RPM. The method owns all Embedded + * Controller access and serialization; this driver deliberately exposes no + * fan control interface. + */ + +#include +#include +#include +#include +#include +#include + +#define HONOR_FMI_GFNS_RESULT_SIZE 3 + +struct honor_fmi_data { + acpi_handle gfns; +}; + +static const struct dmi_system_id honor_fmi_dmi_table[] = { + { + .matches = { + DMI_MATCH(DMI_SYS_VENDOR, "HONOR"), + DMI_EXACT_MATCH(DMI_PRODUCT_NAME, "FMI-XX"), + }, + }, + {} +}; +MODULE_DEVICE_TABLE(dmi, honor_fmi_dmi_table); + +static int honor_fmi_read_rpm(struct honor_fmi_data *data, int channel, + long *rpm) +{ + union acpi_object input = { + .buffer = { + .type = ACPI_TYPE_BUFFER, + .length = 3, + }, + }; + struct acpi_object_list arguments = { + .count = 1, + .pointer = &input, + }; + struct acpi_buffer output = { ACPI_ALLOCATE_BUFFER, NULL }; + union acpi_object *result; + u8 input_bytes[3] = { 0, 0, channel }; + acpi_status status; + int ret = 0; + + input.buffer.pointer = input_bytes; + + status = acpi_evaluate_object(data->gfns, NULL, &arguments, &output); + if (ACPI_FAILURE(status)) + return -EIO; + + result = output.pointer; + if (!result || result->type != ACPI_TYPE_BUFFER || + result->buffer.length < HONOR_FMI_GFNS_RESULT_SIZE) { + ret = -EPROTO; + goto out_free; + } + + if (result->buffer.pointer[0]) { + ret = -EIO; + goto out_free; + } + + *rpm = result->buffer.pointer[1] | + (result->buffer.pointer[2] << 8); + +out_free: + kfree(output.pointer); + return ret; +} + +static umode_t honor_fmi_is_visible(const void *data, + enum hwmon_sensor_types type, u32 attr, + int channel) +{ + return 0444; +} + +static int honor_fmi_read(struct device *dev, enum hwmon_sensor_types type, + u32 attr, int channel, long *value) +{ + struct honor_fmi_data *data = dev_get_drvdata(dev); + + return honor_fmi_read_rpm(data, channel, value); +} + +static const struct hwmon_ops honor_fmi_hwmon_ops = { + .is_visible = honor_fmi_is_visible, + .read = honor_fmi_read, +}; + +static const struct hwmon_channel_info * const honor_fmi_hwmon_info[] = { + HWMON_CHANNEL_INFO(fan, HWMON_F_INPUT, HWMON_F_INPUT), + NULL +}; + +static const struct hwmon_chip_info honor_fmi_chip_info = { + .ops = &honor_fmi_hwmon_ops, + .info = honor_fmi_hwmon_info, +}; + +static int honor_fmi_probe(struct platform_device *pdev) +{ + struct honor_fmi_data *data; + struct device *hwmon_dev; + acpi_status status; + + if (!dmi_check_system(honor_fmi_dmi_table)) + return -ENODEV; + + data = devm_kzalloc(&pdev->dev, sizeof(*data), GFP_KERNEL); + if (!data) + return -ENOMEM; + + status = acpi_get_handle(NULL, "\\GFNS", &data->gfns); + if (ACPI_FAILURE(status)) + return dev_err_probe(&pdev->dev, -ENODEV, + "firmware does not provide \\GFNS\n"); + + hwmon_dev = devm_hwmon_device_register_with_info(&pdev->dev, "honor_fmi", + data, + &honor_fmi_chip_info, + NULL); + return PTR_ERR_OR_ZERO(hwmon_dev); +} + +static struct platform_driver honor_fmi_driver = { + .probe = honor_fmi_probe, + .driver = { + .name = "honor-fmi-hwmon", + }, +}; + +static struct platform_device *honor_fmi_device; + +static int __init honor_fmi_init(void) +{ + int ret; + + if (!dmi_check_system(honor_fmi_dmi_table)) + return -ENODEV; + + ret = platform_driver_register(&honor_fmi_driver); + if (ret) + return ret; + + honor_fmi_device = platform_device_register_simple("honor-fmi-hwmon", + PLATFORM_DEVID_NONE, + NULL, 0); + if (IS_ERR(honor_fmi_device)) { + platform_driver_unregister(&honor_fmi_driver); + return PTR_ERR(honor_fmi_device); + } + + return 0; +} + +static void __exit honor_fmi_exit(void) +{ + platform_device_unregister(honor_fmi_device); + platform_driver_unregister(&honor_fmi_driver); +} + +module_init(honor_fmi_init); +module_exit(honor_fmi_exit); + +MODULE_AUTHOR("Nikita Dubrovskih "); +MODULE_DESCRIPTION("HONOR FMI-XX fan speed monitor"); +MODULE_LICENSE("GPL"); -- 2.55.0