Matthew Garrett wrote:
On Tue, Nov 04, 2008 at 10:58:11AM +0900, Kenji Kaneshige wrote:
- Even with pciehp_passive option, pciehp driver controls hotplug
related registers at the initialization time, enabling software
notification mechanism for hotplug events, trying to turn power
on the slot and so on. Is this your intended behaviour?
Yes, although the most recent version of the patch doesn't do the
forcible power on if no card is detected.
Oh, sorry.
I didn't noticed it was already changed in your v2 patch.
- Maybe I don't understand what "pciehp will listen for events..."
in your patch description means. But if you expect the pciehp's
interrupts for hotplug events, it would not work properly when
hotplug control is not granted through _OSC or OSHP.
What do you mean by "not work properly"? The hardware we've tested with
fires events even without an OSHP method being present. That's the case
we're trying to deal with.
Because the explanation of PCI Express Native Hot Plug control bit in
_OSC return value in PCI firmware spec says
"... If firmware allows the operating system control of this feature,
then in the context of the _OSC method, it must ensure that all hot
plug events are routed to device interrupts as described in the PCI
Express Base Specification. ..."
My understanding is that it is not ensured that all hot plug events
are routed to device interrupts, if hotplug control is not granted.
Thanks,
Kenji Kaneshige
t_slot->hpc_ops->get_adapter_status(t_slot, &value); /* Check if
slot is occupied */
- if (value && pciehp_force) {
+ if (value && (pciehp_force || pciehp_passive)) {
rc = pciehp_enable_slot(t_slot);
if (rc) /* -ENODEV: shouldn't happen, but deal with it */
value = 0;
This code no longer runs in the pciehp_passive case. However, by the
looks of it it still does in the resume case - that probably wants
fixing.
--
To unsubscribe from this list: send the line "unsubscribe linux-pci" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
