Claude review: fbdev: i810: Use safer strscpy() instead of strcpy()

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

Patch Review

**Files:** `i810-i2c.c`, `i810_main.c` — 2 replacements

The string conversions themselves are fine:
- `name` → `chan->adapter.name` (`char[48]`)
- `"I810"` (5 bytes) → `fix->id` (`char[16]`)

**Issue: undeclared whitespace change.** Both hunks silently convert leading spaces to tabs:

```c
-        strcpy(chan->adapter.name, name);
+	strscpy(chan->adapter.name, name);
```

```c
-    	strcpy(fix->id, "I810");
+	strscpy(fix->id, "I810");
```

The commit message says "Only idiomatic code replacement, and no functional changes" but the indentation change is a separate style fix that should either be split out or mentioned. Mixing whitespace changes with functional changes makes `git blame` noisier.

---

---
Generated by Claude Code Patch Reviewer