From: Andrew Morton __must_check goes on the declaration, not the definition. Cc: "Randy.Dunlap" Signed-off-by: Andrew Morton --- drivers/pci/pcie/portdrv.h | 4 +++- drivers/pci/pcie/portdrv_core.c | 3 +-- pci/pcie/portdrv_pci.c | 0 3 files changed, 4 insertions(+), 3 deletions(-) diff -puN drivers/pci/pcie/portdrv_core.c~pcie-check-and-return-bus_register-errors-fix drivers/pci/pcie/portdrv_core.c --- a/drivers/pci/pcie/portdrv_core.c~pcie-check-and-return-bus_register-errors-fix +++ a/drivers/pci/pcie/portdrv_core.c @@ -6,7 +6,6 @@ * Copyright (C) Tom Long Nguyen (tom.l.nguyen@intel.com) */ -#include #include #include #include @@ -403,7 +402,7 @@ void pcie_port_device_remove(struct pci_ pci_disable_msi(dev); } -int __must_check pcie_port_bus_register(void) +int pcie_port_bus_register(void) { return bus_register(&pcie_port_bus_type); } diff -puN drivers/pci/pcie/portdrv.h~pcie-check-and-return-bus_register-errors-fix drivers/pci/pcie/portdrv.h --- a/drivers/pci/pcie/portdrv.h~pcie-check-and-return-bus_register-errors-fix +++ a/drivers/pci/pcie/portdrv.h @@ -9,6 +9,8 @@ #ifndef _PORTDRV_H_ #define _PORTDRV_H_ +#include + #if !defined(PCI_CAP_ID_PME) #define PCI_CAP_ID_PME 1 #endif @@ -39,7 +41,7 @@ extern int pcie_port_device_suspend(stru extern int pcie_port_device_resume(struct pci_dev *dev); #endif extern void pcie_port_device_remove(struct pci_dev *dev); -extern int pcie_port_bus_register(void); +extern int __must_check pcie_port_bus_register(void); extern void pcie_port_bus_unregister(void); #endif /* _PORTDRV_H_ */ diff -puN drivers/pci/pcie/portdrv_pci.c~pcie-check-and-return-bus_register-errors-fix drivers/pci/pcie/portdrv_pci.c _