Date:   Wed, 8 Mar 2023 13:46:52 +0000
From:   Mark Brown <broonie@kernel.org>
To:     Svyatoslav Ryhel <clamor95@gmail.com>
Cc:     Guenter Roeck <linux@roeck-us.net>,
        Jean Delvare <jdelvare@suse.com>,
        Rob Herring <robh+dt@kernel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Liam Girdwood <lgirdwood@gmail.com>,
        linux-hwmon@vger.kernel.org, devicetree@vger.kernel.org,
        linux-kernel@vger.kernel.org
Subject: Re: [PATCH v1 1/2] dt-bindings: hwmon: ina2xx: add supply property
Message-ID: <5cd6764c-9b04-42ea-932d-9f14aa465605@sirena.org.uk>
References: <20230308094024.14115-1-clamor95@gmail.com>
 <20230308094024.14115-2-clamor95@gmail.com>
 <31ca0ede-012c-4849-bf25-d0492b116681@sirena.org.uk>
 <6DBD0F5A-4625-4FCD-8D64-23293D734A82@gmail.com>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
        protocol="application/pgp-signature"; boundary="emhiT6n8SP5ckuD4"
Content-Disposition: inline
In-Reply-To: <6DBD0F5A-4625-4FCD-8D64-23293D734A82@gmail.com>
Precedence: bulk


--emhiT6n8SP5ckuD4
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Wed, Mar 08, 2023 at 02:58:20PM +0200, Svyatoslav Ryhel wrote:
> 8 =D0=B1=D0=B5=D1=80=D0=B5=D0=B7=D0=BD=D1=8F 2023 =D1=80. 14:54:34 GMT+02=
:00, Mark Brown <broonie@kernel.org> =D0=BD=D0=B0=D0=BF=D0=B8=D1=81=D0=B0=
=D0=B2(-=D0=BB=D0=B0):
> >On Wed, Mar 08, 2023 at 11:40:23AM +0200, Svyatoslav Ryhel wrote:
> >> Add supply property.

> >> +  vdd-supply: true
> >> +
> >>  required:
> >>    - compatible
> >>    - reg

> >Unless the device can work without power the supply should be required.

> Device can work without supply defined on most devices, but in my case po=
wer is gated with gpio and devices will not work without fixed regulator.

If there are devices that work without any source of power at all that
would be very surprising.  It doesn't matter if a particular system has
a non-controllable regulator, the binding should still make it mandatory
to describe that.

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

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

iQEzBAABCgAdFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAmQIkcsACgkQJNaLcl1U
h9DLjgf+JwLFER7NTybJIWpDUEcwfYmI40mRfAH7UE8En493f/0pMebcBBHNFa6J
KVUCJkGtFM3EPf3u0uX/kKfgqyIqHQ6NFwz0mBs3tA2jvWTCTlwqgjIrIMtqnyJl
pboxy1NkaLnKWZr7qSmEf5OVgJU5vCv1WIR3FXzBoe3M6HXof3Sxet1xCoJr1yGj
rPIM5Zb+9cwwEC7flYl5VAt8IIDEPbV1u6uFxdNpEuT2439csNp0iOnJpel0Hrlp
EttKtNc23ASQ40l5CTv459qRUOfU+1/wizGfc7LneTI8n/oU8E/hT9R3bpGAaFB5
VCzUnH64yOc+QSGT1cbe7BqGVePF0A==
=iM/u
-----END PGP SIGNATURE-----

--emhiT6n8SP5ckuD4--