[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20200408100536.GB20795@ravnborg.org>
Date: Wed, 8 Apr 2020 12:05:36 +0200
From: Sam Ravnborg <sam@...nborg.org>
To: Paul Cercueil <paul@...pouillou.net>
Cc: Thierry Reding <thierry.reding@...il.com>, od@...c.me,
dri-devel@...ts.freedesktop.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 2/2] drm/panel: NT39016: Add support for 50 Hz refresh
rate
Hi Paul.
On Wed, Apr 08, 2020 at 11:58:30AM +0200, Paul Cercueil wrote:
> By changing the pixel clock and the length of the back porch, it is
> possible to obtain a perfect 50 Hz refresh rate.
>
> Signed-off-by: Paul Cercueil <paul@...pouillou.net>
Lower case in $subject.
Acked-by: Sam Ravnborg <sam@...nborg.org>
> ---
> drivers/gpu/drm/panel/panel-novatek-nt39016.c | 15 ++++++++++++++-
> 1 file changed, 14 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/panel/panel-novatek-nt39016.c b/drivers/gpu/drm/panel/panel-novatek-nt39016.c
> index f1286cf6528b..05cae8d62d56 100644
> --- a/drivers/gpu/drm/panel/panel-novatek-nt39016.c
> +++ b/drivers/gpu/drm/panel/panel-novatek-nt39016.c
> @@ -325,7 +325,7 @@ static int nt39016_remove(struct spi_device *spi)
> }
>
> static const struct drm_display_mode kd035g6_display_modes[] = {
> - {
> + { /* 60 Hz */
> .clock = 6000,
> .hdisplay = 320,
> .hsync_start = 320 + 10,
> @@ -338,6 +338,19 @@ static const struct drm_display_mode kd035g6_display_modes[] = {
> .vrefresh = 60,
> .flags = DRM_MODE_FLAG_NHSYNC | DRM_MODE_FLAG_NVSYNC,
> },
> + { /* 50 Hz */
> + .clock = 5400,
> + .hdisplay = 320,
> + .hsync_start = 320 + 42,
> + .hsync_end = 320 + 42 + 50,
> + .htotal = 320 + 42 + 50 + 20,
> + .vdisplay = 240,
> + .vsync_start = 240 + 5,
> + .vsync_end = 240 + 5 + 1,
> + .vtotal = 240 + 5 + 1 + 4,
> + .vrefresh = 50,
> + .flags = DRM_MODE_FLAG_NHSYNC | DRM_MODE_FLAG_NVSYNC,
> + },
> };
>
> static const struct nt39016_panel_info kd035g6_info = {
> --
> 2.25.1
Powered by blists - more mailing lists