diff options
author | Tuomas Tynkkynen <tuomas.tynkkynen@iki.fi> | 2014-04-08 02:15:22 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2014-04-16 14:41:44 -0400 |
commit | ff8ebe6448e98df59a8c7e7e93876f8f3d6a8b19 (patch) | |
tree | 46fb35b16877afd5c1d2fe1a7219c21b97a4b545 | |
parent | 5ed0a8e667090003fdf7b750296fcfb248349502 (diff) |
staging: goldfish: Call free_irq in error path
If misc_register failed in goldfish_audio_probe, the already requested
IRQ wouldn't get freed. Add a call to free_irq() like there is in
goldfish_audio_remove().
Signed-off-by: Tuomas Tynkkynen <tuomas.tynkkynen@iki.fi>
Acked-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-rw-r--r-- | drivers/staging/goldfish/goldfish_audio.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/staging/goldfish/goldfish_audio.c b/drivers/staging/goldfish/goldfish_audio.c index f96dcec740ae..7ac2602242f1 100644 --- a/drivers/staging/goldfish/goldfish_audio.c +++ b/drivers/staging/goldfish/goldfish_audio.c | |||
@@ -334,6 +334,7 @@ static int goldfish_audio_probe(struct platform_device *pdev) | |||
334 | return 0; | 334 | return 0; |
335 | 335 | ||
336 | err_misc_register_failed: | 336 | err_misc_register_failed: |
337 | free_irq(data->irq, data); | ||
337 | err_request_irq_failed: | 338 | err_request_irq_failed: |
338 | dma_free_coherent(&pdev->dev, COMBINED_BUFFER_SIZE, | 339 | dma_free_coherent(&pdev->dev, COMBINED_BUFFER_SIZE, |
339 | data->buffer_virt, data->buffer_phys); | 340 | data->buffer_virt, data->buffer_phys); |