Claude review: fbdev/offb: fix PCI device reference leak on probe failure

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

Patch Review

**The bug:** In `offb_init_nodriver()`, the original code at line 642 was:

```c
pdev = pci_get_device(vid, did, NULL);
if (!pdev || pci_enable_device(pdev))
    return;
```

`pci_get_device()` increments the device's reference count. When `pdev` is non-NULL but `pci_enable_device()` fails, the function returned without calling `pci_dev_put(pdev)`, leaking the reference.

**The fix:** The patch correctly splits the condition:

```c
pdev = pci_get_device(vid, did, NULL);
if (!pdev)
    return;

if (pci_enable_device(pdev)) {
    pci_dev_put(pdev);
    return;
}
```

This ensures the reference is released on the `pci_enable_device()` failure path while keeping the `!pdev` case unchanged (no reference to release).

**Observation (pre-existing issue, not a problem with this patch):** On the *success* path, `pdev` is also never released. After `pci_enable_device()` succeeds, execution falls through to `offb_init_fb()` at line 655, which does not receive or store `pdev`. The reference from `pci_get_device()` is leaked on the success path too. This is outside the scope of this patch, but worth noting as a potential follow-up fix — the function would need a `pci_dev_put(pdev)` after the `pci_enable_device()` block completes (the device remains enabled even after the reference is dropped).

---
Generated by Claude Code Patch Reviewer