Commit b9ab1f3f authored Mar 31, 2017 by Michal Wajdeczko Committed by Chris Wilson Mar 31, 2017

drm/i915/uc: Drop use of MISSING_CASE on trivial enums



We can rely on compiler to notify us if we miss any case.
This approach may also reduce driver size (reported ~4K).

Signed-off-by: Michal Wajdeczko <michal.wajdeczko@intel.com>
Cc: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Jani Nikula <jani.nikula@intel.com>
Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/20170331102652.177664-1-michal.wajdeczko@intel.com


Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>

parent d7c530b2

drivers/gpu/drm/i915/intel_uc.h

+2 −6

Original line number	Diff line number	Diff line
		@@ -114,10 +114,8 @@ const char *intel_uc_fw_status_repr(enum intel_uc_fw_status status)
		return "PENDING";
		case INTEL_UC_FIRMWARE_SUCCESS:
		return "SUCCESS";
		default:
		MISSING_CASE(status);
		return "<invalid>";
		}
		return "<invalid>";
		}

		enum intel_uc_fw_type {
		@@ -133,10 +131,8 @@ static inline const char *intel_uc_fw_type_repr(enum intel_uc_fw_type type)
		return "GuC";
		case INTEL_UC_FW_TYPE_HUC:
		return "HuC";
		default:
		MISSING_CASE(type);
		return "<invalid>";
		}
		return "uC";
		}

		/*