Thanks very much for picking up the diagnoses we did and joining the pieces together with other APSM issues with other chipsets. :-) I'm glad it's finally being resolved! Adrian On 25 July 2011 17:16, Stanislaw Gruszka <sgruszka@xxxxxxxxxx> wrote: > 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
