[PATCH 13/23] video: da8xx-fb: enable sync lost intr for v2 ip

Subsystems: framebuffer layer, the rest

STALE4785d

2 messages, 2 authors, 2013-06-26 · open the first message on its own page

[PATCH 13/23] video: da8xx-fb: enable sync lost intr for v2 ip

From: Darren Etheridge <hidden>
Date: 2013-06-25 14:22:41

From: Afzal Mohammed <redacted>

interrupt handler is checking for sync lost interrupt, but it was not
enabled, enable it.

Signed-off-by: Afzal Mohammed <redacted>
Signed-off-by: Darren Etheridge <redacted>
---
 drivers/video/da8xx-fb.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/video/da8xx-fb.c b/drivers/video/da8xx-fb.c
index 7f92f37..ca69e01 100644
--- a/drivers/video/da8xx-fb.c
+++ b/drivers/video/da8xx-fb.c
@@ -318,7 +318,7 @@ static void lcd_blit(int load_mode, struct da8xx_fb_par *par)
 			reg_int = lcdc_read(LCD_INT_ENABLE_SET_REG) |
 				LCD_V2_END_OF_FRAME0_INT_ENA |
 				LCD_V2_END_OF_FRAME1_INT_ENA |
-				LCD_FRAME_DONE;
+				LCD_FRAME_DONE | LCD_SYNC_LOST;
 			lcdc_write(reg_int, LCD_INT_ENABLE_SET_REG);
 		}
 		reg_dma |= LCD_DUAL_FRAME_BUFFER_ENABLE;
-- 
1.7.0.4

Re: [PATCH 13/23] video: da8xx-fb: enable sync lost intr for v2 ip

From: Tomi Valkeinen <hidden>
Date: 2013-06-26 08:15:27

On 25/06/13 17:22, Darren Etheridge wrote:
From: Afzal Mohammed <redacted>

interrupt handler is checking for sync lost interrupt, but it was not
enabled, enable it.

Signed-off-by: Afzal Mohammed <redacted>
Signed-off-by: Darren Etheridge <redacted>
If the patch is a fix, say that it's a fix. Tell what the bug is, what
symptoms it causes and how it's fixed.

I really dislike too terse commit descriptions. The descriptions should
be as short as possible, but no shorter! =). For this one, I guess just
a few more lines would be enough.

 Tomi

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help