This is a note to let you know that I've just added the patch titled Revert "drm/i915/opregion: check port number bounds for SWSCI display power state" to the 5.10-stable tree which can be found at: http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary The filename of the patch is: revert-drm-i915-opregion-check-port-number-bounds-for-swsci-display-power-state.patch and it can be found in the queue-5.10 subdirectory. If you, or anyone else, feels it should not be added to the stable tree, please let <stable@xxxxxxxxxxxxxxx> know about it. >From gthelen@xxxxxxxxxx Thu May 19 14:59:02 2022 From: Greg Thelen <gthelen@xxxxxxxxxx> Date: Mon, 16 May 2022 17:08:35 -0700 Subject: Revert "drm/i915/opregion: check port number bounds for SWSCI display power state" To: stable@xxxxxxxxxxxxxxx Cc: Jani Nikula <jani.nikula@xxxxxxxxx>, Greg Thelen <gthelen@xxxxxxxxxx> Message-ID: <20220517000835.2450573-1-gthelen@xxxxxxxxxx> From: Greg Thelen <gthelen@xxxxxxxxxx> This reverts commit b84857c06ef9e72d09fadafdbb3ce9af64af954f. 5.10 stable contains 2 identical commits: 1. commit eb7bf11e8ef1 ("drm/i915/opregion: check port number bounds for SWSCI display power state") 2. commit b84857c06ef9 ("drm/i915/opregion: check port number bounds for SWSCI display power state") Both commits add separate checks for the same condition. Revert the 2nd redundant check to match upstream, which only has one check. Signed-off-by: Greg Thelen <gthelen@xxxxxxxxxx> Signed-off-by: Yu Liao <liaoyu15@xxxxxxxxxx> Signed-off-by: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> --- drivers/gpu/drm/i915/display/intel_opregion.c | 15 --------------- 1 file changed, 15 deletions(-) --- a/drivers/gpu/drm/i915/display/intel_opregion.c +++ b/drivers/gpu/drm/i915/display/intel_opregion.c @@ -376,21 +376,6 @@ int intel_opregion_notify_encoder(struct return -EINVAL; } - /* - * The port numbering and mapping here is bizarre. The now-obsolete - * swsci spec supports ports numbered [0..4]. Port E is handled as a - * special case, but port F and beyond are not. The functionality is - * supposed to be obsolete for new platforms. Just bail out if the port - * number is out of bounds after mapping. - */ - if (port > 4) { - drm_dbg_kms(&dev_priv->drm, - "[ENCODER:%d:%s] port %c (index %u) out of bounds for display power state notification\n", - intel_encoder->base.base.id, intel_encoder->base.name, - port_name(intel_encoder->port), port); - return -EINVAL; - } - if (!enable) parm |= 4 << 8; Patches currently in stable-queue which might be from gthelen@xxxxxxxxxx are queue-5.10/revert-drm-i915-opregion-check-port-number-bounds-for-swsci-display-power-state.patch