Skip to content

Commit 8455dad

Browse files
mikuintickle
authored andcommitted
drm/i915/icl: Don't warn on spurious interrupts
There is a chance we can see spurious interrupts in live now. We have more engines enabled and that with more elaborate access patterns with pm and display, increases the chances hardware just makes a social call, without anything to work on. Remove the error as we have tests to actually probe if we really miss interrupt, instead of getting spurious ones. Note that now we do write to intr_dw even with a zero value. This is considered advantegous as the write is an ack that sw is done. Cc: Chris Wilson <[email protected]> Signed-off-by: Mika Kuoppala <[email protected]> Reviewed-by: Chris Wilson <[email protected]> Signed-off-by: Chris Wilson <[email protected]> Link: https://patchwork.freedesktop.org/patch/msgid/[email protected]
1 parent a087baf commit 8455dad

File tree

1 file changed

+1
-7
lines changed

1 file changed

+1
-7
lines changed

drivers/gpu/drm/i915/i915_irq.c

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -3025,14 +3025,8 @@ gen11_gt_bank_handler(struct drm_i915_private * const i915,
30253025

30263026
intr_dw = raw_reg_read(regs, GEN11_GT_INTR_DW(bank));
30273027

3028-
if (unlikely(!intr_dw)) {
3029-
DRM_ERROR("GT_INTR_DW%u blank!\n", bank);
3030-
return;
3031-
}
3032-
30333028
for_each_set_bit(bit, &intr_dw, 32) {
3034-
const u32 ident = gen11_gt_engine_identity(i915,
3035-
bank, bit);
3029+
const u32 ident = gen11_gt_engine_identity(i915, bank, bit);
30363030

30373031
gen11_gt_identity_handler(i915, ident);
30383032
}

0 commit comments

Comments
 (0)