Commit 8eba2703 authored by Vincent Abriou's avatar Vincent Abriou Committed by Benjamin Gaignard

drm/sti: missing first pixel column on HDMI display

VTG programing updated to fix the shift of one column observed
on HDMI display.
Signed-off-by: default avatarVincent Abriou <vincent.abriou@st.com>
parent d3c8a0b2
...@@ -62,7 +62,7 @@ ...@@ -62,7 +62,7 @@
#define VTG_IRQ_MASK (VTG_IRQ_TOP | VTG_IRQ_BOTTOM) #define VTG_IRQ_MASK (VTG_IRQ_TOP | VTG_IRQ_BOTTOM)
/* Delay introduced by the HDMI in nb of pixel */ /* Delay introduced by the HDMI in nb of pixel */
#define HDMI_DELAY (6) #define HDMI_DELAY (5)
/* delay introduced by the Arbitrary Waveform Generator in nb of pixels */ /* delay introduced by the Arbitrary Waveform Generator in nb of pixels */
#define AWG_DELAY_HD (-9) #define AWG_DELAY_HD (-9)
...@@ -121,6 +121,32 @@ static void vtg_reset(struct sti_vtg *vtg) ...@@ -121,6 +121,32 @@ static void vtg_reset(struct sti_vtg *vtg)
writel(1, vtg->regs + VTG_DRST_AUTOC); writel(1, vtg->regs + VTG_DRST_AUTOC);
} }
static void vtg_set_output_window(void __iomem *regs,
const struct drm_display_mode *mode)
{
u32 video_top_field_start;
u32 video_top_field_stop;
u32 video_bottom_field_start;
u32 video_bottom_field_stop;
u32 xstart = sti_vtg_get_pixel_number(*mode, 0);
u32 ystart = sti_vtg_get_line_number(*mode, 0);
u32 xstop = sti_vtg_get_pixel_number(*mode, mode->hdisplay - 1);
u32 ystop = sti_vtg_get_line_number(*mode, mode->vdisplay - 1);
/* Set output window to fit the display mode selected */
video_top_field_start = (ystart << 16) | xstart;
video_top_field_stop = (ystop << 16) | xstop;
/* Only progressive supported for now */
video_bottom_field_start = video_top_field_start;
video_bottom_field_stop = video_top_field_stop;
writel(video_top_field_start, regs + VTG_VID_TFO);
writel(video_top_field_stop, regs + VTG_VID_TFS);
writel(video_bottom_field_start, regs + VTG_VID_BFO);
writel(video_bottom_field_stop, regs + VTG_VID_BFS);
}
static void vtg_set_mode(struct sti_vtg *vtg, static void vtg_set_mode(struct sti_vtg *vtg,
int type, const struct drm_display_mode *mode) int type, const struct drm_display_mode *mode)
{ {
...@@ -129,18 +155,14 @@ static void vtg_set_mode(struct sti_vtg *vtg, ...@@ -129,18 +155,14 @@ static void vtg_set_mode(struct sti_vtg *vtg,
if (vtg->slave) if (vtg->slave)
vtg_set_mode(vtg->slave, VTG_TYPE_SLAVE_BY_EXT0, mode); vtg_set_mode(vtg->slave, VTG_TYPE_SLAVE_BY_EXT0, mode);
/* Set the number of clock cycles per line */
writel(mode->htotal, vtg->regs + VTG_CLKLN); writel(mode->htotal, vtg->regs + VTG_CLKLN);
writel(mode->vtotal * 2, vtg->regs + VTG_HLFLN);
tmp = (mode->vtotal - mode->vsync_start + 1) << 16; /* Set Half Line Per Field (only progressive supported for now) */
tmp |= mode->htotal - mode->hsync_start; writel(mode->vtotal * 2, vtg->regs + VTG_HLFLN);
writel(tmp, vtg->regs + VTG_VID_TFO);
writel(tmp, vtg->regs + VTG_VID_BFO);
tmp = (mode->vdisplay + mode->vtotal - mode->vsync_start + 1) << 16; /* Program output window */
tmp |= mode->hdisplay + mode->htotal - mode->hsync_start; vtg_set_output_window(vtg->regs, mode);
writel(tmp, vtg->regs + VTG_VID_TFS);
writel(tmp, vtg->regs + VTG_VID_BFS);
/* prepare VTG set 1 for HDMI */ /* prepare VTG set 1 for HDMI */
tmp = (mode->hsync_end - mode->hsync_start + HDMI_DELAY) << 16; tmp = (mode->hsync_end - mode->hsync_start + HDMI_DELAY) << 16;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment