> + if ((driver->port_type != PCIE_ANY_PORT) &&> + (driver->port_type != pci_pcie_type(pciedev->port)))No need for the inner braces here.Otherwise looks good:Reviewed-by: Christoph Hellwig <hch@lst.de>