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: clk: qcom: gxclkctl: Use custom disable callback for gx_gdsc
Date: Tue, 28 Apr 2026 15:05:23 +1000
Message-ID: <review-patch2-20260427-gfx-clk-fixes-v2-2-797e54b3d464@oss.qualcomm.com>
In-Reply-To: <20260427-gfx-clk-fixes-v2-2-797e54b3d464@oss.qualcomm.com>
References: <20260427-gfx-clk-fixes-v2-0-797e54b3d464@oss.qualcomm.com> <20260427-gfx-clk-fixes-v2-2-797e54b3d464@oss.qualcomm.com>
X-Mailer: Claude Code Patch Reviewer
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0

Patch Review

**Author:** Taniya Das

Single-line change wiring the new callback:

```c
 	.pd = {
 		.name = "gx_clkctl_gx_gdsc",
 		.power_on = gdsc_gx_do_nothing_enable,
+		.power_off = gdsc_gx_disable,
 	},
```

This is the correct fix for the reported issue. It has proper `Reported-by`, `Closes`, `Fixes`, and `Reviewed-by` tags.

No issues found.

---

---
Generated by Claude Code Patch Reviewer