Claude review: drm/sysfb: simpledrm: Use __clk_disable_unprepare_counts_only()

[Claude Code Review Bot <claude-review@example.com>]

Patch Review

**Normal teardown path — correct:**
The change in `simpledrm_device_release_clocks()` is the core fix and is correct. When simpledrm is being replaced by the real display driver, clocks should stay on:

```c
	for (i = 0; i < sdev->clk_count; ++i) {
		if (sdev->clks[i]) {
			__clk_disable_unprepare_counts_only(sdev->clks[i]);
			clk_put(sdev->clks[i]);
		}
	}
```

**Error path during init — questionable:**
The error cleanup in `simpledrm_device_init_clocks()` is also changed to counts-only:

```c
	while (i) {
		--i;
		if (sdev->clks[i]) {
			__clk_disable_unprepare_counts_only(sdev->clks[i]);
			clk_put(sdev->clks[i]);
		}
	}
```

This error path runs when `devm_add_action_or_reset()` or a later `of_clk_get()` / `clk_prepare_enable()` fails. In this scenario simpledrm never fully initialized and will not be providing a framebuffer — the clocks it enabled should arguably be properly disabled via `clk_disable_unprepare()`, since simpledrm is aborting, not handing off to a real driver. Using counts-only here means clocks that simpledrm actually turned on (if the firmware hadn't already) will be left running with zero refcount.

Consider keeping `clk_disable_unprepare()` in the error path, since the "don't touch hardware during handoff" rationale doesn't apply when init is failing.

---
Generated by Claude Code Patch Reviewer