Claude review: drm/xe: Handle PUNIT errors by requesting cold-reset recovery

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

Patch Review

This is the core functional patch and is quite small.

```c
+static void punit_error_handler(struct xe_device *xe)
+{
+	xe_device_set_wedged_method(xe, DRM_WEDGE_RECOVERY_COLD_RESET);
+	xe_device_declare_wedged(xe);
+}
```

Clean and correct — sets the recovery method, then declares the device wedged, which triggers the uevent to userspace.

The call site change is also correct:
```c
-			/* TODO: Add PUNIT error handling */
+			punit_error_handler(xe);
 			return XE_RAS_RECOVERY_ACTION_DISCONNECT;
```

The function is properly `static` (as requested in v4 review feedback) and the `xe_` prefix is removed (v5 feedback). 

**Note:** `xe_assert.h` and `xe_device_types.h` are added as includes but neither appears to be directly needed by the code added in this patch. The `xe_assert.h` include is unused. `xe_device_types.h` would already be pulled in transitively through `xe_device.h`. These may be leftover from a previous version.

```c
+#include "xe_assert.h"
+#include "xe_device_types.h"
```

---

---
Generated by Claude Code Patch Reviewer