On ti, 2017-02-28 at 13:11 +0200, Jani Nikula wrote:
> Leave the runtime check in place in case the platform variable itself
> comes from bogus sources.
>
> Signed-off-by: Jani Nikula <jani.nikula@xxxxxxxxx>
<SNIP>
> @@ -890,6 +890,7 @@ enum intel_platform {
> INTEL_BROXTON,
> INTEL_KABYLAKE,
> INTEL_GEMINILAKE,
> + INTEL_MAX_PLATFORMS
> };
Easily confused to be be off by one, INTEL_MAX_PLATFORMS =
INTEL_GEMINILAKE, so I suggest changing it to INTEL_NUM_PLATFORMS which
is explicit.
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
