From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-alma10-1.taild15c8.ts.net [100.103.45.18]) (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 6BC0E49A3C2 for ; Fri, 16 Oct 2026 12:35:58 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=100.103.45.18 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1792154163; cv=none; b=jn4EGmxqdF+3SGNA03iQxLIse4spfSt7cytvGd+AcJJ22zq2kHRNnBZOkwoz2JqZgklE7YkscRbm2yUWQaCDsagY9KZIZqGhQoXgiJUYkDfUfEf76Z6qFJruznOxmjp2622cVYapZgGv/BJc9KEqIh2FZ/Jsukx6SoOyQu+sdvs= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1792154163; c=relaxed/simple; bh=x1FsBrPCSkdZJtG+HdqZMYtb7qe84QtVBUwI7EaPUHw=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=EaOPnG4WK8/Z7STBNhv3UJdJI2kJQqYkSZpKi8Znx/LrO8BB5sf0TfnJn5vw/vdv1hxMiukFV/nIcyYgi8ZxD0oE6hrNC1FboGZGTpiswDKs+N6/LvhQ+jEl4NEm91KyJcCRMTb2egjNH7CAx4s8YYriKrxD7T1dUBa0TmnkSOA= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=lG6tifYO; arc=none smtp.client-ip=100.103.45.18 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="lG6tifYO" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 6D2911F000C4; Fri, 16 Oct 2026 12:35:57 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1792154157; bh=K5ujFTPNI8d5JuoGTXJQu2ch1Jer/YBRO4cp15Sdgus=; h=Date:From:To:Cc:Subject:References:In-Reply-To; b=lG6tifYOh0hiaMRrjZ+4kvFGvg/AC/9H8e5J+neAbbWsfzNugPRd7OVLLfExYahqb 86RGR5ufsE5WVew218k2GvQk7YjDSVXX+agN//lFon5UvI2+z4y5o4PPzsYaHN3Xy+ iZi4RFuGMtYiPIcTSD1OQtr/C+2nTncjFIRpyNdNE0wLbtN5GSX20EfOiAEx9AbpYX w5fbmpdF0T3OwqCZBiNYGmYiazIrKzTdZmSPoEJDcTbJNoEjQIw/IzxFrpHXGyAOPs wq6EZWdVr8EQ9G2nLqgS7swNZMwrHwF/EZVRJvBNjNikea58vqgyngB7xmOvzxlTlq Eiy1bzmmnQNIg== Date: Fri, 16 Oct 2026 14:35:54 +0200 From: Maxime Ripard To: =?utf-8?B?TWHDrXJh?= Canal Cc: Maarten Lankhorst , Thomas Zimmermann , David Airlie , Simona Vetter , dri-devel@lists.freedesktop.org, Chen-Yu Tsai , Dave Stevenson , Raspberry Pi Kernel Maintenance , Samuel Holland , linux-sunxi@lists.linux.dev Subject: Re: [PATCH 56/75] drm/atomic-state-helper: Rename connector_tv_reset to state_init Message-ID: References: <20261014-drm-no-more-connector-reset-v1-0-586c9c026823@kernel.org> <20261014-drm-no-more-connector-reset-v1-56-586c9c026823@kernel.org> <1ae560be-63a2-425e-bd47-ed279448eeec@igalia.com> Precedence: bulk X-Mailing-List: linux-sunxi@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha384; protocol="application/pgp-signature"; boundary="ag4zz2wpi4nkqtta" Content-Disposition: inline In-Reply-To: <1ae560be-63a2-425e-bd47-ed279448eeec@igalia.com> --ag4zz2wpi4nkqtta Content-Type: text/plain; protected-headers=v1; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Subject: Re: [PATCH 56/75] drm/atomic-state-helper: Rename connector_tv_reset to state_init MIME-Version: 1.0 Hi Ma=EDra, On Thu, Oct 15, 2026 at 07:56:05AM -0300, Ma=EDra Canal wrote: > Hi Maxime, >=20 > On 14/10/26 14:27, Maxime Ripard wrote: > > Recent commits renamed other reset helpers to use the state_init > > suffix when they only initialize properties rather than allocate or > > update the object state itself. > >=20 > > drm_atomic_helper_connector_tv_reset() falls in the same category: > > it initializes the analog TV properties from the kernel command line > > defaults without allocating any state. It also reads the connector > > state through connector->state, coupling it to the reset path. > >=20 > > Rename it to drm_atomic_helper_connector_tv_state_init() and pass > > the connector state explicitly as a parameter. This decouples it > > from the reset hook and prepares for the conversion to > > atomic_create_state where connector->state is not yet assigned. > >=20 > > Signed-off-by: Maxime Ripard > > --- > > Cc: "Ma=EDra Canal" > > Cc: Chen-Yu Tsai > > Cc: Dave Stevenson > > Cc: Raspberry Pi Kernel Maintenance > > Cc: Samuel Holland > > Cc: linux-sunxi@lists.linux.dev > > --- > > drivers/gpu/drm/drm_atomic_state_helper.c | 11 ++++++----- > > drivers/gpu/drm/sun4i/sun4i_tv.c | 2 +- > > drivers/gpu/drm/vc4/vc4_vec.c | 2 +- > > include/drm/drm_atomic_state_helper.h | 3 ++- > > 4 files changed, 10 insertions(+), 8 deletions(-) > >=20 > > diff --git a/drivers/gpu/drm/drm_atomic_state_helper.c b/drivers/gpu/dr= m/drm_atomic_state_helper.c > > index 6238066fd5d6..1424c4873251 100644 > > --- a/drivers/gpu/drm/drm_atomic_state_helper.c > > +++ b/drivers/gpu/drm/drm_atomic_state_helper.c > > @@ -499,20 +499,21 @@ void drm_atomic_helper_connector_tv_margins_state= _init(struct drm_connector_stat > > state->tv.margins.bottom =3D cmdline->tv_margins.bottom; > > } > > EXPORT_SYMBOL(drm_atomic_helper_connector_tv_margins_state_init); > > /** > > - * drm_atomic_helper_connector_tv_reset - Resets Analog TV connector p= roperties > > + * drm_atomic_helper_connector_tv_state_init - Initializes Analog TV c= onnector properties > > + * @state: the DRM State object > > * @connector: DRM connector > > * > > - * Resets the analog TV properties attached to a connector > > + * Initializes the analog TV properties attached to a connector > > */ > > -void drm_atomic_helper_connector_tv_reset(struct drm_connector *connec= tor) > > +void drm_atomic_helper_connector_tv_state_init(struct drm_connector_st= ate *state, > > + struct drm_connector *connector) > > { > > struct drm_device *dev =3D connector->dev; > > struct drm_cmdline_mode *cmdline =3D &connector->cmdline_mode; > > - struct drm_connector_state *state =3D connector->state; > > struct drm_property *prop; > > uint64_t val; > > prop =3D dev->mode_config.tv_mode_property; > > if (prop) > > @@ -572,11 +573,11 @@ void drm_atomic_helper_connector_tv_reset(struct = drm_connector *connector) > > state->tv.hue =3D val; > > drm_atomic_helper_connector_tv_margins_state_init(connector->state, >=20 > Shouldn't it be s/connector->state/state here? Yes, it should be. It's going to be fixed in the next version, thanks! Maxime --ag4zz2wpi4nkqtta Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iJUEABMJAB0WIQTkHFbLp4ejekA/qfgnX84Zoj2+dgUCatIaKQAKCRAnX84Zoj2+ dhc2AYDikiyvZ7pA69z4NLHxRQqwbwh3GFryw/2J6W65qf6Bb0XH4sah0UV/tbuZ +qRy8FgBgJoo85c7y2zvSICho+qjJzX/b4MPZIVZygwSOIr3Ld5k641amh1nBZzc HTwG2TLutg== =9+cI -----END PGP SIGNATURE----- --ag4zz2wpi4nkqtta--