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 AUO B133HAN06.6 and BOE NV133FHM-N4F V8.0 Date: Sat, 16 May 2026 15:39:27 +1000 Message-ID: In-Reply-To: <20260511095914.1200201-1-wenst@chromium.org> References: <20260511095914.1200201-1-wenst@chromium.org> <20260511095914.1200201-1-wenst@chromium.org> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Patch Review **Sorted order =E2=80=94 correct.** The AUO entry `0xcb8f` is correctly pla= ced between `0xc9a8` and `0xcdba`. The BOE entry `0x0a25` is correctly plac= ed between `0x0a1b` and `0x0a36`. **Delay profiles =E2=80=94 appropriate.** The AUO entry uses `&delay_200_500_e50`, which is the standard delay profil= e used by nearly all surrounding AUO entries. Reasonable default for a data= sheet-only entry. The BOE entry uses `&delay_200_500_e50_po2e200`, which adds a `.powered_on_= to_enable =3D 200` field beyond the basic `delay_200_500_e50`. This delay s= truct is already used by several other BOE panels (e.g., `NV133FHM-N42` at = `0x0717`, `NV116WHM-N45` at `0x0754`). The choice of a delay with `powered_= on_to_enable` suggests the panel needs extra time after power-on before the= display can be enabled, and since this was tested on actual hardware, this= is a validated choice. **Commit message** is properly formatted and includes an honest caveat that= the AUO entry is datasheet-only. The Signed-off-by is present. **No issues.** This is a minimal, well-ordered data-only patch. Already mer= ged to drm-next. --- Generated by Claude Code Patch Reviewer