Re: [PATCH v5 1/3] dt-bindings: mfd: rk8xx: add board level power hold GPIOs

[Lee Jones <lee@kernel.org>]

On Tue, 27 Oct 2026, Yaozhong Li wrote:

> Some boards route "power hold" lines from the SoC into the board's power
> circuitry. Those lines have to be held asserted while the system is
> running and released before the PMIC shutdown request is written,
> otherwise the rails can drop and come back up instead of staying off.
> 
> Add power-hold-gpios to describe those lines and power-hold-delay-ms for
> the settle time. The driver reads them for every RK8xx variant that is
> the system power controller, so add them to all seven RK8xx schemas.
> 
> The array is bounded at two entries. Firefly's vendor kernel handles
> exactly two such lines, pmic,hold-gpio and pmic,stby-gpio, and its board
> files use one or both of them, with an RK808 and with an RK818.
> 
> The lines are released from the PMIC's power-off path, so they are only
> meaningful when the PMIC is the system power controller. Require that.
> A board adding these lines is expected to switch to the generic
> system-power-controller property at the same time, so the deprecated
> rockchip,system-power-controller spelling is not accepted here.
> 
> Assisted-by: LLM
> Signed-off-by: Yaozhong Li <yaozhonguwl@gmail.com>
> ---
>  .../bindings/mfd/rockchip,rk801.yaml          | 24 +++++++++++++++++++
>  .../bindings/mfd/rockchip,rk805.yaml          | 24 +++++++++++++++++++
>  .../bindings/mfd/rockchip,rk806.yaml          | 24 +++++++++++++++++++
>  .../bindings/mfd/rockchip,rk808.yaml          | 24 +++++++++++++++++++
>  .../bindings/mfd/rockchip,rk816.yaml          | 24 +++++++++++++++++++
>  .../bindings/mfd/rockchip,rk817.yaml          | 24 +++++++++++++++++++
>  .../bindings/mfd/rockchip,rk818.yaml          | 24 +++++++++++++++++++

Whoa! Is duplicating 200 lines really the best solution?

-- 
Lee Jones