drm/xe/topology: Stop trying to fix programming mistakes
authorMichal Wajdeczko <michal.wajdeczko@intel.com>
Wed, 4 Jun 2025 20:29:08 +0000 (22:29 +0200)
committerMatt Roper <matthew.d.roper@intel.com>
Fri, 6 Jun 2025 18:47:05 +0000 (11:47 -0700)
We shouldn't ever pass more DSS registers than our hardcoded limit,
it should be sufficient to just assert that instead of trying to
fix it, as this will never happen in the production driver.

Signed-off-by: Michal Wajdeczko <michal.wajdeczko@intel.com>
Cc: Matt Roper <matthew.d.roper@intel.com>
Reviewed-by: Matt Roper <matthew.d.roper@intel.com>
Link: https://lore.kernel.org/r/20250604202908.769-4-michal.wajdeczko@intel.com
Signed-off-by: Matt Roper <matthew.d.roper@intel.com>
drivers/gpu/drm/xe/xe_gt_topology.c

index 0487439..305939c 100644 (file)
@@ -22,8 +22,7 @@ static void load_dss_mask(struct xe_gt *gt, xe_dss_mask_t mask, int numregs,
        u32 fuse_val[XE_MAX_DSS_FUSE_REGS] = {};
        int i;
 
-       if (drm_WARN_ON(&gt_to_xe(gt)->drm, numregs > XE_MAX_DSS_FUSE_REGS))
-               numregs = XE_MAX_DSS_FUSE_REGS;
+       xe_gt_assert(gt, numregs <= ARRAY_SIZE(fuse_val));
 
        for (i = 0; i < numregs; i++)
                fuse_val[i] = xe_mmio_read32(&gt->mmio, regs[i]);