From mboxrd@z Thu Jan 1 00:00:00 1970 From: Claude Code Review Bot To: dri-devel-reviews@example.com Subject: Claude review: backlight: ktd2801: enable BL_CORE_SUSPENDRESUME Date: Tue, 31 Mar 2026 17:51:56 +1000 Message-ID: In-Reply-To: <20260328-ktd2801-pm-fix-v1-1-007cb103faeb@dujemihanovic.xyz> References: <20260328-ktd2801-pm-fix-v1-1-007cb103faeb@dujemihanovic.xyz> 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: backlight: ktd2801: enable BL_CORE_SUSPENDRESUME Author: =?utf-8?q?Duje_Mihanovi=C4=87?= Patches: 1 Reviewed: 2026-03-31T17:51:56.057017 --- This is a single-patch series that adds `BL_CORE_SUSPENDRESUME` to the KTD2801 backlight driver's ops. The fix is straightforward, correct, and well-motivated. The backlight core's suspend/resume handling will call `update_status` with brightness=0 on suspend and restore brightness on resume, which is the standard mechanism for backlight power management. The driver's `ktd2801_update_status` already handles the blank/unblank transitions correctly via `expresswire_power_off` / `expresswire_enable`, so this flag is all that's needed. **Verdict: Looks good.** This is a minimal, correct fix. --- Generated by Claude Code Patch Reviewer