From: "Pradhan, Sanman" <sanman.pradhan@hpe.com>
To: Guenter Roeck <linux@roeck-us.net>,
Jonathan Corbet <corbet@lwn.net>, "Rob Herring" <robh@kernel.org>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
"Conor Dooley" <conor+dt@kernel.org>
Cc: "linux-hwmon@vger.kernel.org" <linux-hwmon@vger.kernel.org>,
"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
"linux-doc@vger.kernel.org" <linux-doc@vger.kernel.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
Sanman Pradhan <psanman@juniper.net>
Subject: [PATCH 0/3] hwmon: (pmbus/tps53679) Add TPS536C7
Date: Fri, 4 Sep 2026 22:08:51 +0000 [thread overview]
Message-ID: <20260904220843.495082-1-sanman.pradhan@hpe.com> (raw)
From: Sanman Pradhan <psanman@juniper.net>
TPS536C7 is a TI dual-channel D-CAP+ step-down controller. Its two
channels are distributed across a configurable number of phases (N+M,
up to 12 total), and whether channel B (PMBus page 1) exists depends on
the phase configuration, so the page count is derived at probe time.
Patch 1 fixes a pre-existing TPS53676 bug in the phase-configuration
parser (it tested the firing-order bit instead of the PAGE bit), which
miscounts channel-B phases on dual-channel parts; it is also a
prerequisite for reusing that parser for TPS536C7. Patch 2 adds the
compatible string. Patch 3 adds the driver support via a shared helper,
exposing aggregate per-channel telemetry only (TPS536C7 can exceed
PMBUS_PHASES, so per-phase telemetry is not reported).
Tested on a TPS536C7 in a QFX5230 using a Linux 5.15 backport of this
driver; the part was configured for channel A only. CAPABILITY read back
0xd0; block reads of IC_DEVICE_ID and USER_DATA_03 failed with -EBADMSG
when PEC was enabled, while the same reads without PEC returned the
expected 6- and 24-byte payloads, and byte/word telemetry kept working
with PEC. The series itself is built and checked against hwmon-next. The
dual-channel (page 1) path is exercised by inspection only, as no such
part was available.
Note: Senthil Muniyappan, credited in patch 3, has left the company.
Sanman Pradhan (3):
hwmon: (pmbus/tps53679) Fix TPS53676 phase page decoding
dt-bindings: trivial-devices: Add TI TPS536C7
hwmon: (pmbus/tps53679) Add support for TPS536C7
.../devicetree/bindings/trivial-devices.yaml | 2 +
Documentation/hwmon/tps53679.rst | 23 +++-
drivers/hwmon/pmbus/Kconfig | 2 +-
drivers/hwmon/pmbus/tps53679.c | 128 ++++++++++++++++--
4 files changed, 136 insertions(+), 19 deletions(-)
base-commit: ba08432bda66a7889d8f3d1581dabf10f59b25eb
--
2.34.1
next reply other threads:[~2026-09-04 22:09 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-09-04 22:08 Pradhan, Sanman [this message]
2026-09-04 22:08 ` [PATCH 1/3] hwmon: (pmbus/tps53679) Fix TPS53676 phase page decoding Pradhan, Sanman
2026-09-04 22:09 ` [PATCH 2/3] dt-bindings: trivial-devices: Add TI TPS536C7 Pradhan, Sanman
2026-09-07 17:36 ` Conor Dooley
2026-09-04 22:09 ` [PATCH] hwmon: (pmbus/tps53679) Add support for TPS536C7 Pradhan, Sanman
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=20260904220843.495082-1-sanman.pradhan@hpe.com \
--to=sanman.pradhan@hpe.com \
--cc=conor+dt@kernel.org \
--cc=corbet@lwn.net \
--cc=devicetree@vger.kernel.org \
--cc=krzk+dt@kernel.org \
--cc=linux-doc@vger.kernel.org \
--cc=linux-hwmon@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux@roeck-us.net \
--cc=psanman@juniper.net \
--cc=robh@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