pci: make local function static
PATCH 1/2 pci: make local function static
Using 'make namespacecheck' identify code which should be declared static. Checked for users in other driver/archs as well. Compile tested only.
Signed-off-by Stephen Hemminger <[email protected]>
---
Patch against next branch of pci git
drivers/pci/access.c | 3 - drivers/pci/hotplug/pciehp.h | 1 drivers/pci/hotplug/pciehp_core.c | 2 drivers/pci/pci.c | 56 +++++++++---------- drivers/pci/pci.h | 2 drivers/pci/probe.c | 111 +++++++++++++++++++------------------- include/linux/pci.h | 9 --- 7 files changed, 86 insertions(+), 98 deletions(-)
drivers/pci/pci.c +28 -28
@@ -265,7 +265,7 @@ int pci_bus_find_capability(struct pci_b* not support it. Some capabilities can occur several times, e.g., the * vendor-specific capability, and this provides a way to find them all. */ -int pci_find_next_ext_capability (struct pci_dev *dev,int start,int cap) +static int pci_find_next_ext_capability (struct pci_dev *dev,int start,int cap) {u32 header;int ttl;
@@ -304,7 +304,6 @@ int pci_find_next_ext_capability(structreturn 0 ; } -EXPORT_SYMBOL_GPL (pci_find_next_ext_capability);/** * pci_find_ext_capability - Find an extended capability
@@ -683,6 +682,28 @@ static int pci_platform_power_transition }/** +* pci_wakeup - Wake up a PCI device +* @pci_dev: Device to handle. +* @ign: ignored parameter +*/ +static int pci_wakeup (struct pci_dev *pci_dev,void *ign) +{ +pci_wakeup_event (pci_dev); +pm_request_resume (&pci_dev->dev); +return 0 ; +} + +/** +* pci_wakeup_bus - Walk given bus and wake up devices on it +* @bus: Top bus of the subtree to walk. +*/ +static void pci_wakeup_bus (struct pci_bus *bus) +{ +if (bus) +pci_walk_bus (bus, pci_wakeup,NULL ); +} + +/** * __pci_start_power_transition - Start power transition of a PCI device * @dev: PCI device to handle. * @state: State to put the device into.
@@ -873,7 +894,8 @@ static struct pci_cap_saved_state *_pci_return NULL ; } -struct pci_cap_saved_state *pci_find_saved_cap (struct pci_dev *dev,char cap) +static struct pci_cap_saved_state *pci_find_saved_cap (struct pci_dev *dev, +char cap) {return _pci_find_saved_cap (dev, cap, false); }
@@ -1110,7 +1132,8 @@ EXPORT_SYMBOL_GPL(pci_store_saved_state)* @dev: PCI device that we're dealing with * @state: Saved state returned from pci_store_saved_state() */ -int pci_load_saved_state (struct pci_dev *dev,struct pci_saved_state *state) +static int pci_load_saved_state (struct pci_dev *dev, +struct pci_saved_state *state) {struct pci_cap_saved_data *cap;
@@ -1138,7 +1161,6 @@ int pci_load_saved_state(struct pci_dev dev->state_saved = true;return 0 ; } -EXPORT_SYMBOL_GPL (pci_load_saved_state);/** * pci_load_and_free_saved_state - Reload the save state pointed to by state,
@@ -1570,27 +1592,6 @@ void pci_pme_wakeup_bus(struct pci_bus *pci_walk_bus (bus, pci_pme_wakeup, (void *)true); } -/** -* pci_wakeup - Wake up a PCI device -* @pci_dev: Device to handle. -* @ign: ignored parameter -*/ -static int pci_wakeup (struct pci_dev *pci_dev,void *ign) -{ -pci_wakeup_event (pci_dev); -pm_request_resume (&pci_dev->dev); -return 0 ; -} - -/** -* pci_wakeup_bus - Walk given bus and wake up devices on it -* @bus: Top bus of the subtree to walk. -*/ -void pci_wakeup_bus (struct pci_bus *bus) -{ -if (bus) -pci_walk_bus (bus, pci_wakeup,NULL ); -}/** * pci_pme_capable - check the capability of PCI device to generate PME#
@@ -1804,7 +1805,7 @@ int pci_wake_from_d3(struct pci_dev *dev* If the platform can't manage @dev, return the deepest state from which it * can generate wake events, based on any available PME info. */ -pci_power_t pci_target_state (struct pci_dev *dev) +static pci_power_t pci_target_state (struct pci_dev *dev) {pci_power_t target_state = PCI_D3hot;
@@ -4482,7 +4483,6 @@ EXPORT_SYMBOL(pci_restore_state);EXPORT_SYMBOL (pci_pme_capable);EXPORT_SYMBOL (pci_pme_active);EXPORT_SYMBOL (pci_wake_from_d3); -EXPORT_SYMBOL (pci_target_state);EXPORT_SYMBOL (pci_prepare_to_sleep);EXPORT_SYMBOL (pci_back_from_sleep);EXPORT_SYMBOL_GPL (pci_set_pcie_reset_state);
include/linux/pci.h +0 -9
@@ -386,8 +386,6 @@ static inline int pci_channel_offline(streturn (pdev->error_state != pci_channel_io_normal); } -extern struct resource busn_resource; -struct pci_host_bridge_window {struct list_head list;struct resource *res;/* host bridge aperture (CPU address) */
@@ -799,7 +797,6 @@ enum pci_lost_interrupt_reason pci_lost_int pci_find_capability (struct pci_dev *dev,int cap);int pci_find_next_capability (struct pci_dev *dev,u8 pos,int cap);int pci_find_ext_capability (struct pci_dev *dev,int cap); -int pci_find_next_ext_capability (struct pci_dev *dev,int pos,int cap);int pci_find_ht_capability (struct pci_dev *dev,int ht_cap);int pci_find_next_ht_capability (struct pci_dev *dev,int pos,int ht_cap);struct pci_bus *pci_find_next_bus (const struct pci_bus *from);
@@ -864,7 +861,6 @@ static inline int pci_write_config_dwordint pcie_capability_read_word (struct pci_dev *dev,int pos,u16 *val);int pcie_capability_read_dword (struct pci_dev *dev,int pos,u32 *val);int pcie_capability_write_word (struct pci_dev *dev,int pos,u16 val); -int pcie_capability_write_dword (struct pci_dev *dev,int pos,u32 val);int pcie_capability_clear_and_set_word (struct pci_dev *dev,int pos,u16 clear,u16 set); int pcie_capability_clear_and_set_dword(struct pci_dev *dev, int pos,
@@ -974,10 +970,8 @@ void __iomem __must_check *pci_platform_int pci_save_state (struct pci_dev *dev);void pci_restore_state (struct pci_dev *dev);struct pci_saved_state *pci_store_saved_state (struct pci_dev *dev); -int pci_load_saved_state (struct pci_dev *dev,struct pci_saved_state *state);int pci_load_and_free_saved_state (struct pci_dev *dev,struct pci_saved_state **state); -struct pci_cap_saved_state *pci_find_saved_cap (struct pci_dev *dev,char cap);struct pci_cap_saved_state *pci_find_saved_ext_cap (struct pci_dev *dev,u16 cap);int pci_add_cap_save_buffer (struct pci_dev *dev,char cap,unsigned int size);
@@ -991,7 +985,6 @@ void pci_pme_active(struct pci_dev *dev,int __pci_enable_wake (struct pci_dev *dev,pci_power_t state,bool runtime,bool enable);int pci_wake_from_d3 (struct pci_dev *dev,bool enable); -pci_power_t pci_target_state (struct pci_dev *dev);int pci_prepare_to_sleep (struct pci_dev *dev);int pci_back_from_sleep (struct pci_dev *dev);bool pci_dev_run_wake (struct pci_dev *dev);
@@ -1126,8 +1119,6 @@ int pci_scan_bridge(struct pci_bus *bus,void pci_walk_bus (struct pci_bus *top,int (*cb)(struct pci_dev *,void *),void *userdata); -int pci_cfg_space_size_ext (struct pci_dev *dev); -int pci_cfg_space_size (struct pci_dev *dev);unsigned char pci_bus_max_busnr (struct pci_bus *bus);void pci_setup_bridge (struct pci_bus *bus); resource_size_t pcibios_window_alignment(struct pci_bus *bus,
drivers/pci/hotplug/pciehp.h +0 -1
@@ -43,7 +43,6 @@extern bool pciehp_poll_mode;extern int pciehp_poll_time;extern bool pciehp_debug; -extern bool pciehp_force; #define dbg(format, arg...) \ do { \
drivers/pci/hotplug/pciehp_core.c +1 -1
@@ -41,7 +41,7 @@bool pciehp_debug;bool pciehp_poll_mode;int pciehp_poll_time; -bool pciehp_force; +static bool pciehp_force;#define DRIVER_VERSION "0.4"#define DRIVER_AUTHOR "Dan Zink <[email protected]>, Greg Kroah-Hartman <[email protected]>, Dely Sy <[email protected]>"
drivers/pci/access.c +1 -2
@@ -622,7 +622,7 @@ int pcie_capability_write_word(struct pc }EXPORT_SYMBOL (pcie_capability_write_word); -int pcie_capability_write_dword (struct pci_dev *dev,int pos,u32 val) +static int pcie_capability_write_dword (struct pci_dev *dev,int pos,u32 val) {if (pos &3 )return -EINVAL ;
@@ -632,7 +632,6 @@ int pcie_capability_write_dword(struct preturn pci_write_config_dword (dev,pci_pcie_cap (dev) + pos, val); } -EXPORT_SYMBOL (pcie_capability_write_dword);int pcie_capability_clear_and_set_word (struct pci_dev *dev,int pos,u16 clear,u16 set)
drivers/pci/pci.h +0 -2
@@ -6,7 +6,6 @@#define PCI_CFG_SPACE_SIZE 256#define PCI_CFG_SPACE_EXP_SIZE 4096 -extern const unsigned char pcix_bus_speed[];extern const unsigned char pcie_link_speed[];/* Functions internal to the PCI core code */
@@ -68,7 +67,6 @@ void pci_power_up(struct pci_dev *dev);void pci_disable_enabled_device (struct pci_dev *dev);int pci_finish_runtime_suspend (struct pci_dev *dev);int __pci_pme_wakeup (struct pci_dev *dev,void *ign); -void pci_wakeup_bus (struct pci_bus *bus);void pci_config_pm_runtime_get (struct pci_dev *dev);void pci_config_pm_runtime_put (struct pci_dev *dev);void pci_pm_init (struct pci_dev *dev);
drivers/pci/probe.c +56 -55
135 lines not on this page · raw message
Bjorn Helgaas
maintainerQuoted 30 lines · Stephen Hemminger
Using 'make namespacecheck' identify code which should be declared static. Checked for users in other driver/archs as well. Compile tested only.
Signed-off-by Stephen Hemminger <[email protected]>---
Patch against next branch of pci gitdrivers/pci/access.c | 3 - drivers/pci/hotplug/pciehp.h | 1 drivers/pci/hotplug/pciehp_core.c | 2 drivers/pci/pci.c | 56 +++++++++---------- drivers/pci/pci.h | 2 drivers/pci/probe.c | 111 +++++++++++++++++++------------------- include/linux/pci.h | 9 --- 7 files changed, 86 insertions(+), 98 deletions(-)drivers/pci/pci.c 2013-12-27 13:00:54.784294985 -0800 renamed from drivers/pci/pci.c 2013-12-27 12:51:57.592814680 -0800 +1
-1@@ -265,7 +265,7 @@ int pci_bus_find_capability(struct pci_b * not support it. Some capabilities can occur several times, e.g., the * vendor-specific capability, and this provides a way to find them all. */ -int pci_find_next_ext_capability(struct pci_dev *dev, int start, int cap) +static int pci_find_next_ext_capability(struct pci_dev *dev, int start, int cap) { u32 header; int ttl;
drivers/pci/pci.c 2013-12-27 13:00:54.784294985 -0800 int pci_find_next_ext_capability(struct · line 307
Quoted 4 lines
drivers/pci/pci.c 2013-12-27 13:00:54.784294985 -0800 renamed from drivers/pci/pci.c 2013-12-27 12:51:57.592814680 -0800 +0
-1@@ -304,7 +304,6 @@ int pci_find_next_ext_capability(struct return 0; } -EXPORT_SYMBOL_GPL(pci_find_next_ext_capability);
I added pci_find_next_ext_capability() (44a9a36f6be4) because some extended capabilities can occur several times and pci_find_ext_capability() can only find the first one.
The only in-tree user of pci_find_next_ext_capability() is pci_find_ext_capability(), so we *could* make it static. But I don't think there's much to gain by making it static, and I'm afraid doing so will just encourage people to reimplement it in a driver instead of using the core interface.
I am a bit dubious about the GPL export, though. I'm sure I copied that from pci_find_ext_capability(), but I don't know why either one needs to be GPL.
Quoted 20 lines
include/linux/pci.h 2013-12-27 13:03:11.518137906 -0800 renamed from include/linux/pci.h 2013-12-27 12:51:57.592814680 -0800 +0
-3@@ -386,8 +386,6 @@ static inline int pci_channel_offline(st return (pdev->error_state != pci_channel_io_normal); } -extern struct resource busn_resource; - struct pci_host_bridge_window { struct list_head list; struct resource *res; /* host bridge aperture (CPU address) */@@ -799,7 +797,6 @@ enum pci_lost_interrupt_reason pci_lost_ int pci_find_capability(struct pci_dev *dev, int cap); int pci_find_next_capability(struct pci_dev *dev, u8 pos, int cap); int pci_find_ext_capability(struct pci_dev *dev, int cap); -int pci_find_next_ext_capability(struct pci_dev *dev, int pos, int cap); int pci_find_ht_capability(struct pci_dev *dev, int ht_cap); int pci_find_next_ht_capability(struct pci_dev *dev, int pos, int ht_cap); struct pci_bus *pci_find_next_bus(const struct pci_bus *from);
include/linux/pci.h 2013-12-27 13:03:11.518137906 -0800 static inline int pci_write_config_dword · line 867
Quoted 4 lines
include/linux/pci.h 2013-12-27 13:03:11.518137906 -0800 renamed from include/linux/pci.h 2013-12-27 12:51:57.592814680 -0800 +0
-1@@ -864,7 +861,6 @@ static inline int pci_write_config_dword int pcie_capability_read_word(struct pci_dev *dev, int pos, u16 *val); int pcie_capability_read_dword(struct pci_dev *dev, int pos, u32 *val); int pcie_capability_write_word(struct pci_dev *dev, int pos, u16 val); -int pcie_capability_write_dword(struct pci_dev *dev, int pos, u32 val);
I think we should keep this one to preserve the symmetry of this group of interfaces. We could argue about whether drivers should be accessing the PCIe capability directly in the first place (especially to write to it), but that's a different discussion.
Bjorn