diff options
| author | Zhang Jiejing <jiejing.zhang@freescale.com> | 2012-03-06 14:47:57 +0800 |
|---|---|---|
| committer | Zhang Jiejing <jiejing.zhang@freescale.com> | 2012-03-06 14:58:07 +0800 |
| commit | 0ba6e2dccccedde8f5e3f03a9cbce1781fbd1f7b (patch) | |
| tree | f50b1618260d7fcb66b4fa362760613ca277ecc8 /drivers/video | |
| parent | 8b4eb3911f63aa32b74ef8dd79d2ca2a721db983 (diff) | |
ENGR00176159 video: ipuv3-fb: change to timeout semaphore to wait on irq.
change to timeout semaphore to wait on irq.
use no timeout semaphore have below issues:
1. since fbmem.c will hold the console_lock() before call PAN_DISPLAY ioictl,
if have wrong happens on IPU, IRQ not come, any log printk will not ouput,
it will become like a system hang, and developer don't know what's wrong.
2. semaphore don't have timeout, here we can't know irq not come,
so hang it infintly.
3. semaphore lock and unlock in different context is a dangous operation.
To fix these issue, use timedout version to wait on irq.
But for better coding stly to align Kernel Coding Style Doc,
better use complete to wait on irq, use semaphre little ugly.
Signed-off-by: Zhang Jiejing <jiejing.zhang@freescale.com>
Diffstat (limited to 'drivers/video')
| -rw-r--r-- | drivers/video/mxc/mxc_ipuv3_fb.c | 11 |
1 files changed, 9 insertions, 2 deletions
diff --git a/drivers/video/mxc/mxc_ipuv3_fb.c b/drivers/video/mxc/mxc_ipuv3_fb.c index 6434d82d0941..9252537aa7ee 100644 --- a/drivers/video/mxc/mxc_ipuv3_fb.c +++ b/drivers/video/mxc/mxc_ipuv3_fb.c @@ -925,7 +925,11 @@ static int mxcfb_ioctl(struct fb_info *fbi, unsigned int cmd, unsigned long arg) else ipu_alp_ch_irq = IPU_IRQ_BG_ALPHA_SYNC_EOF; - down(&mxc_fbi->alpha_flip_sem); + if (down_timeout(&mxc_fbi->alpha_flip_sem, HZ/2)) { + dev_err(fbi->device, "timeout when waitting for alpha flip irq\n"); + retval = -ETIMEDOUT; + break; + } mxc_fbi->cur_ipu_alpha_buf = !mxc_fbi->cur_ipu_alpha_buf; @@ -1317,7 +1321,10 @@ mxcfb_pan_display(struct fb_var_screeninfo *var, struct fb_info *info) } } - down(&mxc_fbi->flip_sem); + if (down_timeout(&mxc_fbi->flip_sem, HZ/2)) { + dev_err(info->device, "timeout when waitting for flip irq\n"); + return -ETIMEDOUT; + } mxc_fbi->cur_ipu_buf = (++mxc_fbi->cur_ipu_buf) % 3; mxc_fbi->cur_ipu_alpha_buf = !mxc_fbi->cur_ipu_alpha_buf; |