Claude review: drm/tegra: dp: fix kernel-doc warnings in dp.h

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

Patch Review

**Status: Clean**

The patch fixes four distinct kernel-doc issues:

1. **Missing `@` prefix on `tps3_supported`** (line 103 of the diff):
   ```c
   - * tps3_supported:
   + * @tps3_supported:
   ```
   Correct — kernel-doc requires `@` prefix for struct member documentation.

2. **Empty descriptions for `@apply_training` and `@configure`** (lines 113, 119 of the diff):
   ```c
   - * @apply_training:
   + * @apply_training: apply the link training
   ```
   ```c
   - * @configure:
   + * @configure: configure the DP link
   ```
   Correct — kernel-doc warns when `@member:` entries in a struct/ops doc block have no description text. The added descriptions are brief and appropriate.

3. **Misplaced nested struct member docs for `aux_rd_interval`** (lines 126–141 of the diff):
   The old code had `@cr` and `@ce` documented in a separate `/** */` block directly above the anonymous struct, which kernel-doc doesn't associate with the parent struct's doc block. The fix removes that orphaned comment and instead documents `aux_rd_interval.cr` and `aux_rd_interval.ce` as nested member entries in the parent `struct drm_dp_link` doc block:
   ```c
   + * @aux_rd_interval.cr: clock recovery read interval
   + * @aux_rd_interval.ce: channel equalization read interval
   ```
   This is the correct kernel-doc syntax for anonymous nested struct members, and eliminates both the "Excess struct member 'cr'" warning and the implicit missing-member warnings.

All four fixes are correct kernel-doc usage. The patch is well-scoped — it changes only documentation, no functional code. The commit message clearly lists all four warnings being fixed. The Acked-by from Mikko Perttunen is appropriate as the Tegra driver maintainer.

No concerns with this patch.

---
Generated by Claude Code Patch Reviewer