From mboxrd@z Thu Jan 1 00:00:00 1970 From: Claude Code Review Bot To: dri-devel-reviews@example.com Subject: Claude review: drm/mediatek: dsi: Store driver data before invoking mipi_dsi_host_register Date: Fri, 27 Feb 2026 13:42:15 +1000 Message-ID: In-Reply-To: <20260225094047.76780-1-l.scorcia@gmail.com> References: <20260225094047.76780-1-l.scorcia@gmail.com> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Overall Series Review Subject: drm/mediatek: dsi: Store driver data before invoking mipi_dsi_host= _register Author: Luca Leonardo Scorcia Patches: 2 Reviewed: 2026-02-27T13:42:15.167058 --- This is a single-patch fix for a real NULL pointer dereference crash in the= MediaTek DSI driver. The analysis is correct: `mipi_dsi_host_register()` c= an synchronously trigger `mtk_dsi_host_attach()` =E2=86=92 `component_add()= ` =E2=86=92 `component_bind_all()` =E2=86=92 `mtk_dsi_bind()`, which calls = `dev_get_drvdata()`. Since `platform_set_drvdata()` was called *after* `mip= i_dsi_host_register()`, `dsi` was NULL in `mtk_dsi_bind()`, causing the cra= sh. The fix is straightforward and correct. The patch has already been revi= ewed-by AngeloGioacchino Del Regno. The patch looks good and should be applied. --- Generated by Claude Code Patch Reviewer