Claude review: drm/xe/hw_error: Use HW_ERR prefix in log

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

Patch Review

**Verdict: Correct change, questionable Fixes tag.**

The change itself is straightforward and correct — `HW_ERR` is defined in `include/linux/printk.h` as `"[Hardware Error]: "` and is the standard kernel convention for hardware error reporting. The string concatenation with the format literal is fine:

```c
drm_warn(&xe->drm, HW_ERR "%s %s detected\n", name, severity_str);
```

**On the Fixes tag:** The referenced commit `01aab7e1c9d4` ("drm/xe/xe_hw_error: Add support for PVC SoC errors") simply didn't use the `HW_ERR` prefix. Missing a log prefix isn't a functional bug — this is a convention/consistency improvement. It might be more appropriate without a `Fixes:` tag, or at least should not be backported to stable trees.

**Pre-existing issue (not introduced by this patch):** The `log_soc_error` messages are missing a trailing `\n`:

```c
drm_warn(&xe->drm, HW_ERR "%s SOC %s detected", name, severity_str);
```

Compare with `log_hw_error` and `log_gt_err`, which both have `\n`. This inconsistency predates this patch but could be worth fixing in a follow-up.

---
Generated by Claude Code Patch Reviewer