diff options
author | Linas Vepstas <linas@austin.ibm.com> | 2007-04-13 18:34:12 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2007-05-02 22:02:39 -0400 |
commit | 517d5a0417e19101eaa769039d1921d626ee546c (patch) | |
tree | 585258a6a1d8cdd732809237dd94d289b020a262 /drivers/pci | |
parent | 3499f0726ec179afd19669070681ac457d27033f (diff) |
PCI: rpaphp: remove a function that does nothing but wrap debug printks
Remove a stove-pipe-- a function that is called from only one place,
does nothing but wraps another function with debug printk's.
Signed-off-by: Linas Vepstas <linas@austin.ibm.com>
Cc: John Rose <johnrose@austin.ibm.com>
Signed-off-by: Kristen Carlson Accardi <kristen.c.accardi@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/pci')
-rw-r--r-- | drivers/pci/hotplug/rpaphp_pci.c | 37 |
1 files changed, 13 insertions, 24 deletions
diff --git a/drivers/pci/hotplug/rpaphp_pci.c b/drivers/pci/hotplug/rpaphp_pci.c index a669ba3f29c9..881e8073c807 100644 --- a/drivers/pci/hotplug/rpaphp_pci.c +++ b/drivers/pci/hotplug/rpaphp_pci.c | |||
@@ -116,23 +116,6 @@ static void print_slot_pci_funcs(struct pci_bus *bus) | |||
116 | return; | 116 | return; |
117 | } | 117 | } |
118 | 118 | ||
119 | static int setup_pci_hotplug_slot_info(struct slot *slot) | ||
120 | { | ||
121 | struct hotplug_slot_info *hotplug_slot_info = slot->hotplug_slot->info; | ||
122 | |||
123 | dbg("%s Initilize the PCI slot's hotplug->info structure ...\n", | ||
124 | __FUNCTION__); | ||
125 | rpaphp_get_power_status(slot, &hotplug_slot_info->power_status); | ||
126 | rpaphp_get_pci_adapter_status(slot, 1, | ||
127 | &hotplug_slot_info->adapter_status); | ||
128 | if (hotplug_slot_info->adapter_status == NOT_VALID) { | ||
129 | err("%s: NOT_VALID: skip dn->full_name=%s\n", | ||
130 | __FUNCTION__, slot->dn->full_name); | ||
131 | return -EINVAL; | ||
132 | } | ||
133 | return 0; | ||
134 | } | ||
135 | |||
136 | static void set_slot_name(struct slot *slot) | 119 | static void set_slot_name(struct slot *slot) |
137 | { | 120 | { |
138 | struct pci_bus *bus = slot->bus; | 121 | struct pci_bus *bus = slot->bus; |
@@ -200,14 +183,20 @@ exit_rc: | |||
200 | 183 | ||
201 | int rpaphp_register_pci_slot(struct slot *slot) | 184 | int rpaphp_register_pci_slot(struct slot *slot) |
202 | { | 185 | { |
203 | int rc = -EINVAL; | 186 | struct hotplug_slot_info *info = slot->hotplug_slot->info; |
187 | |||
188 | rpaphp_get_power_status(slot, &info->power_status); | ||
189 | rpaphp_get_pci_adapter_status(slot, 1, &info->adapter_status); | ||
190 | |||
191 | if (info->adapter_status == NOT_VALID) { | ||
192 | err("%s: NOT_VALID: skip dn->full_name=%s\n", | ||
193 | __FUNCTION__, slot->dn->full_name); | ||
194 | return -EINVAL; | ||
195 | } | ||
204 | 196 | ||
205 | if (setup_pci_hotplug_slot_info(slot)) | ||
206 | goto exit_rc; | ||
207 | if (setup_pci_slot(slot)) | 197 | if (setup_pci_slot(slot)) |
208 | goto exit_rc; | 198 | return -EINVAL; |
209 | rc = rpaphp_register_slot(slot); | 199 | |
210 | exit_rc: | 200 | return rpaphp_register_slot(slot); |
211 | return rc; | ||
212 | } | 201 | } |
213 | 202 | ||