Messages in this thread | | | From | Andy Shevchenko <> | Date | Sun, 24 Jan 2021 21:53:48 +0200 | Subject | Re: [PATCH v4 11/13] ASoC: arizona-jack: Cleanup logging |
| |
On Sat, Jan 23, 2021 at 2:17 PM Hans de Goede <hdegoede@redhat.com> wrote: > > Cleanup the use of dev_foo functions used for logging: > > 1. Many of these are unnecessarily split over multiple lines > 2. Use dev_err_probe() in cases where we might get a -EPROBE_DEFER > return value
...
> if (IS_ERR(info->micd_pol_gpio)) { > ret = PTR_ERR(info->micd_pol_gpio); > - dev_err(arizona->dev, > - "Failed to get microphone polarity GPIO: %d\n", > - ret); > + dev_err_probe(arizona->dev, ret, "getting microphone polarity GPIO\n"); > return ret; > }
I still think that using dev_err_probe() naturally, i.e. as a part of the return statement is better.
-- With Best Regards, Andy Shevchenko
| |