Claude review: drm/msm/mdp5: Remove MSM8974v1

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

Patch Review

**Correctness**: The patch correctly removes the `msm8x74v1_config` definition (92 lines of dead config) and its handler table entry `{ .revision = 0, ... }`. The rename from `msm8x74v2_config` to `msm8x74_config` is consistent — the `.name` field was already `"msm8x74"` (without any version suffix), so the C symbol now matches the reported name.

The lookup logic in `mdp5_cfg_init()` iterates `cfg_handlers_v1[]` matching on `minor` revision number. With the revision 0 entry removed, any v1 hardware attempting to probe would fail with the existing `"unexpected MDP minor revision"` error at line 1163, which is the correct behavior for unsupported hardware.

**Minor issue — duplicate `Signed-off-by` tag**: The commit message has `Signed-off-by: Konrad Dybcio` appearing twice — once before the `---` separator and once after it (lines 91 and 94 of the mbox). The one after `---` won't be included in the final commit by git-am, but the duplication looks accidental — possibly a b4 artifact.

```
Signed-off-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>
---
related: https://lore.kernel.org/...
Signed-off-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>
---
```

**No functional concerns**: The remaining revision 2 entry (`msm8x74_config`) is untouched in content, only renamed. No other file references `msm8x74v1_config` or `msm8x74v2_config` by name (they are only used via the handler table), so no other files need updating.

Overall: This is a clean removal patch. The rationale is sound (v1 silicon never shipped commercially), and the implementation is correct. The only nit is the duplicate Signed-off-by.

---
Generated by Claude Code Patch Reviewer