8 Jul
2015
8 Jul
'15
1:14 p.m.
On Wed, Jul 08, 2015 at 04:25:50PM +0800, Koro Chen wrote:
The driver does not free irq if snd_soc_register_codec fails. It does not return error when request irq failed, either. Fix this by using devm_request_threaded_irq(), and returns when error.
Unfortunately this isn't safe...
- if (i2c->irq)
free_irq(i2c->irq, rt5645);- cancel_delayed_work_sync(&rt5645->jack_detect_work);
This work item is queued up by the interrupt handler so we need to unregister the interrupt before we cancel any pending work otherwise it's possible that the interrupt may fire after we cancelled the work.