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: mediatek: Add compatibles for MediaTek mt8167 Date: Tue, 24 Feb 2026 10:06:12 +1000 Message-ID: In-Reply-To: <66eafae30f9fe00b469e79d385c1ddd24d209475.1771863641.git.l.scorcia@gmail.com> References: <66eafae30f9fe00b469e79d385c1ddd24d209475.1771863641.git.l.scorcia@gmail.com> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit MIME-Version: 1.0 Patch Review No issues found. This patch adds mt8167 compatible strings to seven display block binding schemas (AAL, CCORR, dither, gamma, OVL, RDMA, WDMA). Each new entry is placed in the correct position within the binding schema -- using fallback compatibles where appropriate (e.g., mt8173 for AAL/gamma/wdma, mt8183 for ccorr/dither, mt2701 for rdma) and standalone for OVL (which gets its own driver data in patch 6). > + - enum: > + - mediatek,mt8167-disp-ccorr > + - mediatek,mt8365-disp-ccorr > - const: mediatek,mt8183-disp-ccorr The conversion from a single `const` to an `enum` for ccorr and wdma is the correct way to add another compatible that shares the same fallback. Already has Reviewed-by tags from both Krzysztof Kozlowski and AngeloGioacchino Del Regno. --- Generated by Claude Code Patch Reviewer