On to, 2016-12-08 at 12:41 +0200, Jani Nikula wrote: > > On Thu, 08 Dec 2016, Tvrtko Ursulin <[email protected]> wrote: > > > > From: Tvrtko Ursulin <[email protected]> > > > > Inspired by the same approach used by Chris Wilson in the self > > test patches. > > > > We add a separate header file containing the list of our > > platforms and then use the pre-processor to generate all > > other places which use that list. This avoids having to > > list them multiple times and avoids the maintenance > > burden. > > > > Signed-off-by: Tvrtko Ursulin <[email protected]> > > Cc: Chris Wilson <[email protected]> > > Cc: Jani Nikula <[email protected]> > > Cc: Joonas Lahtinen <[email protected]>
<SNIP> > I was thinking we could improve the printed names here, for example: > > [INTEL_I965G] = "I965G/Broadwater" > [INTEL_VALLEYVIEW] = "Valleyview/Baytrail" > +1 on this idea! Regards, Joonas -- Joonas Lahtinen Open Source Technology Center Intel Corporation _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
