From mboxrd@z Thu Jan 1 00:00:00 1970 From: Claude Code Review Bot To: dri-devel-reviews@example.com Subject: Claude review: media: renesas: vsp1: Avoid forward function declaration Date: Sat, 16 May 2026 14:29:48 +1000 Message-ID: In-Reply-To: <20260511235637.3468558-2-laurent.pinchart+renesas@ideasonboard.com> References: <20260511235637.3468558-1-laurent.pinchart+renesas@ideasonboard.com> <20260511235637.3468558-2-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 Pure code motion =E2=80=94 `vsp1_du_pipeline_configure()` is moved above it= s first call site, eliminating the forward declaration. The remaining forwa= rd declaration for `vsp1_du_pipeline_setup_inputs` is kept since it has a m= utual recursion with `vsp1_du_pipeline_setup_brx`. No functional change. Straightforward and correct. **Reviewed-by worthy.** --- Generated by Claude Code Patch Reviewer