Re: [PATCH 1/6] drm/i915: export the stolen region as a resource

2017-11-23 Thread Joonas Lahtinen
+ Ville to comment if the removed code loses some meaningful comments or not. I already went through the code doing consolidations, about a year ago, so I may be blind to it. On Wed, 2017-11-22 at 21:19 +, Matthew Auld wrote: > We duplicate the stolen discovery code in early-quirks and in i915

Re: [PATCH 1/6] drm/i915: export the stolen region as a resource

2017-11-23 Thread Ingo Molnar
* Joonas Lahtinen wrote: > + Dave as a heads-up > > On Thu, 2017-11-23 at 07:17 +0100, Ingo Molnar wrote: > > * Matthew Auld wrote: > > > > > We duplicate the stolen discovery code in early-quirks and in i915, > > > however if we just export the region as a resource from early-quirks we > > >

Re: [PATCH 1/6] drm/i915: export the stolen region as a resource

2017-11-23 Thread Ingo Molnar
* Matthew Auld wrote: > We duplicate the stolen discovery code in early-quirks and in i915, > however if we just export the region as a resource from early-quirks we > can nuke the duplication. > > Suggested-by: Joonas Lahtinen > Suggested-by: Chris Wilson > Signed-off-by: Matthew Auld > Cc:

Re: [PATCH 1/6] drm/i915: export the stolen region as a resource

2017-11-22 Thread Joonas Lahtinen
+ Dave as a heads-up On Thu, 2017-11-23 at 07:17 +0100, Ingo Molnar wrote: > * Matthew Auld wrote: > > > We duplicate the stolen discovery code in early-quirks and in i915, > > however if we just export the region as a resource from early-quirks we > > can nuke the duplication. > > > > Suggeste

[PATCH 1/6] drm/i915: export the stolen region as a resource

2017-11-22 Thread Matthew Auld
We duplicate the stolen discovery code in early-quirks and in i915, however if we just export the region as a resource from early-quirks we can nuke the duplication. Suggested-by: Joonas Lahtinen Suggested-by: Chris Wilson Signed-off-by: Matthew Auld Cc: Joonas Lahtinen Cc: Chris Wilson Cc: P