On Tuesday 24 March 2009, Benjamin Herrenschmidt wrote: > On Tue, 2009-03-24 at 12:00 +0100, Rafael J. Wysocki wrote: > > > In fact I have yet another idea. If we use the "retransmission with exponential > > backoff" algorithm in pci_raw_set_power_state(), we won't have to add any > > extra parameters to pci_set_power_state() and the radeon case will be covered > > automatically. That should also cover any other devices having similar > > problems IMO. > > > > A patch to implement that is appended, please tell me what you think. > > This is crazy.... > > Most devices don't need that and those who are a bit "touchy" may well > puke on being written to too fast while they are in the middle of > the transition. > > Let's wait to see if Alex from ATI has an answer here, and if not, I > would suggest keeping the dirt inside radeonfb. OK, so what would you like to do exactly? The only thing I see is not to set current_state to PCI_D2 before calling the final pci_set_power_state(). Rafael -- 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
