On Sat, Nov 30, 2024 at 12:15:25PM +0100, Sasha Finkelstein wrote: > On Sat, 30 Nov 2024 at 10:29, Dmitry Baryshkov > <dmitry.baryshkov@xxxxxxxxxx> wrote: > > > + > > > +static int summit_resume(struct device *dev) > > > +{ > > > + return summit_set_brightness(dev); > > > > Doesn't the generic code already handle that for you? > > Apparently not, I have commented out the pm ops, entered sleep > and the display stayed on. Please add BL_CORE_SUSPENDRESUME option to the backlight device. But interestingly enough, none of the panels set that flag. -- With best wishes Dmitry