Re: [PATCH net-next v1 2/2] net: dsa: realtek: rtl83xx: add support for enabling supplies

Mark Brown <[email protected]>
Newsgroups org.kernel.vger.linux-kernel,org.kernel.vger.linux-devicetree,org.kernel.vger.netdev
Message-ID <[email protected]>
On Fri, Aug 07, 2026 at 10:27:31AM +0200, Ahmad Fatoum wrote:

> Anyways, circling back to the patch: Any objections to leave it as-is
> and suppress the warnings in this particular driver?

Yes, that is a buggy use of regulator_get_optional() so you should not
do that.  Unless the supply may be physically absent it should not be
used.  If you have ideas for how to safely improve the implementation of
the normal regulator_get() then it'd be better to do that, bearing in
mind that it's an API where getting things wrong can easily result in
physical damange to the system.
signature.asc (application/pgp-signature, 488 B)
-----BEGIN PGP SIGNATURE-----

iQEzBAABCgAdFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAmp1zWoACgkQJNaLcl1U
h9B+JAf/QGGxvyiBcW5kulXya+ewiFmKk+tHUnDIGuEgOPsinCZwmjTPrF5ZzdjH
+EyrE8Y9uCG0ZgYpfDjwjZFnCBzbU45zBueQFmati0v/5GFdn9IrMyizA3Wepuo4
OIN+GDdoAZwgl/pTLfGM4ZrVy4WnAr8I52joyYgHCFV7AmJJsxS/0lPlFPuU/DHR
L3PUDb8QUgPv1ZmzcjgJL5OMep/FvrYyYslpLG2e0Uh2Mkkk01l6tb0F7zzzg+QU
B6cfiQmVPwzep+nKIA66qHmG/lyB6zhe7rn41fb/ZjAoHlqs55X0hkgYhmpH6+Ib
4fqOo6m1FZJ9dyiPL7iC/D4p5hbPwg==
=rERM
-----END PGP SIGNATURE-----
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.