Hi Alex, Any concern about this patch? itâ??s split as suggested. â?? Sincerely Yours, Pixel On 25/10/2017, 10:17 AM, "Pixel Ding" <Pixel.Ding at amd.com> wrote: >From: pding <Pixel.Ding at amd.com> > >Hi Alex, > >Split the wait_reset patch to 2. Part 2. > >please review. > >--- > >Signed-off-by: pding <Pixel.Ding at amd.com> >--- > drivers/gpu/drm/amd/amdgpu/mxgpu_ai.c | 1 + > drivers/gpu/drm/amd/amdgpu/mxgpu_vi.c | 6 ++++++ > 2 files changed, 7 insertions(+) > >diff --git a/drivers/gpu/drm/amd/amdgpu/mxgpu_ai.c b/drivers/gpu/drm/amd/amdgpu/mxgpu_ai.c >index b4906d2..f91aab3 100644 >--- a/drivers/gpu/drm/amd/amdgpu/mxgpu_ai.c >+++ b/drivers/gpu/drm/amd/amdgpu/mxgpu_ai.c >@@ -353,5 +353,6 @@ const struct amdgpu_virt_ops xgpu_ai_virt_ops = { > .req_full_gpu = xgpu_ai_request_full_gpu_access, > .rel_full_gpu = xgpu_ai_release_full_gpu_access, > .reset_gpu = xgpu_ai_request_reset, >+ .wait_reset = NULL, > .trans_msg = xgpu_ai_mailbox_trans_msg, > }; >diff --git a/drivers/gpu/drm/amd/amdgpu/mxgpu_vi.c b/drivers/gpu/drm/amd/amdgpu/mxgpu_vi.c >index c25a831..27b03c7 100644 >--- a/drivers/gpu/drm/amd/amdgpu/mxgpu_vi.c >+++ b/drivers/gpu/drm/amd/amdgpu/mxgpu_vi.c >@@ -458,6 +458,11 @@ static int xgpu_vi_request_reset(struct amdgpu_device *adev) > return xgpu_vi_send_access_requests(adev, IDH_REQ_GPU_RESET_ACCESS); > } > >+static int xgpu_vi_wait_reset_cmpl(struct amdgpu_device *adev) >+{ >+ return xgpu_vi_poll_msg(adev, IDH_FLR_NOTIFICATION_CMPL); >+} >+ > static int xgpu_vi_request_full_gpu_access(struct amdgpu_device *adev, > bool init) > { >@@ -613,5 +618,6 @@ const struct amdgpu_virt_ops xgpu_vi_virt_ops = { > .req_full_gpu = xgpu_vi_request_full_gpu_access, > .rel_full_gpu = xgpu_vi_release_full_gpu_access, > .reset_gpu = xgpu_vi_request_reset, >+ .wait_reset = xgpu_vi_wait_reset_cmpl, > .trans_msg = NULL, /* Does not need to trans VF errors to host. */ > }; >-- >2.9.5 >