RE: [PATCH] drm/amd/display: assign fb_location only if bo is pinned

2017-10-26 Thread S, Shirish
...@daenzer.net] Sent: Wednesday, October 25, 2017 3:54 PM To: S, Shirish ; Grodzovsky, Andrey Cc: Deucher, Alexander ; dri-devel@lists.freedesktop.org; amd-...@lists.freedesktop.org Subject: Re: [PATCH] drm/amd/display: assign fb_location only if bo is pinned On 25/10/17 12:05 PM, S, Shirish wrote: >

RE: [PATCH] drm/amd/display: assign fb_location only if bo is pinned

2017-10-25 Thread S, Shirish
Hi Alex, Michel & Andrey, [PATCH] drm/amd/display: assign fb_location only if bo is pinned [PATCH 2/2] drm/amd/display: cleanup addReq and fix fb_location should be dropped and instead: [PATCH] drm/amd/display: fix high part address in dm_plane_helper_prepare_fb() should be reviewed .

Re: [PATCH] drm/amd/display: assign fb_location only if bo is pinned

2017-10-25 Thread Michel Dänzer
On 25/10/17 12:05 PM, S, Shirish wrote: > Hi Alex, Michel & Andrey, > > [PATCH] drm/amd/display: assign fb_location only if bo is pinned > [PATCH 2/2] drm/amd/display: cleanup addReq and fix fb_location > > should be dropped and instead: Since you pushed the latter t

Re: [PATCH] drm/amd/display: assign fb_location only if bo is pinned

2017-10-25 Thread Michel Dänzer
On 25/10/17 07:18 AM, S, Shirish wrote: > > Andrey, > Till now the get_fb_info() never calculated fb_location(as addrReq was > always false), but since now its required to program the high part of > the address structure, > can you move it back into the commit scope and ensure that fb_location > is

Re: [PATCH] drm/amd/display: assign fb_location only if bo is pinned

2017-10-24 Thread S, Shirish
On 10/25/2017 1:13 AM, Andrey Grodzovsky wrote: On 10/24/2017 12:06 PM, Michel Dänzer wrote: On 24/10/17 06:00 PM, Andrey Grodzovsky wrote: On 10/24/2017 11:43 AM, Michel Dänzer wrote: On 24/10/17 04:58 PM, Andrey Grodzovsky wrote: On 10/24/2017 10:36 AM, S, Shirish wrote: On 10/24/2017

Re: [PATCH] drm/amd/display: assign fb_location only if bo is pinned

2017-10-24 Thread Andrey Grodzovsky
On 10/24/2017 12:06 PM, Michel Dänzer wrote: On 24/10/17 06:00 PM, Andrey Grodzovsky wrote: On 10/24/2017 11:43 AM, Michel Dänzer wrote: On 24/10/17 04:58 PM, Andrey Grodzovsky wrote: On 10/24/2017 10:36 AM, S, Shirish wrote: On 10/24/2017 7:48 PM, Andrey Grodzovsky wrote: On 10/24/2017 09

Re: [PATCH] drm/amd/display: assign fb_location only if bo is pinned

2017-10-24 Thread Andrey Grodzovsky
On 10/24/2017 11:43 AM, Michel Dänzer wrote: [ Adding the dri-devel list ] On 24/10/17 04:58 PM, Andrey Grodzovsky wrote: On 10/24/2017 10:36 AM, S, Shirish wrote: On 10/24/2017 7:48 PM, Andrey Grodzovsky wrote: On 10/24/2017 09:51 AM, S, Shirish wrote: From: Shirish S On some systems am

Re: [PATCH] drm/amd/display: assign fb_location only if bo is pinned

2017-10-24 Thread Michel Dänzer
On 24/10/17 06:00 PM, Andrey Grodzovsky wrote: > On 10/24/2017 11:43 AM, Michel Dänzer wrote: >> On 24/10/17 04:58 PM, Andrey Grodzovsky wrote: >>> On 10/24/2017 10:36 AM, S, Shirish wrote: On 10/24/2017 7:48 PM, Andrey Grodzovsky wrote: > On 10/24/2017 09:51 AM, S, Shirish wrote: >> F

Re: [PATCH] drm/amd/display: assign fb_location only if bo is pinned

2017-10-24 Thread Michel Dänzer
[ Adding the dri-devel list ] On 24/10/17 04:58 PM, Andrey Grodzovsky wrote: > On 10/24/2017 10:36 AM, S, Shirish wrote: >> On 10/24/2017 7:48 PM, Andrey Grodzovsky wrote: >>> On 10/24/2017 09:51 AM, S, Shirish wrote: From: Shirish S On some systems amdgpu_bo_gpu_offset seems to b