[PATCH] PCI/AER: Move pci_uevent_ers() out of pci.h
Bryant G. Ly
bryantly at linux.vnet.ibm.com
Fri Feb 9 02:05:45 AEDT 2018
On 2/8/18 6:20 AM, Michael Ellerman wrote:
> There's no reason pci_uevent_ers() needs to be inline in pci.h, so
> move it out to a C file.
>
> Given it's used by AER the obvious location would be somewhere in
> drivers/pci/pcie/aer, but because it's also used by powerpc EEH code
> unfortunately that doesn't work in the case where EEH is enabled but
> PCIEPORTBUS is not.
>
> So for now put it in pci-driver.c, next to pci_uevent(), with an
> appropriate #ifdef so it's not built if AER and EEH are both disabled.
>
> While we're moving it also fix up the kernel doc comment for @pdev to
> be accurate.
>
> Reported-by: Linus Torvalds <torvalds at linux-foundation.org>
> Signed-off-by: Michael Ellerman <mpe at ellerman.id.au>
> ---
> drivers/pci/pci-driver.c | 36 ++++++++++++++++++++++++++++++++++++
> include/linux/pci.h | 38 +++-----------------------------------
> 2 files changed, 39 insertions(+), 35 deletions(-)
Looks good, thanks for fixing it!
Signed-off-by: Bryant G. Ly <bryantly at linux.vnet.ibm.com>
-Bryant
More information about the Linuxppc-dev
mailing list