Claude review: drm/amdgpu/jpeg: deduplicate jpeg_v3_0 process_interrupt

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

Patch Review

**Correctness:** The deleted `jpeg_v3_0_process_interrupt` is verified identical to `jpeg_v2_0_process_interrupt` — both use `VCN_2_0__SRCID__JPEG_DECODE`, `DRM_ERROR`, and `adev->jpeg.inst->ring_dec`. The deduplication is valid.

**Style concern — macro alias is unnecessary indirection:**

```c
#define jpeg_v3_0_process_interrupt jpeg_v2_0_process_interrupt
```

This macro exists solely so the struct initializer can still say `.process = jpeg_v3_0_process_interrupt`. This is over-engineering — it hides the actual function being called and makes code search harder (grepping for the definition of `jpeg_v3_0_process_interrupt` now lands on a macro, not a function). It would be simpler and more readable to just use the function directly in the struct initializer:

```c
static const struct amdgpu_irq_src_funcs jpeg_v3_0_irq_funcs = {
    .set = jpeg_v3_0_set_interrupt_state,
    .process = jpeg_v2_0_process_interrupt,
};
```

This is the standard kernel pattern for reusing a function from another version — no macro needed.

**Tag ordering issue:** The Signed-off-by / Co-developed-by order is wrong per `Documentation/process/submitting-patches.rst`. Currently:

```
Signed-off-by: Tiago Dourado <tiagodourado@usp.br>
Co-developed-by: Luiz Fernandes <luiz.f.f.fernandes@usp.br>
Signed-off-by: Luiz Fernandes <luiz.f.f.fernandes@usp.br>
```

The submitter's `Signed-off-by` must be **last**, and every `Co-developed-by` must be immediately followed by its matching `Signed-off-by`. The correct order is:

```
Co-developed-by: Luiz Fernandes <luiz.f.f.fernandes@usp.br>
Signed-off-by: Luiz Fernandes <luiz.f.f.fernandes@usp.br>
Signed-off-by: Tiago Dourado <tiagodourado@usp.br>
```

---

---
Generated by Claude Code Patch Reviewer