Claude review: pci: check if VGA decoding was really activated

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

Patch Review

**Verdict: Has a coding style issue.**

This is the core functional change — reading back `PCI_BRIDGE_CONTROL` after writing to verify the bridge actually accepted the VGA enable. The logic is correct: only check on `decode == true` (enabling), since failure to disable is less critical.

**Coding style bug** at `drivers/pci/pci.c:6203`:

```c
if(!(cmd & PCI_BRIDGE_CTL_VGA))
```

Missing space between `if` and `(`. Kernel coding style requires `if (condition)`. This should be `if (!(cmd & PCI_BRIDGE_CTL_VGA))`.

The choice of `-EIO` as the error code is reasonable for a hardware communication issue.

One consideration: if there are multiple bridges in the hierarchy and an intermediate one fails, the bridges already configured earlier in the `while (bus)` loop will have `PCI_BRIDGE_CTL_VGA` set but the function returns `-EIO`. The caller (`__vga_tryget`) will propagate the error without undoing those earlier bridge writes. This is similar to the state-consistency concern from patch 2 and is probably acceptable in practice, but worth noting.

---
Generated by Claude Code Patch Reviewer