video: da8xx: Register IRQ as last thing in driver probing.
authorCaglar Akyuz <caglar@bilkon-kontrol.com.tr>
Tue, 30 Nov 2010 20:04:14 +0000 (20:04 +0000)
committerPaul Mundt <lethal@linux-sh.org>
Wed, 1 Dec 2010 09:18:52 +0000 (18:18 +0900)
commit93c176f39fedaeff854ccb7681d626d65bdffe52
tree71beb067554e1cd0a73d3e92c0baead2e1b5b579
parent22a95949d03548cb346d81d71ce6c64dc273f421
video: da8xx: Register IRQ as last thing in driver probing.

Following commit exposed a bug in driver:

"fbdev: da8xx/omap-l1xx: implement double buffering"

Bug is, if interrupt handler is called before initialization is
finished, raster controller is enabled and following register
modifications causes hardware to stay in a broken state.

By looking at this one may say that proper locking is missing in
this driver, and a more proper fix should be prepared. However,
aformentioned commit causes a regression in the driver and some
fix to current one should be applied first.

Signed-off-by: Caglar Akyuz <caglar@bilkon-kontrol.com.tr>
Signed-off-by: Paul Mundt <lethal@linux-sh.org>
drivers/video/da8xx-fb.c