Re: [PATCH 2/2] drm/amdgpu: clean up UVD instance handling v2

2018-07-19 Thread Leo Liu
On 07/18/2018 02:44 PM, Christian König wrote: The whole handle, filp and entity handling is superfluous here. We should have reviewed that more thoughtfully. It looks like somebody just made the code instance aware without knowing the background. Yeah It's only required for UVD without VMID

RE: [PATCH 2/2] drm/amdgpu: clean up UVD instance handling v2

2018-07-18 Thread Zhou, David(ChunMing)
Acked-by: Chunming Zhou -Original Message- From: amd-gfx [mailto:amd-gfx-boun...@lists.freedesktop.org] On Behalf Of Christian K?nig Sent: Thursday, July 19, 2018 2:45 AM To: amd-gfx@lists.freedesktop.org Subject: [PATCH 2/2] drm/amdgpu: clean up UVD instance handling v2 The whole

[PATCH 2/2] drm/amdgpu: clean up UVD instance handling v2

2018-07-18 Thread Christian König
The whole handle, filp and entity handling is superfluous here. We should have reviewed that more thoughtfully. It looks like somebody just made the code instance aware without knowing the background. v2: fix one more missed case in amdgpu_uvd_suspend Signed-off-by: Christian König ---