On Mon, Mar 12, 2018 at 01:01:47AM -0700, Christoph Hellwig wrote:
> > + if ((driver->port_type != PCIE_ANY_PORT) &&
> > + (driver->port_type != pci_pcie_type(pciedev->port)))
>
> No need for the inner braces here.
Thanks, I removed them.
On Mon, Mar 12, 2018 at 01:01:47AM -0700, Christoph Hellwig wrote:
> > + if ((driver->port_type != PCIE_ANY_PORT) &&
> > + (driver->port_type != pci_pcie_type(pciedev->port)))
>
> No need for the inner braces here.
Thanks, I removed them.
> + 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
> + 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
4 matches
Mail list logo