]> git.ipfire.org Git - thirdparty/linux.git/commitdiff
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 048743913b368d11954a5319d664713090fb549f..305939c697475db2faf759799e23bff87e4810ff 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]);