Re: [PATCH 04/10] ASoC: rockchip: spdif: Use dev_err_probe

From: Sebastian Reichel

Date: Tue Jan 27 2026 - 17:13:14 EST


Hello Alexey,

On Tue, Jan 27, 2026 at 09:31:18PM +0400, Alexey Charkov wrote:
> > ret = devm_snd_dmaengine_pcm_register(&pdev->dev, NULL, 0);
> > - if (ret) {
> > - dev_err(&pdev->dev, "Could not register PCM\n");
> > - return ret;
> > - }
> > + if (ret)
> > + dev_err_probe(&pdev->dev, ret, "Could not register PCM\n");
>
> Shouldn't this one return the error code, instead of falling through
> to 'return 0'?

Yes, good catch. I will fix that up in v2.

Thanks,

-- Sebastian

Attachment: signature.asc
Description: PGP signature