Re: [PATCH] ASoC: core: delete component->card_list in soc_remove_component only

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Liao

Thank you for your feedback

> > > To me, the easiest way to see the issue is force return - EPROBE_DEFER
> > > on codec driver's probe function and call
> > > list_del(&component->card_list); before calling
> > > soc_cleanup_component(component); So list_del() will be called twice and you
> > will see the issue.
> > 
> > OK, I could reproduce your issue.
> > And I think it will be solved if you can use
> > list_del_init() instead of list_del() at soc_cleanup_component() ?
> > (= without your patch)
> > 
> > 	- list_del()
> > 	+ list_del_init()
> > 
> > If possible, I want to cleanup all component related resource at
> > soc_cleanup_component(). Because it is easy to read / understand.
> 
> Thanks Morimoto san, I verified the solution works and agree with
> you. Will you send a patch to upstream?

Good to know !!
I'm happy if you can update it :)

Thank you for your help !!
Best regards
---
Kuninori Morimoto
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel



[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Pulse Audio]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux