From mboxrd@z Thu Jan  1 00:00:00 1970
From: Claude Code Review Bot <claude-review@example.com>
To: dri-devel-reviews@example.com
Subject: Claude review: drm/bridge: synopsys: dw-dp: Drop useless memory allocation
Date: Tue, 10 Mar 2026 11:47:02 +1000
Message-ID: <review-patch8-20260310-drm-rk-fixes-v2-8-645ecfb43f49@collabora.com>
In-Reply-To: <20260310-drm-rk-fixes-v2-8-645ecfb43f49@collabora.com>
References: <20260310-drm-rk-fixes-v2-0-645ecfb43f49@collabora.com> <20260310-drm-rk-fixes-v2-8-645ecfb43f49@collabora.com>
X-Mailer: Claude Code Patch Reviewer
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0

Patch Review

**Verdict: Good**

Removes a dead `devm_kzalloc()` whose result is immediately overwritten by `devm_drm_bridge_alloc()`:

```c
-	dp = devm_kzalloc(dev, sizeof(*dp), GFP_KERNEL);
-	if (!dp)
-		return ERR_PTR(-ENOMEM);
-
 	dp = devm_drm_bridge_alloc(dev, struct dw_dp, bridge, &dw_dp_bridge_funcs);
```

This was clearly a leftover from the conversion to `devm_drm_bridge_alloc()`. The removed allocation leaks on every successful call. Good cleanup.

---
Generated by Claude Code Patch Reviewer