Re: [PATCH 1/6] dt-bindings: regulator: pca9450: Document new usage of sd-vsel-gpios
From: Rob Herring <robh@kernel.org>
Date: 2023-02-15 20:02:20
Also in:
lkml
On Mon, Feb 13, 2023 at 04:58:19PM +0100, Frieder Schrempf wrote:
quoted hunk ↗ jump to hunk
From: Frieder Schrempf <redacted> The sd-vsel-gpios property is abandoned in its current meaning as an output. We now use it to specify an optional signal that can be evaluated by the driver in order to retrieve the current status of the SD_VSEL signal that is used to select the control register of LDO5. Signed-off-by: Frieder Schrempf <redacted> --- .../regulator/nxp,pca9450-regulator.yaml | 23 ++++++++++++++----- 1 file changed, 17 insertions(+), 6 deletions(-)diff --git a/Documentation/devicetree/bindings/regulator/nxp,pca9450-regulator.yaml b/Documentation/devicetree/bindings/regulator/nxp,pca9450-regulator.yaml index 835b53302db8..c86534538a4e 100644 --- a/Documentation/devicetree/bindings/regulator/nxp,pca9450-regulator.yaml +++ b/Documentation/devicetree/bindings/regulator/nxp,pca9450-regulator.yaml@@ -40,8 +40,24 @@ properties: description: | list of regulators provided by this controller + properties: + LDO5: + type: object + $ref: regulator.yaml# + description: + Properties for single LDO5 regulator. + + properties: + sd-vsel-gpios:
It is a pin on the device, right? Then it belongs in the device node as it was. Can't the direction of the signal tell you how it is used? Assuming the pin is bidirectional? The binding should support any possible way the device is wired, not just what's been seen so far on some boards. Rob