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-----