Re: [PATCH 11/11] ASoC: fsl: mpc5200_psc_ac97: Use guard() for mutex locks

[PATCH 00/11] ASoC: fsl: Use guard() for mutex & spin locks · <hidden> · 2026-06-12
[PATCH 01/11] ASoC: fsl_asrc: Use guard() for spin locks · <hidden> · 2026-06-12
[PATCH 02/11] ASoC: fsl_audmix: Use guard() for spin locks · <hidden> · 2026-06-12
[PATCH 03/11] ASoC: fsl_easrc: Use guard() for spin locks · <hidden> · 2026-06-12
[PATCH 04/11] ASoC: fsl_esai: Use guard() for spin locks · <hidden> · 2026-06-12
[PATCH 05/11] ASoC: fsl_spdif: Use guard() for spin locks · <hidden> · 2026-06-12
[PATCH 06/11] ASoC: fsl_ssi: Use guard() for mutex locks · <hidden> · 2026-06-12
[PATCH 07/11] ASoC: fsl_xcvr: Use guard() for spin locks · <hidden> · 2026-06-12
[PATCH 08/11] ASoC: imx-audio-rpmsg: Use guard() for spin locks · <hidden> · 2026-06-12
[PATCH 09/11] ASoC: fsl_rpmsg: Use guard() for mutex & spin locks · <hidden> · 2026-06-12
[PATCH 10/11] ASoC: fsl: mpc5200_dma: Use guard() for spin locks · <hidden> · 2026-06-12
[PATCH 11/11] ASoC: fsl: mpc5200_psc_ac97: Use guard() for mutex locks · <hidden> · 2026-06-12
Re: [PATCH 11/11] ASoC: fsl: mpc5200_psc_ac97: Use guard() for mutex locks · Mark Brown <broonie@kernel.org> · 2026-06-12
Re: [PATCH 11/11] ASoC: fsl: mpc5200_psc_ac97: Use guard() for mutex locks · Bui Duc Phuc <hidden> · 2026-06-12

From: Bui Duc Phuc <hidden>
Date: 2026-06-12 21:42:56
Also in: imx, linux-arm-kernel, linux-sound, lkml

Hi Mark,

Thank you for the review.

quoted

-     dev_dbg(psc_dma->dev, "cold reset\n");
+     scoped_guard(mutex_lock, &psc_dma->mutex) {
+             dev_dbg(psc_dma->dev, "cold reset\n");

mutex_lock not mutex?

Sorry for this oversight.

As mentioned in the cover letter, all files were compile-tested except
mpc5200_dma and mpc5200_psc_ac97.
I'll take a closer look next week and send a v2.

Best regards,
Phuc

`h`	back out one level
`j`	next message in thread
`k`	previous message in thread
`l`	drill in
`Esc`	close help / fold thread tree
`?`	toggle this help