* [PATCH 0/3] Generic DPI panel on Colibri iMX7 / Col.Eval.board @ 2020-01-15 12:34 Oleksandr Suvorov 2020-01-15 12:34 ` [PATCH 1/3] drm/panel: make LVDS panel driver DPI capable Oleksandr Suvorov 2020-01-15 12:34 ` [PATCH 2/3] drm/panel: pass video modes bus_flags Oleksandr Suvorov 0 siblings, 2 replies; 6+ messages in thread From: Oleksandr Suvorov @ 2020-01-15 12:34 UTC (permalink / raw) To: linux-kernel Cc: Mark Rutland, devicetree, Pengutronix Kernel Team, Marcel Ziswiler, Sam Ravnborg, Sascha Hauer, dri-devel, Oleksandr Suvorov, David Airlie, Igor Opanyuk, Rob Herring, Thierry Reding, NXP Linux Team, Shawn Guo, linux-arm-kernel Make LVDS panel driver DPI capable and use it to add generic RGB (DPI) panel support for Colibri iMX7 SoM on Colibri Evaluation Board. Oleksandr Suvorov (1): ARM: dts: imx7-colibri: add generic RGB (DPI) panel Stefan Agner (2): drm/panel: make LVDS panel driver DPI capable drm/panel: pass video modes bus_flags arch/arm/boot/dts/imx7-colibri-eval-v3.dtsi | 29 +++++++++++++- drivers/gpu/drm/panel/panel-lvds.c | 44 ++++++++++++++++----- 2 files changed, 62 insertions(+), 11 deletions(-) -- 2.24.1 _______________________________________________ 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
* [PATCH 1/3] drm/panel: make LVDS panel driver DPI capable 2020-01-15 12:34 [PATCH 0/3] Generic DPI panel on Colibri iMX7 / Col.Eval.board Oleksandr Suvorov @ 2020-01-15 12:34 ` Oleksandr Suvorov 2020-01-18 13:04 ` Sam Ravnborg 2020-01-15 12:34 ` [PATCH 2/3] drm/panel: pass video modes bus_flags Oleksandr Suvorov 1 sibling, 1 reply; 6+ messages in thread From: Oleksandr Suvorov @ 2020-01-15 12:34 UTC (permalink / raw) To: linux-kernel Cc: Marcel Ziswiler, Oleksandr Suvorov, David Airlie, Igor Opanyuk, Thierry Reding, dri-devel, Sam Ravnborg From: Stefan Agner <stefan@agner.ch> The LVDS panel driver has almost everything which is required to describe a simple parallel RGB panel (also known as DPI, Display Pixel Interface). Signed-off-by: Stefan Agner <stefan@agner.ch> Signed-off-by: Oleksandr Suvorov <oleksandr.suvorov@toradex.com> --- drivers/gpu/drm/panel/panel-lvds.c | 40 +++++++++++++++++++++++------- 1 file changed, 31 insertions(+), 9 deletions(-) diff --git a/drivers/gpu/drm/panel/panel-lvds.c b/drivers/gpu/drm/panel/panel-lvds.c index 5ce3f4a2b7a1..21a169aa3ae4 100644 --- a/drivers/gpu/drm/panel/panel-lvds.c +++ b/drivers/gpu/drm/panel/panel-lvds.c @@ -22,6 +22,11 @@ #include <drm/drm_crtc.h> #include <drm/drm_panel.h> +enum panel_type { + PANEL_LVDS, + PANEL_DPI +}; + struct panel_lvds { struct drm_panel panel; struct device *dev; @@ -115,6 +120,7 @@ static int panel_lvds_parse_dt(struct panel_lvds *lvds) struct display_timing timing; const char *mapping; int ret; + enum panel_type type; ret = of_get_display_timing(np, "panel-timing", &timing); if (ret < 0) { @@ -147,13 +153,28 @@ static int panel_lvds_parse_dt(struct panel_lvds *lvds) return -ENODEV; } - if (!strcmp(mapping, "jeida-18")) { - lvds->bus_format = MEDIA_BUS_FMT_RGB666_1X7X3_SPWG; - } else if (!strcmp(mapping, "jeida-24")) { - lvds->bus_format = MEDIA_BUS_FMT_RGB888_1X7X4_JEIDA; - } else if (!strcmp(mapping, "vesa-24")) { - lvds->bus_format = MEDIA_BUS_FMT_RGB888_1X7X4_SPWG; - } else { + type = (enum panel_type)of_device_get_match_data(lvds->dev); + switch (type) { + case PANEL_LVDS: + if (!strcmp(mapping, "jeida-18")) + lvds->bus_format = MEDIA_BUS_FMT_RGB666_1X7X3_SPWG; + else if (!strcmp(mapping, "jeida-24")) + lvds->bus_format = MEDIA_BUS_FMT_RGB888_1X7X4_JEIDA; + else if (!strcmp(mapping, "vesa-24")) + lvds->bus_format = MEDIA_BUS_FMT_RGB888_1X7X4_SPWG; + break; + case PANEL_DPI: + if (!strcmp(mapping, "rgb24")) + lvds->bus_format = MEDIA_BUS_FMT_RGB888_1X24; + else if (!strcmp(mapping, "rgb565")) + lvds->bus_format = MEDIA_BUS_FMT_RGB565_1X16; + else if (!strcmp(mapping, "bgr666")) + lvds->bus_format = MEDIA_BUS_FMT_RGB666_1X18; + else if (!strcmp(mapping, "lvds666")) + lvds->bus_format = MEDIA_BUS_FMT_RGB666_1X24_CPADHI; + }; + + if (!lvds->bus_format) { dev_err(lvds->dev, "%pOF: invalid or missing %s DT property\n", np, "data-mapping"); return -EINVAL; @@ -247,7 +268,8 @@ static int panel_lvds_remove(struct platform_device *pdev) } static const struct of_device_id panel_lvds_of_table[] = { - { .compatible = "panel-lvds", }, + { .compatible = "panel-lvds", .data = (void *)PANEL_LVDS }, + { .compatible = "panel-dpi", .data = (void *)PANEL_DPI }, { /* Sentinel */ }, }; @@ -257,7 +279,7 @@ static struct platform_driver panel_lvds_driver = { .probe = panel_lvds_probe, .remove = panel_lvds_remove, .driver = { - .name = "panel-lvds", + .name = "panel-generic", .of_match_table = panel_lvds_of_table, }, }; -- 2.24.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 1/3] drm/panel: make LVDS panel driver DPI capable 2020-01-15 12:34 ` [PATCH 1/3] drm/panel: make LVDS panel driver DPI capable Oleksandr Suvorov @ 2020-01-18 13:04 ` Sam Ravnborg 2020-01-20 9:57 ` Oleksandr Suvorov 0 siblings, 1 reply; 6+ messages in thread From: Sam Ravnborg @ 2020-01-18 13:04 UTC (permalink / raw) To: Oleksandr Suvorov Cc: Marcel Ziswiler, linux-kernel, David Airlie, Igor Opanyuk, Thierry Reding, dri-devel Hi Oleksandr & Stafan. Thanks for the update to panel-lvds. On Wed, Jan 15, 2020 at 12:34:17PM +0000, Oleksandr Suvorov wrote: > From: Stefan Agner <stefan@agner.ch> > > The LVDS panel driver has almost everything which is required to > describe a simple parallel RGB panel (also known as DPI, Display > Pixel Interface). > > Signed-off-by: Stefan Agner <stefan@agner.ch> > Signed-off-by: Oleksandr Suvorov <oleksandr.suvorov@toradex.com> > --- There are a few high-level things we need to have sorted out. The driver, when this patch is added, assumes that certain properties are now mandatory when using the panel-dpi compatible. - data-mapping - width-mm - height-mm - panel-timing But this does not match the panel-dpi binding. So we need the panel-dpi binding updated first. The current driver specify the connector type in drm_panel_init(). But a DPI panel is assumed to use a DRM_MODE_CONNECTOR_DPI, and not a DRM_MODE_CONNECTOR_LVDS. So the drm_panel_init() call needs to take into account the type of binding. > @@ -257,7 +279,7 @@ static struct platform_driver panel_lvds_driver = { > .probe = panel_lvds_probe, > .remove = panel_lvds_remove, > .driver = { > - .name = "panel-lvds", > + .name = "panel-generic", I think changing the name of the driver like this is an UAPI change, which is not OK > .of_match_table = panel_lvds_of_table, > }, > }; 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 1/3] drm/panel: make LVDS panel driver DPI capable 2020-01-18 13:04 ` Sam Ravnborg @ 2020-01-20 9:57 ` Oleksandr Suvorov 2020-01-20 20:20 ` Sam Ravnborg 0 siblings, 1 reply; 6+ messages in thread From: Oleksandr Suvorov @ 2020-01-20 9:57 UTC (permalink / raw) To: Sam Ravnborg Cc: Marcel Ziswiler, linux-kernel, dri-devel, Oleksandr Suvorov, David Airlie, Igor Opanyuk, Thierry Reding Hi Sam, On Sat, Jan 18, 2020 at 3:04 PM Sam Ravnborg <sam@ravnborg.org> wrote: > > > > The LVDS panel driver has almost everything which is required to > > describe a simple parallel RGB panel (also known as DPI, Display > > Pixel Interface). > > > > --- > > There are a few high-level things we need to have sorted out. > > The driver, when this patch is added, assumes that certain properties > are now mandatory when using the panel-dpi compatible. > - data-mapping > - width-mm > - height-mm > - panel-timing > > But this does not match the panel-dpi binding. > So we need the panel-dpi binding updated first. > > > The current driver specify the connector type in drm_panel_init(). > But a DPI panel is assumed to use a DRM_MODE_CONNECTOR_DPI, > and not a DRM_MODE_CONNECTOR_LVDS. > So the drm_panel_init() call needs to take into account the type > of binding. > Thanks, I'll fix it in 2nd version. > > > @@ -257,7 +279,7 @@ static struct platform_driver panel_lvds_driver = { > > .probe = panel_lvds_probe, > > .remove = panel_lvds_remove, > > .driver = { > > - .name = "panel-lvds", > > + .name = "panel-generic", > > I think changing the name of the driver like this is an UAPI change, > which is not OK I see 2 simple ways there: - keep the original platform driver name; - fork panel-lvds driver as panel-generic driver with dpi support. What solution do you prefer? > > .of_match_table = panel_lvds_of_table, > > }, > > }; > > Sam -- Best regards Oleksandr Suvorov Toradex AG Altsagenstrasse 5 | 6048 Horw/Luzern | Switzerland | T: +41 41 500 4800 (main line) _______________________________________________ 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 1/3] drm/panel: make LVDS panel driver DPI capable 2020-01-20 9:57 ` Oleksandr Suvorov @ 2020-01-20 20:20 ` Sam Ravnborg 0 siblings, 0 replies; 6+ messages in thread From: Sam Ravnborg @ 2020-01-20 20:20 UTC (permalink / raw) To: Oleksandr Suvorov Cc: Marcel Ziswiler, linux-kernel, dri-devel, David Airlie, Igor Opanyuk, Thierry Reding Hi Oleksandr. Thanks for the quick reply. On Mon, Jan 20, 2020 at 10:03:20AM +0000, Oleksandr Suvorov wrote: > Hi Sam, > > On Sat, Jan 18, 2020 at 3:04 PM Sam Ravnborg <sam@ravnborg.org> wrote: > > > > > > The LVDS panel driver has almost everything which is required to > > > describe a simple parallel RGB panel (also known as DPI, Display > > > Pixel Interface). > > > > > > --- > > > > There are a few high-level things we need to have sorted out. > > > > The driver, when this patch is added, assumes that certain properties > > are now mandatory when using the panel-dpi compatible. > > - data-mapping > > - width-mm > > - height-mm > > - panel-timing > > > > But this does not match the panel-dpi binding. > > So we need the panel-dpi binding updated first. I just sent a patch-set converting this binding to DT schema. Let's land this and you can make your changes on top of it. Care to review it? > > > > > > The current driver specify the connector type in drm_panel_init(). > > But a DPI panel is assumed to use a DRM_MODE_CONNECTOR_DPI, > > and not a DRM_MODE_CONNECTOR_LVDS. > > So the drm_panel_init() call needs to take into account the type > > of binding. > > > Thanks, I'll fix it in 2nd version. > > > > > @@ -257,7 +279,7 @@ static struct platform_driver panel_lvds_driver = { > > > .probe = panel_lvds_probe, > > > .remove = panel_lvds_remove, > > > .driver = { > > > - .name = "panel-lvds", > > > + .name = "panel-generic", > > > > I think changing the name of the driver like this is an UAPI change, > > which is not OK > > I see 2 simple ways there: > - keep the original platform driver name; Please keep the original platform driver name. It is a bit confusing but this is the best option I see. 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
* [PATCH 2/3] drm/panel: pass video modes bus_flags 2020-01-15 12:34 [PATCH 0/3] Generic DPI panel on Colibri iMX7 / Col.Eval.board Oleksandr Suvorov 2020-01-15 12:34 ` [PATCH 1/3] drm/panel: make LVDS panel driver DPI capable Oleksandr Suvorov @ 2020-01-15 12:34 ` Oleksandr Suvorov 1 sibling, 0 replies; 6+ messages in thread From: Oleksandr Suvorov @ 2020-01-15 12:34 UTC (permalink / raw) To: linux-kernel Cc: Marcel Ziswiler, Oleksandr Suvorov, David Airlie, Igor Opanyuk, Thierry Reding, dri-devel, Sam Ravnborg From: Stefan Agner <stefan@agner.ch> Make sure that the bus_flags parsed from the display timings are passed to the connector display info. Signed-off-by: Stefan Agner <stefan@agner.ch> Signed-off-by: Oleksandr Suvorov <oleksandr.suvorov@toradex.com> --- drivers/gpu/drm/panel/panel-lvds.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/panel/panel-lvds.c b/drivers/gpu/drm/panel/panel-lvds.c index 21a169aa3ae4..334a67c1a52f 100644 --- a/drivers/gpu/drm/panel/panel-lvds.c +++ b/drivers/gpu/drm/panel/panel-lvds.c @@ -101,7 +101,9 @@ static int panel_lvds_get_modes(struct drm_panel *panel, connector->display_info.height_mm = lvds->height; drm_display_info_set_bus_formats(&connector->display_info, &lvds->bus_format, 1); - connector->display_info.bus_flags = lvds->data_mirror + drm_bus_flags_from_videomode(&lvds->video_mode, + &connector->display_info.bus_flags); + connector->display_info.bus_flags |= lvds->data_mirror ? DRM_BUS_FLAG_DATA_LSB_TO_MSB : DRM_BUS_FLAG_DATA_MSB_TO_LSB; -- 2.24.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
end of thread, other threads:[~2020-01-20 20:20 UTC | newest] Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2020-01-15 12:34 [PATCH 0/3] Generic DPI panel on Colibri iMX7 / Col.Eval.board Oleksandr Suvorov 2020-01-15 12:34 ` [PATCH 1/3] drm/panel: make LVDS panel driver DPI capable Oleksandr Suvorov 2020-01-18 13:04 ` Sam Ravnborg 2020-01-20 9:57 ` Oleksandr Suvorov 2020-01-20 20:20 ` Sam Ravnborg 2020-01-15 12:34 ` [PATCH 2/3] drm/panel: pass video modes bus_flags Oleksandr Suvorov
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).