From: Guenter Roeck <linux@roeck-us.net>
To: James Calligeros <jcalligeros99@gmail.com>
Cc: Sven Peter <sven@kernel.org>, Janne Grunau <j@jannau.net>,
Alyssa Rosenzweig <alyssa@rosenzweig.io>,
Neal Gompa <neal@gompa.dev>, Lee Jones <lee@kernel.org>,
Rob Herring <robh@kernel.org>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
Conor Dooley <conor+dt@kernel.org>,
Alexandre Belloni <alexandre.belloni@bootlin.com>,
Jean Delvare <jdelvare@suse.com>,
Dmitry Torokhov <dmitry.torokhov@gmail.com>,
Jonathan Corbet <corbet@lwn.net>,
asahi@lists.linux.dev, linux-arm-kernel@lists.infradead.org,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-rtc@vger.kernel.org, linux-hwmon@vger.kernel.org,
linux-input@vger.kernel.org, linux-doc@vger.kernel.org
Subject: Re: [PATCH v5 06/11] hwmon: Add Apple Silicon SMC hwmon driver
Date: Mon, 17 Nov 2025 11:00:48 -0800 [thread overview]
Message-ID: <7a445358-ce9e-4180-99dd-1a771c8ffa98@roeck-us.net> (raw)
In-Reply-To: <20251112-macsmc-subdevs-v5-6-728e4b91fe81@gmail.com>
On Wed, Nov 12, 2025 at 09:16:52PM +1000, James Calligeros wrote:
> The System Management Controller on Apple Silicon devices is responsible
> for integrating and exposing the data reported by the vast array of
> hardware monitoring sensors present on these devices. It is also
> responsible for fan control, and allows users to manually set fan
> speeds if they so desire. Add a hwmon driver to expose current,
> power, temperature, and voltage monitoring sensors, as well as
> fan speed monitoring and control via the SMC on Apple Silicon devices.
>
> The SMC firmware has no consistency between devices, even when they
> share an SoC. The FourCC keys used to access sensors are almost
> random. An M1 Mac mini will have different FourCCs for its CPU core
> temperature sensors to an M1 MacBook Pro, for example. For this
> reason, the valid sensors for a given device are specified in a
> child of the SMC Devicetree node. The driver uses this information
> to determine which sensors to make available at runtime.
>
> Reviewed-by: Neal Gompa <neal@gompa.dev>
> Acked-by: Guenter Roeck <linux@roeck-us.net>
> Co-developed-by: Janne Grunau <j@jannau.net>
> Signed-off-by: Janne Grunau <j@jannau.net>
> Signed-off-by: James Calligeros <jcalligeros99@gmail.com>
Applied to hwmon-next.
Note that I can not apply the devicetree patch (2/11), presumably since it depends
on the first patch of the series.
Guenter
next prev parent reply other threads:[~2025-11-17 19:00 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-11-12 11:16 [PATCH v5 00/11] mfd: macsmc: add rtc, hwmon and hid subdevices James Calligeros
2025-11-12 11:16 ` [PATCH v5 01/11] dt-bindings: rtc: Add Apple SMC RTC James Calligeros
2025-11-12 11:16 ` [PATCH v5 02/11] dt-bindings: hwmon: Add Apple System Management Controller hwmon schema James Calligeros
2026-05-25 13:33 ` Guenter Roeck
2025-11-12 11:16 ` [PATCH v5 03/11] rtc: Add new rtc-macsmc driver for Apple Silicon Macs James Calligeros
2025-11-12 11:16 ` [PATCH v5 04/11] mfd: macsmc: Wire up Apple SMC RTC subdevice James Calligeros
2025-11-12 11:16 ` [PATCH v5 05/11] mfd: macsmc: Add new __SMC_KEY macro James Calligeros
2025-11-20 13:44 ` Lee Jones
2025-11-28 10:36 ` James Calligeros
2025-11-12 11:16 ` [PATCH v5 06/11] hwmon: Add Apple Silicon SMC hwmon driver James Calligeros
2025-11-17 19:00 ` Guenter Roeck [this message]
2025-11-28 8:10 ` Dan Carpenter
2025-11-12 11:16 ` [PATCH v5 07/11] mfd: macsmc: Wire up Apple SMC hwmon subdevice James Calligeros
2025-11-12 11:16 ` [PATCH v5 08/11] input: macsmc-input: New driver to handle the Apple Mac SMC buttons/lid James Calligeros
2025-12-22 13:50 ` Nick Chan
2025-11-12 11:16 ` [PATCH v5 09/11] mfd: macsmc: Wire up Apple SMC input subdevice James Calligeros
2025-11-12 11:16 ` [PATCH v5 10/11] arm64: dts: apple: t8103,t60xx,t8112: Add SMC RTC node James Calligeros
2025-11-12 11:16 ` [PATCH v5 11/11] arm64: dts: apple: t8103, t8112, t60xx: Add hwmon SMC subdevice James Calligeros
2025-11-20 13:47 ` [PATCH v5 00/11] mfd: macsmc: add rtc, hwmon and hid subdevices Lee Jones
2025-12-08 22:09 ` (subset) " Alexandre Belloni
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=7a445358-ce9e-4180-99dd-1a771c8ffa98@roeck-us.net \
--to=linux@roeck-us.net \
--cc=alexandre.belloni@bootlin.com \
--cc=alyssa@rosenzweig.io \
--cc=asahi@lists.linux.dev \
--cc=conor+dt@kernel.org \
--cc=corbet@lwn.net \
--cc=devicetree@vger.kernel.org \
--cc=dmitry.torokhov@gmail.com \
--cc=j@jannau.net \
--cc=jcalligeros99@gmail.com \
--cc=jdelvare@suse.com \
--cc=krzk+dt@kernel.org \
--cc=lee@kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-doc@vger.kernel.org \
--cc=linux-hwmon@vger.kernel.org \
--cc=linux-input@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-rtc@vger.kernel.org \
--cc=neal@gompa.dev \
--cc=robh@kernel.org \
--cc=sven@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®