* [PATCH v1 0/2] Improve descriptions of a few simple-panels
@ 2020-06-21 22:27 Dmitry Osipenko
2020-06-21 22:27 ` [PATCH v1 1/2] drm/panel-simple: Correct EDT ET057090DHU connector type Dmitry Osipenko
` (2 more replies)
0 siblings, 3 replies; 12+ messages in thread
From: Dmitry Osipenko @ 2020-06-21 22:27 UTC (permalink / raw)
To: Thierry Reding, Sam Ravnborg, Laurent Pinchart
Cc: dri-devel, linux-tegra, linux-kernel
Hello,
This is a follow up to [1], which was already applied to drm-misc and then
Laurent Pinchart spotted some problems. This series addresses those problems.
[1] https://patchwork.ozlabs.org/project/linux-tegra/patch/20200617222703.17080-8-digetx@gmail.com/
Dmitry Osipenko (2):
drm/panel-simple: Correct EDT ET057090DHU connector type
drm/panel-simple: Add missing BUS descriptions for some panels
drivers/gpu/drm/panel/panel-simple.c | 14 +++++++++++++-
1 file changed, 13 insertions(+), 1 deletion(-)
--
2.26.0
^ permalink raw reply [flat|nested] 12+ messages in thread* [PATCH v1 1/2] drm/panel-simple: Correct EDT ET057090DHU connector type 2020-06-21 22:27 [PATCH v1 0/2] Improve descriptions of a few simple-panels Dmitry Osipenko @ 2020-06-21 22:27 ` Dmitry Osipenko 2020-06-21 22:29 ` Laurent Pinchart 2020-06-21 22:27 ` [PATCH v1 2/2] drm/panel-simple: Add missing BUS descriptions for some panels Dmitry Osipenko 2020-06-27 20:19 ` [PATCH v1 0/2] Improve descriptions of a few simple-panels Sam Ravnborg 2 siblings, 1 reply; 12+ messages in thread From: Dmitry Osipenko @ 2020-06-21 22:27 UTC (permalink / raw) To: Thierry Reding, Sam Ravnborg, Laurent Pinchart Cc: dri-devel, linux-tegra, linux-kernel The EDT ET057090DHU panel has a DPI connector and not LVDS. This patch corrects the panel's description. Reported-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> Fixes: 94f07917ebe8 ("drm/panel-simple: Add missing connector type for some panels") Signed-off-by: Dmitry Osipenko <digetx@gmail.com> --- drivers/gpu/drm/panel/panel-simple.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/panel/panel-simple.c b/drivers/gpu/drm/panel/panel-simple.c index ea6973d5cf54..87edd2bdf09a 100644 --- a/drivers/gpu/drm/panel/panel-simple.c +++ b/drivers/gpu/drm/panel/panel-simple.c @@ -1588,7 +1588,7 @@ static const struct panel_desc edt_et057090dhu = { }, .bus_format = MEDIA_BUS_FMT_RGB666_1X18, .bus_flags = DRM_BUS_FLAG_DE_HIGH | DRM_BUS_FLAG_PIXDATA_DRIVE_NEGEDGE, - .connector_type = DRM_MODE_CONNECTOR_LVDS, + .connector_type = DRM_MODE_CONNECTOR_DPI, }; static const struct drm_display_mode edt_etm0700g0dh6_mode = { -- 2.26.0 ^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [PATCH v1 1/2] drm/panel-simple: Correct EDT ET057090DHU connector type 2020-06-21 22:27 ` [PATCH v1 1/2] drm/panel-simple: Correct EDT ET057090DHU connector type Dmitry Osipenko @ 2020-06-21 22:29 ` Laurent Pinchart 2020-06-22 13:40 ` Dmitry Osipenko 0 siblings, 1 reply; 12+ messages in thread From: Laurent Pinchart @ 2020-06-21 22:29 UTC (permalink / raw) To: Dmitry Osipenko Cc: Thierry Reding, Sam Ravnborg, dri-devel, linux-tegra, linux-kernel Hi Dmitry, Thank you for the patch. On Mon, Jun 22, 2020 at 01:27:41AM +0300, Dmitry Osipenko wrote: > The EDT ET057090DHU panel has a DPI connector and not LVDS. This patch > corrects the panel's description. > > Reported-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > Fixes: 94f07917ebe8 ("drm/panel-simple: Add missing connector type for some panels") > Signed-off-by: Dmitry Osipenko <digetx@gmail.com> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > --- > drivers/gpu/drm/panel/panel-simple.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/panel/panel-simple.c b/drivers/gpu/drm/panel/panel-simple.c > index ea6973d5cf54..87edd2bdf09a 100644 > --- a/drivers/gpu/drm/panel/panel-simple.c > +++ b/drivers/gpu/drm/panel/panel-simple.c > @@ -1588,7 +1588,7 @@ static const struct panel_desc edt_et057090dhu = { > }, > .bus_format = MEDIA_BUS_FMT_RGB666_1X18, > .bus_flags = DRM_BUS_FLAG_DE_HIGH | DRM_BUS_FLAG_PIXDATA_DRIVE_NEGEDGE, > - .connector_type = DRM_MODE_CONNECTOR_LVDS, > + .connector_type = DRM_MODE_CONNECTOR_DPI, > }; > > static const struct drm_display_mode edt_etm0700g0dh6_mode = { -- Regards, Laurent Pinchart ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH v1 1/2] drm/panel-simple: Correct EDT ET057090DHU connector type 2020-06-21 22:29 ` Laurent Pinchart @ 2020-06-22 13:40 ` Dmitry Osipenko 0 siblings, 0 replies; 12+ messages in thread From: Dmitry Osipenko @ 2020-06-22 13:40 UTC (permalink / raw) To: Laurent Pinchart Cc: Thierry Reding, Sam Ravnborg, dri-devel, linux-tegra, linux-kernel 22.06.2020 01:29, Laurent Pinchart пишет: > Hi Dmitry, > > Thank you for the patch. > > On Mon, Jun 22, 2020 at 01:27:41AM +0300, Dmitry Osipenko wrote: >> The EDT ET057090DHU panel has a DPI connector and not LVDS. This patch >> corrects the panel's description. >> >> Reported-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> >> Fixes: 94f07917ebe8 ("drm/panel-simple: Add missing connector type for some panels") >> Signed-off-by: Dmitry Osipenko <digetx@gmail.com> > > Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> Thank you! :) ^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH v1 2/2] drm/panel-simple: Add missing BUS descriptions for some panels 2020-06-21 22:27 [PATCH v1 0/2] Improve descriptions of a few simple-panels Dmitry Osipenko 2020-06-21 22:27 ` [PATCH v1 1/2] drm/panel-simple: Correct EDT ET057090DHU connector type Dmitry Osipenko @ 2020-06-21 22:27 ` Dmitry Osipenko 2020-06-27 20:43 ` Laurent Pinchart 2020-06-27 20:19 ` [PATCH v1 0/2] Improve descriptions of a few simple-panels Sam Ravnborg 2 siblings, 1 reply; 12+ messages in thread From: Dmitry Osipenko @ 2020-06-21 22:27 UTC (permalink / raw) To: Thierry Reding, Sam Ravnborg, Laurent Pinchart Cc: dri-devel, linux-tegra, linux-kernel This patch adds missing BUS fields to the display panel descriptions of the panels which are found on NVIDIA Tegra devices: 1. AUO B101AW03 2. Chunghwa CLAA070WP03XG 3. Chunghwa CLAA101WA01A 4. Chunghwa CLAA101WB01 5. Innolux N156BGE L21 6. Samsung LTN101NT05 Suggested-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> Signed-off-by: Dmitry Osipenko <digetx@gmail.com> --- drivers/gpu/drm/panel/panel-simple.c | 12 ++++++++++++ 1 file changed, 12 insertions(+) diff --git a/drivers/gpu/drm/panel/panel-simple.c b/drivers/gpu/drm/panel/panel-simple.c index 87edd2bdf09a..986df9937650 100644 --- a/drivers/gpu/drm/panel/panel-simple.c +++ b/drivers/gpu/drm/panel/panel-simple.c @@ -698,6 +698,8 @@ static const struct panel_desc auo_b101aw03 = { .width = 223, .height = 125, }, + .bus_format = MEDIA_BUS_FMT_RGB666_1X7X3_SPWG, + .bus_flags = DRM_BUS_FLAG_DE_HIGH | DRM_BUS_FLAG_PIXDATA_DRIVE_POSEDGE, .connector_type = DRM_MODE_CONNECTOR_LVDS, }; @@ -1352,6 +1354,8 @@ static const struct panel_desc chunghwa_claa070wp03xg = { .width = 94, .height = 150, }, + .bus_format = MEDIA_BUS_FMT_RGB666_1X7X3_SPWG, + .bus_flags = DRM_BUS_FLAG_DE_HIGH | DRM_BUS_FLAG_PIXDATA_DRIVE_POSEDGE, .connector_type = DRM_MODE_CONNECTOR_LVDS, }; @@ -1375,6 +1379,8 @@ static const struct panel_desc chunghwa_claa101wa01a = { .width = 220, .height = 120, }, + .bus_format = MEDIA_BUS_FMT_RGB666_1X7X3_SPWG, + .bus_flags = DRM_BUS_FLAG_DE_HIGH | DRM_BUS_FLAG_PIXDATA_DRIVE_POSEDGE, .connector_type = DRM_MODE_CONNECTOR_LVDS, }; @@ -1398,6 +1404,8 @@ static const struct panel_desc chunghwa_claa101wb01 = { .width = 223, .height = 125, }, + .bus_format = MEDIA_BUS_FMT_RGB666_1X7X3_SPWG, + .bus_flags = DRM_BUS_FLAG_DE_HIGH | DRM_BUS_FLAG_PIXDATA_DRIVE_POSEDGE, .connector_type = DRM_MODE_CONNECTOR_LVDS, }; @@ -2071,6 +2079,8 @@ static const struct panel_desc innolux_n156bge_l21 = { .width = 344, .height = 193, }, + .bus_format = MEDIA_BUS_FMT_RGB666_1X7X3_SPWG, + .bus_flags = DRM_BUS_FLAG_DE_HIGH | DRM_BUS_FLAG_PIXDATA_DRIVE_POSEDGE, .connector_type = DRM_MODE_CONNECTOR_LVDS, }; @@ -3018,6 +3028,8 @@ static const struct panel_desc samsung_ltn101nt05 = { .width = 223, .height = 125, }, + .bus_format = MEDIA_BUS_FMT_RGB666_1X7X3_SPWG, + .bus_flags = DRM_BUS_FLAG_DE_HIGH | DRM_BUS_FLAG_PIXDATA_DRIVE_POSEDGE, .connector_type = DRM_MODE_CONNECTOR_LVDS, }; -- 2.26.0 ^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [PATCH v1 2/2] drm/panel-simple: Add missing BUS descriptions for some panels 2020-06-21 22:27 ` [PATCH v1 2/2] drm/panel-simple: Add missing BUS descriptions for some panels Dmitry Osipenko @ 2020-06-27 20:43 ` Laurent Pinchart 2020-06-27 23:44 ` Dmitry Osipenko 0 siblings, 1 reply; 12+ messages in thread From: Laurent Pinchart @ 2020-06-27 20:43 UTC (permalink / raw) To: Dmitry Osipenko Cc: Thierry Reding, Sam Ravnborg, dri-devel, linux-tegra, linux-kernel Hi Dmitry, Thank you for the patch. On Mon, Jun 22, 2020 at 01:27:42AM +0300, Dmitry Osipenko wrote: > This patch adds missing BUS fields to the display panel descriptions of > the panels which are found on NVIDIA Tegra devices: > > 1. AUO B101AW03 > 2. Chunghwa CLAA070WP03XG > 3. Chunghwa CLAA101WA01A > 4. Chunghwa CLAA101WB01 > 5. Innolux N156BGE L21 > 6. Samsung LTN101NT05 > > Suggested-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > Signed-off-by: Dmitry Osipenko <digetx@gmail.com> > --- > drivers/gpu/drm/panel/panel-simple.c | 12 ++++++++++++ > 1 file changed, 12 insertions(+) > > diff --git a/drivers/gpu/drm/panel/panel-simple.c b/drivers/gpu/drm/panel/panel-simple.c > index 87edd2bdf09a..986df9937650 100644 > --- a/drivers/gpu/drm/panel/panel-simple.c > +++ b/drivers/gpu/drm/panel/panel-simple.c > @@ -698,6 +698,8 @@ static const struct panel_desc auo_b101aw03 = { > .width = 223, > .height = 125, > }, > + .bus_format = MEDIA_BUS_FMT_RGB666_1X7X3_SPWG, > + .bus_flags = DRM_BUS_FLAG_DE_HIGH | DRM_BUS_FLAG_PIXDATA_DRIVE_POSEDGE, Does DRM_BUS_FLAG_PIXDATA_DRIVE_* make sense for LVDS ? The rest looks good, except the Samsung panel for which I haven't been able to locate a datasheet. Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > .connector_type = DRM_MODE_CONNECTOR_LVDS, > }; > > @@ -1352,6 +1354,8 @@ static const struct panel_desc chunghwa_claa070wp03xg = { > .width = 94, > .height = 150, > }, > + .bus_format = MEDIA_BUS_FMT_RGB666_1X7X3_SPWG, > + .bus_flags = DRM_BUS_FLAG_DE_HIGH | DRM_BUS_FLAG_PIXDATA_DRIVE_POSEDGE, > .connector_type = DRM_MODE_CONNECTOR_LVDS, > }; > > @@ -1375,6 +1379,8 @@ static const struct panel_desc chunghwa_claa101wa01a = { > .width = 220, > .height = 120, > }, > + .bus_format = MEDIA_BUS_FMT_RGB666_1X7X3_SPWG, > + .bus_flags = DRM_BUS_FLAG_DE_HIGH | DRM_BUS_FLAG_PIXDATA_DRIVE_POSEDGE, > .connector_type = DRM_MODE_CONNECTOR_LVDS, > }; > > @@ -1398,6 +1404,8 @@ static const struct panel_desc chunghwa_claa101wb01 = { > .width = 223, > .height = 125, > }, > + .bus_format = MEDIA_BUS_FMT_RGB666_1X7X3_SPWG, > + .bus_flags = DRM_BUS_FLAG_DE_HIGH | DRM_BUS_FLAG_PIXDATA_DRIVE_POSEDGE, > .connector_type = DRM_MODE_CONNECTOR_LVDS, > }; > > @@ -2071,6 +2079,8 @@ static const struct panel_desc innolux_n156bge_l21 = { > .width = 344, > .height = 193, > }, > + .bus_format = MEDIA_BUS_FMT_RGB666_1X7X3_SPWG, > + .bus_flags = DRM_BUS_FLAG_DE_HIGH | DRM_BUS_FLAG_PIXDATA_DRIVE_POSEDGE, > .connector_type = DRM_MODE_CONNECTOR_LVDS, > }; > > @@ -3018,6 +3028,8 @@ static const struct panel_desc samsung_ltn101nt05 = { > .width = 223, > .height = 125, > }, > + .bus_format = MEDIA_BUS_FMT_RGB666_1X7X3_SPWG, > + .bus_flags = DRM_BUS_FLAG_DE_HIGH | DRM_BUS_FLAG_PIXDATA_DRIVE_POSEDGE, > .connector_type = DRM_MODE_CONNECTOR_LVDS, > }; > -- Regards, Laurent Pinchart ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH v1 2/2] drm/panel-simple: Add missing BUS descriptions for some panels 2020-06-27 20:43 ` Laurent Pinchart @ 2020-06-27 23:44 ` Dmitry Osipenko 2020-06-28 7:07 ` Laurent Pinchart 0 siblings, 1 reply; 12+ messages in thread From: Dmitry Osipenko @ 2020-06-27 23:44 UTC (permalink / raw) To: Laurent Pinchart Cc: Thierry Reding, Sam Ravnborg, dri-devel, linux-tegra, linux-kernel 27.06.2020 23:43, Laurent Pinchart пишет: > Hi Dmitry, > > Thank you for the patch. > > On Mon, Jun 22, 2020 at 01:27:42AM +0300, Dmitry Osipenko wrote: >> This patch adds missing BUS fields to the display panel descriptions of >> the panels which are found on NVIDIA Tegra devices: >> >> 1. AUO B101AW03 >> 2. Chunghwa CLAA070WP03XG >> 3. Chunghwa CLAA101WA01A >> 4. Chunghwa CLAA101WB01 >> 5. Innolux N156BGE L21 >> 6. Samsung LTN101NT05 >> >> Suggested-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> >> Signed-off-by: Dmitry Osipenko <digetx@gmail.com> >> --- >> drivers/gpu/drm/panel/panel-simple.c | 12 ++++++++++++ >> 1 file changed, 12 insertions(+) >> >> diff --git a/drivers/gpu/drm/panel/panel-simple.c b/drivers/gpu/drm/panel/panel-simple.c >> index 87edd2bdf09a..986df9937650 100644 >> --- a/drivers/gpu/drm/panel/panel-simple.c >> +++ b/drivers/gpu/drm/panel/panel-simple.c >> @@ -698,6 +698,8 @@ static const struct panel_desc auo_b101aw03 = { >> .width = 223, >> .height = 125, >> }, >> + .bus_format = MEDIA_BUS_FMT_RGB666_1X7X3_SPWG, >> + .bus_flags = DRM_BUS_FLAG_DE_HIGH | DRM_BUS_FLAG_PIXDATA_DRIVE_POSEDGE, > > Does DRM_BUS_FLAG_PIXDATA_DRIVE_* make sense for LVDS ? To be honest I don't know whether it make sense or not for LVDS. I saw that other LVDS panels in panel-simple.c use the PIXDATA flag and then looked at what timing diagrams in the datasheets show. > The rest looks good, except the Samsung panel for which I haven't been > able to locate a datasheet. > > Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> Thanks to you and Sam! ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH v1 2/2] drm/panel-simple: Add missing BUS descriptions for some panels 2020-06-27 23:44 ` Dmitry Osipenko @ 2020-06-28 7:07 ` Laurent Pinchart 2020-06-28 7:52 ` Sam Ravnborg 0 siblings, 1 reply; 12+ messages in thread From: Laurent Pinchart @ 2020-06-28 7:07 UTC (permalink / raw) To: Dmitry Osipenko Cc: Thierry Reding, Sam Ravnborg, dri-devel, linux-tegra, linux-kernel Hi Dmitry, On Sun, Jun 28, 2020 at 02:44:15AM +0300, Dmitry Osipenko wrote: > 27.06.2020 23:43, Laurent Pinchart пишет: > > On Mon, Jun 22, 2020 at 01:27:42AM +0300, Dmitry Osipenko wrote: > >> This patch adds missing BUS fields to the display panel descriptions of > >> the panels which are found on NVIDIA Tegra devices: > >> > >> 1. AUO B101AW03 > >> 2. Chunghwa CLAA070WP03XG > >> 3. Chunghwa CLAA101WA01A > >> 4. Chunghwa CLAA101WB01 > >> 5. Innolux N156BGE L21 > >> 6. Samsung LTN101NT05 > >> > >> Suggested-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > >> Signed-off-by: Dmitry Osipenko <digetx@gmail.com> > >> --- > >> drivers/gpu/drm/panel/panel-simple.c | 12 ++++++++++++ > >> 1 file changed, 12 insertions(+) > >> > >> diff --git a/drivers/gpu/drm/panel/panel-simple.c b/drivers/gpu/drm/panel/panel-simple.c > >> index 87edd2bdf09a..986df9937650 100644 > >> --- a/drivers/gpu/drm/panel/panel-simple.c > >> +++ b/drivers/gpu/drm/panel/panel-simple.c > >> @@ -698,6 +698,8 @@ static const struct panel_desc auo_b101aw03 = { > >> .width = 223, > >> .height = 125, > >> }, > >> + .bus_format = MEDIA_BUS_FMT_RGB666_1X7X3_SPWG, > >> + .bus_flags = DRM_BUS_FLAG_DE_HIGH | DRM_BUS_FLAG_PIXDATA_DRIVE_POSEDGE, > > > > Does DRM_BUS_FLAG_PIXDATA_DRIVE_* make sense for LVDS ? > > To be honest I don't know whether it make sense or not for LVDS. I saw > that other LVDS panels in panel-simple.c use the PIXDATA flag and then > looked at what timing diagrams in the datasheets show. I think we should drop DRM_BUS_FLAG_PIXDATA_DRIVE_* for LVDS panels. I'll submit a patch. > > The rest looks good, except the Samsung panel for which I haven't been > > able to locate a datasheet. > > > > Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > > Thanks to you and Sam! -- Regards, Laurent Pinchart ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH v1 2/2] drm/panel-simple: Add missing BUS descriptions for some panels 2020-06-28 7:07 ` Laurent Pinchart @ 2020-06-28 7:52 ` Sam Ravnborg 2020-06-28 8:02 ` Laurent Pinchart 0 siblings, 1 reply; 12+ messages in thread From: Sam Ravnborg @ 2020-06-28 7:52 UTC (permalink / raw) To: Laurent Pinchart Cc: Dmitry Osipenko, Thierry Reding, dri-devel, linux-tegra, linux-kernel Hi Laurent. On Sun, Jun 28, 2020 at 10:07:45AM +0300, Laurent Pinchart wrote: > Hi Dmitry, > > On Sun, Jun 28, 2020 at 02:44:15AM +0300, Dmitry Osipenko wrote: > > 27.06.2020 23:43, Laurent Pinchart пишет: > > > On Mon, Jun 22, 2020 at 01:27:42AM +0300, Dmitry Osipenko wrote: > > >> This patch adds missing BUS fields to the display panel descriptions of > > >> the panels which are found on NVIDIA Tegra devices: > > >> > > >> 1. AUO B101AW03 > > >> 2. Chunghwa CLAA070WP03XG > > >> 3. Chunghwa CLAA101WA01A > > >> 4. Chunghwa CLAA101WB01 > > >> 5. Innolux N156BGE L21 > > >> 6. Samsung LTN101NT05 > > >> > > >> Suggested-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > > >> Signed-off-by: Dmitry Osipenko <digetx@gmail.com> > > >> --- > > >> drivers/gpu/drm/panel/panel-simple.c | 12 ++++++++++++ > > >> 1 file changed, 12 insertions(+) > > >> > > >> diff --git a/drivers/gpu/drm/panel/panel-simple.c b/drivers/gpu/drm/panel/panel-simple.c > > >> index 87edd2bdf09a..986df9937650 100644 > > >> --- a/drivers/gpu/drm/panel/panel-simple.c > > >> +++ b/drivers/gpu/drm/panel/panel-simple.c > > >> @@ -698,6 +698,8 @@ static const struct panel_desc auo_b101aw03 = { > > >> .width = 223, > > >> .height = 125, > > >> }, > > >> + .bus_format = MEDIA_BUS_FMT_RGB666_1X7X3_SPWG, > > >> + .bus_flags = DRM_BUS_FLAG_DE_HIGH | DRM_BUS_FLAG_PIXDATA_DRIVE_POSEDGE, > > > > > > Does DRM_BUS_FLAG_PIXDATA_DRIVE_* make sense for LVDS ? > > > > To be honest I don't know whether it make sense or not for LVDS. I saw > > that other LVDS panels in panel-simple.c use the PIXDATA flag and then > > looked at what timing diagrams in the datasheets show. > > I think we should drop DRM_BUS_FLAG_PIXDATA_DRIVE_* for LVDS panels. > I'll submit a patch. We should also clean up all the DRM_BUS_FLAG_* one day. No need for the deprecated values, so a few files needs an update. And we could document what flags makes sense for LVDS etc. On the TODO list... Sam > > > > The rest looks good, except the Samsung panel for which I haven't been > > > able to locate a datasheet. > > > > > > Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > > > > Thanks to you and Sam! > > -- > Regards, > > Laurent Pinchart ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH v1 2/2] drm/panel-simple: Add missing BUS descriptions for some panels 2020-06-28 7:52 ` Sam Ravnborg @ 2020-06-28 8:02 ` Laurent Pinchart 2020-06-28 8:53 ` Sam Ravnborg 0 siblings, 1 reply; 12+ messages in thread From: Laurent Pinchart @ 2020-06-28 8:02 UTC (permalink / raw) To: Sam Ravnborg Cc: Dmitry Osipenko, Thierry Reding, dri-devel, linux-tegra, linux-kernel Hi Sam, On Sun, Jun 28, 2020 at 09:52:45AM +0200, Sam Ravnborg wrote: > On Sun, Jun 28, 2020 at 10:07:45AM +0300, Laurent Pinchart wrote: > > On Sun, Jun 28, 2020 at 02:44:15AM +0300, Dmitry Osipenko wrote: > >> 27.06.2020 23:43, Laurent Pinchart пишет: > >>> On Mon, Jun 22, 2020 at 01:27:42AM +0300, Dmitry Osipenko wrote: > >>>> This patch adds missing BUS fields to the display panel descriptions of > >>>> the panels which are found on NVIDIA Tegra devices: > >>>> > >>>> 1. AUO B101AW03 > >>>> 2. Chunghwa CLAA070WP03XG > >>>> 3. Chunghwa CLAA101WA01A > >>>> 4. Chunghwa CLAA101WB01 > >>>> 5. Innolux N156BGE L21 > >>>> 6. Samsung LTN101NT05 > >>>> > >>>> Suggested-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > >>>> Signed-off-by: Dmitry Osipenko <digetx@gmail.com> > >>>> --- > >>>> drivers/gpu/drm/panel/panel-simple.c | 12 ++++++++++++ > >>>> 1 file changed, 12 insertions(+) > >>>> > >>>> diff --git a/drivers/gpu/drm/panel/panel-simple.c b/drivers/gpu/drm/panel/panel-simple.c > >>>> index 87edd2bdf09a..986df9937650 100644 > >>>> --- a/drivers/gpu/drm/panel/panel-simple.c > >>>> +++ b/drivers/gpu/drm/panel/panel-simple.c > >>>> @@ -698,6 +698,8 @@ static const struct panel_desc auo_b101aw03 = { > >>>> .width = 223, > >>>> .height = 125, > >>>> }, > >>>> + .bus_format = MEDIA_BUS_FMT_RGB666_1X7X3_SPWG, > >>>> + .bus_flags = DRM_BUS_FLAG_DE_HIGH | DRM_BUS_FLAG_PIXDATA_DRIVE_POSEDGE, > >>> > >>> Does DRM_BUS_FLAG_PIXDATA_DRIVE_* make sense for LVDS ? > >> > >> To be honest I don't know whether it make sense or not for LVDS. I saw > >> that other LVDS panels in panel-simple.c use the PIXDATA flag and then > >> looked at what timing diagrams in the datasheets show. > > > > I think we should drop DRM_BUS_FLAG_PIXDATA_DRIVE_* for LVDS panels. > > I'll submit a patch. > > We should also clean up all the DRM_BUS_FLAG_* one day. > No need for the deprecated values, so a few files needs an update. > And we could document what flags makes sense for LVDS etc. Where would you add that documentation ? The hardest part is to find a place that will be noticed by developers :-) I've just submitted a patch that adds a WARN_ON to catch similar issues in the panel-simple driver. It's not ideal as we really shouldn't have such code in the kernel, this is something that should be caught as part of the integration process. > On the TODO list... > > >>> The rest looks good, except the Samsung panel for which I haven't been > >>> able to locate a datasheet. > >>> > >>> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> -- Regards, Laurent Pinchart ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH v1 2/2] drm/panel-simple: Add missing BUS descriptions for some panels 2020-06-28 8:02 ` Laurent Pinchart @ 2020-06-28 8:53 ` Sam Ravnborg 0 siblings, 0 replies; 12+ messages in thread From: Sam Ravnborg @ 2020-06-28 8:53 UTC (permalink / raw) To: Laurent Pinchart Cc: Dmitry Osipenko, Thierry Reding, dri-devel, linux-tegra, linux-kernel On Sun, Jun 28, 2020 at 11:02:53AM +0300, Laurent Pinchart wrote: > Hi Sam, > > > We should also clean up all the DRM_BUS_FLAG_* one day. > > No need for the deprecated values, so a few files needs an update. > > And we could document what flags makes sense for LVDS etc. > > Where would you add that documentation ? The hardest part is to find a > place that will be noticed by developers :-) I will try to extend drm_bus_flags documentation in drm_connector.h And then add a few comments in panel-simple as well. Sam > > I've just submitted a patch that adds a WARN_ON to catch similar issues > in the panel-simple driver. It's not ideal as we really shouldn't have > such code in the kernel, this is something that should be caught as part > of the integration process. > > > On the TODO list... > > > > >>> The rest looks good, except the Samsung panel for which I haven't been > > >>> able to locate a datasheet. > > >>> > > >>> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > > -- > Regards, > > Laurent Pinchart ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH v1 0/2] Improve descriptions of a few simple-panels 2020-06-21 22:27 [PATCH v1 0/2] Improve descriptions of a few simple-panels Dmitry Osipenko 2020-06-21 22:27 ` [PATCH v1 1/2] drm/panel-simple: Correct EDT ET057090DHU connector type Dmitry Osipenko 2020-06-21 22:27 ` [PATCH v1 2/2] drm/panel-simple: Add missing BUS descriptions for some panels Dmitry Osipenko @ 2020-06-27 20:19 ` Sam Ravnborg 2 siblings, 0 replies; 12+ messages in thread From: Sam Ravnborg @ 2020-06-27 20:19 UTC (permalink / raw) To: Dmitry Osipenko Cc: Thierry Reding, Laurent Pinchart, dri-devel, linux-tegra, linux-kernel On Mon, Jun 22, 2020 at 01:27:40AM +0300, Dmitry Osipenko wrote: > Hello, > > This is a follow up to [1], which was already applied to drm-misc and then > Laurent Pinchart spotted some problems. This series addresses those problems. > > [1] https://patchwork.ozlabs.org/project/linux-tegra/patch/20200617222703.17080-8-digetx@gmail.com/ > > Dmitry Osipenko (2): > drm/panel-simple: Correct EDT ET057090DHU connector type > drm/panel-simple: Add missing BUS descriptions for some panels Thanks for the quick fixes, both are now applied to drm-misc-next. Sam ^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2020-06-28 8:53 UTC | newest] Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2020-06-21 22:27 [PATCH v1 0/2] Improve descriptions of a few simple-panels Dmitry Osipenko 2020-06-21 22:27 ` [PATCH v1 1/2] drm/panel-simple: Correct EDT ET057090DHU connector type Dmitry Osipenko 2020-06-21 22:29 ` Laurent Pinchart 2020-06-22 13:40 ` Dmitry Osipenko 2020-06-21 22:27 ` [PATCH v1 2/2] drm/panel-simple: Add missing BUS descriptions for some panels Dmitry Osipenko 2020-06-27 20:43 ` Laurent Pinchart 2020-06-27 23:44 ` Dmitry Osipenko 2020-06-28 7:07 ` Laurent Pinchart 2020-06-28 7:52 ` Sam Ravnborg 2020-06-28 8:02 ` Laurent Pinchart 2020-06-28 8:53 ` Sam Ravnborg 2020-06-27 20:19 ` [PATCH v1 0/2] Improve descriptions of a few simple-panels Sam Ravnborg
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).