drm/i915/uc: Drop use of MISSING_CASE on trivial enums
authorMichal Wajdeczko <michal.wajdeczko@intel.com>
Fri, 31 Mar 2017 10:26:52 +0000 (10:26 +0000)
committerChris Wilson <chris@chris-wilson.co.uk>
Fri, 31 Mar 2017 20:08:06 +0000 (21:08 +0100)
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>
drivers/gpu/drm/i915/intel_uc.h

index d2dcde79669e812251ed286c7c0ad3488bd1daef..4b7f73aeddac6475db31d184853f833c8ba3d510 100644 (file)
@@ -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";
 }
 
 /*