[alsa-devel] [PATCH v2 07/23] ASoC: au1x: psc-ac97.c: use devm_snd_soc_register_component()

Kuninori Morimoto kuninori.morimoto.gx at renesas.com
Wed Jul 3 02:36:14 CEST 2019


Hi Mark

> > @@ -426,8 +426,6 @@ static int au1xpsc_ac97_drvremove(struct platform_device *pdev)
> >  {
> >  	struct au1xpsc_audio_data *wd = platform_get_drvdata(pdev);
> >  
> > -	snd_soc_unregister_component(&pdev->dev);
> > -
> >  	/* disable PSC completely */
> >  	__raw_writel(0, AC97_CFG(wd));
> >  	wmb(); /* drain writebuffer */
> 
> I don't think this is 100% safe - with this change we will turn
> off the hardware block before we unregister the component which
> means that we might try to interact with it.  If we're converting
> to devm then we need to be careful the ordering of the removal
> won't be affected.

Hmm indeed.
Thank you for your review.

Thank you for your help !!
Best regards
---
Kuninori Morimoto


More information about the Alsa-devel mailing list