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: verisilicon: DRM_VERISILICON_DC should depend on ARCH_THEAD Date: Fri, 27 Feb 2026 12:27:47 +1000 Message-ID: In-Reply-To: References: X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Patch Review **Commit message:** Clear and well-written. Correctly explains the rational= e (driver only supports DC8200 in TH1520), notes the Fixes tag, and mention= s that the dependency can be relaxed later. **Code change:** ```diff - depends on RISCV || COMPILE_TEST + depends on ARCH_THEAD || COMPILE_TEST ``` This is correct. `ARCH_THEAD` is defined under `arch/riscv/Kconfig.socs` an= d already implies `RISCV`, so the architecture constraint is preserved whil= e being more specific. The `COMPILE_TEST` alternative is properly kept so b= uild-testing coverage is not lost. The pattern matches what other T-HEAD peripheral drivers use throughout the= tree. For example, `drivers/firmware/Kconfig:216` and `drivers/clk/thead/K= config:5` both have the identical `depends on ARCH_THEAD || COMPILE_TEST` c= onstruct. **Fixes tag:** References `dbf21777caa8b8c8 ("drm: verisilicon: add a drive= r for Verisilicon display controllers")` which is appropriate =E2=80=94 thi= s is the commit that introduced the overly broad dependency. **No issues found.**=20 Reviewed-by worthy as-is. --- Generated by Claude Code Patch Reviewer