Claude review: fbdev: clps711x-fb: Request memory region for MMIO

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

Patch Review

**Status: Good**

Makes resource 0 (MMIO registers) consistent with resource 1 (framebuffer), which already uses `devm_platform_get_and_ioremap_resource()` at line 231 of the current source. Good consistency improvement.

```c
cfb->base = devm_platform_get_and_ioremap_resource(pdev, 0, &res);
if (IS_ERR(cfb->base)) {
    ret = PTR_ERR(cfb->base);
    goto out_fb_release;
}
```

The original code had a subtle deficiency: when `platform_get_resource()` returned NULL, it jumped to `out_fb_release` with `ret` still at its init value of `-ENOENT` (line 206 of current source). While not a bug per se, the new code is better — `ret` is explicitly set to the actual error from the helper.

`res` is still used afterward (lines 228-229: `info->fix.mmio_start = res->start; info->fix.mmio_len = resource_size(res)`), and it's correctly populated by the `&res` output parameter.

No issues.

---
Generated by Claude Code Patch Reviewer