On Tue, Dec 05, 2017 at 05:50:37PM +0800, Dongdong Liu wrote: > + if ((pci_pcie_type(dev) != PCI_EXP_TYPE_UPSTREAM) && > + (pci_pcie_type(dev) != PCI_EXP_TYPE_DOWNSTREAM)) > + pci_disable_pcie_error_reporting(dev); No need for the inner braces here.
On Tue, Dec 05, 2017 at 05:50:37PM +0800, Dongdong Liu wrote: > + if ((pci_pcie_type(dev) != PCI_EXP_TYPE_UPSTREAM) && > + (pci_pcie_type(dev) != PCI_EXP_TYPE_DOWNSTREAM)) > + pci_disable_pcie_error_reporting(dev); No need for the inner braces here.