Claude review: video: cirrusfb: Make use of struct zorro_device_id::driver_data_ptr

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

Patch Review

**Status: Looks good.**

This patch converts cirrusfb to use the new `driver_data_ptr` field. The table initialization changes from:

```c
-		.driver_data	= (unsigned long)&zcl_sd64,
+		.driver_data_ptr = &zcl_sd64,
```

And the consumption site changes from:

```c
-	zcl = (const struct zorrocl *)ent->driver_data;
+	zcl = ent->driver_data_ptr;
```

Both directions of the conversion are correct:

1. **Initialization side:** Drops the `(unsigned long)` cast. The `const struct zorrocl *` is implicitly converted to `const void *` — this is safe and preserves const.

2. **Usage side:** `ent->driver_data_ptr` is `const void *`, which is implicitly converted to `const struct zorrocl *` in the assignment to `zcl`. This is valid C and preserves the const qualifier that was previously being explicitly cast (actually correctly in the old code, but the cast was fragile — e.g., if someone changed the type of `zcl`, the old `(const struct zorrocl *)` cast would silently hide any mismatch).

3. **Terminator cleanup:** `{ 0 }` → `{ }` is consistent with patch 5.

4. **Alignment reformatting:** The patch also drops the column-aligned `=` signs (e.g., `.id\t\t=` → `.id =`). This is a minor style preference; the kernel doesn't mandate column alignment in struct initializers, so this is fine as a drive-by cleanup.

No concerns.

---
Generated by Claude Code Patch Reviewer