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/mode-config: Mention drm_mode_config_reset() culprits Date: Sun, 22 Mar 2026 03:32:25 +1000 Message-ID: In-Reply-To: <20260320-drm-mode-config-init-v2-3-c63f1134e76c@kernel.org> References: <20260320-drm-mode-config-init-v2-0-c63f1134e76c@kernel.org> <20260320-drm-mode-config-init-v2-3-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 Good documentation addition. Notes that `drm_private_obj` structures are expected to be stable across suspend/resume and that `drm_mode_config_reset()` doesn't affect them. Minor nit: the doc uses `@drm_private_obj` and `@drm_mode_config_reset()` with `@` prefix, but these aren't function parameters -- kernel-doc convention uses `&` for cross-references to structs/functions. Should be `&drm_private_obj` and `&drm_mode_config_reset()`: ``` + * Note that @drm_private_obj structures are expected to be stable across + * suspend/resume cycles, and @drm_mode_config_reset() does not affect these ``` Already has Laurent's R-b. --- Generated by Claude Code Patch Reviewer