[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-Id: <20260106164247.472544-3-sean.anderson@linux.dev>
Date: Tue, 6 Jan 2026 11:42:45 -0500
From: Sean Anderson <sean.anderson@...ux.dev>
To: Laurent Pinchart <laurent.pinchart@...asonboard.com>,
Tomi Valkeinen <tomi.valkeinen@...asonboard.com>,
dri-devel@...ts.freedesktop.org
Cc: Simona Vetter <simona@...ll.ch>,
Thomas Zimmermann <tzimmermann@...e.de>,
linux-kernel@...r.kernel.org,
Maxime Ripard <mripard@...nel.org>,
David Airlie <airlied@...il.com>,
linux-arm-kernel@...ts.infradead.org,
Michal Simek <michal.simek@....com>,
Anatoliy Klymenko <anatoliy.klymenko@....com>,
Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>,
Mike Looijmans <mike.looijmans@...ic.nl>,
Sean Anderson <sean.anderson@...ux.dev>
Subject: [PATCH v2 2/4] drm: zynqmp: Check property creation status
Make sure to return an error in the event that we can't create our
properties.
Fixes: 650f12042b85 ("drm: xlnx: zynqmp_dpsub: Add global alpha support")
Fixes: 8c772f0b2b8e ("drm: xlnx: zynqmp_dpsub: Expose plane ordering to userspace")
Signed-off-by: Sean Anderson <sean.anderson@...ux.dev>
Reviewed-by: Anatoliy Klymenko <anatoliy.klymenko@....com>
Reviewed-by: Thomas Zimmermann <tzimmermann@...e.de>
---
(no changes since v1)
drivers/gpu/drm/xlnx/zynqmp_kms.c | 12 +++++++++---
1 file changed, 9 insertions(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/xlnx/zynqmp_kms.c b/drivers/gpu/drm/xlnx/zynqmp_kms.c
index 02f3a7d78cf8..816bea4ea986 100644
--- a/drivers/gpu/drm/xlnx/zynqmp_kms.c
+++ b/drivers/gpu/drm/xlnx/zynqmp_kms.c
@@ -175,9 +175,15 @@ static int zynqmp_dpsub_create_planes(struct zynqmp_dpsub *dpsub)
drm_plane_helper_add(plane, &zynqmp_dpsub_plane_helper_funcs);
- drm_plane_create_zpos_immutable_property(plane, i);
- if (i == ZYNQMP_DPSUB_LAYER_GFX)
- drm_plane_create_alpha_property(plane);
+ ret = drm_plane_create_zpos_immutable_property(plane, i);
+ if (ret)
+ return ret;
+
+ if (i == ZYNQMP_DPSUB_LAYER_GFX) {
+ ret = drm_plane_create_alpha_property(plane);
+ if (ret)
+ return ret;
+ }
}
return 0;
--
2.35.1.1320.gc452695387.dirty
Powered by blists - more mailing lists