From nobody Thu Apr 9 12:08:22 2026 Received: from mgamail.intel.com (mgamail.intel.com [192.198.163.12]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 0724E265298 for ; Mon, 9 Mar 2026 07:32:35 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=192.198.163.12 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773041557; cv=none; b=I/fAgRCNtFEIyy5GAxJ9bn1ok8IYvWXSfYqVe7Kfes51zAsgQm77crrozrAvpiaxI6pfm6M7FCr96/JHvX6E6ZL+hHbM+hBl2iFlQDfixfJi4XDHqORJpwZ/7z/vdTcXdmn2sOgFcqkTyY1k0C2SLPAidLtsv98DDVXvqNXtD2I= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773041557; c=relaxed/simple; bh=0qNNpnP3lyZH3HdmWjAPRNRTyd2jswfseZV6vLDwdXM=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=eG2AsMqFHm5FG6kc8PHHPlV0FsXiib8qobQhrY1bcYZcKEzbm6j34Dj0YPynZZyUkDE7QHF/UVDZl5E4MidHSwaZujD1IbUaamKLCWn7he44XvFla+QRUy0azFrjt+5s4oRBZLrBmVPbvJhJrX0mdjxmFqEG/S9hneysb4Gw92o= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=intel.com; spf=pass smtp.mailfrom=intel.com; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b=a2qj8pCD; arc=none smtp.client-ip=192.198.163.12 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=intel.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=intel.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b="a2qj8pCD" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1773041556; x=1804577556; h=from:date:subject:mime-version:content-transfer-encoding: message-id:references:in-reply-to:to:cc; bh=0qNNpnP3lyZH3HdmWjAPRNRTyd2jswfseZV6vLDwdXM=; b=a2qj8pCDqIDmlO7i2L8y7VZ39nQn8/opSV2o2TQLzlf4bAA6rzHb9bOA nMhC5Paxn+ec/mUyMGz4aWej/VrsVw5gHPlGuzufK1ME75ohkXrznD1Fp AvrTWigLaxEI3sSLok6aLg5oJU5+Hz27CfcA5MBmcCdXKe0x9ynHAw/OG ue6DSkLE9Cv6HP9pH8f0mFrfChdvJrbNmFlhaS0b/vLAgoO+mA7EWco0y s8+CjiyVjwGJatBisRm9PuBKVy4yz6iS7tR3QyYwt4KwcUB7QggnhsEdT OpfU07wtSl4g9zJN0fERFcDTWsSidXyMD7cRSSXYxRfCjZOHZnzQAUf3Y A==; X-CSE-ConnectionGUID: XRS28remSXCDGkvXm5r1Lg== X-CSE-MsgGUID: hBCAL0kGTfaMa1qGlL8ooQ== X-IronPort-AV: E=McAfee;i="6800,10657,11723"; a="77935646" X-IronPort-AV: E=Sophos;i="6.23,109,1770624000"; d="scan'208";a="77935646" Received: from fmviesa007.fm.intel.com ([10.60.135.147]) by fmvoesa106.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 09 Mar 2026 00:31:06 -0700 X-CSE-ConnectionGUID: /iEYHDqqQSyCz+NV4+ewNg== X-CSE-MsgGUID: zdJlzA9pTQSXq1tTIPc88Q== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.23,109,1770624000"; d="scan'208";a="217286718" Received: from srr4-3-linux-106-armuthy.iind.intel.com ([10.190.238.56]) by fmviesa007.fm.intel.com with ESMTP; 09 Mar 2026 00:31:01 -0700 From: Arun R Murthy Date: Mon, 09 Mar 2026 12:59:24 +0530 Subject: [PATCH RFC 1/3] drm/display/dp: Export function to wake the sink AUX_CH Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20260309-dp_aux_timeout-v1-1-08c610a63a84@intel.com> References: <20260309-dp_aux_timeout-v1-0-08c610a63a84@intel.com> In-Reply-To: <20260309-dp_aux_timeout-v1-0-08c610a63a84@intel.com> To: Simona Vetter , Jani Nikula , ville.syrjala@linux.intel.com, suraj.kandpal@intel.com, imre.deak@intel.com Cc: dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, intel-gfx@lists.freedesktop.org, intel-xe@lists.freedesktop.org, Arun R Murthy X-Mailer: b4 0.15-dev On a native AUX_CH transaction timeout, as per Spec 2.1 Section 2.3.4 says that if the AUX Reply Timeout timer timesout, DPTX can rety for 3 times because the No Reply may be due to the DPRX waking up from power-saving state. Export a function to wake the DP sink AUX_CH. Signed-off-by: Arun R Murthy --- drivers/gpu/drm/display/drm_dp_helper.c | 36 +++++++++++++++++++++++++++++= ++++ include/drm/display/drm_dp_helper.h | 1 + 2 files changed, 37 insertions(+) diff --git a/drivers/gpu/drm/display/drm_dp_helper.c b/drivers/gpu/drm/disp= lay/drm_dp_helper.c index a697cc227e28964cd8322803298178e7d788e820..1b8aa543ec3dc5819948af6f272= 4f3514b96b918 100644 --- a/drivers/gpu/drm/display/drm_dp_helper.c +++ b/drivers/gpu/drm/display/drm_dp_helper.c @@ -701,6 +701,42 @@ void drm_dp_dpcd_set_powered(struct drm_dp_aux *aux, b= ool powered) } EXPORT_SYMBOL(drm_dp_dpcd_set_powered); =20 +/** + * drm_dp_wake_sink() - Try to wake up the sink device AUX_CH if in sleep + * + * @aux: DisplayPort AUX channel + */ +void drm_dp_wake_sink(struct drm_dp_aux *aux) +{ + const u8 wake_retry =3D 6; + u8 value =3D 0; + int ret =3D 0; + + /* + * Wake the sink device + * Spec DP2.1 section 2.3.1.2 if AUX CH is powered down by writing 0x02 + * to DP_SET_POWER dpcd reg, 1ms time would be required to wake it up + */ + ret =3D poll_timeout_us(ret =3D drm_dp_dpcd_readb(aux, DP_SET_POWER, &val= ue), + ret > 0, + 1000, wake_retry * 1000, true); + + /* + * If sink is in D3 then it may not respond to the AUX tx so + * wake it up to D3_AUX_ON state + * If the above poll_timeout_us fails, try waking the sink. + */ + if (value =3D=3D DP_SET_POWER_D3 || ret < 0) { + /* After setting to D0 need a min of 1ms to wake(Spec DP2.1 sec 2.3.1.2)= */ + drm_dp_dpcd_writeb(aux, DP_SET_POWER, + DP_SET_POWER_D0); + fsleep(1000); + drm_dp_dpcd_writeb(aux, DP_SET_POWER, + DP_SET_POWER_D3_AUX_ON); + } +} +EXPORT_SYMBOL(drm_dp_wake_sink); + /** * drm_dp_dpcd_set_probe() - Set whether a probing before DPCD access is d= one * @aux: DisplayPort AUX channel diff --git a/include/drm/display/drm_dp_helper.h b/include/drm/display/drm_= dp_helper.h index 1d0acd58f48676f60ff6a07cc6812f72cbb452e8..933428025c4d75ab7a0aa63039e= d5f8e45ca625b 100644 --- a/include/drm/display/drm_dp_helper.h +++ b/include/drm/display/drm_dp_helper.h @@ -546,6 +546,7 @@ struct drm_dp_aux { bool dpcd_probe_disabled; }; =20 +void drm_dp_wake_sink(struct drm_dp_aux *aux); int drm_dp_dpcd_probe(struct drm_dp_aux *aux, unsigned int offset); void drm_dp_dpcd_set_powered(struct drm_dp_aux *aux, bool powered); void drm_dp_dpcd_set_probe(struct drm_dp_aux *aux, bool enable); --=20 2.25.1 From nobody Thu Apr 9 12:08:22 2026 Received: from mgamail.intel.com (mgamail.intel.com [192.198.163.12]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 1F0B236494E for ; Mon, 9 Mar 2026 07:32:37 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=192.198.163.12 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773041558; cv=none; b=hDIv4xP5QSCRy+fmYECyLhwCIp+1r7qEjECeDspPteraH7MYsEHyP/yNkvg0i+TTDsKZDGm4243UMFsZvf48CoUqeKJVxnrIOaVBvR3XYqB1We23SD5saF6TrBBn3zqIZbtG/6IAyPqQRYnbKbU5Uq2wzQ4fVfuq5R1m25tlIJ4= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773041558; c=relaxed/simple; bh=lJ8E4ZxIoPsvLDyPf5xPOgjs/2rkmGqWz7hirornyf8=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=GzzEquv6lfODOyWeNrPfYc37GkcZRwioirrJH/qNpwRq/Iay6qjM1QQbbv2TcQu8Z2JHbERDUrRlhchAbO88mOWH623fPplvRYHNNTX+vDJYXQwqiriWus2ogZHsfIbLm9GaoEp2VoNsmquH5gEVTboW5m7rk3qIy4Jjzg/US/A= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=intel.com; spf=pass smtp.mailfrom=intel.com; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b=fl7JSZia; arc=none smtp.client-ip=192.198.163.12 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=intel.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=intel.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b="fl7JSZia" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1773041557; x=1804577557; h=from:date:subject:mime-version:content-transfer-encoding: message-id:references:in-reply-to:to:cc; bh=lJ8E4ZxIoPsvLDyPf5xPOgjs/2rkmGqWz7hirornyf8=; b=fl7JSZiam88XiCnBzgXIXz2f6sqIOUucHeme4A4mWSHpk8dsT/MD2deB u4yQVNperqGaJG7f9r/XLy77ia+qCLTKWsER1HJwWKQSpy9pQ6Ky6T3Qv 5L7fIcXasINTSPm5eMAUPrfqw5K6E0AQZfrhuJ7s79y60Vf3t4fQHJNpH 120ba+kl0CNSrFokqOmK0Fyk2StDGPN9LZjnpQlE3L/jlq3BLGBNQQkuH FpaiAgr0CrrIH/T5qXZMev7n+GE2EUEdqEcYEXwWelLAVcSX4uRJe4vCA 0DAcqUAHMqtbanwzVuCuXktsaF3lMoM6xyjA0tOrIVvs2dM4WBev8UTfn Q==; X-CSE-ConnectionGUID: bfwjxs5GQ4C+gMoVXiPuIQ== X-CSE-MsgGUID: D+BxZgRDRIu+kGWf+RPI3A== X-IronPort-AV: E=McAfee;i="6800,10657,11723"; a="77935658" X-IronPort-AV: E=Sophos;i="6.23,109,1770624000"; d="scan'208";a="77935658" Received: from fmviesa007.fm.intel.com ([10.60.135.147]) by fmvoesa106.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 09 Mar 2026 00:31:07 -0700 X-CSE-ConnectionGUID: 7US0M7c8TTS+CzJ9UDVhAA== X-CSE-MsgGUID: dBau2nXoTPeYdXKgGdjwuw== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.23,109,1770624000"; d="scan'208";a="217286736" Received: from srr4-3-linux-106-armuthy.iind.intel.com ([10.190.238.56]) by fmviesa007.fm.intel.com with ESMTP; 09 Mar 2026 00:31:04 -0700 From: Arun R Murthy Date: Mon, 09 Mar 2026 12:59:25 +0530 Subject: [PATCH RFC 2/3] drm/i915/dp: On AUX_CH tx timeout, wake up the sink Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20260309-dp_aux_timeout-v1-2-08c610a63a84@intel.com> References: <20260309-dp_aux_timeout-v1-0-08c610a63a84@intel.com> In-Reply-To: <20260309-dp_aux_timeout-v1-0-08c610a63a84@intel.com> To: Simona Vetter , Jani Nikula , ville.syrjala@linux.intel.com, suraj.kandpal@intel.com, imre.deak@intel.com Cc: dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, intel-gfx@lists.freedesktop.org, intel-xe@lists.freedesktop.org, Arun R Murthy X-Mailer: b4 0.15-dev On a native AUX_CH transaction when the timer on DPTX timesout this can be due to DPTX sending invalid command or the sink is waking up from sleept state and hence there is a No Reply from DPRX. In this case try to wake up the sink device. Signed-off-by: Arun R Murthy --- drivers/gpu/drm/i915/display/intel_dp_aux.c | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/display/intel_dp_aux.c b/drivers/gpu/drm/= i915/display/intel_dp_aux.c index b20ec3e589fadc4972efb77286978f38a527bd1c..0a9e2d6cdbc5d9e0d17b2db60a3= 2cf20a3bad6b6 100644 --- a/drivers/gpu/drm/i915/display/intel_dp_aux.c +++ b/drivers/gpu/drm/i915/display/intel_dp_aux.c @@ -372,8 +372,17 @@ intel_dp_aux_xfer(struct intel_dp *intel_dp, * Timeout errors from the HW already meet this * requirement so skip to next iteration */ - if (status & DP_AUX_CH_CTL_TIME_OUT_ERROR) + if (status & DP_AUX_CH_CTL_TIME_OUT_ERROR) { + /* + * Timeout can occur when there is no reply + * from the sink and this can be either due to + * DPTX sending wrong command or sink is sleep + * state/waking up from sleep state. + * Spec DP2.1 Section 2.11.7.1.5.8 + */ + drm_dp_wake_sink(&intel_dp->aux); continue; + } =20 if (status & DP_AUX_CH_CTL_RECEIVE_ERROR) { usleep_range(400, 500); --=20 2.25.1 From nobody Thu Apr 9 12:08:22 2026 Received: from mgamail.intel.com (mgamail.intel.com [192.198.163.12]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id C9152366DB5 for ; Mon, 9 Mar 2026 07:32:37 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=192.198.163.12 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773041559; cv=none; b=MiXojE8WMD4T4rhY+6w7VsFhzosZREYrWQNdoz9XmaoyRjQU+mpK9AmrOfhzU+DjJCrS4sjlH8yBRCsGwfnyqumY/zaVCB7WekreG60pADmVLCDSZQFHWhMyOqQVY4GLxoLxQ98dK9Rx3hFQGR6t4jd7n4JQpbyB0JJwbv+kaBM= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773041559; c=relaxed/simple; bh=aYMdgR5L9YxLHQ/YR7xUnnbgKI39rEzCYiTECzTdEjY=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=L5eAMc+d4GVj8TVmKPdS9XrEY8/o9XXo55p7SPwV+kZMXaj7jUYsIMibs1V9oFBuO5WiVKSQSrqBUEFthF21AxWxZ+ZxELjbvc4z3DC80bzRIlqkjXpUbHa2KKPl9fNPvOz0xUXuBKUpYWeiS10Ky7YhhjcNSOAaMCHRdL7+qrA= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=intel.com; spf=pass smtp.mailfrom=intel.com; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b=TSnmY4Bx; arc=none smtp.client-ip=192.198.163.12 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=intel.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=intel.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b="TSnmY4Bx" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1773041558; x=1804577558; h=from:date:subject:mime-version:content-transfer-encoding: message-id:references:in-reply-to:to:cc; bh=aYMdgR5L9YxLHQ/YR7xUnnbgKI39rEzCYiTECzTdEjY=; b=TSnmY4BxaCldT7KoVrh3Akf/UIWJmVNcQu99fhPs7oYDRpQ3xIheel7X GgLRuSgdXq22reG4vSDrUurJyNVmUIgwJYHEtfHnhZa2asAH2QYlozKfH 8+ep/V6hOkIaU5CBxWzoyLdhnVD8Pehf7FvG07ZxS4qO88F6YHMTjWFmo KJQQLppBK3X+ozR0f+Vle2hqqTQJE1+3PuXoEzB0yqNzm7HtufWiZD7WR vzGy1xX27Y2HF4xyF4/U9s3jmKZbpI+mZXhnHXNhumJ35cj3ZvraTTWlF dT5Hw/lQY7ErrCIQYPgyXFQuC6EXvUI6Zw0ee2h/CHCzPDrq4AMxAmQGP g==; X-CSE-ConnectionGUID: P0hAwBEcSJGHGZ5J3mXtLg== X-CSE-MsgGUID: q5+aXFwpTvaMwxOgoIT9Xg== X-IronPort-AV: E=McAfee;i="6800,10657,11723"; a="77935665" X-IronPort-AV: E=Sophos;i="6.23,109,1770624000"; d="scan'208";a="77935665" Received: from fmviesa007.fm.intel.com ([10.60.135.147]) by fmvoesa106.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 09 Mar 2026 00:31:12 -0700 X-CSE-ConnectionGUID: kmjpPaESSD2tE+Lsr5/ytg== X-CSE-MsgGUID: Oog5+2vIStG8YZqBs4Z+Iw== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.23,109,1770624000"; d="scan'208";a="217286758" Received: from srr4-3-linux-106-armuthy.iind.intel.com ([10.190.238.56]) by fmviesa007.fm.intel.com with ESMTP; 09 Mar 2026 00:31:07 -0700 From: Arun R Murthy Date: Mon, 09 Mar 2026 12:59:26 +0530 Subject: [PATCH RFC 3/3] drm/i915/dp: Configure PORT_AUX_CTL and then trigger the tx Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20260309-dp_aux_timeout-v1-3-08c610a63a84@intel.com> References: <20260309-dp_aux_timeout-v1-0-08c610a63a84@intel.com> In-Reply-To: <20260309-dp_aux_timeout-v1-0-08c610a63a84@intel.com> To: Simona Vetter , Jani Nikula , ville.syrjala@linux.intel.com, suraj.kandpal@intel.com, imre.deak@intel.com Cc: dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, intel-gfx@lists.freedesktop.org, intel-xe@lists.freedesktop.org, Arun R Murthy X-Mailer: b4 0.15-dev Use re_rmw and update the required bits for PORT_AUX_CTL and drop the bit configurations that are not required(AUX Power Request setting of bit 19). Also break writing to PORT_AUX_CTL into 2 steps with first step for doing the configuration/settings and then second write to trigger the AUX transaction. Signed-off-by: Arun R Murthy --- drivers/gpu/drm/i915/display/intel_display_types.h | 6 +- drivers/gpu/drm/i915/display/intel_dp_aux.c | 83 ++++++++++++++----= ---- drivers/gpu/drm/i915/display/intel_psr.c | 29 +++++--- 3 files changed, 76 insertions(+), 42 deletions(-) diff --git a/drivers/gpu/drm/i915/display/intel_display_types.h b/drivers/g= pu/drm/i915/display/intel_display_types.h index e189f8c39ccb440f99cd642de177b18f3b605753..341749452579acfc3e08715d2f0= b211bf6489dd9 100644 --- a/drivers/gpu/drm/i915/display/intel_display_types.h +++ b/drivers/gpu/drm/i915/display/intel_display_types.h @@ -1882,10 +1882,10 @@ struct intel_dp { =20 u32 (*get_aux_clock_divider)(struct intel_dp *dp, int index); /* - * This function returns the value we have to program the AUX_CTL - * register with to kick off an AUX transaction. + * This function programs the configuration/settings for the AUX_CTL + * register but dont kick off an AUX transaction. */ - u32 (*get_aux_send_ctl)(struct intel_dp *dp, int send_bytes, + void (*get_aux_send_ctl)(struct intel_dp *dp, int send_bytes, u32 aux_clock_divider); =20 i915_reg_t (*aux_ch_ctl_reg)(struct intel_dp *dp); diff --git a/drivers/gpu/drm/i915/display/intel_dp_aux.c b/drivers/gpu/drm/= i915/display/intel_dp_aux.c index 0a9e2d6cdbc5d9e0d17b2db60a32cf20a3bad6b6..4fef378e0a8fbf79211fd98913e= 507e90b2b48ea 100644 --- a/drivers/gpu/drm/i915/display/intel_dp_aux.c +++ b/drivers/gpu/drm/i915/display/intel_dp_aux.c @@ -175,12 +175,13 @@ static int g4x_dp_aux_precharge_len(void) precharge_min - preamble) / 2; } =20 -static u32 g4x_get_aux_send_ctl(struct intel_dp *intel_dp, - int send_bytes, - u32 aux_clock_divider) +static void g4x_get_aux_send_ctl(struct intel_dp *intel_dp, + int send_bytes, + u32 aux_clock_divider) { struct intel_display *display =3D to_intel_display(intel_dp); - u32 timeout; + i915_reg_t ch_ctl =3D intel_dp->aux_ch_ctl_reg(intel_dp); + u32 timeout, value; =20 /* Max timeout value on G4x-BDW: 1.6ms */ if (display->platform.broadwell) @@ -188,8 +189,7 @@ static u32 g4x_get_aux_send_ctl(struct intel_dp *intel_= dp, else timeout =3D DP_AUX_CH_CTL_TIME_OUT_400us; =20 - return DP_AUX_CH_CTL_SEND_BUSY | - DP_AUX_CH_CTL_DONE | + value =3D DP_AUX_CH_CTL_DONE | DP_AUX_CH_CTL_INTERRUPT | DP_AUX_CH_CTL_TIME_OUT_ERROR | timeout | @@ -197,23 +197,35 @@ static u32 g4x_get_aux_send_ctl(struct intel_dp *inte= l_dp, DP_AUX_CH_CTL_MESSAGE_SIZE(send_bytes) | DP_AUX_CH_CTL_PRECHARGE_2US(g4x_dp_aux_precharge_len()) | DP_AUX_CH_CTL_BIT_CLOCK_2X(aux_clock_divider); + + intel_de_rmw(display, ch_ctl, + (DP_AUX_CH_CTL_DONE | + DP_AUX_CH_CTL_INTERRUPT | + DP_AUX_CH_CTL_TIME_OUT_ERROR | + DP_AUX_CH_CTL_TIME_OUT_MASK | + DP_AUX_CH_CTL_RECEIVE_ERROR | + DP_AUX_CH_CTL_MESSAGE_SIZE_MASK | + DP_AUX_CH_CTL_PRECHARGE_2US_MASK | + DP_AUX_CH_CTL_BIT_CLOCK_2X_MASK), + value); + return; } =20 -static u32 skl_get_aux_send_ctl(struct intel_dp *intel_dp, - int send_bytes, - u32 unused) +static void skl_get_aux_send_ctl(struct intel_dp *intel_dp, + int send_bytes, + u32 unused) { struct intel_display *display =3D to_intel_display(intel_dp); struct intel_digital_port *dig_port =3D dp_to_dig_port(intel_dp); - u32 ret; + i915_reg_t ch_ctl =3D intel_dp->aux_ch_ctl_reg(intel_dp); + u32 value; =20 /* * Max timeout values: * SKL-GLK: 1.6ms * ICL+: 4ms */ - ret =3D DP_AUX_CH_CTL_SEND_BUSY | - DP_AUX_CH_CTL_DONE | + value =3D DP_AUX_CH_CTL_DONE | DP_AUX_CH_CTL_INTERRUPT | DP_AUX_CH_CTL_TIME_OUT_ERROR | DP_AUX_CH_CTL_TIME_OUT_MAX | @@ -222,17 +234,22 @@ static u32 skl_get_aux_send_ctl(struct intel_dp *inte= l_dp, DP_AUX_CH_CTL_FW_SYNC_PULSE_SKL(intel_dp_aux_fw_sync_len(intel_dp)) | DP_AUX_CH_CTL_SYNC_PULSE_SKL(intel_dp_aux_sync_len()); =20 - if (intel_tc_port_in_tbt_alt_mode(dig_port)) - ret |=3D DP_AUX_CH_CTL_TBT_IO; + intel_de_rmw(display, ch_ctl, + (DP_AUX_CH_CTL_DONE | + DP_AUX_CH_CTL_INTERRUPT | + DP_AUX_CH_CTL_TIME_OUT_ERROR | + DP_AUX_CH_CTL_TIME_OUT_MASK | + DP_AUX_CH_CTL_RECEIVE_ERROR | + DP_AUX_CH_CTL_MESSAGE_SIZE_MASK | + DP_AUX_CH_CTL_FW_SYNC_PULSE_SKL_MASK | + DP_AUX_CH_CTL_SYNC_PULSE_SKL_MASK), + value); =20 - /* - * Power request bit is already set during aux power well enable. - * Preserve the bit across aux transactions. - */ - if (DISPLAY_VER(display) >=3D 14) - ret |=3D XELPDP_DP_AUX_CH_CTL_POWER_REQUEST; + if (intel_tc_port_in_tbt_alt_mode(dig_port)) + intel_de_rmw(display, ch_ctl, DP_AUX_CH_CTL_TBT_IO, + DP_AUX_CH_CTL_TBT_IO); =20 - return ret; + return; } =20 static int @@ -341,11 +358,12 @@ intel_dp_aux_xfer(struct intel_dp *intel_dp, } =20 while ((aux_clock_divider =3D intel_dp->get_aux_clock_divider(intel_dp, c= lock++))) { - u32 send_ctl =3D intel_dp->get_aux_send_ctl(intel_dp, - send_bytes, - aux_clock_divider); + intel_dp->get_aux_send_ctl(intel_dp, send_bytes, + aux_clock_divider); =20 - send_ctl |=3D aux_send_ctl_flags; + /* Update the flags */ + intel_de_rmw(display, ch_ctl, DP_AUX_CH_CTL_AUX_AKSV_SELECT, + aux_send_ctl_flags); =20 /* Must try at least 3 times according to DP spec */ for (try =3D 0; try < 5; try++) { @@ -356,15 +374,20 @@ intel_dp_aux_xfer(struct intel_dp *intel_dp, send_bytes - i)); =20 /* Send the command and wait for it to complete */ - intel_de_write(display, ch_ctl, send_ctl); + intel_de_rmw(display, ch_ctl, + DP_AUX_CH_CTL_SEND_BUSY, + DP_AUX_CH_CTL_SEND_BUSY); =20 status =3D intel_dp_aux_wait_done(intel_dp); =20 /* Clear done status and any errors */ - intel_de_write(display, ch_ctl, - status | DP_AUX_CH_CTL_DONE | - DP_AUX_CH_CTL_TIME_OUT_ERROR | - DP_AUX_CH_CTL_RECEIVE_ERROR); + intel_de_rmw(display, ch_ctl, + (DP_AUX_CH_CTL_DONE | + DP_AUX_CH_CTL_TIME_OUT_ERROR | + DP_AUX_CH_CTL_RECEIVE_ERROR), + (DP_AUX_CH_CTL_DONE | + DP_AUX_CH_CTL_TIME_OUT_ERROR | + DP_AUX_CH_CTL_RECEIVE_ERROR)); =20 /* * DP CTS 1.2 Core Rev 1.1, 4.2.1.1 & 4.2.1.2 diff --git a/drivers/gpu/drm/i915/display/intel_psr.c b/drivers/gpu/drm/i91= 5/display/intel_psr.c index 9296ca3a4ff468a6e61babd81217e4ba19b15062..e06e04f20355d511e5c58fc2886= 6aa763fd65a4b 100644 --- a/drivers/gpu/drm/i915/display/intel_psr.c +++ b/drivers/gpu/drm/i915/display/intel_psr.c @@ -722,7 +722,9 @@ static void hsw_psr_setup_aux(struct intel_dp *intel_dp) { struct intel_display *display =3D to_intel_display(intel_dp); enum transcoder cpu_transcoder =3D intel_dp->psr.transcoder; + i915_reg_t ch_ctl =3D psr_aux_ctl_reg(display, cpu_transcoder); u32 aux_clock_divider, aux_ctl; + /* write DP_SET_POWER=3DD0 */ static const u8 aux_msg[] =3D { [0] =3D (DP_AUX_NATIVE_WRITE << 4) | ((DP_SET_POWER >> 16) & 0xf), @@ -742,17 +744,26 @@ static void hsw_psr_setup_aux(struct intel_dp *intel_= dp) aux_clock_divider =3D intel_dp->get_aux_clock_divider(intel_dp, 0); =20 /* Start with bits set for DDI_AUX_CTL register */ - aux_ctl =3D intel_dp->get_aux_send_ctl(intel_dp, sizeof(aux_msg), - aux_clock_divider); + intel_dp->get_aux_send_ctl(intel_dp, sizeof(aux_msg), + aux_clock_divider); =20 /* Select only valid bits for SRD_AUX_CTL */ - aux_ctl &=3D EDP_PSR_AUX_CTL_TIME_OUT_MASK | - EDP_PSR_AUX_CTL_MESSAGE_SIZE_MASK | - EDP_PSR_AUX_CTL_PRECHARGE_2US_MASK | - EDP_PSR_AUX_CTL_BIT_CLOCK_2X_MASK; - - intel_de_write(display, psr_aux_ctl_reg(display, cpu_transcoder), - aux_ctl); + aux_ctl =3D EDP_PSR_AUX_CTL_TIME_OUT_MASK | + EDP_PSR_AUX_CTL_MESSAGE_SIZE_MASK | + EDP_PSR_AUX_CTL_PRECHARGE_2US_MASK | + EDP_PSR_AUX_CTL_BIT_CLOCK_2X_MASK; + + intel_de_rmw(display, ch_ctl, + (EDP_PSR_AUX_CTL_TIME_OUT_MASK | + EDP_PSR_AUX_CTL_MESSAGE_SIZE_MASK | + EDP_PSR_AUX_CTL_PRECHARGE_2US_MASK | + EDP_PSR_AUX_CTL_BIT_CLOCK_2X_MASK), + aux_ctl); + + /* Send the command or intitate the AUX transaction */ + intel_de_rmw(display, ch_ctl, + DP_AUX_CH_CTL_SEND_BUSY, + DP_AUX_CH_CTL_SEND_BUSY); } =20 static bool psr2_su_region_et_valid(struct intel_connector *connector, boo= l panel_replay) --=20 2.25.1