Claude review: drm/panthor: Replace the panthor_irq macro machinery by inline helpers

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

Patch Review

**R-b: Steven Price**

Large but mechanical refactoring. The `PANTHOR_IRQ_HANDLER(__name, __handler)` macro is replaced by:
- `panthor_irq_default_raw_handler()` — shared raw handler
- `panthor_irq_default_threaded_handler()` — takes a callback
- `panthor_irq_suspend()` / `panthor_irq_resume()` / `panthor_irq_enable_events()` / `panthor_irq_disable_events()` — no longer name-stamped
- `panthor_irq_request()` — replaces `panthor_request_xxx_irq()`

The IRQ handler callbacks change signature from `void (*)(struct panthor_device *, u32)` to `void (*)(struct panthor_irq *, u32)`. Each call site (`panthor_fw.c`, `panthor_gpu.c`, `panthor_mmu.c`, `panthor_pwr.c`) is updated to extract `ptdev` from `pirq->ptdev`. This is a sensible change since `pirq` is the natural data context.

One note: the `panthor_irq_request()` adds a `devm_kasprintf()` call before `panthor_irq_resume()`. The v2 changelog notes this was fixed per review feedback (moved before `panthor_irq_resume()`). Good.

The forward declaration that existed for `panthor_mmu_irq_handler` / `PANTHOR_IRQ_HANDLER(mmu, ...)` in `panthor_mmu.c` is cleanly eliminated since the threaded handler wrapper is now a small static function defined after the actual handler.

No concerns.

---

---
Generated by Claude Code Patch Reviewer