2020-07-03 09:51:56

by Jitao Shi

[permalink] [raw]
Subject: [PATCH] drm/panel: auo,b116xw03: fix flash backlight when power on

Delay the backlight on to make sure the video stable.

Signed-off-by: Jitao Shi <[email protected]>
---
drivers/gpu/drm/panel/panel-simple.c | 3 +++
1 file changed, 3 insertions(+)

diff --git a/drivers/gpu/drm/panel/panel-simple.c b/drivers/gpu/drm/panel/panel-simple.c
index 3ad828eaefe1..18f34f286d3d 100644
--- a/drivers/gpu/drm/panel/panel-simple.c
+++ b/drivers/gpu/drm/panel/panel-simple.c
@@ -734,6 +734,9 @@ static const struct panel_desc auo_b116xw03 = {
.width = 256,
.height = 144,
},
+ .delay = {
+ .enable = 400,
+ },
};

static const struct drm_display_mode auo_b133xtn01_mode = {
--
2.25.1


2020-07-05 08:08:13

by Sam Ravnborg

[permalink] [raw]
Subject: Re: [PATCH] drm/panel: auo,b116xw03: fix flash backlight when power on

Hi Jitao.

On Fri, Jul 03, 2020 at 05:51:13PM +0800, Jitao Shi wrote:
> Delay the backlight on to make sure the video stable.
>
> Signed-off-by: Jitao Shi <[email protected]>
> ---
> drivers/gpu/drm/panel/panel-simple.c | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/drivers/gpu/drm/panel/panel-simple.c b/drivers/gpu/drm/panel/panel-simple.c
> index 3ad828eaefe1..18f34f286d3d 100644
> --- a/drivers/gpu/drm/panel/panel-simple.c
> +++ b/drivers/gpu/drm/panel/panel-simple.c
> @@ -734,6 +734,9 @@ static const struct panel_desc auo_b116xw03 = {
> .width = 256,
> .height = 144,
> },
> + .delay = {
> + .enable = 400,
> + },
> };
>
> static const struct drm_display_mode auo_b133xtn01_mode = {

Patch did not apply to drm-misc-next.
Please update - and when you do so also add:
.bus_flags
.bus_format
.connector_type

So we have this panel properly defined.

Sam

2020-07-05 09:45:57

by Jitao Shi

[permalink] [raw]
Subject: Re: [PATCH] drm/panel: auo,b116xw03: fix flash backlight when power on

On Sun, 2020-07-05 at 10:06 +0200, Sam Ravnborg wrote:
> Hi Jitao.
>
> On Fri, Jul 03, 2020 at 05:51:13PM +0800, Jitao Shi wrote:
> > Delay the backlight on to make sure the video stable.
> >
> > Signed-off-by: Jitao Shi <[email protected]>
> > ---
> > drivers/gpu/drm/panel/panel-simple.c | 3 +++
> > 1 file changed, 3 insertions(+)
> >
> > diff --git a/drivers/gpu/drm/panel/panel-simple.c b/drivers/gpu/drm/panel/panel-simple.c
> > index 3ad828eaefe1..18f34f286d3d 100644
> > --- a/drivers/gpu/drm/panel/panel-simple.c
> > +++ b/drivers/gpu/drm/panel/panel-simple.c
> > @@ -734,6 +734,9 @@ static const struct panel_desc auo_b116xw03 = {
> > .width = 256,
> > .height = 144,
> > },
> > + .delay = {
> > + .enable = 400,
> > + },
> > };
> >
> > static const struct drm_display_mode auo_b133xtn01_mode = {
>
> Patch did not apply to drm-misc-next.
> Please update - and when you do so also add:
> .bus_flags
> .bus_format
> .connector_type
>
> So we have this panel properly defined.
>
> Sam

Thanks for your review.
I'll add those next version.

Jitao