Claude review: drm/connector: Make drm_connector_attach_hdr_output_metadata_property() return void

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

Patch Review

**Reviewed-by worthy** — clean and correct.

The commit message accurately describes the change and the rationale. Two things are done here:

1. **Return type change to void**: The function body just calls `drm_object_attach_property()` (which is void) and then `return 0`. Removing the fake return value is the right thing to do. Confirmed that none of the 6 callers (amdgpu_dm, dw-hdmi, intel_dp ×2, intel_hdmi, meson_encoder_hdmi) check the return value.

2. **Typo fix**: `"HDR_OUTPUT_METADA"` → `"HDR_OUTPUT_METADATA"` in the kerneldoc — straightforward correction.

The diff is minimal and correct:

```c
-int drm_connector_attach_hdr_output_metadata_property(struct drm_connector *connector)
+void drm_connector_attach_hdr_output_metadata_property(struct drm_connector *connector)
```

The removal of the `Returns:` section from the kerneldoc and the trailing blank line + `return 0` are appropriate.

No concerns with this patch.

---
Generated by Claude Code Patch Reviewer