diff options
author | Mark Lord <lkml@rtr.ca> | 2007-11-28 18:11:28 -0500 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2008-02-01 18:04:22 -0500 |
commit | 0a3c33d77ff7ad5b988997536a8f09c49e35ad20 (patch) | |
tree | 62fa58c6aa0e51ee1e01a27580cfddb24aa88acf /drivers/pci/hotplug | |
parent | a6f29a98af278a8d784a870ae9f08e530de73ffe (diff) |
PCIE: fix PCIe Hotplug so that it works with ExpressCard slots on Dell notebooks (and others?) in conjunction with modparam of pciehp_force=1.
Fix pciehp_probe() to deal with ExpressCard cards
that were inserted prior to the driver being loaded.
Signed-off-by: Mark Lord <mlord@pobox.com>
Signed-off-by: Kristen Carlson Accardi <kristen.c.accardi@intel.com>
Cc: Andrew Morton <akpm@linux-foundation.org>
Cc: Theodore Ts'o <tytso@mit.edu>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/pci/hotplug')
-rw-r--r-- | drivers/pci/hotplug/pciehp.h | 1 | ||||
-rw-r--r-- | drivers/pci/hotplug/pciehp_core.c | 5 | ||||
-rw-r--r-- | drivers/pci/hotplug/pciehp_ctrl.c | 1 |
3 files changed, 6 insertions, 1 deletions
diff --git a/drivers/pci/hotplug/pciehp.h b/drivers/pci/hotplug/pciehp.h index 7959c222dc24..968f57087ce0 100644 --- a/drivers/pci/hotplug/pciehp.h +++ b/drivers/pci/hotplug/pciehp.h | |||
@@ -161,6 +161,7 @@ extern int pciehp_configure_device(struct slot *p_slot); | |||
161 | extern int pciehp_unconfigure_device(struct slot *p_slot); | 161 | extern int pciehp_unconfigure_device(struct slot *p_slot); |
162 | extern void pciehp_queue_pushbutton_work(struct work_struct *work); | 162 | extern void pciehp_queue_pushbutton_work(struct work_struct *work); |
163 | int pcie_init(struct controller *ctrl, struct pcie_device *dev); | 163 | int pcie_init(struct controller *ctrl, struct pcie_device *dev); |
164 | int pciehp_enable_slot(struct slot *p_slot); | ||
164 | 165 | ||
165 | static inline struct slot *pciehp_find_slot(struct controller *ctrl, u8 device) | 166 | static inline struct slot *pciehp_find_slot(struct controller *ctrl, u8 device) |
166 | { | 167 | { |
diff --git a/drivers/pci/hotplug/pciehp_core.c b/drivers/pci/hotplug/pciehp_core.c index 6462ac3b405f..74f4f7bc8db9 100644 --- a/drivers/pci/hotplug/pciehp_core.c +++ b/drivers/pci/hotplug/pciehp_core.c | |||
@@ -471,6 +471,11 @@ static int pciehp_probe(struct pcie_device *dev, const struct pcie_port_service_ | |||
471 | t_slot = pciehp_find_slot(ctrl, ctrl->slot_device_offset); | 471 | t_slot = pciehp_find_slot(ctrl, ctrl->slot_device_offset); |
472 | 472 | ||
473 | t_slot->hpc_ops->get_adapter_status(t_slot, &value); /* Check if slot is occupied */ | 473 | t_slot->hpc_ops->get_adapter_status(t_slot, &value); /* Check if slot is occupied */ |
474 | if (value) { | ||
475 | rc = pciehp_enable_slot(t_slot); | ||
476 | if (rc) /* -ENODEV: shouldn't happen, but deal with it */ | ||
477 | value = 0; | ||
478 | } | ||
474 | if ((POWER_CTRL(ctrl->ctrlcap)) && !value) { | 479 | if ((POWER_CTRL(ctrl->ctrlcap)) && !value) { |
475 | rc = t_slot->hpc_ops->power_off_slot(t_slot); /* Power off slot if not occupied*/ | 480 | rc = t_slot->hpc_ops->power_off_slot(t_slot); /* Power off slot if not occupied*/ |
476 | if (rc) | 481 | if (rc) |
diff --git a/drivers/pci/hotplug/pciehp_ctrl.c b/drivers/pci/hotplug/pciehp_ctrl.c index f1e0966cee95..342cfc6c1de8 100644 --- a/drivers/pci/hotplug/pciehp_ctrl.c +++ b/drivers/pci/hotplug/pciehp_ctrl.c | |||
@@ -37,7 +37,6 @@ | |||
37 | #include "pciehp.h" | 37 | #include "pciehp.h" |
38 | 38 | ||
39 | static void interrupt_event_handler(struct work_struct *work); | 39 | static void interrupt_event_handler(struct work_struct *work); |
40 | static int pciehp_enable_slot(struct slot *p_slot); | ||
41 | static int pciehp_disable_slot(struct slot *p_slot); | 40 | static int pciehp_disable_slot(struct slot *p_slot); |
42 | 41 | ||
43 | static int queue_interrupt_event(struct slot *p_slot, u32 event_type) | 42 | static int queue_interrupt_event(struct slot *p_slot, u32 event_type) |