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:22:52 +1000 Message-ID: In-Reply-To: <20260422-hot-plug-passup-v9-1-aef804255986@collabora.com> References: <20260422-hot-plug-passup-v9-0-aef804255986@collabora.com> <20260422-hot-plug-passup-v9-1-aef804255986@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.** This is a straightforward documentation fix. The old comment: ```c /** @epoch_counter: used to detect any other changes in connector, besides status */ ``` is replaced with: ```c /** * @epoch_counter: Used to detect changes in connector. Increased when * the connector, including its status, is changed. */ ``` The fix is correct. The `epoch_counter` is bumped in `drm_connector_set_status_locked()` (among other places), so it very much does track status changes. The Fixes tag referencing the original commit `5186421cbfe2` is appropriate since the doc was wrong from the start. No issues. --- Generated by Claude Code Patch Reviewer