Hi, On Fri, Sep 27, 2024 at 2:44 AM Zhaoxiong Lv <lvzhaoxiong@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxx> wrote: > > In MTK chips, if the DRM runtime resume has not yet completed and the > system enters sleep mode at the same time, there is a possibility of > a black screen after waking the machine. Reduce the disable delay > resolves this issue, Similar to patch #1, this sounds very suspicious and I think you need to root cause the problem and submit a proper fix instead of just playing with timings. > The "backlight_off_to_display_off_delay_ms" was added between > "backlight off" and "display off" to prevent "display off" from being > executed when the backlight is not fully powered off, which may cause > a white screen. However, we removed this > "backlight_off_to_display_off_delay_ms" and found that this situation > did not occur. Therefore, in order to solve the problem mentioned > above, we removed this delay, and the delay between "display off" and > "enter sleep" is not defined in the spec, so we reduce it from 120ms > to 50ms. > > In addition, T14 >= 120ms, so we change > "enter_sleep_to_reset_down_delay_ms" from 100ms to 120ms. > > The panel spec: > 1. https://github.com/Vme5o/power-on-off-sequential > > Fixes: e4bd1db1c1f7 ("drm/panel: boe-th101mb31ig002: Support for starry-er88577 MIPI-DSI panel") > > Signed-off-by: Zhaoxiong Lv <lvzhaoxiong@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxx> Similar to patch #1, no blank space between the "Fixes:" tag and the "Signed-off-by:" tag. -Doug