Claude review: drm/msm: Add sysprof accessors

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

Patch Review

Good refactor splitting the raw `refcount_read()` checks into two semantically distinct accessors: `msm_gpu_sysprof_no_perfcntr_zap()` and `msm_gpu_sysprof_no_ifpc()`. The comment on `msm_gpu_sysprof_no_ifpc()` clearly explains the planned future divergence.

**Issue**: In `a8xx_gpu.c` `hw_init()`, the change uses `msm_gpu_sysprof_no_perfcntr_zap()`:
```c
if (!ret && msm_gpu_sysprof_no_perfcntr_zap(gpu)) {
    ret = a6xx_gmu_set_oob(gmu, GMU_OOB_PERFCOUNTER_SET);
```
But the `a6xx_gpu.c` `hw_init()` uses `msm_gpu_sysprof_no_ifpc(gpu)` for the same `GMU_OOB_PERFCOUNTER_SET` call. Since this is about disabling IFPC, the a8xx path should also use `msm_gpu_sysprof_no_ifpc()` for consistency, even though the two are currently equivalent. Using the wrong accessor here means when the planned divergence happens, the a8xx path won't be updated correctly.

Also: adding `.sysprof_setup = a6xx_gmu_sysprof_setup` to `a8xx_gpu_funcs` seems like a bug fix that was previously missing -- worth mentioning in the commit message.

---
Generated by Claude Code Patch Reviewer