From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from bali.collaboradmins.com (bali.collaboradmins.com [148.251.105.195]) (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 8E73338B7B1 for ; Fri, 12 Jun 2026 19:23:23 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=148.251.105.195 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781292204; cv=none; b=bo3kHG/YDb7z4fqWO0d615w3sD4ynkFdrxrtooDBy63qJZNGBRMo5c6pFRshWpf+SRfym/QhNHB3wj9WDA1Z22QJp/ctCszdIMAfIE6l4GpaxNTxrG9kqM6RwJUlM7PJqmbhqiXTHUuxji/UJzVVMjYc9ip1yz0hMBj0jHYJy0c= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781292204; c=relaxed/simple; bh=0gYiI0nuF3TbeK77gJZHu37hGbu8poCEiD0ZSHVH6BI=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=uZmScxnEwckXz6yUJt4jJwuhyd/Kvtjo2mVl1yWf/GTxq432e7ZTWJXLcGA+prUJtLj2U3noZkqmA2H8mC+LfrLM574atJ2R5GCpzMIl6aB+GR+9Q7GJ2r4gh6Ba8nXVdeBXk7v6UptFZB8rhseQE+nO19RiqZhzAcLb62Z2OBA= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=collabora.com; spf=pass smtp.mailfrom=collabora.com; dkim=pass (2048-bit key) header.d=collabora.com header.i=@collabora.com header.b=Gm3y3rLl; arc=none smtp.client-ip=148.251.105.195 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=collabora.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=collabora.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=collabora.com header.i=@collabora.com header.b="Gm3y3rLl" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=collabora.com; s=mail; t=1781292201; bh=0gYiI0nuF3TbeK77gJZHu37hGbu8poCEiD0ZSHVH6BI=; h=Date:Subject:To:Cc:References:From:In-Reply-To:From; b=Gm3y3rLlS68dIE9jBDPmevDg42WfdulXApUp3vAv7VyBiVDX5lBLsVz4Ooo6x8oiQ fVzGhxPc15D3LNsmcxF3UzV/dX+t5U3yv66pQR7P6saOQI7//NVDOgS1YT7jq2v6mp XrVqT3wv2DbKwu3I691jheVivA51j13/bTKpUU3udFEyHqbvv33cDmhSi5CsfiEJo1 sLuG0ZWupmhJBmyeVBs2hceJeHsPa2ciUBo3fHa/LXSE55Wvsn4SzScBoYeEKbDCNg yXVwsudNWFnlsrDmQsR5WVP11ouXRXzoT8w0+dhe22v2tqkinnK5WM7JZdmcNl9iIL wkeyr6clHNf+w== Received: from [100.64.0.241] (unknown [100.64.0.241]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) (Authenticated sender: cristicc) by bali.collaboradmins.com (Postfix) with ESMTPSA id DA53517E040C; Fri, 12 Jun 2026 21:23:20 +0200 (CEST) Message-ID: Date: Fri, 12 Jun 2026 22:23:20 +0300 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v7 05/30] drm/display: hdmi_state_helper: Add ctx-aware hotplug helper for SCDC sync To: Maxime Ripard Cc: Maarten Lankhorst , Thomas Zimmermann , David Airlie , Simona Vetter , Andrzej Hajda , Neil Armstrong , Robert Foss , Laurent Pinchart , Jonas Karlman , Jernej Skrabec , Luca Ceresoli , Sandy Huang , =?UTF-8?Q?Heiko_St=C3=BCbner?= , Andy Yan , Daniel Stone , Dave Stevenson , =?UTF-8?Q?Ma=C3=ADra_Canal?= , Raspberry Pi Kernel Maintenance , kernel@collabora.com, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-rockchip@lists.infradead.org References: <20260602-dw-hdmi-qp-scramb-v7-0-445eb54ee1ed@collabora.com> <20260602-dw-hdmi-qp-scramb-v7-5-445eb54ee1ed@collabora.com> <20260611-refined-idealistic-sunfish-3df3a4@houat> Content-Language: en-US From: Cristian Ciocaltea In-Reply-To: <20260611-refined-idealistic-sunfish-3df3a4@houat> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit On 6/11/26 6:25 PM, Maxime Ripard wrote: > On Tue, Jun 02, 2026 at 01:44:05AM +0300, Cristian Ciocaltea wrote: >> Introduce drm_atomic_helper_connector_hdmi_hotplug_ctx(), a variant of >> drm_atomic_helper_connector_hdmi_hotplug() that accepts a >> drm_modeset_acquire_ctx. >> >> This enables SCDC status synchronization on hotplug events, which >> requires lock acquisition context for performing the CRTC reset >> triggered by drm_scdc_sync_status(). [...] >> diff --git a/include/drm/display/drm_hdmi_state_helper.h b/include/drm/display/drm_hdmi_state_helper.h >> index 13375bd0f4ae..75fedd4a3ba8 100644 >> --- a/include/drm/display/drm_hdmi_state_helper.h >> +++ b/include/drm/display/drm_hdmi_state_helper.h >> @@ -7,6 +7,7 @@ struct drm_atomic_commit; >> struct drm_connector; >> struct drm_connector_state; >> struct drm_display_mode; >> +struct drm_modeset_acquire_ctx; >> struct hdmi_audio_infoframe; >> >> enum drm_connector_status; >> @@ -24,6 +25,9 @@ int drm_atomic_helper_connector_hdmi_update_infoframes(struct drm_connector *con >> struct drm_atomic_commit *state); >> void drm_atomic_helper_connector_hdmi_hotplug(struct drm_connector *connector, >> enum drm_connector_status status); >> +int drm_atomic_helper_connector_hdmi_hotplug_ctx(struct drm_connector *connector, >> + enum drm_connector_status status, >> + struct drm_modeset_acquire_ctx *ctx); > > There's not a lot of users, so I'd prefer if we were just changing the > prototype of drm_atomic_helper_connector_hdmi_hotplug() Ack - that was actually my first attempt, but I later thought it would be more desirable to handle it incrementally. Cristian