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/rockchip: dw_hdmi_qp: Use local dev variable consistently in bind() Date: Tue, 28 Apr 2026 15:25:21 +1000 Message-ID: In-Reply-To: <20260426-dw-hdmi-qp-scramb-v5-8-d778e70c317b@collabora.com> References: <20260426-dw-hdmi-qp-scramb-v5-0-d778e70c317b@collabora.com> <20260426-dw-hdmi-qp-scramb-v5-8-d778e70c317b@collabora.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 This replaces `hdmi->dev`, `pdev->dev`, etc. with the `dev` parameter alrea= dy available in `dw_hdmi_qp_rockchip_bind()`. All three expressions (`dev`,= `hdmi->dev`, `&pdev->dev`) refer to the same `struct device`, so the chang= e is purely cosmetic. The ordering is correct =E2=80=94 `hdmi->dev` is set = to `dev` earlier in the function, so they're interchangeable. **No issues. Clean mechanical cleanup.** --- --- Generated by Claude Code Patch Reviewer