Re: [PATCH v2 07/14] dt-bindings: pincfg-node: Add input-voltage-microvolt property
From: Krzysztof Kozlowski
Date: Sat Mar 07 2026 - 07:52:59 EST
On Fri, Mar 06, 2026 at 03:52:37PM +0800, Yu-Chun Lin wrote:
> From: Tzuyi Chang <tychang@xxxxxxxxxxx>
>
> Add a generic pin configuration property "input-voltage-microvolt" to
> specify the input voltage level of a pin in microvolts.
Why?
>
> Signed-off-by: Tzuyi Chang <tychang@xxxxxxxxxxx>
> Co-developed-by: Yu-Chun Lin <eleanor.lin@xxxxxxxxxxx>
> Signed-off-by: Yu-Chun Lin <eleanor.lin@xxxxxxxxxxx>
> ---
> Documentation/devicetree/bindings/pinctrl/pincfg-node.yaml | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/pinctrl/pincfg-node.yaml b/Documentation/devicetree/bindings/pinctrl/pincfg-node.yaml
> index a916d0fc79a9..da182c8a1d00 100644
> --- a/Documentation/devicetree/bindings/pinctrl/pincfg-node.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/pincfg-node.yaml
> @@ -162,6 +162,10 @@ properties:
> this affects the expected delay in ps before latching a value to
> an output pin.
>
> + input-voltage-microvolt:
> + description: Specifies the input voltage level of the pin in microvolts.
> + This defines the reference for VIH/VIL.
What is VIH/VIL? Why generic binding would define the voltage of pins
for pin control?
This patch misses actual background why you are doing it and what sort
of common/typical pinctrl setup you describe.
Best regards,
Krzysztof