From de1b77e7cd8c56733dab390a39b0f45b6959adb3 Mon Sep 17 00:00:00 2001 From: Leandro Ribeiro Date: Tue, 8 Sep 2020 10:05:23 -0300 Subject: [PATCH] drm-backend: do not set output->crtc to NULL in drm_output_init_planes() failure After commit "drm-backend: move code to init/deinit planes to specific functions" we have a specific function to init planes. As this function does not set output->crtc, it should not set it to NULL in case of failure. This is caller's responsibility. Signed-off-by: Leandro Ribeiro --- libweston/backend-drm/drm.c | 1 - 1 file changed, 1 deletion(-) diff --git a/libweston/backend-drm/drm.c b/libweston/backend-drm/drm.c index 1517e1da6..7a8bc2388 100644 --- a/libweston/backend-drm/drm.c +++ b/libweston/backend-drm/drm.c @@ -1664,7 +1664,6 @@ drm_output_init_planes(struct drm_output *output) if (!output->scanout_plane) { weston_log("Failed to find primary plane for output %s\n", output->base.name); - output->crtc = NULL; return -1; }