On Thu, 19 Apr 2012, Stephen Warren wrote:
> >> Note: I found that the immediate resume is because hcd_bus_suspend()
> >> aborts part-way through due to "suspend raced with wakeup event", which
> >> is due to ehci_hub_status_data() finding that port_status has all of
> >> PORT_CSC | PORT_PEC | PORT_OCC set. Is it worth tracking that down
> >> further? I have no idea if it's an ehci-tegra.c issue or quirk in our
> >> EHCI HW.
> >
> > It's probably because of the recent change to hcd_bus_suspend().
> > Before that change, it did not call ehci_hub_status_data() after doing
> > the suspend. But now you see the problem: ehci_hub_status_data()
> > returns bogus information because the controller has been powered down.
> > That's the reason why I wanted to update ehci-tegra.c.
>
> Aha, I'm beginning to understand!
>
> > If you revert commit 879d38e6bc36d73b0ac40ec9b0d839fda9fa8b1a, how well
> > does the current driver behave?
>
> It's basically the same as the patch you posted at the start of this
> thread. In other words:
>
> * When nothing is plugged in, the port(hub) suspends and stays
> suspended; no oscillating.
>
> * When I plug something in, the system doesn't see it.
>
> * If I then manually resume the hub ("echo on > usb3/power/control"),
> new devices are seen, and devices can be unplugged/replugged and seen.
>
> * While a device is plugged in, I can "echo auto > usb3/power/control"
> and it stays operational.
>
> So, I guess the only question is: the patch you posted was to solve the
> problem of seeing bogus status during suspend, which causes the
> immediate resume; Why doesn't that work?
According to what you said earlier, it _does_ work. That is, you said
with my patch to ehci-tegra the controller does not resume immediately;
it goes to low power and stays that way even when a new device is
plugged in. Which means that wakeup is a problem, but then it's also a
problem with the existing driver.
Thus the combination of 879d38e6b plus my ehci-tegra patch leaves
the behavior basically unchanged. Which is what I wanted to verify.
Unless you have an objection, I'll submit the patch (including the
change that calls pm_runtime_forbid() unconditionally).
Alan Stern
--
To unsubscribe from this list: send the line "unsubscribe linux-tegra" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
