Claude review: media: iris: migrate to using global UBWC config

[Claude Code Review Bot <claude-review@example.com>]

Overall Series Review

Subject: media: iris: migrate to using global UBWC config
Author: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>
Patches: 8
Reviewed: 2026-05-07T12:40:16.438595

---

This is a clean, well-structured 7-patch series that migrates the Qualcomm Iris video decoder driver from maintaining its own UBWC (Universal Bandwidth Compression) configuration to using the centralized `QCOM_UBWC_CONFIG` registry. The motivation is sound: duplicating UBWC parameters (HBB, swizzle, bank spreading, etc.) per-driver is error-prone, especially since values like `highest_bank_bit` can depend on DDR type at runtime rather than being fixed constants.

The series is incrementally decomposed -- each patch replaces one parameter, which makes bisection and review straightforward. All patches carry Reviewed-by from Konrad Dybcio, Bryan O'Donoghue, and Dikshita Agarwal, plus a Tested-by from Wangao Wang. The cover letter notes this is compile-tested only by the author due to lack of Gen2 hardware, which is an honest disclosure.

The API mappings from the old per-driver values to the new central UBWC helpers are correct based on my verification of the `<linux/soc/qcom/ubwc.h>` header. One minor style concern noted below.

**Verdict: Series looks good. One minor nit on probe error handling.**

---

---
Generated by Claude Code Patch Reviewer