On 7/11/2020 2:07 PM, Linus Walleij wrote: > On Wed, Jul 1, 2020 at 6:44 AM Florian Fainelli <f.fainelli@xxxxxxxxx> wrote: >> On 6/30/2020 9:37 PM, Mark Tomlinson wrote: > >>> That was one of my thoughts too. I found someone had tried that >>> earlier, but it was rejected: >>> >>> >>> https://patchwork.ozlabs.org/project/linux-gpio/patch/1516566774-1786-1-git-send-email-david@xxxxxxxxxxxxxx/ >> >> clk or reset APIs do not complain loudly on EPROBE_DEFER, it seems to me >> that GPIO should follow here. Also, it does look like Linus was in >> agreement in the end, not sure why it was not applied though. > > I never got an updated patch. My last message was: > >>> so you mean something like this? >>> >>> if (err == -EPROBE_DEFER) >>> dev_info(dev, "deferring probe\n") >>> else >>> dev_err(dev, "... failed to register\n") >> >> Yes exactly. > > Patches welcome :D Not sure how useful the dev_info(dev, "deferring probe\n") is nowadays given that the device driver core will show which devices are on the probe deferral list, maybe we can turn this into a dev_dbg() instead? -- Florian