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/colorop: Create drm_atomic_helper_colorop_create_state() Date: Sun, 22 Mar 2026 03:32:26 +1000 Message-ID: In-Reply-To: <20260320-drm-mode-config-init-v2-5-c63f1134e76c@kernel.org> References: <20260320-drm-mode-config-init-v2-0-c63f1134e76c@kernel.org> <20260320-drm-mode-config-init-v2-5-c63f1134e76c@kernel.org> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit MIME-Version: 1.0 Patch Review Creates the new `drm_atomic_helper_colorop_create_state()` helper. Uses `kzalloc_obj(*state)` for allocation, calls `__drm_colorop_state_init()`, returns `ERR_PTR(-ENOMEM)` on failure. Pattern is consistent with the later patches and the existing `drm_private_obj` pattern. Note: colorop doesn't get an `atomic_create_state` callback in the `drm_colorop_funcs` -- the function is called directly. This makes sense since colorops have no driver-specific subclassing currently. --- Generated by Claude Code Patch Reviewer