Thanks, applied to drm-misc-fixes. Regards, Qiang On Sun, Jul 19, 2020 at 6:41 PM Erico Nunes <nunes.erico@xxxxxxxxx> wrote: > > On Sun, Jul 19, 2020 at 9:31 AM Qiang Yu <yuq825@xxxxxxxxx> wrote: > > > > PP bcast is marked as doing async reset after job is done. > > When resume after suspend, each PP is reset individually, > > so no need to reset in PP bcast resume. But I forgot to > > clear the PP bcast async reset mark so call into async wait > > before job run and gets timeout. > > > > Fixes: 3446d7e9883d ("drm/lima: add resume/suspend callback for each ip") > > Signed-off-by: Qiang Yu <yuq825@xxxxxxxxx> > > --- > > drivers/gpu/drm/lima/lima_pp.c | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/drivers/gpu/drm/lima/lima_pp.c b/drivers/gpu/drm/lima/lima_pp.c > > index 33f01383409c..a5c95bed08c0 100644 > > --- a/drivers/gpu/drm/lima/lima_pp.c > > +++ b/drivers/gpu/drm/lima/lima_pp.c > > @@ -271,6 +271,8 @@ void lima_pp_fini(struct lima_ip *ip) > > > > int lima_pp_bcast_resume(struct lima_ip *ip) > > { > > + /* PP has been reset by individual PP resume */ > > + ip->data.async_reset = false; > > return 0; > > } > > > > -- > > Reviewed-by: Erico Nunes <nunes.erico@xxxxxxxxx> > > This fixes the issue reported at > https://gitlab.freedesktop.org/lima/linux/-/issues/34 . _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel