On to, 2016-12-08 at 12:41 +0200, Jani Nikula wrote: > > On Thu, 08 Dec 2016, Tvrtko Ursulin <tursulin@xxxxxxxxxxx> wrote: > > > > From: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxx> > > > > 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 <tvrtko.ursulin@xxxxxxxxx> > > Cc: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> > > Cc: Jani Nikula <jani.nikula@xxxxxxxxx> > > Cc: Joonas Lahtinen <joonas.lahtinen@xxxxxxxxxxxxxxx> <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 Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx