Messages in this thread | | | Date | Wed, 17 Apr 2019 16:46:38 +0100 | From | Mark Brown <> | Subject | Re: [PATCH] regulator: core: do not report EPROBE_DEFER as error. |
| |
On Wed, Apr 17, 2019 at 10:54:11AM +0200, Jorge Ramirez-Ortiz wrote: > Do not log a temporary failure to get a regulator (EPROBE_DEFER) while > the driver is requesting retries.
> - dev_err(dev, "Failed to get supply '%s': %d\n", > - consumers[i].supply, ret); > + if (ret != -EPROBE_DEFER) > + dev_err(dev, "Failed to get supply '%s': %d\n", > + consumers[i].supply, ret);
Please leave at least a debug log in place, it's not good to just silently fail - even if we will retry someone might still need some help debugging (eg, figuring out that they need to enable whatever driver is providing the supply in their config) so we should tell them why we're deferring.
BTW does an e-mail address need updating somewhere here? [unhandled content-type:application/pgp-signature] | |