Claude review: drm/exynos: Make exynos_drm_framebuffer_init() an internal interface

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

Patch Review

**Status: Good**

After patch 4 removed the fbdev caller of `exynos_drm_framebuffer_init()`, the only remaining caller is `exynos_user_fb_create()` in the same file. Making it `static` is the right thing to do:

```c
-struct drm_framebuffer *
+static struct drm_framebuffer *
 exynos_drm_framebuffer_init(...)
```

The header cleanup is also correct — replacing the `#include "exynos_drm_gem.h"` (which was only needed for the now-removed declaration) with minimal forward declarations and `<linux/types.h>` for `dma_addr_t`:

```c
-#include "exynos_drm_gem.h"
+#include <linux/types.h>

-struct drm_framebuffer *
-exynos_drm_framebuffer_init(...);
+struct drm_device;
+struct drm_framebuffer;
```

The compensating `#include "exynos_drm_gem.h"` additions in `exynos_drm_fb.c` and `exynos_drm_fbdev.c` are correct — both files use `exynos_drm_gem` types and were previously getting the header transitively through `exynos_drm_fb.h`.

No concerns.

---
Generated by Claude Code Patch Reviewer