Claude review: drm: Remove unused header in drm_dumb_buffers.c

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

Patch Review

**Result: No issues.**

The patch removes:
```c
-#include "drm_internal.h"
```

I verified this is correct by cross-referencing the symbols used in `drm_dumb_buffers.c` against the declarations in `drm_internal.h`:

- `drm_gem_handle_delete()` — declared in `<drm/drm_gem.h>` (line 584), **not** in `drm_internal.h`
- `drm_gem_dumb_map_offset()` — declared in `<drm/drm_gem.h>`, **not** in `drm_internal.h`
- `drm_mode_create_dumb_ioctl()`, `drm_mode_mmap_dumb_ioctl()`, `drm_mode_destroy_dumb_ioctl()` — declared in `drm_crtc_internal.h` (lines 116-120), which remains included
- All other symbols come from the public `<drm/*.h>` headers already included

No symbol used by `drm_dumb_buffers.c` is declared in `drm_internal.h`. The remaining include of `drm_crtc_internal.h` covers the internal declarations this file needs.

The `Fixes:` tag referencing `96a7b60f6ddb` is reasonable — that's the commit that removed `drm_gem_dumb_destroy()`, which was the last function requiring `drm_internal.h`. Minor nit: some maintainers might prefer not to use a `Fixes:` tag for a cosmetic cleanup since nothing is functionally broken, but this is a matter of style/preference and not worth blocking on.

---
Generated by Claude Code Patch Reviewer