From mboxrd@z Thu Jan  1 00:00:00 1970
From: Claude Code Review Bot <claude-review@example.com>
To: dri-devel-reviews@example.com
Subject: Claude review: fbdev: simplefb: Use __clk_disable_unprepare_counts_only()
Date: Thu, 28 May 2026 12:34:30 +1000
Message-ID: <review-patch3-20260527094811.116977-4-johannes.goede@oss.qualcomm.com>
In-Reply-To: <20260527094811.116977-4-johannes.goede@oss.qualcomm.com>
References: <20260527094811.116977-1-johannes.goede@oss.qualcomm.com> <20260527094811.116977-4-johannes.goede@oss.qualcomm.com>
X-Mailer: Claude Code Patch Reviewer
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0

Patch Review

**Straightforward and correct:**

```c
	if (par->clks_enabled)
		__clk_disable_unprepare_counts_only(par->clks[i]);
```

The `par->clks_enabled` guard ensures this only runs when clocks were succe=
ssfully enabled. This path is only for the normal teardown/handoff case. No=
 error-path concern here.

Minor: the commit message has a typo =E2=80=94 "non work display output" sh=
ould be "non working display output."

---
Generated by Claude Code Patch Reviewer