Claude review: fbcon: Use correct type for vc_resize() return value

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

Patch Review

**Assessment: Correct and clean.**

The change is straightforward:

```c
-	int rows, cols;
-	unsigned long ret = 0;
+	int rows, cols, ret;
```

This is the right fix. `vc_resize()` is declared in `include/linux/vt_kern.h:46` as returning `int`, so `ret` should be `int`. The usage at line 1494-1496 is:

```c
	ret = vc_resize(vc, cols, rows);

	if (con_is_visible(vc) && !ret)
		update_screen(vc);
```

The `!ret` check works identically for both types when the value comes from `vc_resize()`, so this is purely a type-correctness fix with no behavioral change. The `= 0` initializer is correctly dropped since `ret` is always assigned before use (at line 1494), and every path that reaches line 1494 will set it.

**Minor nit on the Fixes tag:** The commit message claims `Fixes: af0db3c1f898 ("fbdev: Fix vmalloc out-of-bounds write in fast_imageblit")`. That commit title is about `fast_imageblit`, which seems unrelated to the `unsigned long ret` declaration in `fbcon_set_disp()`. I'd recommend the author double-check that the Fixes tag points to the correct commit — it should point to whichever commit introduced the `unsigned long ret` variable in this function. If it was always `unsigned long` since the function's inception, a Fixes tag may not be appropriate at all (a simple cleanup doesn't necessarily need one).

**No other issues.** The patch is minimal, correctly scoped, and the commit message accurately describes the warning being fixed.

---
Generated by Claude Code Patch Reviewer