On Fri, Mar 15, 2024 at 12:27:36PM +0100, Bastien Curutchet wrote: > - dev_err(&pdev->dev, "no mem resource?\n"); > - return -ENODEV; > + return dev_err_probe(&pdev->dev, -ENODEV, "no mem resource?\n"); > } dev_err_probe() with a fixed error code doesn't seem to make much sense, the whole point is to handle deferral but for a straight lookup like this that can't happen.
Attachment:
signature.asc
Description: PGP signature
- Follow-Ups:
- Re: [PATCH 04/13] ASoC: ti: davinci-i2s: Replace dev_err with dev_err_probe
- From: Herve Codina
- Re: [PATCH 04/13] ASoC: ti: davinci-i2s: Replace dev_err with dev_err_probe
- References:
- [PATCH 00/13] ASoC: ti: davinci-i2s: Add features to McBSP driver
- From: Bastien Curutchet
- [PATCH 04/13] ASoC: ti: davinci-i2s: Replace dev_err with dev_err_probe
- From: Bastien Curutchet
- [PATCH 00/13] ASoC: ti: davinci-i2s: Add features to McBSP driver
- Prev by Date: Re: [PATCH 12/25] ASoC: meson: t9015: add support for A1 SoC family
- Next by Date: Re: [PATCH 09/13] ASoC: ti: davinci-i2s: Enable unexpected frame pulses detection
- Previous by thread: [PATCH 04/13] ASoC: ti: davinci-i2s: Replace dev_err with dev_err_probe
- Next by thread: Re: [PATCH 04/13] ASoC: ti: davinci-i2s: Replace dev_err with dev_err_probe
- Index(es):