+ 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 +0000, 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 <joonas.lahti...@linux.intel.com> > Suggested-by: Chris Wilson <ch...@chris-wilson.co.uk> > Signed-off-by: Matthew Auld <matthew.a...@intel.com> > Cc: Joonas Lahtinen <joonas.lahti...@linux.intel.com> > Cc: Chris Wilson <ch...@chris-wilson.co.uk> > Cc: Paulo Zanoni <paulo.r.zan...@intel.com> > Cc: Ingo Molnar <mi...@kernel.org> > Cc: H. Peter Anvin <h...@zytor.com> > Cc: dri-devel@lists.freedesktop.org > Cc: x...@kernel.org <SNIP> > @@ -548,6 +551,9 @@ intel_graphics_stolen(int num, int slot, int func, > printk(KERN_INFO "Reserving Intel graphics memory at %pa-%pa\n", > &base, &end); > > + intel_graphics_stolen_res.start = base; > + intel_graphics_stolen_res.end = end; You can take advantage of the newly establisted resource by using %pR in the printk. I sent a patch to convert the function signatures to resource_size_t for a less painful future. Maybe squash just the early quirks/header change portion of this patch to that patch, then we can iterate on the i915 changes on a reminder of the series on top of that. Regards, Joonas -- Joonas Lahtinen Open Source Technology Center Intel Corporation _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
