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/amd/display: clean up reversed type specifiers in dml2_core Date: Thu, 30 Apr 2026 10:53:42 +1000 Message-ID: In-Reply-To: <20260429210209.42737-1-adri.vero.dev@gmail.com> References: <20260429210209.42737-1-adri.vero.dev@gmail.com> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Overall Series Review Subject: drm/amd/display: clean up reversed type specifiers in dml2_core Author: Adriano Vero Patches: 1 Reviewed: 2026-04-30T10:53:42.592152 --- This is a single-patch cleanup that fixes non-standard C type specifier ord= ering in the DML2 core code under `drivers/gpu/drm/amd/display/`. The chang= es are purely cosmetic: - `int unsigned` =E2=86=92 `unsigned int` - `long int unsigned` =E2=86=92 `unsigned long` The patch is **correct and straightforward**. All replacements are semantic= ally identical in C (the standard allows specifiers in any order, but canon= ical ordering is `unsigned int` / `unsigned long`). The cast targets (`unsi= gned long`) match the actual field type (`unsigned long`). The function sig= nature changes in the `.h` file are consistent with the `.c` file changes. The patch also takes the opportunity to reformat the `dml2_core_util_get_nu= m_active_pipes` declaration/definition to fit within 80 columns, which is a= reasonable companion change. **Verdict:** Looks good. No functional change, no issues found. --- Generated by Claude Code Patch Reviewer