Re: [PATCH v2 0/3] ASoC: xilinx: xlnx_i2s: Use dev_err_probe() and drop redundant error handling

[Michal Simek]

On 7/10/26 12:21, phucduc.bui@xxxxxxxxx wrote:
> From: bui duc phuc <phucduc.bui@xxxxxxxxx>
>
> Hi all,
>
> This series cleans up error handling in the Xilinx ASoC drivers.
> It replaces dev_err() followed by return with dev_err_probe() where
> appropriate in probe paths, allowing deferred probe to be handled
> correctly while simplifying the code.
> It also removes redundant dev_err() calls after helper functions that
> already report failures, avoiding duplicate error messages.
>
> Changes in v2:
>   - Split out a separate patch to preserve devm_request_irq() error codes.
>   - Adjust line wrapping.
>   - Add the Reviewed-by tag.
>
> Best regards,
> Phuc
>
> bui duc phuc (3):
>    ASoC: xilinx: xlnx_i2s: Use dev_err_probe() and drop redundant error
>      handling
>    ASoC: xilinx: xlnx_spdif: Preserve devm_request_irq() error codes
>    ASoC: xilinx: xlnx_spdif: Use dev_err_probe() and drop redundant error
>      handling
>
>   sound/soc/xilinx/xlnx_i2s.c   | 18 +++++++-----------
>   sound/soc/xilinx/xlnx_spdif.c | 31 +++++++++++--------------------
>   2 files changed, 18 insertions(+), 31 deletions(-)

Reviewed-by: Michal Simek <michal.simek@xxxxxxx>

Thanks,
Michal