Re: [PATCH] drm/panel-edp: Add BOE NE140QDM-NX2

[Doug Anderson]

Hi,

On Sat, Jul 25, 2026 at 1:58 PM Weiji Wang <nebclllo0444@xxxxxxxxx> wrote:
>
> Hi,
>
>
> On 7/26/26 00:29, Doug Anderson wrote:
> > Right that using the "enable" delay is more conservative, but it does
> > sound as if "prepare_to_enable" is more correct in this case if you
> > are provided "(T4+T5+T6+T8)-min".
> >
> > I re-looked at the one panel using "delay_200_500_p2e80"
> > (NV116WHM-T01), since I still have access to that datasheet. There, I
> > see:
> >
> > T4+T5+T6+T8 > 80ms
> > 0ms < T7 <= 50ms
> > 50ms < T8
> >
> > In other words, T7-max is 50 and T8-min is 50.
> >
> > Based on that, I believe it would be more correct for this panel to
> > use "delay_200_500_e50_p2e80". I'll try to send a patch for that,
> > which would get rid of the need for "delay_200_500_p2e80".
> >
> > Does that match your panel, too?
> >
> > -Doug
>
>
> Yes, the constraints of T4+T5+T6+T8, T7 and T8 is exactly same as mine.
> However, my copy of datasheet did not mention min/max value of T6, and it
> feels like assuming T6-min=0 if setting 'enable' to 50ms, or
> max(T7-max, T8-min). Is that proper?

If it doesn't mention T6-min, then the assumption is that the panel
has no constraints on T6. Thus, T6-min would be 0.


> Sorry to inconvenience you due to my inexperience in LCD panels.

No worries. This stuff is pretty arcane.

-Doug