> +static inline > int __pci_read_base(struct pci_dev *dev, enum pci_bar_type type, > + struct resource *res, unsigned int reg) This looks weird. Normal kernel style would be: static inline int __pci_read_base(struct pci_dev *dev, enum pci_bar_type type, struct resource *res, unsigned int reg) or static inline int __pci_read_base(struct pci_dev *dev, enum pci_bar_type type, that being said, there seems to be no good agument to even make this and inline function. > + return __pci_bus_read_base(dev->bus, dev->devfn, type, res, reg, dev->mmio_always_on); Please avoid pointless overly long lines.
