On 08/12/2016 13:16, Joonas Lahtinen wrote:
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!
Too much prettiness for the kernel I would have thought. Don't see the
benefit personally.
Regards,
Tvrtko
_______________________________________________
Intel-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
