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: Wed, 27 May 2026 14:54:39 +1000 Message-ID: In-Reply-To: <20260526-hot-plug-passup-v10-1-f62351a9ea3e@collabora.com> References: <20260526-hot-plug-passup-v10-0-f62351a9ea3e@collabora.com> <20260526-hot-plug-passup-v10-1-f62351a9ea3e@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 **Verdict: Looks good.** Trivial documentation fix. The original kerneldoc for `epoch_counter` said: ```c /** @epoch_counter: used to detect any other changes in connector, besides status */ ``` But `epoch_counter` is also bumped on status changes (confirmed in `drm_probe_helper.c` where `drm_helper_probe_detect` changes `epoch_counter` alongside `connector->status`). The replacement is accurate: ```c /** * @epoch_counter: Used to detect changes in connector. Increased when * the connector, including its status, is changed. */ ``` Has `Fixes:` tag and `Reviewed-by:` from Daniel Stone. No issues. --- --- Generated by Claude Code Patch Reviewer