[lkml]   [2008]   [Apr]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
Subjectpci commands resume order [Was: 2.6.25-git2: BUG: unable to handle kernel paging request at ffffffffffffffff]
On 04/25/2008 09:45 AM, David Miller wrote:
> I notice Jiri, in your hardware list, you have an ath5k Atheros AR5212 chip
> in there.
> I took a look at the resume code for ath5k but nothing really suspicious
> there except:
> err = pci_enable_device(pdev);
> if (err)
> return err;
> pci_restore_state(pdev);
> Shouldn't we restore state before we turn the chip back on and thus
> potentially let it start DMA'ing all over the place?

Hmm, actually every second wireless driver do this :/. I think it's wrong too.

BTW pci_set_power_state(pdev, PCI_D0); in resume isn't needed at all, right?
It's done in pci_enable_device, isn't it?

 \ /
  Last update: 2008-04-25 10:23    [W:0.238 / U:6.148 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site