[PATCH v1 2/2] drm/panel: boe-th101mb31ig002: Modify Starry panel timing

Dmitry Baryshkov dmitry.baryshkov at linaro.org
Mon Sep 23 13:44:06 UTC 2024


On Mon, Sep 23, 2024 at 09:38:59PM GMT, zhaoxiong lv wrote:
> On Mon, Sep 23, 2024 at 8:07 PM Dmitry Baryshkov
> <dmitry.baryshkov at linaro.org> wrote:
> >
> > On Mon, Sep 23, 2024 at 05:22:20PM GMT, zhaoxiong lv wrote:
> > > On Mon, Sep 16, 2024 at 1:15 PM Dmitry Baryshkov
> > > <dmitry.baryshkov at linaro.org> wrote:
> > > >
> > > > Same comment as the one that I've provided to the other patch, plus:
> > > >
> > > > On Sun, 15 Sept 2024 at 10:11, Zhaoxiong Lv
> > > > <lvzhaoxiong at huaqin.corp-partner.google.com> wrote:
> > > > >
> > > > > In order to meet the timing, remove the delay between "backlight off"
> > > > > and "display off", and reduce the delay between "display_off" and
> > > > > "enter_sleep"
> > > >
> > > > Separate commit, separate _justification_. Why, not what.
> > > >
> > > > >
> > > > > Removing variables: display_off_to_enter_sleep_delay_ms
> > > >
> > > > This phrase is useless.
> > > >
> > > hi Dmitry
> > >
> > > As in another patch reply, in order to solve a black screen problem,
> > > the delay is reduced.
> > > The panel spec:
> > > 1. https://github.com/Vme5o/power-on-off-sequential
> >
> > It should be described in the commit message(s). You have removed one
> > delay and added another one. Is that a single fix or two separate fixes?
> hi Dmitry
> We only modify one of the drivers (starry), and the other driver
> (boe_th101mb31ig002) is just to keep the original value.

Well, the question was about fixes, not about the drivers.

-- 
With best wishes
Dmitry


More information about the dri-devel mailing list