Claude review: fbdev: Wrap fbcon updates from vga-switcheroo in helper

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

Patch Review

**Bug — `#if CONFIG_FB`**: The patch adds:
```c
+#if CONFIG_FB
 	if (new_client->fb_info)
-		fbcon_remap_all(new_client->fb_info);
+		fb_switch_outputs(new_client->fb_info);
+#endif
```
This should be `#ifdef CONFIG_FB`. While `#if CONFIG_FB` happens to work for bool Kconfig symbols (undefined macros evaluate to 0 in `#if` expressions), it's not standard kernel preprocessor style. The kernel overwhelmingly uses `#ifdef CONFIG_*` for presence checks. `#if CONFIG_FB` could also trigger compiler warnings with `-Wundef` if enabled.

Moreover, looking at the Kconfig:
```
config VGA_SWITCHEROO
	depends on (FRAMEBUFFER_CONSOLE=n || FB=y)
```
When `VGA_SWITCHEROO` is enabled, either `FB=y` (so CONFIG_FB is always defined) or `FRAMEBUFFER_CONSOLE=n` (FB may or may not be set). The guard is functionally needed for the `FRAMEBUFFER_CONSOLE=n && FB=n` case (though that seems unlikely given the dependency implies you need *some* display path).

**Design**: The `fb_switch_outputs()` function is a thin wrapper around `fbcon_remap_all()`. The commit message explains this is a stepping stone toward moving the call into DRM's fbdev emulation, which justifies the indirection. The function is exported with `EXPORT_SYMBOL` since `vga_switcheroo` is outside the fbdev core.

---
Generated by Claude Code Patch Reviewer