Messages in this thread | | | From | Dmitry Baryshkov <> | Date | Wed, 15 Dec 2021 22:16:29 +0300 | Subject | Re: [PATCH] drm/msm: Don't use autosuspend for display |
| |
On Wed, 15 Dec 2021 at 22:10, Stephen Boyd <swboyd@chromium.org> wrote: > > Quoting Rob Clark (2021-12-15 09:59:02) > > From: Rob Clark <robdclark@chromium.org> > > > > No functional change, as we only actually enable autosuspend for the GPU > > device. But lets not encourage thinking that autosuspend is a good idea > > for anything display related. > > I'd prefer to see a small blurb about why it's not a good idea to use > autosuspend for display things. Then this commit can be dug out of the > history and someone new can quickly understand the reasoning behind it. > Just saying it's not a good idea doesn't really help.
I'd second this request, Nevertheless Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> > > > > Signed-off-by: Rob Clark <robdclark@chromium.org> > > --- > > Reviewed-by: Stephen Boyd <swboyd@chromium.org>
-- With best wishes Dmitry
| |