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 <shiris...@amd.com>; Grodzovsky, Andrey <andrey.grodzov...@amd.com> Cc: Deucher, Alexander <alexander.deuc...@amd.com>; dri-devel@lists.freedesktop.org; amd-...@lists.freedesktop.org Subject: Re: [PATCH] drm/a

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

2017-10-25 Thread S, Shirish
;shiris...@amd.com>; Grodzovsky, Andrey <andrey.grodzov...@amd.com> Cc: Deucher, Alexander <alexander.deuc...@amd.com>; amd-...@lists.freedesktop.org; dri-devel@lists.freedesktop.org Subject: Re: [PATCH] drm/amd/display: assign fb_location only if bo is pinned On 25/10/17 07:1

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 to amd-staging-drm-next,

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 >

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

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

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: >>

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