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/atomic-state-helper: Add drm_atomic_helper_bridge_create_state() Date: Thu, 04 Jun 2026 15:45:42 +1000 Message-ID: In-Reply-To: <20260530-drm-no-more-bridge-reset-v1-5-875d828d31bc@kernel.org> References: <20260530-drm-no-more-bridge-reset-v1-0-875d828d31bc@kernel.org> <20260530-drm-no-more-bridge-reset-v1-5-875d828d31bc@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 Adds the new helper, which is functionally identical to `drm_atomic_helper_bridge_reset()`. Clean implementation. No issues. ### PATCHES 6-71: Mechanical driver conversions All follow the same pattern -- changing one line in the `drm_bridge_funcs` initializer: ```c - .atomic_reset = drm_atomic_helper_bridge_reset, + .atomic_create_state = drm_atomic_helper_bridge_create_state, ``` Drivers with multiple bridge_funcs structs (tc358767, msm dp, verisilicon) have 2 instances each -- all correctly converted. These are all straightforward, correct, mechanical changes. No issues with any individual conversion. --- Generated by Claude Code Patch Reviewer