From mboxrd@z Thu Jan 1 00:00:00 1970 From: Claude Code Review Bot To: dri-devel-reviews@example.com Subject: Claude review: Support for the Pixel 3a XL with the Tianma panel Date: Tue, 17 Feb 2026 15:51:43 +1000 Message-ID: In-Reply-To: <20260217000854.131242-1-mailingradian@gmail.com> References: <20260217000854.131242-1-mailingradian@gmail.com> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit MIME-Version: 1.0 Overall Series Review Subject: Support for the Pixel 3a XL with the Tianma panel Author: Richard Acayan Patches: 7 Reviewed: 2026-02-17T15:51:43.373818 --- This series adds support for the Google Pixel 3a XL (codename "bonito") with the Tianma/Novatek NT37700F panel variant. It consists of DT bindings additions (patches 1-2), a new panel driver (patch 3), preparatory DTS label additions (patch 4), refactoring of the existing sargo DTS into a common include (patch 5), and the new bonito-tianma DTS (patch 6). The overall approach is clean and follows established patterns. The DTS refactoring to extract common elements into a shared include is a sensible way to support the closely-related 3a and 3a XL devices. The panel driver is generated from the vendor's DSI command sequences using the well-known `linux-mdss-dsi-panel-driver-generator` tool and follows the modern `mipi_dsi_multi_context` pattern. There are a few issues to address. The most notable is a comment in patch 6 that says the opposite of what the file actually describes. There is also a value change in patch 5 that changes the Pixel 3a's touchscreen-x-mm from 62 to 63 without explanation. Both should be clarified. --- Generated by Claude Code Patch Reviewer