From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mgamail.intel.com (mgamail.intel.com [192.198.163.17]) (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 E4837395DAD for ; Thu, 26 Feb 2026 09:12:45 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=192.198.163.17 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1772097169; cv=none; b=LzmBSvKyRtmNtEnTBWbUkKHfhHXtZzIdzgdlkHWyrHTL5FMl+0uCSD6FfqLw8M7c0S5U4SJp0wL+TTt5YRpU+YyzmYIRSv1SgxnZy1CLqO6baTMtdRKndCvca9zQiq0eurS+IK2iFsu31MDHoXWAqRMRY2oSd+WBpvVnNGSWb1M= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1772097169; c=relaxed/simple; bh=blMXsWAvuofUkGygXR2xg4VS4c6FdlmUnwcyorXruTo=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=m36gKcoxRAcFA3Zze/ksLGfCBv/MRX2qmrrJ8R+HLYvy5qdqkRE+kCwMASCJ3OG8peH1y4dSwPmJgDjJlmW13BwuaRfggJCgniQjV/IbSgMJixhp0CmPpFpN4UMiYxHckUYOgAnbQh8tpnBoXe+HaPKPLOzyjGRN0JjdvAztCz0= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.intel.com; spf=pass smtp.mailfrom=intel.com; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b=Aaq7/Zgs; arc=none smtp.client-ip=192.198.163.17 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.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="Aaq7/Zgs" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1772097166; x=1803633166; h=date:from:to:cc:subject:message-id:references: mime-version:content-transfer-encoding:in-reply-to; bh=blMXsWAvuofUkGygXR2xg4VS4c6FdlmUnwcyorXruTo=; b=Aaq7/ZgsdbfB7KgOHY16a+3qxRpL0SEztBzzDeiTIT7XnCpi5aShMvKZ zPlYitfXHcFvyO2/zRxu7KpeGT1Hu0UfhKTLXbJM+Ccds6CqjeL7CsBe1 5Nho9ve0/A4hMuCNAhZYBNT+dvTA5q+g03XIIFuz3H2KLDojJW7W/98EO ehiAvKJ3Rp2HHRxv4a/MqEi6XkZ7yOB5oS1NEOrBZQZk+2Jv4J+BVvQ1c HSD8LvWiWOHIfBbjqSeGFNScRpD5Lv/QORGruTt4wDKmnTtC5oGeoF7mc dwAvieh7/rsurC4T48qHGQ0ubWSLhqjn3xQHyaPpnTuhKoRlEmpeh+3ly Q==; X-CSE-ConnectionGUID: fbgaHTFgQnuI4X10GyEGCA== X-CSE-MsgGUID: B4moeO43Q++D2OAiVZ97zw== X-IronPort-AV: E=McAfee;i="6800,10657,11712"; a="73065115" X-IronPort-AV: E=Sophos;i="6.21,312,1763452800"; d="scan'208";a="73065115" Received: from orviesa007.jf.intel.com ([10.64.159.147]) by fmvoesa111.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 26 Feb 2026 01:12:45 -0800 X-CSE-ConnectionGUID: SuBJDBysQ32+DR0sRD4BBg== X-CSE-MsgGUID: ToLYcNFdS0q2cG3BEaoPUA== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.21,312,1763452800"; d="scan'208";a="216648273" Received: from egrumbac-mobl6.ger.corp.intel.com (HELO localhost) ([10.245.244.224]) by orviesa007-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 26 Feb 2026 01:12:43 -0800 Date: Thu, 26 Feb 2026 11:12:39 +0200 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= To: Jouni =?iso-8859-1?Q?H=F6gander?= Cc: intel-gfx@lists.freedesktop.org, intel-xe@lists.freedesktop.org, Animesh Manna , Jani Nikula , stable@vger.kernel.org, Ankit Nautiyal Subject: Re: [PATCH] drm/i915/alpm: ALPM disable fixes Message-ID: References: <20260212062731.397801-1-jouni.hogander@intel.com> Precedence: bulk X-Mailing-List: stable@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20260212062731.397801-1-jouni.hogander@intel.com> X-Patchwork-Hint: comment Organization: Intel Finland Oy - BIC 0357606-4 - c/o Alberga Business Park, 6 krs Bertel Jungin Aukio 5, 02600 Espoo, Finland On Thu, Feb 12, 2026 at 08:27:31AM +0200, Jouni Högander wrote: > PORT_ALPM_CTL is supposed to be written only before link training. Remove > writing it from ALPM disable. > > Also clearing ALPM_CTL_ALPM_AUX_LESS_ENABLE and is not about disabling ALPM > but switching to AUX-Wake ALPM. Stop touching this bit on ALPM disable. There's another open coded variant of this in intel_alpm_lobf_disable(), and that thing also gets passed the wrong crtc_state (new instead of old). Also LOBF enable is now being done from two places (intel_alpm_lobf_enable() and intel_psr_enable_source()). How those two interact is anyone's guess. > > Closes: https://gitlab.freedesktop.org/drm/xe/kernel/-/issues/7153 > Fixes: 1ccbf135862b ("drm/i915/psr: Enable ALPM on source side for eDP Panel replay") > Cc: Animesh Manna > Cc: Jani Nikula > Cc: # v6.10+ > Signed-off-by: Jouni Högander > --- > drivers/gpu/drm/i915/display/intel_alpm.c | 7 +------ > 1 file changed, 1 insertion(+), 6 deletions(-) > > diff --git a/drivers/gpu/drm/i915/display/intel_alpm.c b/drivers/gpu/drm/i915/display/intel_alpm.c > index e0a4a59dc025..b3334bc4d0f9 100644 > --- a/drivers/gpu/drm/i915/display/intel_alpm.c > +++ b/drivers/gpu/drm/i915/display/intel_alpm.c > @@ -604,12 +604,7 @@ void intel_alpm_disable(struct intel_dp *intel_dp) > mutex_lock(&intel_dp->alpm.lock); > > intel_de_rmw(display, ALPM_CTL(display, cpu_transcoder), > - ALPM_CTL_ALPM_ENABLE | ALPM_CTL_LOBF_ENABLE | > - ALPM_CTL_ALPM_AUX_LESS_ENABLE, 0); > - > - intel_de_rmw(display, > - PORT_ALPM_CTL(cpu_transcoder), > - PORT_ALPM_CTL_ALPM_AUX_LESS_ENABLE, 0); > + ALPM_CTL_ALPM_ENABLE | ALPM_CTL_LOBF_ENABLE, 0); > > drm_dbg_kms(display->drm, "Disabling ALPM\n"); > mutex_unlock(&intel_dp->alpm.lock); > -- > 2.43.0 -- Ville Syrjälä Intel