Re: [PATCH v3 09/14] ASoC: tegra: Use dev_err_probe() in OPE, PEQ and MBDRC drivers

From: Mark Brown

Date: Wed Mar 25 2026 - 09:31:25 EST


On Wed, Mar 25, 2026 at 10:14:32AM +0000, Sheetal wrote:
> Log errors in the Tegra210 OPE, PEQ and MBDRC probe paths using
> dev_err_probe().

> err = tegra210_peq_regmap_init(pdev);
> - if (err < 0) {
> - dev_err(dev, "PEQ init failed\n");
> - return err;
> - }
> + if (err < 0)
> + return dev_err_probe(dev, err, "PEQ init failed\n");

There's dev_err_probe() logging in the called function, the
dev_err_probe() here is redundant and overwrites the failure reason.

Attachment: signature.asc
Description: PGP signature