Thread (12 messages) 12 messages, 5 authors, 2026-01-19

Re: [PATCH v2 2/5] accel/thames: Add driver for the C7x DSPs in TI SoCs

From: Markus Elfring <hidden>
Date: 2026-01-14 17:02:14
Also in: dri-devel, linux-devicetree, linux-doc, linux-media, lkml

quoted hunk ↗ jump to hunk
+++ b/drivers/accel/thames/thames_drv.c
@@ -0,0 +1,155 @@
+static int thames_open(struct drm_device *dev, struct drm_file *file)
+{
+	if (!try_module_get(THIS_MODULE))
+		return -EINVAL;
+
+	thames_priv = kzalloc(sizeof(*thames_priv), GFP_KERNEL);
+	if (!thames_priv) {
+		ret = -ENOMEM;
+		goto err_put_mod;
+	}
…

I find the identifiers “ret” and “err_put_mod” redundant here because the desired
exception handling can be specified in this if branch directly.

Regards,
Markus
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help