Hi Satendra Singh Thakur, On 07.05.2018 05:32, Satendra Singh Thakur wrote: > To avoid duplicate logic for the same > > Signed-off-by: Satendra Singh Thakur <satendr...@samsung.com> > Acked-by: Madhur Verma <madhur.ve...@samsung.com> > Cc: Hemanshu Srivastava <hemansh...@samsung.com>
Whole exynos_dsi_mode_set callback is redundant, so I have posted patch removing it [1], so this patch can be dropped. [1]: https://marc.info/?l=dri-devel&m=152568538400712 Regards Andrzej > --- > > v2: Removed Mr Robin from reviewed-by field > > drivers/gpu/drm/exynos/exynos_drm_dsi.c | 13 ++----------- > 1 file changed, 2 insertions(+), 11 deletions(-) > > diff --git a/drivers/gpu/drm/exynos/exynos_drm_dsi.c > b/drivers/gpu/drm/exynos/exynos_drm_dsi.c > index 7904ffa..7fe84fd 100644 > --- a/drivers/gpu/drm/exynos/exynos_drm_dsi.c > +++ b/drivers/gpu/drm/exynos/exynos_drm_dsi.c > @@ -1490,17 +1490,8 @@ static void exynos_dsi_mode_set(struct drm_encoder > *encoder, > struct drm_display_mode *adjusted_mode) > { > struct exynos_dsi *dsi = encoder_to_dsi(encoder); > - struct videomode *vm = &dsi->vm; > - struct drm_display_mode *m = adjusted_mode; > - > - vm->hactive = m->hdisplay; > - vm->vactive = m->vdisplay; > - vm->vfront_porch = m->vsync_start - m->vdisplay; > - vm->vback_porch = m->vtotal - m->vsync_end; > - vm->vsync_len = m->vsync_end - m->vsync_start; > - vm->hfront_porch = m->hsync_start - m->hdisplay; > - vm->hback_porch = m->htotal - m->hsync_end; > - vm->hsync_len = m->hsync_end - m->hsync_start; > + > + drm_display_mode_to_videomode(adjusted_mode, &dsi->vm); > } > > static const struct drm_encoder_helper_funcs exynos_dsi_encoder_helper_funcs > = { _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
