diff options
| author | Andrey Gusakov <andrey.gusakov@cogentembedded.com> | 2017-07-05 20:18:06 +0300 |
|---|---|---|
| committer | Lucas Stach <l.stach@pengutronix.de> | 2017-07-06 14:43:16 +0200 |
| commit | 166d95d37d4a128fc76fc08eed7bd60d61b6c9fe (patch) | |
| tree | af0e4afaad9a612b50f8081cf23350c0ba4980ff /drivers | |
| parent | a9f5b45b213d1059000f44764d6b8fcfb6bd80fe (diff) | |
| download | barebox-166d95d37d4a128fc76fc08eed7bd60d61b6c9fe.tar.gz barebox-166d95d37d4a128fc76fc08eed7bd60d61b6c9fe.tar.xz | |
video: tc358767: fix timing calculation
Fields in HTIM01 and HTIM02 regs should be even.
Recomended thresh_dly value is max_tu_symbol.
Signed-off-by: Andrey Gusakov <andrey.gusakov@cogentembedded.com>
Signed-off-by: Lucas Stach <l.stach@pengutronix.de>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/video/tc358767.c | 23 |
1 files changed, 13 insertions, 10 deletions
diff --git a/drivers/video/tc358767.c b/drivers/video/tc358767.c index e0b09af4f7..cc6a35e70b 100644 --- a/drivers/video/tc358767.c +++ b/drivers/video/tc358767.c @@ -717,7 +717,7 @@ static int tc_set_video_mode(struct tc_data *tc, struct fb_videomode *mode) int htotal; int vtotal; int vid_sync_dly; - int max_tu_symbol; + int max_tu_symbol = TU_SIZE_RECOMMENDED - 1; htotal = mode->hsync_len + mode->left_margin + mode->xres + mode->right_margin; @@ -731,14 +731,18 @@ static int tc_set_video_mode(struct tc_data *tc, struct fb_videomode *mode) mode->upper_margin, mode->lower_margin, mode->vsync_len); dev_dbg(tc->dev, "total: %dx%d\n", htotal, vtotal); - - /* LCD Ctl Frame Size */ - tc_write(VPCTRL0, (0x40 << 20) /* VSDELAY */ | + /* + * Datasheet is not clear of vsdelay in case of DPI + * assume we do not need any delay when DPI is a source of + * sync signals + */ + tc_write(VPCTRL0, (0 << 20) /* VSDELAY */ | OPXLFMT_RGB888 | FRMSYNC_DISABLED | MSF_DISABLED); - tc_write(HTIM01, (mode->left_margin << 16) | /* H back porch */ - (mode->hsync_len << 0)); /* Hsync */ - tc_write(HTIM02, (mode->right_margin << 16) | /* H front porch */ - (mode->xres << 0)); /* width */ + /* LCD Ctl Frame Size */ + tc_write(HTIM01, (ALIGN(mode->left_margin, 2) << 16) | /* H back porch */ + (ALIGN(mode->hsync_len, 2) << 0)); /* Hsync */ + tc_write(HTIM02, (ALIGN(mode->right_margin, 2) << 16) | /* H front porch */ + (ALIGN(mode->xres, 2) << 0)); /* width */ tc_write(VTIM01, (mode->upper_margin << 16) | /* V back porch */ (mode->vsync_len << 0)); /* Vsync */ tc_write(VTIM02, (mode->lower_margin << 16) | /* V front porch */ @@ -757,7 +761,7 @@ static int tc_set_video_mode(struct tc_data *tc, struct fb_videomode *mode) /* DP Main Stream Attributes */ vid_sync_dly = mode->hsync_len + mode->left_margin + mode->xres; tc_write(DP0_VIDSYNCDELAY, - (0x003e << 16) | /* thresh_dly */ + (max_tu_symbol << 16) | /* thresh_dly */ (vid_sync_dly << 0)); tc_write(DP0_TOTALVAL, (vtotal << 16) | (htotal)); @@ -779,7 +783,6 @@ static int tc_set_video_mode(struct tc_data *tc, struct fb_videomode *mode) * (output active video bandwidth in bytes)) * Must be less than tu_size. */ - max_tu_symbol = TU_SIZE_RECOMMENDED - 1; tc_write(DP0_MISC, (max_tu_symbol << 23) | (TU_SIZE_RECOMMENDED << 16) | BPC_8); return 0; |