Claude review: drm: Use trace_invoke_##name() at guarded tracepoint call sites

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

Patch Review

Converts 6 call sites across 4 DRM files:

- **`amdgpu_cs.c:719`**: `trace_amdgpu_cs()` → `trace_invoke_amdgpu_cs()`. The guard is the early return at line 1001 (`if (!trace_amdgpu_cs_enabled()) return;`), which is a function-level guard in `trace_amdgpu_cs_ibs()`. ✓

- **`amdgpu_vm.c:1394`**: `trace_amdgpu_vm_bo_mapping()` → `trace_invoke_amdgpu_vm_bo_mapping()`, guarded by `trace_amdgpu_vm_bo_mapping_enabled()` ✓

- **`amdgpu_vm.c:2167`**: `trace_amdgpu_vm_bo_cs()` → `trace_invoke_amdgpu_vm_bo_cs()`. Looking at the upstream source, the function `amdgpu_vm_bo_trace_cs()` has a guard `if (!trace_amdgpu_vm_bo_cs_enabled()) return;` at the function entry. ✓

- **`amdgpu_dm.c:5190`**: `trace_amdgpu_dm_brightness()` → `trace_invoke_amdgpu_dm_brightness()`, guarded by `trace_amdgpu_dm_brightness_enabled()` ✓. Minor alignment change due to longer function name — looks fine.

- **`sched_entity.c:429`** and **`sched_entity.c:586`**: Both correctly guarded by their respective `_enabled()` checks. ✓

No issues with this patch.

---
Generated by Claude Code Patch Reviewer