On Sat, Jul 23, 2011 at 10:21:01AM +0530, Rajkumar Manoharan wrote: > On Fri, Jul 22, 2011 at 03:31:51PM +0200, Stanislaw Gruszka wrote: > > + pci_read_config_byte(parent, PCI_EXP_LNKCTL, &aspm); > > + aspm &= ~(PCIE_LINK_STATE_L0S | PCIE_LINK_STATE_L1); > > + pci_write_config_byte(parent, PCI_EXP_LNKCTL, aspm); > > + > Why dont you use pci_disable_link_state instead? Because pci_disable_link_state is noop for CONFIG_PCIEASPM disabled. I have further patches (based on stuff in e1000e, not finished yet) that integrate this into pci core and work on both cases, CONFIG_PCIEASPM disabled and enabled. -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html