From mboxrd@z Thu Jan 1 00:00:00 1970 From: Claude Code Review Bot To: dri-devel-reviews@example.com Subject: Claude review: The Apple iMac11, 1 (late 2009) has an integrated ATI Mobility Radeon HD 4850. This machine suffers from a similar problem as the iMac10, 1 (late 2009) and the iMac11, 2 (mid 2010). This small patch fixes the issue on this machine. Date: Sat, 16 May 2026 15:30:57 +1000 Message-ID: In-Reply-To: <20260510190505.4810-2-gilles.risch@gmail.com> References: <20260510190505.4810-2-gilles.risch@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: The Apple iMac11, 1 (late 2009) has an integrated ATI Mobility Radeon HD 4850. This machine suffers from a similar problem as the iMac10, 1 (late 2009) and the iMac11, 2 (mid 2010). This small patch fixes the issue on this machine. Author: Gilles Risch Patches: 1 Reviewed: 2026-05-16T15:30:57.705528 --- This is a single-patch series that fixes a dark internal eDP panel on the Apple iMac11,1 (late 2009, ATI Mobility Radeon HD 4850 / RV770). The approach introduces a new `ASIC_IS_DCE31` macro to include the RV770 chip in code paths that previously only covered DCE 3.2 (RV730 and later), and adds the iMac11,1 to an existing Apple DMI quirk list for encoder selection. The fix addresses a real hardware issue and the core logic is sound, but there are several problems with the submission that need addressing before it can be accepted: 1. **Severely malformed subject line** -- the entire patch description has been placed in the Subject header, violating standard kernel patch formatting. 2. **DCE version naming is questionable** -- historical AMD documentation associates DCE 3.1 with the RS780/RS880 IGPs, not RV770. The RV770 is widely understood to be DCE 3.2, and the existing `ASIC_IS_DCE32` macro appears to have a longstanding off-by-one in the chip enum that excludes it. Creating a new macro with a debatable name rather than fixing the existing one warrants discussion. 3. **PLL behavior changes affect all RV770 systems**, not just Apple iMacs -- this is not called out in the commit message and represents a regression risk. 4. **Thin commit message** with no technical explanation of the root cause. --- --- Generated by Claude Code Patch Reviewer