Messages in this thread | | | From | Doug Anderson <> | Date | Fri, 17 Jul 2020 13:29:18 -0700 | Subject | Re: [PATCH v2] drm: msm: a6xx: fix gpu failure after system resume |
| |
Hi,
On Fri, Jul 17, 2020 at 1:24 PM Rob Clark <robdclark@gmail.com> wrote: > > On Fri, Jul 17, 2020 at 10:39 AM Doug Anderson <dianders@chromium.org> wrote: > > > > Hi, > > > > On Fri, Jul 17, 2020 at 7:46 AM Jordan Crouse <jcrouse@codeaurora.org> wrote: > > > > > > On Fri, Jul 17, 2020 at 08:04:18PM +0530, Akhil P Oommen wrote: > > > > On targets where GMU is available, GMU takes over the ownership of GX GDSC > > > > during its initialization. So, move the refcount-get on GX PD before we > > > > initialize the GMU. This ensures that nobody can collapse the GX GDSC > > > > once GMU owns the GX GDSC. This patch fixes some GMU OOB errors seen > > > > during GPU wake up during a system resume. > > > > > > > Signed-off-by: Akhil P Oommen <akhilpo@codeaurora.org> > > > > Reported-by: Matthias Kaehlcke <mka@chromium.org> > > > > Tested-by: Matthias Kaehlcke <mka@chromium.org> > > > > > > The Signed-off-by needs to be at the end but I think Rob can do that for you. > > > > It does? I've always been told that this is supposed to be roughly a > > log of what happens. In that sense you added your SoB before the > > review/test happened so it should come before. I know some > > maintainers seem to do things differently but that seems to be the > > most common. In that sense, I think the order that Akhil has is > > correct. ...but, obviously, it's up to the maintainer. ;-) > > yeah, I chronological order was my understanding too.. but presumably > the Reported-by happened before the Signed-of-by (which is how I > reordered things when applying the patch)
Doh! Yeah, I somehow read that as Reviewed-by. Thanks! :-)
-Doug
| |