* [PATCH v3] drm/panel: simple: add SGD GKTW70SDAD1SD
@ 2021-02-02 17:35 ` Oliver Graute
0 siblings, 0 replies; 10+ messages in thread
From: Oliver Graute @ 2021-02-02 17:35 UTC (permalink / raw)
To: thierry.reding
Cc: oliver.graute, Marco Felsch, Fabio Estevam, Sam Ravnborg,
David Airlie, Daniel Vetter, dri-devel, linux-kernel
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>
---
v3:
- added flags
- added delay
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 better (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..c63f6a8 100644
--- a/drivers/gpu/drm/panel/panel-simple.c
+++ b/drivers/gpu/drm/panel/panel-simple.c
@@ -3336,6 +3336,36 @@ 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},
+ .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,
+ },
+ .delay = {
+ .prepare = 20 + 20 + 10 + 10,
+ .enable = 50,
+ .disable = 50,
+ .unprepare = 10 + 10 + 20 + 20,
+ },
+};
+
static const struct drm_display_mode sharp_ld_d5116z01b_mode = {
.clock = 168480,
.hdisplay = 1920,
@@ -4222,6 +4252,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
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH v3] drm/panel: simple: add SGD GKTW70SDAD1SD
@ 2021-02-02 17:35 ` Oliver Graute
0 siblings, 0 replies; 10+ messages in thread
From: Oliver Graute @ 2021-02-02 17:35 UTC (permalink / raw)
To: thierry.reding
Cc: David Airlie, Sam Ravnborg, Marco Felsch, dri-devel,
linux-kernel, oliver.graute
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>
---
v3:
- added flags
- added delay
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 better (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..c63f6a8 100644
--- a/drivers/gpu/drm/panel/panel-simple.c
+++ b/drivers/gpu/drm/panel/panel-simple.c
@@ -3336,6 +3336,36 @@ 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},
+ .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,
+ },
+ .delay = {
+ .prepare = 20 + 20 + 10 + 10,
+ .enable = 50,
+ .disable = 50,
+ .unprepare = 10 + 10 + 20 + 20,
+ },
+};
+
static const struct drm_display_mode sharp_ld_d5116z01b_mode = {
.clock = 168480,
.hdisplay = 1920,
@@ -4222,6 +4252,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
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH v3] drm/panel: simple: add SGD GKTW70SDAD1SD
2021-02-02 17:35 ` Oliver Graute
@ 2021-02-02 17:59 ` Marco Felsch
-1 siblings, 0 replies; 10+ messages in thread
From: Marco Felsch @ 2021-02-02 17:59 UTC (permalink / raw)
To: Oliver Graute
Cc: thierry.reding, Fabio Estevam, Sam Ravnborg, David Airlie,
Daniel Vetter, dri-devel, linux-kernel
Hi Oliver,
On 21-02-02 18:35, 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>
> ---
>
> v3:
>
> - added flags
> - added delay
Thanks, did you test the changes?
I just picked it from the datasheet.
Regards,
Marco
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v3] drm/panel: simple: add SGD GKTW70SDAD1SD
@ 2021-02-02 17:59 ` Marco Felsch
0 siblings, 0 replies; 10+ messages in thread
From: Marco Felsch @ 2021-02-02 17:59 UTC (permalink / raw)
To: Oliver Graute
Cc: David Airlie, linux-kernel, dri-devel, thierry.reding, Sam Ravnborg
Hi Oliver,
On 21-02-02 18:35, 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>
> ---
>
> v3:
>
> - added flags
> - added delay
Thanks, did you test the changes?
I just picked it from the datasheet.
Regards,
Marco
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v3] drm/panel: simple: add SGD GKTW70SDAD1SD
2021-02-02 17:59 ` Marco Felsch
@ 2021-02-04 18:15 ` Oliver Graute
-1 siblings, 0 replies; 10+ messages in thread
From: Oliver Graute @ 2021-02-04 18:15 UTC (permalink / raw)
To: Marco Felsch
Cc: thierry.reding, Fabio Estevam, Sam Ravnborg, David Airlie,
Daniel Vetter, dri-devel, linux-kernel
On 02/02/21, Marco Felsch wrote:
> Hi Oliver,
>
> On 21-02-02 18:35, 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>
> > ---
> >
> > v3:
> >
> > - added flags
> > - added delay
>
> Thanks, did you test the changes?
> I just picked it from the datasheet.
yes, it didn't break anything.
Best regards,
Oliver
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v3] drm/panel: simple: add SGD GKTW70SDAD1SD
@ 2021-02-04 18:15 ` Oliver Graute
0 siblings, 0 replies; 10+ messages in thread
From: Oliver Graute @ 2021-02-04 18:15 UTC (permalink / raw)
To: Marco Felsch
Cc: David Airlie, linux-kernel, dri-devel, thierry.reding, Sam Ravnborg
On 02/02/21, Marco Felsch wrote:
> Hi Oliver,
>
> On 21-02-02 18:35, 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>
> > ---
> >
> > v3:
> >
> > - added flags
> > - added delay
>
> Thanks, did you test the changes?
> I just picked it from the datasheet.
yes, it didn't break anything.
Best regards,
Oliver
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v3] drm/panel: simple: add SGD GKTW70SDAD1SD
2021-02-02 17:35 ` Oliver Graute
@ 2021-02-04 18:22 ` Fabio Estevam
-1 siblings, 0 replies; 10+ messages in thread
From: Fabio Estevam @ 2021-02-04 18:22 UTC (permalink / raw)
To: Oliver Graute
Cc: Thierry Reding, Marco Felsch, Sam Ravnborg, David Airlie,
Daniel Vetter, DRI mailing list, linux-kernel
Hi Oliver,
On Tue, Feb 2, 2021 at 2:35 PM Oliver Graute <oliver.graute@gmail.com> wrote:
> +static const struct panel_desc sgd_gktw70sdad1sd = {
> + .timings = &sgd_gktw70sdad1sd_timing,
> + .num_timings = 1,
> + .bpc = 6,
> + .size = {
> + .width = 153,
> + .height = 86,
> + },
> + .delay = {
> + .prepare = 20 + 20 + 10 + 10,
Adding the datasheet label like Marco suggested make it clearer where
these numbers come from : /* T0 + T2 + T3 + T4 */
Glad you got it working on your imx6ul board:
Reviewed-by: Fabio Estevam <festevam@gmail.com>
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v3] drm/panel: simple: add SGD GKTW70SDAD1SD
@ 2021-02-04 18:22 ` Fabio Estevam
0 siblings, 0 replies; 10+ messages in thread
From: Fabio Estevam @ 2021-02-04 18:22 UTC (permalink / raw)
To: Oliver Graute
Cc: David Airlie, Marco Felsch, DRI mailing list, linux-kernel,
Thierry Reding, Sam Ravnborg
Hi Oliver,
On Tue, Feb 2, 2021 at 2:35 PM Oliver Graute <oliver.graute@gmail.com> wrote:
> +static const struct panel_desc sgd_gktw70sdad1sd = {
> + .timings = &sgd_gktw70sdad1sd_timing,
> + .num_timings = 1,
> + .bpc = 6,
> + .size = {
> + .width = 153,
> + .height = 86,
> + },
> + .delay = {
> + .prepare = 20 + 20 + 10 + 10,
Adding the datasheet label like Marco suggested make it clearer where
these numbers come from : /* T0 + T2 + T3 + T4 */
Glad you got it working on your imx6ul board:
Reviewed-by: Fabio Estevam <festevam@gmail.com>
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v3] drm/panel: simple: add SGD GKTW70SDAD1SD
2021-02-04 18:15 ` Oliver Graute
@ 2021-02-05 9:09 ` Marco Felsch
-1 siblings, 0 replies; 10+ messages in thread
From: Marco Felsch @ 2021-02-05 9:09 UTC (permalink / raw)
To: Oliver Graute
Cc: thierry.reding, Fabio Estevam, Sam Ravnborg, David Airlie,
Daniel Vetter, dri-devel, linux-kernel
On 21-02-04 19:15, Oliver Graute wrote:
> On 02/02/21, Marco Felsch wrote:
> > Hi Oliver,
> >
> > On 21-02-02 18:35, 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>
> > > ---
> > >
> > > v3:
> > >
> > > - added flags
> > > - added delay
> >
> > Thanks, did you test the changes?
> > I just picked it from the datasheet.
>
> yes, it didn't break anything.
Feel free to add my:
Reviewed-by: Marco Felsch <m.felsch@pengutronix.de>
PS:
+1 for Fabio's comment ^^
Regards,
Marco
>
> Best regards,
>
> Oliver
>
--
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 |
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v3] drm/panel: simple: add SGD GKTW70SDAD1SD
@ 2021-02-05 9:09 ` Marco Felsch
0 siblings, 0 replies; 10+ messages in thread
From: Marco Felsch @ 2021-02-05 9:09 UTC (permalink / raw)
To: Oliver Graute
Cc: David Airlie, linux-kernel, dri-devel, thierry.reding, Sam Ravnborg
On 21-02-04 19:15, Oliver Graute wrote:
> On 02/02/21, Marco Felsch wrote:
> > Hi Oliver,
> >
> > On 21-02-02 18:35, 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>
> > > ---
> > >
> > > v3:
> > >
> > > - added flags
> > > - added delay
> >
> > Thanks, did you test the changes?
> > I just picked it from the datasheet.
>
> yes, it didn't break anything.
Feel free to add my:
Reviewed-by: Marco Felsch <m.felsch@pengutronix.de>
PS:
+1 for Fabio's comment ^^
Regards,
Marco
>
> Best regards,
>
> Oliver
>
--
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
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2021-02-05 9:14 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-02-02 17:35 [PATCH v3] drm/panel: simple: add SGD GKTW70SDAD1SD Oliver Graute
2021-02-02 17:35 ` Oliver Graute
2021-02-02 17:59 ` Marco Felsch
2021-02-02 17:59 ` Marco Felsch
2021-02-04 18:15 ` Oliver Graute
2021-02-04 18:15 ` Oliver Graute
2021-02-05 9:09 ` Marco Felsch
2021-02-05 9:09 ` Marco Felsch
2021-02-04 18:22 ` Fabio Estevam
2021-02-04 18:22 ` Fabio Estevam
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.