From mboxrd@z Thu Jan  1 00:00:00 1970
From: Claude Code Review Bot <claude-review@example.com>
To: dri-devel-reviews@example.com
Subject: Claude review: media: renesas: vsp1: Simplify iteration over format arrays
Date: Sat, 16 May 2026 14:29:50 +1000
Message-ID: <review-patch9-20260511235637.3468558-10-laurent.pinchart+renesas@ideasonboard.com>
In-Reply-To: <20260511235637.3468558-10-laurent.pinchart+renesas@ideasonboard.com>
References: <20260511235637.3468558-1-laurent.pinchart+renesas@ideasonboard.com> <20260511235637.3468558-10-laurent.pinchart+renesas@ideasonboard.com>
X-Mailer: Claude Code Patch Reviewer
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0

Patch Review

Introduces:
```c
#define vsp1_for_each_format(info, formats) \
    for (const struct vsp1_format_info *info =3D &formats[0]; \
         info < formats + ARRAY_SIZE(formats); ++info)
```

Used 6 times in `vsp1_get_format_info()` and `vsp1_get_format_info_by_index=
()`. The macro is file-scoped (static arrays only =E2=80=94 `ARRAY_SIZE` en=
forces this at compile time). The `const` qualifier on the iterator is appr=
opriate since the format tables are const.

**Clean and correct.**

---
Generated by Claude Code Patch Reviewer