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/panel-edp: Add panel for Surface Pro 12in Date: Sat, 16 May 2026 09:57:53 +1000 Message-ID: In-Reply-To: <9e749a3a483e4a3c684eac3ee6a4b241c94a0362.1778822464.git.harrison.vanderbyl@gmail.com> References: <9e749a3a483e4a3c684eac3ee6a4b241c94a0362.1778822464.git.harrison.vanderbyl@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 Adds a BOE NE120DRM-N28 panel entry using `delay_200_500_e50` timing: ```c + EDP_PANEL_ENTRY('B', 'O', 'E', 0x0cc9, &delay_200_500_e50, "NE120DRM-N28"), ``` The panel ID `0x0cc9` is correctly derived from the EDID hex dump provided in the commit message (bytes 8-9 of the EDID: `09 e5` is the manufacturer, `c9 0c` little-endian is product code `0x0cc9`). The entry is in the correct sorted position (between `0x0cb6` and `0x0cf2`). The timing choice `delay_200_500_e50` is described as tested for wake from sleep and hibernation. The commit message helpfully includes the raw EDID which is good practice for panel patches. **No issues.** --- --- Generated by Claude Code Patch Reviewer