Re: [PATCH v3 00/37] drm/connector: Convert all drivers to atomic_create_state and remove reset

From: Maxime Ripard <mripard@kernel.org>
To: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>,
	 Thomas Zimmermann <tzimmermann@suse.de>,
	David Airlie <airlied@gmail.com>,
	 Simona Vetter <simona@ffwll.ch>
Cc: dri-devel@lists.freedesktop.org,
	"Christian König" <christian.koenig@amd.com>,
	"Alex Deucher" <alexander.deucher@amd.com>,
	amd-gfx@lists.freedesktop.org,
	"Russell King" <linux@armlinux.org.uk>,
	"Alim Akhtar" <alim.akhtar@samsung.com>,
	"Inki Dae" <inki.dae@samsung.com>,
	"Krzysztof Kozlowski" <krzk@kernel.org>,
	"Kyungmin Park" <kyungmin.park@samsung.com>,
	"Peter Griffin" <peter.griffin@linaro.org>,
	"Seung-Woo Kim" <sw0312.kim@samsung.com>,
	linux-samsung-soc@vger.kernel.org,
	"Alison Wang" <alison.wang@nxp.com>,
	"Stefan Agner" <stefan@agner.ch>,
	"Huacai Chen" <chenhuacai@kernel.org>,
	"Icenowy Zheng" <zhengxingda@iscas.ac.cn>,
	"Jianmin Lv" <lvjianmin@loongson.cn>,
	"Mingcong Bai" <jeffbai@aosc.io>,
	"Qianhai Wu" <wuqianhai@loongson.cn>,
	"Xi Ruoyao" <xry111@xry111.site>,
	"Geert Uytterhoeven" <geert+renesas@glider.be>,
	linux-renesas-soc@vger.kernel.org,
	"Marcus Folkesson" <marcus.folkesson@gmail.com>,
	"Alain Volmat" <alain.volmat@foss.st.com>,
	"Raphael Gallais-Pou" <rgallaispou@gmail.com>,
	"Chen-Yu Tsai" <wens@kernel.org>,
	"Samuel Holland" <samuel@sholland.org>,
	linux-sunxi@lists.linux.dev,
	"Jonathan Hunter" <jonathanh@nvidia.com>,
	"Mikko Perttunen" <mperttunen@nvidia.com>,
	"Thierry Reding" <thierry.reding@kernel.org>,
	linux-tegra@vger.kernel.org,
	"Alexey Brodkin" <abrodkin@synopsys.com>,
	"Alex Lanzano" <lanzano.alex@gmail.com>,
	"Chia-I Wu" <olvaffe@gmail.com>,
	"David Airlie" <airlied@redhat.com>,
	"Dmitry Osipenko" <dmitry.osipenko@collabora.com>,
	"Gerd Hoffmann" <kraxel@redhat.com>,
	"Gurchetan Singh" <gurchetansingh@chromium.org>,
	virtualization@lists.linux.dev,
	"Haneen Mohammed" <hamohammed.sa@gmail.com>,
	"Louis Chauvet" <louis.chauvet@bootlin.com>,
	"Melissa Wen" <melissa.srw@gmail.com>,
	"Oleksandr Andrushchenko" <oleksandr_andrushchenko@epam.com>,
	"Jerome Brunet" <jbrunet@baylibre.com>,
	"Kevin Hilman" <khilman@baylibre.com>,
	"Martin Blumenstingl" <martin.blumenstingl@googlemail.com>,
	linux-amlogic@lists.infradead.org,
	"Harry Wentland" <harry.wentland@amd.com>,
	"Leo Li" <sunpeng.li@amd.com>,
	"Rodrigo Siqueira" <siqueira@igalia.com>,
	"Jani Nikula" <jani.nikula@linux.intel.com>,
	"Joonas Lahtinen" <joonas.lahtinen@linux.intel.com>,
	"Rodrigo Vivi" <rodrigo.vivi@intel.com>,
	"Tvrtko Ursulin" <tursulin@ursulin.net>,
	intel-gfx@lists.freedesktop.org, intel-xe@lists.freedesktop.org,
	"Danilo Krummrich" <dakr@kernel.org>,
	"Lyude Paul" <lyude@redhat.com>,
	nouveau@lists.freedesktop.org
Subject: Re: [PATCH v3 00/37] drm/connector: Convert all drivers to atomic_create_state and remove reset
Date: Thu, 5 Nov 2026 10:15:08 +0100	[thread overview]
Message-ID: <auxJBtexUg6oHcs9@houat> (raw)
In-Reply-To: <20261030-drm-no-more-connector-reset-v3-0-30e71d92c327@kernel.org>

[-- Attachment #1: Type: text/plain, Size: 1400 bytes --]

On Fri, Oct 30, 2026 at 04:13:30PM +0100, Maxime Ripard wrote:
> This is a follow-up to the plane and CRTC reset removal series, and
> part of a larger effort to remove the reset hook from all KMS objects.
> 
> The connector reset hook is overloaded: it is called both at probe
> time to create the initial software state and during suspend/resume to
> reset hardware and software state. These two roles have different
> requirements, and the reset hook is not fallible, making error
> handling difficult for the initial state allocation path.
> 
> While reset has the semantics to reset both the software and hardware
> state, the vast majority of implementations and all the helpers only
> reset the software state, making them equivalent to
> atomic_create_state in practice. The atomic_create_state hook makes
> this explicit: it only allocates and initializes a pristine state
> without any side effect, and returns the state pointer or an ERR_PTR
> on failure.
> 
> This series converts all connector drivers tree-wide from the reset hook
> to atomic_create_state, then removes the reset hook from struct
> drm_connector_funcs entirely. The conversions were done using a
> combination of Coccinelle semantic patches, mechanical one-line
> replacements and manual adjustments.
> 
> Signed-off-by: Maxime Ripard <mripard@kernel.org>

Applied to drm-misc-next

Maxime

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 273 bytes --]