Re: [PATCH net-next v1 2/2] net: dsa: realtek: rtl83xx: add support for enabling supplies
From: Mark Brown
Date: Fri Aug 07 2026 - 08:21:42 EST
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.
Attachment:
signature.asc
Description: PGP signature