Skip to content

Commit 5b838a2

Browse files
sfrothwellbroonie
authored andcommitted
ASoC: SDCA: Check devm_mutex_init() return value
Fix interaction with commit daec29d ("locking/mutex: Mark devm_mutex_init() as __must_check"), add return value check. There is no need for additional complex error handling here, failure to init the mutex means the code can't progress, so the failure just needs to be passed up to the caller. Fixes: b126394 ("ASoC: SDCA: Generic interrupt support") Signed-off-by: Stephen Rothwell <sfr@canb.auug.org.au> Signed-off-by: Charles Keepax <ckeepax@opensource.cirrus.com> Link: https://patch.msgid.link/20250722102754.2514351-1-ckeepax@opensource.cirrus.com Signed-off-by: Mark Brown <broonie@kernel.org>
1 parent 1788272 commit 5b838a2

1 file changed

Lines changed: 3 additions & 1 deletion

File tree

sound/soc/sdca/sdca_interrupts.c

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -419,7 +419,9 @@ struct sdca_interrupt_info *sdca_irq_allocate(struct device *dev,
419419

420420
info->irq_chip = sdca_irq_chip;
421421

422-
devm_mutex_init(dev, &info->irq_lock);
422+
ret = devm_mutex_init(dev, &info->irq_lock);
423+
if (ret)
424+
return ERR_PTR(ret);
423425

424426
ret = devm_regmap_add_irq_chip(dev, regmap, irq, IRQF_ONESHOT, 0,
425427
&info->irq_chip, &info->irq_data);

0 commit comments

Comments
 (0)