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: writeback: Modify drm_writeback_queue_job helper Date: Mon, 25 May 2026 19:19:15 +1000 Message-ID: In-Reply-To: <20260522050313.1800378-4-suraj.kandpal@intel.com> References: <20260522050313.1800378-1-suraj.kandpal@intel.com> <20260522050313.1800378-4-suraj.kandpal@intel.com> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit MIME-Version: 1.0 Patch Review Changes `drm_writeback_queue_job()` to take `struct drm_connector *`. **Analysis:** Clean and correct. All callers updated. The header declaration has a minor inconsistency -- the parameter is named `wb_connector` rather than `connector`: ```c -void drm_writeback_queue_job(struct drm_writeback_connector *wb_connector, +void drm_writeback_queue_job(struct drm_connector *wb_connector, ``` It would be slightly better to rename to `connector` for consistency with the rest of the series, but this is cosmetic. --- Generated by Claude Code Patch Reviewer