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: Drop memset from __drm_atomic_helper_bridge_state_init() Date: Thu, 04 Jun 2026 15:45:42 +1000 Message-ID: In-Reply-To: <20260530-drm-no-more-bridge-reset-v1-3-875d828d31bc@kernel.org> References: <20260530-drm-no-more-bridge-reset-v1-0-875d828d31bc@kernel.org> <20260530-drm-no-more-bridge-reset-v1-3-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 Removes the `memset(state, 0, sizeof(*state))` since all callers use `kzalloc`. Adds documentation: `@state is assumed to be zeroed.` Minor concern: this creates a subtle contract that callers must ensure. If someone later calls this on a non-zeroed buffer, they'll get silent corruption. The documentation addition mitigates this, and it matches how similar helpers work in DRM. No issues. --- Generated by Claude Code Patch Reviewer