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

[Weiji Wang]

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?

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