From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pj2-f11.google.com (mail-pj2-f11.google.com [74.125.227.139]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 9C4EB37F74A for ; Wed, 26 Aug 2026 05:24:33 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=74.125.227.139 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1787721874; cv=none; b=emOHit4q+LiO9mI0FO9K5SzfDJ0fOIYp1rWya62Lj26ZF4tItR9t2DScSQu36lNP/hy/PkBIQbSCwmKZxnctXSX4WVxNUIdeHRSTaO/NY01HjviT9wF9rkLgrgmQ2/+rayZvC73+ns9XWKFt1FH+K5GuaGco8UgccO6lemrDZFg= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1787721874; c=relaxed/simple; bh=VjbjrLwydDG0Hmo8qHb9IpE5Qvor6tpTfBgFJ2NgbEw=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=P1pwkoz3dElXlC26jZrPL/u7UfTGSNVwq43whzixA3x7fYPv+Rf4Z37bIxpBaIuFd/ncVBFJgk6Ao9Zmxv0979S8rM6q7UehECKdjtvnCJl4y2v+Tz2esDI3qrHFBa++1PkEqEmOCHw9szYwkAhAGKTwivTQ+i73cmraTxlDUyM= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=ZBqNSBCt; arc=none smtp.client-ip=74.125.227.139 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="ZBqNSBCt" Received: by mail-pj2-f11.google.com with SMTP id d9443c01a7336-2ceb5a39c48so1935255ad.1 for ; Tue, 25 Aug 2026 22:24:33 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1787721873; x=1788326673; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to:content-type; bh=2iIl3uA7kRHPn8Sz3ycjaBFmA0XngwesOSx8TRlbXC0=; b=ZBqNSBCtnLTywAtMUo8Y/Plb1EcWlvC8pxzz/cGKK60Ut5qcBg2TWjfsHzl7esCRh7 yMeJQ38adaiT8CRH1RS6DJ5bZo8xpOkS8fkYaAXDwLK45stRxtnM2j8GptBVVpvybnBS gd8yPHkPIu2qNZ5EY3/FhrWLi5UsdkvqWXt1VmzKrOCxQ6nQ/QGADPLNSvUCey+nCNfq naE4mliUbTfPDWB4vDIHcYXVfcSIPxv7QZK50IJ5WzmLAjx7B9LgEcPjfP/wW1qsqEjF ToJzQczAiD/3JIZ/ldd4A8wzd2SyuHsMCEvC4kvqtzb6zxPQnJ83UPJ8gMWuStA+HOeV 9m1A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1787721873; x=1788326673; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-gg:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to:content-type; bh=2iIl3uA7kRHPn8Sz3ycjaBFmA0XngwesOSx8TRlbXC0=; b=ELXOWzDw1iylio2q4+Ldix8yxrEjxcV8rUVWOBQuQEnBd5OnKclJeMYjZYmYeBBAZl hypcP0RV3s6Lm7Qu6Pd7zKm+1kTLDLEZJaKZnA9cldlCOuhDennzG8EmjhTWoEpbqpti Xx6GI952ii4fYh91qi4JzQzfljvRHvTh0nzN9e3aUekbENeYRpi+ReTwVcB+pk/mP7vd uVSrmmzJPlrih38jJdEDjzElK2yHZu7fNDvUQhO/SHAztArbt/tg6XgSiyYW3R0cMbfD U1rBbVW/ja0PP3oc/7+vTlU6O8X5KyPHfd96zPqgGY7SuqASMLAKPzy5fB/615N0QKod XnKg== X-Forwarded-Encrypted: i=1; AHgh+RpT6zs4BadQfgkWNhj1Wmv8RBtKljkVlaGeCgU8PXc6QE1i6EPWXl5he3XGepCF9GBb1i9oD5+6OXFlEao=@vger.kernel.org X-Gm-Message-State: AFuF++no0vzyovPMpzLYTahAI/ZnRMTfkOoKWNTOA5KaNvwEs7F32OGs bklSliviLpeBF5cE0ucrGSeaRfUT6KirlFsec3SehzXFQFeKOIdDLDJ5 X-Gm-Gg: AR+sD10SbXgDdfrOFa4GULvDv1eYWowNpO+s+ARSNvVssTRHJAY1reymz4l2EC3lYVb /g/7IlcLSn4TKbSMVFb5jCzJOZeBafxw6h/h8WzRD8lQAESFfYUH+KP6xRorkz1Z/MeIFZBbFCC 8O2SJzCuwQ8JqwSkySOOaSLduA9Whh8gk6In0wh89Cgr65wh2v718JosbvTxmhAtz4oOS1w1GDR MHiTHAVagSgoS/Ta/O0Yx+9FGjH/XOnkkYnA1n64PtGkrZjiasKC+1aSaeB6B5Gn8m0FO3vm/q+ 2nqCjRvpNg/82+AwTLEhUGReUDr1+sIHO9P0+gIzrBOWm9DK7jbghgl+nEKXN2MNM8kwWseE5yX RdJeDSa0NDuNhjG3HOZvkJEoBjTgcBDwewpMah/0IpPdxggIMb2cZlooWLiP0XTbCv+xI80a31K L0dK4XdaKvYY9lskcBoRvnXe9t//ue5GkMXs5SqdfRQpqHb4Wjdw9vRIMY25GUnB7lJ/ANLrrTa DCB320N+/g= X-Received: by 2002:a05:6a20:a106:b0:3cd:602e:f8ab with SMTP id adf61e73a8af0-3cf7576bc90mr6714349637.4.1787721872732; Tue, 25 Aug 2026 22:24:32 -0700 (PDT) Received: from DESKTOP-BDFNLNM.localdomain ([27.63.245.244]) by smtp.gmail.com with ESMTPSA id 5a478bee46e88-3283d8dc465sm4505557eec.24.2026.08.25.22.24.28 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 25 Aug 2026 22:24:32 -0700 (PDT) From: tarund18303@gmail.com To: jani.nikula@linux.intel.com Cc: rodrigo.vivi@intel.com, joonas.lahtinen@linux.intel.com, tursulin@ursulin.net, airlied@gmail.com, simona@ffwll.ch, intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, tarun-tarun143 Subject: [PATCH] drm/i915/dp: remove redundant link state check Date: Wed, 26 Aug 2026 05:23:57 +0000 Message-ID: <20260826052357.10151-1-tarund18303@gmail.com> X-Mailer: git-send-email 2.43.0 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit From: tarun-tarun143 The DP link state is checked and recovered through the encoder link check work from the modeset and HPD paths. The detect handler also used to queue the same link check work for non-eDP sinks. This was added when link recovery was moved to the encoder's check link state work, and the original change explicitly left a TODO to remove the detect-time check once it became redundant. Remove the detect-time link state check and rely on the existing link recovery paths. Signed-off-by: tarun-tarun143 --- drivers/gpu/drm/i915/display/intel_dp.c | 12 ------------ 1 file changed, 12 deletions(-) diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c index 113d767e62e9..ffd39206b958 100644 --- a/drivers/gpu/drm/i915/display/intel_dp.c +++ b/drivers/gpu/drm/i915/display/intel_dp.c @@ -6379,18 +6379,6 @@ intel_dp_detect(struct drm_connector *_connector, status = connector_status_disconnected; goto out_unset_edid; } - - /* - * Some external monitors do not signal loss of link synchronization - * with an IRQ_HPD, so force a link status check. - * - * TODO: this probably became redundant, so remove it: the link state - * is rechecked/recovered now after modesets, where the loss of - * synchronization tends to occur. - */ - if (!intel_dp_is_edp(intel_dp)) - intel_dp_check_link_state(intel_dp); - /* * Clearing NACK and defer counts to get their exact values * while reading EDID which are required by Compliance tests -- 2.43.0