From mboxrd@z Thu Jan 1 00:00:00 1970 From: Claude Code Review Bot To: dri-devel-reviews@example.com Subject: Claude review: dt-bindings: display: panel: Add Samsung S6E8FC0-M1906F9 Date: Sun, 22 Mar 2026 03:41:50 +1000 Message-ID: In-Reply-To: <20260320-panel-patches-v7-1-3eaefc4b3878@gmail.com> References: <20260320-panel-patches-v7-0-3eaefc4b3878@gmail.com> <20260320-panel-patches-v7-1-3eaefc4b3878@gmail.com> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Patch Review This patch extends the existing `samsung,s6e8aa5x01-ams561ra01.yaml` bindin= g to add the new `samsung,s6e8fc0-m1906f9` compatible, converting `const` t= o `enum`. **Looks good.** Already has `Reviewed-by: Krzysztof Kozlowski`. The approac= h of reusing an existing binding with a compatible panel interface is appro= priate. One minor observation: the YAML title still says "Samsung AMS561RA01 panel = with S6E8AA5X01 controller" =E2=80=94 it might be worth updating the title = to be more generic now that the binding covers two different panels, but th= is is purely cosmetic and not blocking. --- --- Generated by Claude Code Patch Reviewer