]> git.ipfire.org Git - thirdparty/linux.git/commitdiff
drm/tidss: crtc: Change variable name
authorMaxime Ripard <mripard@kernel.org>
Tue, 2 Sep 2025 08:32:48 +0000 (10:32 +0200)
committerMaxime Ripard <mripard@kernel.org>
Tue, 30 Sep 2025 09:56:34 +0000 (11:56 +0200)
The tidss_crtc_reset() function stores a pointer to struct
tidss_crtc_state in a variable called tcrtc, while it uses tcrtc as a
pointer to struct tidss_crtc in the rest of the driver.

This is confusing, so let's change the variable name.

Reviewed-by: Thomas Zimmermann <tzimmermann@suse.de>
Link: https://lore.kernel.org/r/20250902-drm-state-readout-v1-20-14ad5315da3f@kernel.org
Signed-off-by: Maxime Ripard <mripard@kernel.org>
Link: https://lore.kernel.org/r/20250902-drm-state-readout-v1-20-14ad5315da3f@kernel.org
drivers/gpu/drm/tidss/tidss_crtc.c

index 091f82c86f53bc76c572de4723746af2e35ce1c1..db7c5e4225e6247047087a35a2e6422950fc0111 100644 (file)
@@ -345,20 +345,20 @@ static void tidss_crtc_disable_vblank(struct drm_crtc *crtc)
 
 static void tidss_crtc_reset(struct drm_crtc *crtc)
 {
-       struct tidss_crtc_state *tcrtc;
+       struct tidss_crtc_state *tstate;
 
        if (crtc->state)
                __drm_atomic_helper_crtc_destroy_state(crtc->state);
 
        kfree(crtc->state);
 
-       tcrtc = kzalloc(sizeof(*tcrtc), GFP_KERNEL);
-       if (!tcrtc) {
+       tstate = kzalloc(sizeof(*tstate), GFP_KERNEL);
+       if (!tstate) {
                crtc->state = NULL;
                return;
        }
 
-       __drm_atomic_helper_crtc_reset(crtc, &tcrtc->base);
+       __drm_atomic_helper_crtc_reset(crtc, &tstate->base);
 }
 
 static struct drm_crtc_state *tidss_crtc_duplicate_state(struct drm_crtc *crtc)