Commit d6a0311c authored by Michal Wajdeczko's avatar Michal Wajdeczko
Browse files

drm/xe/hw_engine_group: Don't use drm_warn to catch missed case 



Since hwe->class is an enumeration we can rely on the compiler
to catch any unhandled engine class case at compile time thanks
to [-Werror=switch]. Any unexpected use of a special CLASS_MAX
enum case can be guarded by our xe_gt_assert() instead, which
will be compiled-out on the production builds.

Signed-off-by: default avatarMichal Wajdeczko <michal.wajdeczko@intel.com>
Cc: Francois Dugast <francois.dugast@intel.com>
Cc: Matthew Brost <matthew.brost@intel.com>
Reviewed-by: default avatarFrancois Dugast <francois.dugast@intel.com>
Link: https://lore.kernel.org/r/20250725090508.571-1-michal.wajdeczko@intel.com
parent 4df0bd5e

drivers/gpu/drm/xe/xe_hw_engine_group.c

+2 −2
Original line number Diff line number Diff line
@@ -103,8 +103,8 @@ int xe_hw_engine_setup_groups(struct xe_gt *gt) 
			break;

		case XE_ENGINE_CLASS_OTHER:

			break;

		default:

			drm_warn(&xe->drm, "NOT POSSIBLE");

		case XE_ENGINE_CLASS_MAX:

			xe_gt_assert(gt, false);

		}

	}