> +void pcie_do_fatal_recovery(struct pci_dev *dev, u32 service) > +{ > + pcie_do_recovery(dev, pci_channel_io_frozen, service); > +} > + > +void pcie_do_nonfatal_recovery(struct pci_dev *dev) > +{ > + pcie_do_recovery(dev, pci_channel_io_normal, PCIE_PORT_SERVICE_AER); > +} Is there any good reason to not just expose pcie_do_recovery directly and skip these wrappers?