On Mon, 2009-02-09 at 11:51 -0800, Helmut Schaa wrote: > Am Montag, 9. Februar 2009 schrieb reinette chatre: > > On Mon, 2009-02-09 at 11:17 -0800, drago01 wrote: > > > On Tue, Feb 3, 2009 at 7:20 PM, Reinette Chatre > > > <reinette.chatre@xxxxxxxxx> wrote: > > > > This is the right thing to do and fixes the following warning: > > > > [...] > > > > > > With 2.6.29-rc4-git1 (which has this patch) I sometimes get: > > > > > > iwl3945: Wait for START_ALIVE timeout after 2000ms. > > > iwl3945 0000:07:00.0: PCI INT A disabled > > > > > > After resume with results into no working wireless until the module is reloaded. > > > I am not sure if this patch is the culprit (have not tried to revert > > > it), but that's the only change that might have caused this (with rc3 > > > it was working fine, it just triggered the warning that this patch > > > supposed to solve). > > > > That patch is the culprit. Please see current status at > > http://bugzilla.kernel.org/show_bug.cgi?id=12656 > > Reinette, maybe we could just use the rfkill-patch [1] I've submitted > some time ago. Of course it does a bit more then just saving and restoring > the pci state ;) but it changes iwl3945 to only use pci_save_state and > pci_restore_state in the suspend and resume handlers which should fix the > issue you've pointed out in comment #10. indeed ... thanks to you we do not need to fix 2.6.30. As you say this is a big change - I am going to try to get the one line fix in before going this route because 2.6.29 is only looking for fixes right now. Your changes are much cleaner though and I am very happy that it is in wireless-testing already. Reinette -- 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
