Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 9160095c authored by Jani Nikula's avatar Jani Nikula
Browse files

drm/i915: use BUILD_BUG_ON to ensure platform name has been set up 



Leave the runtime check in place in case the platform variable itself
comes from bogus sources.

Reviewed-by: default avatarChris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: default avatarJani Nikula <jani.nikula@intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/1488280303-9323-1-git-send-email-jani.nikula@intel.com
parent 0129936d

drivers/gpu/drm/i915/i915_drv.h

+1 −0
Original line number Original line Diff line number Diff line
@@ -889,6 +889,7 @@ enum intel_platform { 
	INTEL_BROXTON,

	INTEL_BROXTON,

	INTEL_KABYLAKE,

	INTEL_KABYLAKE,

	INTEL_GEMINILAKE,

	INTEL_GEMINILAKE,

	INTEL_MAX_PLATFORMS

};

};





struct intel_device_info {

struct intel_device_info {

drivers/gpu/drm/i915/intel_device_info.c

+2 −0
Original line number Original line Diff line number Diff line
@@ -56,6 +56,8 @@ static const char * const platform_names[] = { 




const char *intel_platform_name(enum intel_platform platform)

const char *intel_platform_name(enum intel_platform platform)

{

{

	BUILD_BUG_ON(ARRAY_SIZE(platform_names) != INTEL_MAX_PLATFORMS);



	if (WARN_ON_ONCE(platform >= ARRAY_SIZE(platform_names) ||

	if (WARN_ON_ONCE(platform >= ARRAY_SIZE(platform_names) ||

			 platform_names[platform] == NULL))

			 platform_names[platform] == NULL))

		return "<unknown>";

		return "<unknown>";