On 21/08/02 05:44PM, Bjorn Helgaas wrote: > On Sun, Aug 01, 2021 at 07:55:11PM +0530, Amey Narkhede wrote: > > Currently there is separate function pcie_has_flr() to probe if PCIe FLR > > is supported by the device which does not match the calling convention > > followed by reset methods which use second function argument to decide > > whether to probe or not. Add new function pcie_reset_flr() that follows > > the calling convention of reset methods. > > > > Signed-off-by: Amey Narkhede <ameynarkhede03@xxxxxxxxx> > > --- > > drivers/crypto/cavium/nitrox/nitrox_main.c | 4 +-- > > drivers/pci/pci.c | 40 +++++++++++++++------- > > drivers/pci/pcie/aer.c | 12 +++---- > > drivers/pci/quirks.c | 9 ++--- > > include/linux/pci.h | 2 +- > > 5 files changed, 38 insertions(+), 29 deletions(-) > > > int pcie_flr(struct pci_dev *dev) > > { > > @@ -4655,7 +4653,26 @@ int pcie_flr(struct pci_dev *dev) > > > > return pci_dev_wait(dev, "FLR", PCIE_RESET_READY_POLL_MS); > > } > > -EXPORT_SYMBOL_GPL(pcie_flr); > > +EXPORT_SYMBOL(pcie_flr); > > Why this change? If it's unintentional and there's no other reason to > repost, I can fix it up locally. Yeah my bad it was unintentional.