All of lore.kernel.org
 help / color / mirror / Atom feed
From: Oliver Graute <oliver.graute@gmail.com>
To: Marco Felsch <m.felsch@pengutronix.de>
Cc: David Airlie <airlied@linux.ie>,
	linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org,
	thierry.reding@gmail.com, Sam Ravnborg <sam@ravnborg.org>
Subject: Re: [PATCH v2] drm/panel: simple: add SGD GKTW70SDAD1SD
Date: Mon, 1 Feb 2021 20:10:01 +0100	[thread overview]
Message-ID: <20210201191001.GA22816@portage> (raw)
In-Reply-To: <20210201132736.dbqnyzn7dbu7dsmq@pengutronix.de>

On 01/02/21, Marco Felsch wrote:
> Hi Oliver,
> 
> thanks for the patch :)
> 
> On 21-01-29 20:09, Oliver Graute wrote:
> > Add support for the Solomon Goldentek Display Model: GKTW70SDAD1SD
> > to panel-simple.
> > 
> > The panel spec from Variscite can be found at:
> > https://www.variscite.com/wp-content/uploads/2017/12/VLCD-CAP-GLD-RGB.pdf
> > 
> > Signed-off-by: Oliver Graute <oliver.graute@gmail.com>
> > Cc: Marco Felsch <m.felsch@pengutronix.de>
> > Cc: Fabio Estevam <festevam@gmail.com>
> > ---
> > 
> > v2:
> > 
> > - changed bpc to 6
> > - set max value of pixelclock
> > - increased hfront_porch and hback_porch
> > - dropped connector-type
> > 
> > adding of bus_format = MEDIA_BUS_FMT_RGB666_1X18 results in wrong colors.
> > omitting bus_format and using some default is good (Tux Pinguin is colored
> > fine)
> > 
> >  drivers/gpu/drm/panel/panel-simple.c | 26 ++++++++++++++++++++++++++
> >  1 file changed, 26 insertions(+)
> > 
> > diff --git a/drivers/gpu/drm/panel/panel-simple.c b/drivers/gpu/drm/panel/panel-simple.c
> > index 2be358f..c129a8c 100644
> > --- a/drivers/gpu/drm/panel/panel-simple.c
> > +++ b/drivers/gpu/drm/panel/panel-simple.c
> > @@ -3336,6 +3336,28 @@ static const struct panel_desc satoz_sat050at40h12r2 = {
> >  	.connector_type = DRM_MODE_CONNECTOR_LVDS,
> >  };
> >  
> > +static const struct display_timing sgd_gktw70sdad1sd_timing = {
> > +	.pixelclock = {30000000, 30000000, 40000000},
> > +	.hactive = { 800, 800, 800},
> > +	.hfront_porch = {40, 40, 40},
> > +	.hback_porch = {40, 40, 40},
> > +	.hsync_len = {48, 48, 48},
> > +	.vactive = {480, 480, 480},
> > +	.vfront_porch = {13, 13, 13},
> > +	.vback_porch = {29, 29, 29},
> > +	.vsync_len = {3, 3, 3},
> 
> Please add also:
> 
> 	.flags = DISPLAY_FLAGS_HSYNC_LOW | DISPLAY_FLAGS_VSYNC_LOW |
> 		 DISPLAY_FLAGS_DE_HIGH | DISPLAY_FLAGS_PIXDATA_NEGEDGE,
ok will do

> 
> > +};
> > +
> > +static const struct panel_desc sgd_gktw70sdad1sd = {
> > +	.timings = &sgd_gktw70sdad1sd_timing,
> > +	.num_timings = 1,
> > +	.bpc = 6,
> > +	.size = {
> > +		.width = 153,
> > +		.height = 86,
> > +	},
> 
> and:
> 
> 	.delay = {
> 		.prepare = 20 + 20 + 10 + 10, /* T0 + T2 + T3 + T4 */
> 		.enable = 50, /* T5 */
> 		.disable = 50, /* T5 */
> 		.unprepare =  10 + 10 + 20 + 20, /* T4 + T3 + T2 + T0 */
> 	};

ok will do

thx for your review.

Best regards,

Oliver
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

WARNING: multiple messages have this Message-ID (diff)
From: Oliver Graute <oliver.graute@gmail.com>
To: Marco Felsch <m.felsch@pengutronix.de>
Cc: thierry.reding@gmail.com, Fabio Estevam <festevam@gmail.com>,
	Sam Ravnborg <sam@ravnborg.org>, David Airlie <airlied@linux.ie>,
	Daniel Vetter <daniel@ffwll.ch>,
	dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v2] drm/panel: simple: add SGD GKTW70SDAD1SD
Date: Mon, 1 Feb 2021 20:10:01 +0100	[thread overview]
Message-ID: <20210201191001.GA22816@portage> (raw)
In-Reply-To: <20210201132736.dbqnyzn7dbu7dsmq@pengutronix.de>

On 01/02/21, Marco Felsch wrote:
> Hi Oliver,
> 
> thanks for the patch :)
> 
> On 21-01-29 20:09, Oliver Graute wrote:
> > Add support for the Solomon Goldentek Display Model: GKTW70SDAD1SD
> > to panel-simple.
> > 
> > The panel spec from Variscite can be found at:
> > https://www.variscite.com/wp-content/uploads/2017/12/VLCD-CAP-GLD-RGB.pdf
> > 
> > Signed-off-by: Oliver Graute <oliver.graute@gmail.com>
> > Cc: Marco Felsch <m.felsch@pengutronix.de>
> > Cc: Fabio Estevam <festevam@gmail.com>
> > ---
> > 
> > v2:
> > 
> > - changed bpc to 6
> > - set max value of pixelclock
> > - increased hfront_porch and hback_porch
> > - dropped connector-type
> > 
> > adding of bus_format = MEDIA_BUS_FMT_RGB666_1X18 results in wrong colors.
> > omitting bus_format and using some default is good (Tux Pinguin is colored
> > fine)
> > 
> >  drivers/gpu/drm/panel/panel-simple.c | 26 ++++++++++++++++++++++++++
> >  1 file changed, 26 insertions(+)
> > 
> > diff --git a/drivers/gpu/drm/panel/panel-simple.c b/drivers/gpu/drm/panel/panel-simple.c
> > index 2be358f..c129a8c 100644
> > --- a/drivers/gpu/drm/panel/panel-simple.c
> > +++ b/drivers/gpu/drm/panel/panel-simple.c
> > @@ -3336,6 +3336,28 @@ static const struct panel_desc satoz_sat050at40h12r2 = {
> >  	.connector_type = DRM_MODE_CONNECTOR_LVDS,
> >  };
> >  
> > +static const struct display_timing sgd_gktw70sdad1sd_timing = {
> > +	.pixelclock = {30000000, 30000000, 40000000},
> > +	.hactive = { 800, 800, 800},
> > +	.hfront_porch = {40, 40, 40},
> > +	.hback_porch = {40, 40, 40},
> > +	.hsync_len = {48, 48, 48},
> > +	.vactive = {480, 480, 480},
> > +	.vfront_porch = {13, 13, 13},
> > +	.vback_porch = {29, 29, 29},
> > +	.vsync_len = {3, 3, 3},
> 
> Please add also:
> 
> 	.flags = DISPLAY_FLAGS_HSYNC_LOW | DISPLAY_FLAGS_VSYNC_LOW |
> 		 DISPLAY_FLAGS_DE_HIGH | DISPLAY_FLAGS_PIXDATA_NEGEDGE,
ok will do

> 
> > +};
> > +
> > +static const struct panel_desc sgd_gktw70sdad1sd = {
> > +	.timings = &sgd_gktw70sdad1sd_timing,
> > +	.num_timings = 1,
> > +	.bpc = 6,
> > +	.size = {
> > +		.width = 153,
> > +		.height = 86,
> > +	},
> 
> and:
> 
> 	.delay = {
> 		.prepare = 20 + 20 + 10 + 10, /* T0 + T2 + T3 + T4 */
> 		.enable = 50, /* T5 */
> 		.disable = 50, /* T5 */
> 		.unprepare =  10 + 10 + 20 + 20, /* T4 + T3 + T2 + T0 */
> 	};

ok will do

thx for your review.

Best regards,

Oliver

  reply	other threads:[~2021-02-02  8:27 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-01-29 19:09 [PATCH v2] drm/panel: simple: add SGD GKTW70SDAD1SD Oliver Graute
2021-01-29 19:09 ` Oliver Graute
2021-02-01 13:27 ` Marco Felsch
2021-02-01 13:27   ` Marco Felsch
2021-02-01 19:10   ` Oliver Graute [this message]
2021-02-01 19:10     ` Oliver Graute

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20210201191001.GA22816@portage \
    --to=oliver.graute@gmail.com \
    --cc=airlied@linux.ie \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=m.felsch@pengutronix.de \
    --cc=sam@ravnborg.org \
    --cc=thierry.reding@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.