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

phucduc.bui@gmail.com posted 3 patches 2 weeks, 1 day ago
sound/soc/xilinx/xlnx_i2s.c   | 18 +++++++-----------
sound/soc/xilinx/xlnx_spdif.c | 31 +++++++++++--------------------
2 files changed, 18 insertions(+), 31 deletions(-)
[PATCH v2 0/3] ASoC: xilinx: xlnx_i2s: Use dev_err_probe() and drop redundant error handling
Posted by phucduc.bui@gmail.com 2 weeks, 1 day ago
From: bui duc phuc <phucduc.bui@gmail.com>

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(-)

-- 
2.43.0
Re: [PATCH v2 0/3] ASoC: xilinx: xlnx_i2s: Use dev_err_probe() and drop redundant error handling
Posted by Mark Brown 2 weeks, 1 day ago
On Fri, 10 Jul 2026 17:21:35 +0700, phucduc.bui@gmail.com wrote:
> ASoC: xilinx: xlnx_i2s: Use dev_err_probe() and drop redundant error handling
> 
> From: bui duc phuc <phucduc.bui@gmail.com>
> 
> 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.
> 
> [...]

Applied to

   https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-7.3

Thanks!

[1/3] ASoC: xilinx: xlnx_i2s: Use dev_err_probe() and drop redundant error handling
      https://git.kernel.org/broonie/sound/c/14b68141bcff
[2/3] ASoC: xilinx: xlnx_spdif: Preserve devm_request_irq() error codes
      https://git.kernel.org/broonie/sound/c/cb32e3acb103
[3/3] ASoC: xilinx: xlnx_spdif: Use dev_err_probe() and drop redundant error handling
      https://git.kernel.org/broonie/sound/c/f9620327ba7e

All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent to Linus during
the next merge window (or sooner if it is a bug fix), however if
problems are discovered then the patch may be dropped or reverted.

You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.

If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.

Please add any relevant lists and maintainers to the CCs when replying
to this mail.

Thanks,
Mark
Re: [PATCH v2 0/3] ASoC: xilinx: xlnx_i2s: Use dev_err_probe() and drop redundant error handling
Posted by Michal Simek 2 weeks, 1 day ago

On 7/10/26 12:21, phucduc.bui@gmail.com wrote:
> From: bui duc phuc <phucduc.bui@gmail.com>
> 
> 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@amd.com>

Thanks,
Michal