From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (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 C3A4B19ABDE for ; Tue, 25 Feb 2025 16:44:16 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1740501857; cv=none; b=eoLTd58qQ3GJBBbszZkF/yVKen2m0dnu+MYhwMQJi1pMAZLwvNVFfT/A5DZhYvyBUXikdYVWlkmfp55An8DAFUYrXP568QE3/rs6dfMVZVBDipeCdutkB9Rw2BH2eohiq35PWeUpKHNVsQbVtyOW8DwyegJURgwY0v3RxjmbW6E= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1740501857; c=relaxed/simple; bh=A6ouo6ovEkcY8CdYz8vQoee+XUsbhVfQnidOvEECRgU=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=dJoNobTMoEwWQ+4qXctSUMT+6THptRAYQYRxeGMsiqt9BYkpmxnJmfBjObfVVmIQHSzG8LMUqi6fC+Zdyrk68H9HLKjfWJV0MCr9z+U3CTfRuILfpG7/dkCivYmlVqWRaobQX7eQoyY49l4AX+zd5vnFr3aeru2nzRDhOYjYxNk= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=s+IXwaoY; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="s+IXwaoY" Received: by smtp.kernel.org (Postfix) with ESMTPSA id CF8E8C4CEE8; Tue, 25 Feb 2025 16:44:15 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1740501856; bh=A6ouo6ovEkcY8CdYz8vQoee+XUsbhVfQnidOvEECRgU=; h=From:Date:Subject:References:In-Reply-To:To:Cc:From; b=s+IXwaoYdFeKkcJffl8R9V5dSbVPd4IlRZTXF6RgYarT2dC7guRBzzXovboh1LdRT 8XiFvNsAylmvROLx1mbPCaxhs6tm85FakEyXvFnRX8rB2Mbij9DrbJDMxlFR2ls4N+ BfgcH2zs2w+mP2yxOaJwyZ1FIwB2QHPlJmI2fetNky/J4jmwcSFgrFVksY2Sxk1l15 1SwveUAENLSxnFF+8S7N50xSsWjGIhfxRvFVs32VqZOL0PglZkDSTomHLeayyzHkmz pHjq3ZxVzO+xA152u4X1gBDtgPSQRPjx5isNyql3IG65aoJmhc3SU66chJchSIfhNW WQjFSIDNBZzcA== From: Maxime Ripard Date: Tue, 25 Feb 2025 17:43:52 +0100 Subject: [PATCH v4 04/15] drm/atomic: Introduce helper to lookup connector by encoder 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: 7bit Message-Id: <20250225-bridge-connector-v4-4-7ecb07b09cad@kernel.org> References: <20250225-bridge-connector-v4-0-7ecb07b09cad@kernel.org> In-Reply-To: <20250225-bridge-connector-v4-0-7ecb07b09cad@kernel.org> To: Maarten Lankhorst , Thomas Zimmermann , David Airlie , Simona Vetter , Andrzej Hajda , Neil Armstrong , Robert Foss , Laurent Pinchart , Jonas Karlman , Jernej Skrabec , Douglas Anderson Cc: Herve Codina , dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, Maxime Ripard , Simona Vetter X-Mailer: b4 0.14.2 X-Developer-Signature: v=1; a=openpgp-sha256; l=3895; i=mripard@kernel.org; h=from:subject:message-id; bh=A6ouo6ovEkcY8CdYz8vQoee+XUsbhVfQnidOvEECRgU=; b=owGbwMvMwCmsHn9OcpHtvjLG02pJDOl7P3sbrjY5aWW9et5xFk2TL/mrHi+7brhI1/WXFnMEr 0VAju+8jqksDMKcDLJiiixPZMJOL29fXOVgv/IHzBxWJpAhDFycAjART0bGhlffLH33ZSZaWbL+ ie+fMCnL7eHB3WYn9k5PbeUsCT/8QDvNeWvfvKrT/OfXOPEvF/u7hLE++3DCd+skw+T7z2dVqnO 9kGfiq3WW/rI+p8P0vG+EZtYWk4QvQT/1JL071zyLzj3rPhkA X-Developer-Key: i=mripard@kernel.org; a=openpgp; fpr=BE5675C37E818C8B5764241C254BCFC56BF6CE8D With the bridges switching over to drm_bridge_connector, the direct association between a bridge driver and its connector was lost. This is mitigated for atomic bridge drivers by the fact you can access the encoder, and then call drm_atomic_get_old_connector_for_encoder() or drm_atomic_get_new_connector_for_encoder() with drm_atomic_state. This was also made easier by providing drm_atomic_state directly to all atomic hooks bridges can implement. However, bridge drivers don't have a way to access drm_atomic_state outside of the modeset path, like from the hotplug interrupt path or any interrupt handler. Let's introduce a function to retrieve the connector currently assigned to an encoder, without using drm_atomic_state, to make these drivers' life easier. Co-developed-by: Simona Vetter Signed-off-by: Maxime Ripard --- drivers/gpu/drm/drm_atomic.c | 45 ++++++++++++++++++++++++++++++++++++++++++++ include/drm/drm_atomic.h | 3 +++ 2 files changed, 48 insertions(+) diff --git a/drivers/gpu/drm/drm_atomic.c b/drivers/gpu/drm/drm_atomic.c index 9ea2611770f43ce7ccba410406d5f2c528aab022..b926b132590e78f8d41d48eb4da4bccf170ee236 100644 --- a/drivers/gpu/drm/drm_atomic.c +++ b/drivers/gpu/drm/drm_atomic.c @@ -985,10 +985,55 @@ drm_atomic_get_new_connector_for_encoder(const struct drm_atomic_state *state, return NULL; } EXPORT_SYMBOL(drm_atomic_get_new_connector_for_encoder); +/** + * drm_atomic_get_connector_for_encoder - Get connector currently assigned to an encoder + * @encoder: The encoder to find the connector of + * @ctx: Modeset locking context + * + * This function finds and returns the connector currently assigned to + * an @encoder. + * + * Returns: + * The connector connected to @encoder, or an error pointer otherwise. + * When the error is EDEADLK, a deadlock has been detected and the + * sequence must be restarted. + */ +struct drm_connector * +drm_atomic_get_connector_for_encoder(const struct drm_encoder *encoder, + struct drm_modeset_acquire_ctx *ctx) +{ + struct drm_connector_list_iter conn_iter; + struct drm_connector *out_connector = ERR_PTR(-EINVAL); + struct drm_connector *connector; + struct drm_device *dev = encoder->dev; + int ret; + + ret = drm_modeset_lock(&dev->mode_config.connection_mutex, ctx); + if (ret) + return ERR_PTR(ret); + + drm_connector_list_iter_begin(dev, &conn_iter); + drm_for_each_connector_iter(connector, &conn_iter) { + if (!connector->state) + continue; + + if (encoder == connector->state->best_encoder) { + out_connector = connector; + break; + } + } + drm_connector_list_iter_end(&conn_iter); + drm_modeset_unlock(&dev->mode_config.connection_mutex); + + return out_connector; +} +EXPORT_SYMBOL(drm_atomic_get_connector_for_encoder); + + /** * drm_atomic_get_old_crtc_for_encoder - Get old crtc for an encoder * @state: Atomic state * @encoder: The encoder to fetch the crtc state for * diff --git a/include/drm/drm_atomic.h b/include/drm/drm_atomic.h index 4c673f0698fef6b60f77db980378d5e88e0e250e..38636a593c9d98cadda85ccd67326cb152f0dd27 100644 --- a/include/drm/drm_atomic.h +++ b/include/drm/drm_atomic.h @@ -623,10 +623,13 @@ struct drm_connector * drm_atomic_get_old_connector_for_encoder(const struct drm_atomic_state *state, struct drm_encoder *encoder); struct drm_connector * drm_atomic_get_new_connector_for_encoder(const struct drm_atomic_state *state, struct drm_encoder *encoder); +struct drm_connector * +drm_atomic_get_connector_for_encoder(const struct drm_encoder *encoder, + struct drm_modeset_acquire_ctx *ctx); struct drm_crtc * drm_atomic_get_old_crtc_for_encoder(struct drm_atomic_state *state, struct drm_encoder *encoder); struct drm_crtc * -- 2.48.1