summaryrefslogtreecommitdiffstats
path: root/security
diff options
context:
space:
mode:
authorAnatolij Gustschin <agust@denx.de>2013-01-19 10:59:10 +0100
committerAnatolij Gustschin <agust@denx.de>2013-01-29 08:29:57 +0000
commitb2639b5f1d01f218dc95537a1c352b3a551c4dd5 (patch)
tree220965a5adad447c7cce418035c12c4cfc97be7c /security
parent5d3cc311a76073f6e0a27c0752f7e41f69e95ea7 (diff)
drivers/video: fsl-diu-fb: fix bugs in interrupt handling
Since commit f74de500 "drivers/video: fsl-diu-fb: streamline enabling of interrupts" the interrupt handling in the driver is broken. Enabling diu interrupt causes an interrupt storm and results in system lockup. The cookie for the interrupt handler function passed to request_irq() is wrong (it must be a pointer to the diu struct, and not the address of the pointer to the diu struct). As a result the interrupt handler can not read diu registers and acknowledge the interrupt. Fix cookie arguments for request_irq() and free_irq(). Registering the diu interrupt handler in probe() must happen before install_fb() calls since this function registers framebuffer devices and if fbcon tries to take over framebuffer after registering a frame buffer device, it will call fb_open of the diu driver and enable the interrupts. At this time the diu interrupt handler must be registered already. Disabling the interrupts in fsl_diu_release() must happen only if all other AOIs are closed. Otherwise closing an overlay plane will disable the interrupts even if the primary frame buffer plane is opened. Add an appropriate check in the release function. Signed-off-by: Anatolij Gustschin <agust@denx.de> Cc: Timur Tabi <timur@tabi.org> Cc: stable@vger.kernel.org
Diffstat (limited to 'security')
0 files changed, 0 insertions, 0 deletions