[PATCH 2/2] drm/amdgpu/virtual_dce: No need to pin the cursor bo

[Michel Dänzer]

On 2018-12-24 7:08 a.m., Emily Deng wrote:
> For virtual display feature, no need to pin cursor bo.
> 
> Signed-off-by: Emily Deng <Emily.Deng at amd.com>
> ---
>  drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
> index 8a078f4..98df8e4 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
> @@ -2798,7 +2798,7 @@ int amdgpu_device_suspend(struct drm_device *dev, bool suspend, bool fbcon)
>  			struct drm_framebuffer *fb = crtc->primary->fb;
>  			struct amdgpu_bo *robj;
>  
> -			if (amdgpu_crtc->cursor_bo) {
> +			if (amdgpu_crtc->cursor_bo && !adev->enable_virtual_display) {
>  				struct amdgpu_bo *aobj = gem_to_amdgpu_bo(amdgpu_crtc->cursor_bo);
>  				r = amdgpu_bo_reserve(aobj, true);
>  				if (r == 0) {
> @@ -2906,7 +2906,7 @@ int amdgpu_device_resume(struct drm_device *dev, bool resume, bool fbcon)
>  		list_for_each_entry(crtc, &dev->mode_config.crtc_list, head) {
>  			struct amdgpu_crtc *amdgpu_crtc = to_amdgpu_crtc(crtc);
>  
> -			if (amdgpu_crtc->cursor_bo) {
> +			if (amdgpu_crtc->cursor_bo && !adev->enable_virtual_display) {
>  				struct amdgpu_bo *aobj = gem_to_amdgpu_bo(amdgpu_crtc->cursor_bo);
>  				r = amdgpu_bo_reserve(aobj, true);
>  				if (r == 0) {
> 

Reviewed-by: Michel Dänzer <michel.daenzer at amd.com>


-- 
Earthling Michel Dänzer               |               http://www.amd.com
Libre software enthusiast             |             Mesa and X developer