* [PATCH] drm/panel: simple: fix AUO g185han01 horizontal blanking @ 2018-11-12 17:41 Lucas Stach 2018-12-14 13:20 ` Lucas Stach 2019-01-25 11:20 ` Philipp Zabel 0 siblings, 2 replies; 6+ messages in thread From: Lucas Stach @ 2018-11-12 17:41 UTC (permalink / raw) To: Thierry Reding; +Cc: kernel, dri-devel, patchwork-lst The horizontal blanking periods are too short, as the values are specified for a single LVDS channel. Since this panel is dual LVDS they need to be doubled. With this change the panel reaches its nominal vrefresh rate of 60Fps, instead of the 64Fps with the current wrong blanking. Signed-off-by: Lucas Stach <l.stach@pengutronix.de> --- drivers/gpu/drm/panel/panel-simple.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/panel/panel-simple.c b/drivers/gpu/drm/panel/panel-simple.c index 97964f7f2ace..2c89792e91e2 100644 --- a/drivers/gpu/drm/panel/panel-simple.c +++ b/drivers/gpu/drm/panel/panel-simple.c @@ -663,9 +663,9 @@ static const struct panel_desc auo_g133han01 = { static const struct display_timing auo_g185han01_timings = { .pixelclock = { 120000000, 144000000, 175000000 }, .hactive = { 1920, 1920, 1920 }, - .hfront_porch = { 18, 60, 74 }, - .hback_porch = { 12, 44, 54 }, - .hsync_len = { 10, 24, 32 }, + .hfront_porch = { 36, 120, 148 }, + .hback_porch = { 24, 88, 108 }, + .hsync_len = { 20, 48, 64 }, .vactive = { 1080, 1080, 1080 }, .vfront_porch = { 6, 10, 40 }, .vback_porch = { 2, 5, 20 }, -- 2.19.1 _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] drm/panel: simple: fix AUO g185han01 horizontal blanking 2018-11-12 17:41 [PATCH] drm/panel: simple: fix AUO g185han01 horizontal blanking Lucas Stach @ 2018-12-14 13:20 ` Lucas Stach 2019-07-10 13:07 ` Lucas Stach 2019-01-25 11:20 ` Philipp Zabel 1 sibling, 1 reply; 6+ messages in thread From: Lucas Stach @ 2018-12-14 13:20 UTC (permalink / raw) To: Thierry Reding; +Cc: dri-devel, kernel, patchwork-lst Hi Thierry, can you please have a look at this one? Regards, Lucas Am Montag, den 12.11.2018, 18:41 +0100 schrieb Lucas Stach: > The horizontal blanking periods are too short, as the values are > specified for a single LVDS channel. Since this panel is dual LVDS > they need to be doubled. With this change the panel reaches its > nominal vrefresh rate of 60Fps, instead of the 64Fps with the > current wrong blanking. > > Signed-off-by: Lucas Stach <l.stach@pengutronix.de> > --- > drivers/gpu/drm/panel/panel-simple.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/drivers/gpu/drm/panel/panel-simple.c > b/drivers/gpu/drm/panel/panel-simple.c > index 97964f7f2ace..2c89792e91e2 100644 > --- a/drivers/gpu/drm/panel/panel-simple.c > +++ b/drivers/gpu/drm/panel/panel-simple.c > @@ -663,9 +663,9 @@ static const struct panel_desc auo_g133han01 = { > static const struct display_timing auo_g185han01_timings = { > .pixelclock = { 120000000, 144000000, 175000000 }, > .hactive = { 1920, 1920, 1920 }, > - .hfront_porch = { 18, 60, 74 }, > - .hback_porch = { 12, 44, 54 }, > - .hsync_len = { 10, 24, 32 }, > + .hfront_porch = { 36, 120, 148 }, > + .hback_porch = { 24, 88, 108 }, > + .hsync_len = { 20, 48, 64 }, > .vactive = { 1080, 1080, 1080 }, > .vfront_porch = { 6, 10, 40 }, > .vback_porch = { 2, 5, 20 }, _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] drm/panel: simple: fix AUO g185han01 horizontal blanking 2018-12-14 13:20 ` Lucas Stach @ 2019-07-10 13:07 ` Lucas Stach 2019-07-10 13:32 ` Sam Ravnborg 2019-07-12 16:18 ` Sam Ravnborg 0 siblings, 2 replies; 6+ messages in thread From: Lucas Stach @ 2019-07-10 13:07 UTC (permalink / raw) To: Sam Ravnborg; +Cc: Thierry Reding, kernel, dri-devel, patchwork-lst Hi Sam, since you've been picking up some panel patches lately, might I ask you to take a look at this patch? Regards, Lucas Am Freitag, den 14.12.2018, 14:20 +0100 schrieb Lucas Stach: > Hi Thierry, > > can you please have a look at this one? > > Regards, > Lucas > > Am Montag, den 12.11.2018, 18:41 +0100 schrieb Lucas Stach: > > The horizontal blanking periods are too short, as the values are > > specified for a single LVDS channel. Since this panel is dual LVDS > > they need to be doubled. With this change the panel reaches its > > nominal vrefresh rate of 60Fps, instead of the 64Fps with the > > current wrong blanking. > > > > Signed-off-by: Lucas Stach <l.stach@pengutronix.de> > > --- > > drivers/gpu/drm/panel/panel-simple.c | 6 +++--- > > 1 file changed, 3 insertions(+), 3 deletions(-) > > > > diff --git a/drivers/gpu/drm/panel/panel-simple.c > > b/drivers/gpu/drm/panel/panel-simple.c > > index 97964f7f2ace..2c89792e91e2 100644 > > --- a/drivers/gpu/drm/panel/panel-simple.c > > +++ b/drivers/gpu/drm/panel/panel-simple.c > > @@ -663,9 +663,9 @@ static const struct panel_desc auo_g133han01 = > > { > > static const struct display_timing auo_g185han01_timings = { > > .pixelclock = { 120000000, 144000000, 175000000 }, > > .hactive = { 1920, 1920, 1920 }, > > - .hfront_porch = { 18, 60, 74 }, > > - .hback_porch = { 12, 44, 54 }, > > - .hsync_len = { 10, 24, 32 }, > > + .hfront_porch = { 36, 120, 148 }, > > + .hback_porch = { 24, 88, 108 }, > > + .hsync_len = { 20, 48, 64 }, > > .vactive = { 1080, 1080, 1080 }, > > .vfront_porch = { 6, 10, 40 }, > > .vback_porch = { 2, 5, 20 }, > > _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] drm/panel: simple: fix AUO g185han01 horizontal blanking 2019-07-10 13:07 ` Lucas Stach @ 2019-07-10 13:32 ` Sam Ravnborg 2019-07-12 16:18 ` Sam Ravnborg 1 sibling, 0 replies; 6+ messages in thread From: Sam Ravnborg @ 2019-07-10 13:32 UTC (permalink / raw) To: Lucas Stach; +Cc: Thierry Reding, kernel, dri-devel, patchwork-lst On Wed, Jul 10, 2019 at 03:07:40PM +0200, Lucas Stach wrote: > Hi Sam, > > since you've been picking up some panel patches lately, might I ask you > to take a look at this patch? > > Regards, > Lucas > > Am Freitag, den 14.12.2018, 14:20 +0100 schrieb Lucas Stach: > > Hi Thierry, > > > > can you please have a look at this one? > > > > Regards, > > Lucas > > > > Am Montag, den 12.11.2018, 18:41 +0100 schrieb Lucas Stach: > > > The horizontal blanking periods are too short, as the values are > > > specified for a single LVDS channel. Since this panel is dual LVDS > > > they need to be doubled. With this change the panel reaches its > > > nominal vrefresh rate of 60Fps, instead of the 64Fps with the > > > current wrong blanking. > > > > > > Signed-off-by: Lucas Stach <l.stach@pengutronix.de> > > > --- > > > drivers/gpu/drm/panel/panel-simple.c | 6 +++--- > > > 1 file changed, 3 insertions(+), 3 deletions(-) > > > > > > diff --git a/drivers/gpu/drm/panel/panel-simple.c > > > b/drivers/gpu/drm/panel/panel-simple.c > > > index 97964f7f2ace..2c89792e91e2 100644 > > > --- a/drivers/gpu/drm/panel/panel-simple.c > > > +++ b/drivers/gpu/drm/panel/panel-simple.c > > > @@ -663,9 +663,9 @@ static const struct panel_desc auo_g133han01 = > > > { > > > static const struct display_timing auo_g185han01_timings = { > > > .pixelclock = { 120000000, 144000000, 175000000 }, > > > .hactive = { 1920, 1920, 1920 }, > > > - .hfront_porch = { 18, 60, 74 }, > > > - .hback_porch = { 12, 44, 54 }, > > > - .hsync_len = { 10, 24, 32 }, > > > + .hfront_porch = { 36, 120, 148 }, > > > + .hback_porch = { 24, 88, 108 }, > > > + .hsync_len = { 20, 48, 64 }, > > > .vactive = { 1080, 1080, 1080 }, > > > .vfront_porch = { 6, 10, 40 }, > > > .vback_porch = { 2, 5, 20 }, Looks good to me. I have not worked with dual scan but your explanations makes sense. and I checked that all vertical timings are multiplied by 2. So: Reviewed-by: Sam Ravnborg <sam@ravnborg.org> _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] drm/panel: simple: fix AUO g185han01 horizontal blanking 2019-07-10 13:07 ` Lucas Stach 2019-07-10 13:32 ` Sam Ravnborg @ 2019-07-12 16:18 ` Sam Ravnborg 1 sibling, 0 replies; 6+ messages in thread From: Sam Ravnborg @ 2019-07-12 16:18 UTC (permalink / raw) To: Lucas Stach; +Cc: Thierry Reding, kernel, dri-devel, patchwork-lst Hi Lucas. > Hi Sam, > > since you've been picking up some panel patches lately, might I ask you > to take a look at this patch? > > Regards, > Lucas > > Am Freitag, den 14.12.2018, 14:20 +0100 schrieb Lucas Stach: > > Hi Thierry, > > > > can you please have a look at this one? > > > > Regards, > > Lucas > > > > Am Montag, den 12.11.2018, 18:41 +0100 schrieb Lucas Stach: > > > The horizontal blanking periods are too short, as the values are > > > specified for a single LVDS channel. Since this panel is dual LVDS > > > they need to be doubled. With this change the panel reaches its > > > nominal vrefresh rate of 60Fps, instead of the 64Fps with the > > > current wrong blanking. > > > > > > Signed-off-by: Lucas Stach <l.stach@pengutronix.de> > > > --- > > > drivers/gpu/drm/panel/panel-simple.c | 6 +++--- > > > 1 file changed, 3 insertions(+), 3 deletions(-) > > > > > > diff --git a/drivers/gpu/drm/panel/panel-simple.c > > > b/drivers/gpu/drm/panel/panel-simple.c > > > index 97964f7f2ace..2c89792e91e2 100644 > > > --- a/drivers/gpu/drm/panel/panel-simple.c > > > +++ b/drivers/gpu/drm/panel/panel-simple.c > > > @@ -663,9 +663,9 @@ static const struct panel_desc auo_g133han01 = > > > { > > > static const struct display_timing auo_g185han01_timings = { > > > .pixelclock = { 120000000, 144000000, 175000000 }, > > > .hactive = { 1920, 1920, 1920 }, > > > - .hfront_porch = { 18, 60, 74 }, > > > - .hback_porch = { 12, 44, 54 }, > > > - .hsync_len = { 10, 24, 32 }, > > > + .hfront_porch = { 36, 120, 148 }, > > > + .hback_porch = { 24, 88, 108 }, > > > + .hsync_len = { 20, 48, 64 }, > > > .vactive = { 1080, 1080, 1080 }, > > > .vfront_porch = { 6, 10, 40 }, > > > .vback_porch = { 2, 5, 20 }, Updated commit log to include comment form Philipp and applied to drm-misc-next. Sam _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] drm/panel: simple: fix AUO g185han01 horizontal blanking 2018-11-12 17:41 [PATCH] drm/panel: simple: fix AUO g185han01 horizontal blanking Lucas Stach 2018-12-14 13:20 ` Lucas Stach @ 2019-01-25 11:20 ` Philipp Zabel 1 sibling, 0 replies; 6+ messages in thread From: Philipp Zabel @ 2019-01-25 11:20 UTC (permalink / raw) To: Lucas Stach, Thierry Reding; +Cc: dri-devel, kernel, patchwork-lst On Mon, 2018-11-12 at 18:41 +0100, Lucas Stach wrote: > The horizontal blanking periods are too short, as the values are > specified for a single LVDS channel. Since this panel is dual LVDS > they need to be doubled. With this change the panel reaches its > nominal vrefresh rate of 60Fps, instead of the 64Fps with the > current wrong blanking. > > Signed-off-by: Lucas Stach <l.stach@pengutronix.de> > --- > drivers/gpu/drm/panel/panel-simple.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/drivers/gpu/drm/panel/panel-simple.c b/drivers/gpu/drm/panel/panel-simple.c > index 97964f7f2ace..2c89792e91e2 100644 > --- a/drivers/gpu/drm/panel/panel-simple.c > +++ b/drivers/gpu/drm/panel/panel-simple.c > @@ -663,9 +663,9 @@ static const struct panel_desc auo_g133han01 = { > static const struct display_timing auo_g185han01_timings = { > .pixelclock = { 120000000, 144000000, 175000000 }, > .hactive = { 1920, 1920, 1920 }, > - .hfront_porch = { 18, 60, 74 }, > - .hback_porch = { 12, 44, 54 }, > - .hsync_len = { 10, 24, 32 }, > + .hfront_porch = { 36, 120, 148 }, > + .hback_porch = { 24, 88, 108 }, > + .hsync_len = { 20, 48, 64 }, The datasheet specifies 960 active clocks + 40/128/160 clocks blanking on each of the two LVDS channels (min/typical/max), so doubled this is now correct. Reviewed-by: Philipp Zabel <p.zabel@pengutronix.de> regards Philipp _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2019-07-12 16:18 UTC | newest] Thread overview: 6+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2018-11-12 17:41 [PATCH] drm/panel: simple: fix AUO g185han01 horizontal blanking Lucas Stach 2018-12-14 13:20 ` Lucas Stach 2019-07-10 13:07 ` Lucas Stach 2019-07-10 13:32 ` Sam Ravnborg 2019-07-12 16:18 ` Sam Ravnborg 2019-01-25 11:20 ` Philipp Zabel
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox