From: Guenter Roeck <linux@roeck-us.net>
To: u8813345@gmail.com, Rob Herring <robh@kernel.org>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
Conor Dooley <conor+dt@kernel.org>,
Kevin Chang <kevin.chang2@amd.com>,
Jonathan Corbet <corbet@lwn.net>,
Shuah Khan <skhan@linuxfoundation.org>
Cc: linux-hwmon@vger.kernel.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org, linux-doc@vger.kernel.org,
Colin Huang <colin.huang2@amd.com>
Subject: Re: [PATCH v3 2/3] Documentation/hwmon: add Delta E50SN12051 documentation
Date: Thu, 30 Apr 2026 13:18:49 -0700 [thread overview]
Message-ID: <f2cc69a9-9211-42d6-a349-17e67d1ae4b1@roeck-us.net> (raw)
In-Reply-To: <20260430-add-e50sn12051-v3-2-f6d4e043ec7c@gmail.com>
On 4/29/26 23:16, Colin Huang via B4 Relay wrote:
> From: Colin Huang <u8813345@gmail.com>
>
> Document the hardware monitoring support for the Delta E50SN12051
> device.
>
> The documentation describes the supported sensors exposed via the
> hwmon subsystem, including voltage, current, and temperature measurements.
>
> Signed-off-by: Colin Huang <u8813345@gmail.com>
For my reference:
Reviewed-by: Guenter Roeck <linux@roeck-us.net>
> ---
> Documentation/hwmon/e50sn12051.rst | 81 ++++++++++++++++++++++++++++++++++++++
> Documentation/hwmon/index.rst | 1 +
> 2 files changed, 82 insertions(+)
>
> diff --git a/Documentation/hwmon/e50sn12051.rst b/Documentation/hwmon/e50sn12051.rst
> new file mode 100644
> index 000000000000..cbbfa7895d82
> --- /dev/null
> +++ b/Documentation/hwmon/e50sn12051.rst
> @@ -0,0 +1,81 @@
> +.. SPDX-License-Identifier: GPL-2.0-or-later
> +
> +Kernel driver e50sn12051
> +========================
> +
> +Supported chips:
> +
> + * Delta E50SN12051
> +
> + Prefix: 'e50sn12051'
> +
> + Addresses scanned: -
> +
> + Datasheet:
> +
> + Provided by Delta upon request and NDA
> +
> +Description
> +-----------
> +
> +E50SN12051 is a 600W non-isolated 1/8th brick DC-DC power module.
> +
> +This driver supports the E50SN12051 PMBus compliant monitor device.
> +
> +The device provides measurements for:
> +
> +* Input voltage
> +* Output current
> +* Output voltage
> +* Temperature
> +
> +The driver is based on the PMBus core and exposes standard hwmon
> +sysfs attributes.
> +
> +Sysfs attributes
> +----------------
> +
> +======================= =======================================================
> +curr1_label "iout1"
> +curr1_crit Critical maximum current.
> + From IOUT_OC_FAULT_LIMIT register.
> +curr1_crit_alarm Current critical high alarm.
> + From IOUT_OC_FAULT status.
> +curr1_input Measured current.
> + From READ_IOUT register.
> +curr1_max Maximum current.
> + From IOUT_OC_WARN_LIMIT register.
> +curr1_max_alarm Current high alarm.
> + From IOUT_OC_WARN_LIMIT status.
> +
> +in1_label "vin"
> +in1_alarm Input voltage alarm.
> +in1_input Measured voltage.
> + From READ_VIN register.
> +
> +in2_label "vout1"
> +in2_crit Critical maximum voltage.
> + From VOUT_OV_FAULT_LIMIT register.
> +in2_crit_alarm Voltage critical high alarm.
> + From VOLTAGE_OV_FAULT status.
> +in2_input Measured voltage.
> + From READ_VOUT register.
> +in2_max Maximum voltage.
> + From VOUT_OV_WARN_LIMIT register.
> +in2_max_alarm Voltage high alarm.
> + From VOLTAGE_OV_WARNING status.
> +
> +temp1_crit Critical high temperature.
> + From OT_FAULT_LIMIT register.
> +temp1_crit_alarm Module temperature critical high alarm.
> + Set by comparing READ_TEMPERATURE_1 with OT_FAULT_LIMIT
> + if TEMP_OT_FAULT status is set.
> +temp1_input Measured module's hot spot temperature.
> + From READ_TEMPERATURE_1 register.
> +temp1_max Maximum temperature.
> + From OT_WARN_LIMIT register.
> +temp1_max_alarm Module temperature high alarm.
> + Set by comparing READ_TEMPERATURE_1 with OT_WARN_LIMIT if
> + TEMP_OT_WARNING status is set.
> +======================= =======================================================
> +
> diff --git a/Documentation/hwmon/index.rst b/Documentation/hwmon/index.rst
> index 8b655e5d6b68..ba08655fc496 100644
> --- a/Documentation/hwmon/index.rst
> +++ b/Documentation/hwmon/index.rst
> @@ -68,6 +68,7 @@ Hardware Monitoring Kernel Drivers
> drivetemp
> ds1621
> ds620
> + e50sn12051
> emc1403
> emc2103
> emc2305
>
next prev parent reply other threads:[~2026-04-30 20:18 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-04-30 6:16 [PATCH v3 0/3] add support for Delta E50SN12051 Colin Huang via B4 Relay
2026-04-30 6:16 ` [PATCH v3 1/3] dt-bindings: hwmon: pmbus: add Delta E50SN12051 binding Colin Huang via B4 Relay
2026-04-30 6:16 ` [PATCH v3 2/3] Documentation/hwmon: add Delta E50SN12051 documentation Colin Huang via B4 Relay
2026-04-30 20:18 ` Guenter Roeck [this message]
2026-04-30 6:16 ` [PATCH v3 3/3] hwmon: (pmbus) add support for Delta E50SN12051 Colin Huang via B4 Relay
2026-04-30 20:17 ` Guenter Roeck
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=f2cc69a9-9211-42d6-a349-17e67d1ae4b1@roeck-us.net \
--to=linux@roeck-us.net \
--cc=colin.huang2@amd.com \
--cc=conor+dt@kernel.org \
--cc=corbet@lwn.net \
--cc=devicetree@vger.kernel.org \
--cc=kevin.chang2@amd.com \
--cc=krzk+dt@kernel.org \
--cc=linux-doc@vger.kernel.org \
--cc=linux-hwmon@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=robh@kernel.org \
--cc=skhan@linuxfoundation.org \
--cc=u8813345@gmail.com \
/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