[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.025 / U:77.240 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site