summaryrefslogtreecommitdiff
path: root/drivers/video
diff options
context:
space:
mode:
authorChristophe JAILLET <christophe.jaillet@wanadoo.fr>2017-11-09 20:09:29 +0300
committerBartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>2017-11-09 20:09:29 +0300
commit703a4af427a9bf3028f1a9369fcec7bfcf56f0ed (patch)
treeb8476e4702686d2d3149377dd7613c0c5f4170ac /drivers/video
parent451f130602619a17c8883dd0b71b11624faffd51 (diff)
downloadlinux-703a4af427a9bf3028f1a9369fcec7bfcf56f0ed.tar.xz
video: fbdev: au1200fb: Fix incorrect IRQ freeing
'au1200fb_drv_probe()' can not fail after a successful call to 'request_irq()'. So there is no point to call 'free_irq()' in the error handling path. Moreover, the hard coded AU1200_LCD_INT looks boggus since commit 1630d85a8312 ("au1200fb: fix hardcoded IRQ"). So, remove it. Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr> Cc: Tejun Heo <tj@kernel.org> [b.zolnierkie: patch summary and description fixups] Signed-off-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
Diffstat (limited to 'drivers/video')
-rw-r--r--drivers/video/fbdev/au1200fb.c2
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/video/fbdev/au1200fb.c b/drivers/video/fbdev/au1200fb.c
index cf54168d44dc..0d8ed0ef9183 100644
--- a/drivers/video/fbdev/au1200fb.c
+++ b/drivers/video/fbdev/au1200fb.c
@@ -1766,8 +1766,6 @@ failed:
fb_dealloc_cmap(&fbi->cmap);
kfree(fbi->pseudo_palette);
}
- if (plane == 0)
- free_irq(AU1200_LCD_INT, (void*)dev);
return ret;
}