From: Marco Felsch <m.felsch@pengutronix.de> To: Oliver Graute <oliver.graute@gmail.com> 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 14:27:36 +0100 [thread overview] Message-ID: <20210201132736.dbqnyzn7dbu7dsmq@pengutronix.de> (raw) In-Reply-To: <1611947364-30688-1-git-send-email-oliver.graute@gmail.com> 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, > +}; > + > +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 */ }; Regards, Marco > +}; > + > static const struct drm_display_mode sharp_ld_d5116z01b_mode = { > .clock = 168480, > .hdisplay = 1920, > @@ -4222,6 +4244,9 @@ static const struct of_device_id platform_of_match[] = { > .compatible = "satoz,sat050at40h12r2", > .data = &satoz_sat050at40h12r2, > }, { > + .compatible = "sgd,gktw70sdad1sd", > + .data = &sgd_gktw70sdad1sd, > + }, { > .compatible = "sharp,ld-d5116z01b", > .data = &sharp_ld_d5116z01b, > }, { > -- > 2.7.4 > > -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
WARNING: multiple messages have this Message-ID (diff)
From: Marco Felsch <m.felsch@pengutronix.de> To: Oliver Graute <oliver.graute@gmail.com> 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 14:27:36 +0100 [thread overview] Message-ID: <20210201132736.dbqnyzn7dbu7dsmq@pengutronix.de> (raw) In-Reply-To: <1611947364-30688-1-git-send-email-oliver.graute@gmail.com> 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, > +}; > + > +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 */ }; Regards, Marco > +}; > + > static const struct drm_display_mode sharp_ld_d5116z01b_mode = { > .clock = 168480, > .hdisplay = 1920, > @@ -4222,6 +4244,9 @@ static const struct of_device_id platform_of_match[] = { > .compatible = "satoz,sat050at40h12r2", > .data = &satoz_sat050at40h12r2, > }, { > + .compatible = "sgd,gktw70sdad1sd", > + .data = &sgd_gktw70sdad1sd, > + }, { > .compatible = "sharp,ld-d5116z01b", > .data = &sharp_ld_d5116z01b, > }, { > -- > 2.7.4 > > -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
next prev parent reply other threads:[~2021-02-01 13:29 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 [this message] 2021-02-01 13:27 ` Marco Felsch 2021-02-01 19:10 ` Oliver Graute 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=20210201132736.dbqnyzn7dbu7dsmq@pengutronix.de \ --to=m.felsch@pengutronix.de \ --cc=airlied@linux.ie \ --cc=daniel@ffwll.ch \ --cc=dri-devel@lists.freedesktop.org \ --cc=festevam@gmail.com \ --cc=linux-kernel@vger.kernel.org \ --cc=oliver.graute@gmail.com \ --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: linkBe 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.