Re: [PATCH v9 3/9] regulator: dt-bindings: Add MediaTek MT6392 PMIC
From: sashiko-bot@kernel.org
Date: 2026-08-20 12:39:32
Also in:
linux-devicetree
Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider: - [Low] The devicetree binding exposes `avddldo-supply` as a configurable property, but the description explicitly notes it is an 'internal supply derived from vsys'. Devicetree is intended to describe board-level hardware configurations (like external pin connections or PCB routing). Hardwired internal IC routing is a static property of the chip and should be modeled internally within the driver, rather than exposed as a configurable devicetree property. --
--- Patch [3]: [PATCH v9 3/9] regulator: dt-bindings: Add MediaTek MT6392 PMIC ---commit 2d8836e1efcb0c65952c3a56b41f16c4a50e3f1e Author: Luca Leonardo Scorcia [off-list ref] regulator: dt-bindings: Add MediaTek MT6392 PMIC This commit adds devicetree bindings for the voltage regulators found in the MediaTek MT6392 PMIC. It describes the buck converters, LDOs, and their allowed modes for board designs using MT8516/MT8167 SoCs.
quoted hunk ↗ jump to hunk
diff --git a/Documentation/devicetree/bindings/regulator/mediatek,mt6392-regulator.yaml b/Documentation/devicetree/bindings/regulator/mediatek,mt6392-regulator.yaml new file mode 100644 index 0000000000000..d74721d8f2ffc --- /dev/null +++ b/Documentation/devicetree/bindings/regulator/mediatek,mt6392-regulator.yaml
[ ... ]
quoted hunk ↗ jump to hunk
@@ -21,12 +21,15 @@ properties: vproc-supply: description: Supply for buck regulator vproc vcore-supply: description: Supply for buck regulator vcore vsys-supply: description: Supply for buck regulator vsys avddldo-supply: description: Supply for AVDD LDOs (vm, vio18, vcn18, vcamd, vcamio). According to the data sheet this is an internal supply derived from vsys.
[Severity: Low] Since this is an internal supply derived from vsys, should it be exposed as a configurable devicetree property? Devicetree is typically intended to describe board-level hardware configurations like external pin connections. Since the hardwired internal IC routing is a static property of the chip, would it be better to model this internally within the driver rather than exposing it as a configurable devicetree property?
ldo1-supply:
description: Supply for LDOs group 1 (vaud28, vxo22, vaud22, vadc18, vcama, vrtc)-- Sashiko AI review · https://sashiko.dev/#/patchset/20260820122242.120039-1-l.scorcia@gmail.com?part=3