Claude review: drm/colorop: Add DRM_COLOROP_CSC_FF

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

Patch Review

Good addition to the DRM colorop framework. The implementation follows existing patterns well.

Minor observations:

- The `drm_get_colorop_csc_ff_type_name` doc comment says "return a string for interpolation type" — this is a copy-paste error from the LUT3D interpolation function. It should say "return a string for CSC FF type".

```c
/**
 * drm_get_colorop_csc_ff_type_name: return a string for interpolation type
 * @type: csc ff type to compute name of
```

- In `drm_plane_colorop_csc_ff_init`, if the `drm_plane_colorop_init` call succeeds but `drm_property_create_enum` returns NULL, the function returns `-ENOMEM` without cleaning up the partially initialized colorop. The existing patterns in other `*_init` functions have the same issue, so this is pre-existing, but worth noting.

- The `WARN_ON(len <= 0)` check after the loop is dead code since `supported_csc_ff` was already validated non-zero above. It's a reasonable defensive check though.

---
Generated by Claude Code Patch Reviewer