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/bridge: samsung-dsim: enable MFLUSH_VS for Exynos 7870 DSIM Date: Sat, 16 May 2026 08:42:29 +1000 Message-ID: In-Reply-To: <20260516-exynos-dsim-fixes-v2-1-db9bf96ae641@disroot.org> References: <20260516-exynos-dsim-fixes-v2-0-db9bf96ae641@disroot.org> <20260516-exynos-dsim-fixes-v2-1-db9bf96ae641@disroot.org> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Patch Review **Verdict: Looks good.** This patch fixes a regression introduced in v6.17 by commit `a36c533ad3e1`,= which assumed MFLUSH_VS is active-low across all variants. On Exynos 7870 = the bit is active-high, so the bit must be explicitly set. The fix is minimal and correctly scoped =E2=80=94 it adds a hw_type check i= nside the existing video-mode configuration block: ```c if (dsi->plat_data->hw_type =3D=3D DSIM_TYPE_EXYNOS7870) reg |=3D DSIM_MFLUSH_VS; ``` The downstream kernel reference is provided and the patch carries both Test= ed-by and stable Cc tags, which is appropriate for a regression fix. The co= mment is adequate. No issues found. --- --- Generated by Claude Code Patch Reviewer