From mboxrd@z Thu Jan 1 00:00:00 1970 From: Claude Code Review Bot To: dri-devel-reviews@example.com Subject: Claude review: soc: qcom: ubwc: Add config for Milos Date: Tue, 05 May 2026 09:35:34 +1000 Message-ID: In-Reply-To: <20260501-milos-mdss-v3-5-58bfc58c0e13@fairphone.com> References: <20260501-milos-mdss-v3-0-58bfc58c0e13@fairphone.com> <20260501-milos-mdss-v3-5-58bfc58c0e13@fairphone.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 UBWC configuration for Milos: UBWC 4.0 encode/decode, level 2+3 swizzle, bank spread enabled, highest_bank_bit=15, macrotile mode. ```c /* TODO: highest_bank_bit = 14 for LP_DDR4 */ .highest_bank_bit = 15, ``` This TODO is worth tracking. If any Milos-based device ships with LPDDR4 instead of LPDDR5, this will need runtime detection or a separate config. For now this is fine since the known devices (Fairphone 6, Nothing Phone 3a) use LPDDR5. Has Reviewed-by from both Konrad and Dmitry. No blocking issues. --- Generated by Claude Code Patch Reviewer