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/connector: Fix epoch_counter docs to reflect reality Date: Thu, 23 Apr 2026 07:36:45 +1000 Message-ID: In-Reply-To: <20260422-hot-plug-passup-v8-1-5cfae6ba4119@collabora.com> References: <20260422-hot-plug-passup-v8-0-5cfae6ba4119@collabora.com> <20260422-hot-plug-passup-v8-1-5cfae6ba4119@collabora.com> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit MIME-Version: 1.0 Patch Review **Reviewed-by worthy.** This is a straightforward documentation fix. The old doc says: ```c /** @epoch_counter: used to detect any other changes in connector, besides status */ ``` The new doc says: ```c /** * @epoch_counter: Used to detect changes in connector. Increased when * the connector, including its status, is changed. */ ``` This is correct. Looking at the code in `drm_probe_helper.c`, `check_connector_changed()` calls `drm_helper_probe_detect()` which updates the epoch counter when status changes, so the old "besides status" wording was wrong since introduction. The `Fixes:` tag pointing to the original commit is appropriate for a doc fix. No issues. --- --- Generated by Claude Code Patch Reviewer