Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Wed, 4 May 2022 13:55:53 +0100
From:   Mark Brown <broonie@kernel.org>
To:     Zev Weiss <zev@bewilderbeest.net>
Cc:     Liam Girdwood <lgirdwood@gmail.com>,
        Rob Herring <robh+dt@kernel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        linux-kernel@vger.kernel.org, devicetree@vger.kernel.org,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        openbmc@lists.ozlabs.org
Subject: Re: [PATCH 2/6] dt-bindings: regulator: Add reg-external-output
 binding
Message-ID: <YnJ32bG4IkSrRtHV@sirena.org.uk>
References: <20220504065252.6955-1-zev@bewilderbeest.net>
 <20220504065252.6955-2-zev@bewilderbeest.net>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
        protocol="application/pgp-signature"; boundary="OeS2mizHPjpFFiLa"
Content-Disposition: inline
In-Reply-To: <20220504065252.6955-2-zev@bewilderbeest.net>
Precedence: bulk


--OeS2mizHPjpFFiLa
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline

On Tue, May 03, 2022 at 11:52:48PM -0700, Zev Weiss wrote:
> This describes an external output supplied by a regulator, such as a
> power outlet on a power distribution unit (PDU).

OK, so this is that represnetation of the connection - which raises the
question about why the regulator needs a property?

> +description: |
> +  This describes an external output supplied by a regulator, such as
> +  a power outlet on a power distribution unit (PDU).
> +
> +properties:
> +  compatible:
> +    const: reg-external-output
> +
> +  vout-supply:
> +    description:
> +      Phandle of the regulator supplying the output, which should have
> +      the regulator-external-output property.
> +
> +required:

I think at a minimum anything like this would need some sort of
representation of how the output physically appears so that people can
work out how outputs are mapped to the hardware they see.  However we
already have a subsystem for external connectors - extcon.  Perhaps this
should be a regulator client in the extcon API?  It's common for
connectors to include some sort of power provision so it seems like this
would fit right in.

--OeS2mizHPjpFFiLa
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQEzBAABCgAdFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAmJyd9gACgkQJNaLcl1U
h9AKZwf+Kv5LJYapRQXolMBLWWzK2Nn85PgabLCHlDg312x1LS4BqxJGYDX9mRTF
UAYoRIgM8itBAMqd/p2ia6jbpfynvjQ4R+hMx1rIKcPl3qIPV8X7MsLnutU9fjvg
StTLlWzLW/at2L1yGq3ca3PAqJZh/jZOaxU1EPdZNiuXUT0xet0lfvhxt4dGWiHP
ZoGd+muGLfNDflUHBs5jR87rD0WmL4c5/daPJoK47ZFXk2S2wJ0fBa/5l1kPLDkq
QfRmtNC5zFFAj3f5tZfJFt54ozljaCEPuaj7Jt4LVU8/DJDnkioYB5QiYe767X4P
uSBZMYJvEyIzr0gXgk9WJrVRthK1zA==
=VA9P
-----END PGP SIGNATURE-----

--OeS2mizHPjpFFiLa--