RE: [PATCH] Add pci_save_state() to ALSA
From: Zhu, Yi
Date: Fri Nov 12 2004 - 08:02:21 EST
Takashi Iwai wrote:
> pci_save_state() is called internally in
> drivers/pci/pci-driver.c:pci_device_suspend(), so it's redundant.
>> My laptop doesn't resume (gets what I assume is an ACPI timeout and
>> hangs solid) without this small obvious patch.
> I'm wondering how this can fix your problem...
For example, some devices call pci_save_state before pci_disable_device
->suspend, but don't pci_enable_device in ->resume. This works before,
is broken after the pci_save_state() change. We need to find those
and change the individual drivers instead of this simple fix.
Martin, which sound driver do you use?
>> Signed-off-by: Martin Josefsson <gandalf@xxxxxxxxxxxxxx>
>> --- linux-2.6.10-rc1-bk21.orig/sound/core/init.c
> 2004-11-11 18:51:17.000000000 +0100
>> +++ linux-2.6.10-rc1-bk21/sound/core/init.c 2004-11-11
>> 20:57:52.000000000 +0100 @@ -789,6 +789,8 @@ int
>> snd_card_pci_suspend(struct pci_dev return 0; if
>> (card->power_state == SNDRV_CTL_POWER_D3hot) return 0;
>> + /* save the PCI config space */
>> + pci_save_state(dev);
>> /* FIXME: correct state value? */
>> return card->pm_suspend(card, 0);
> To unsubscribe from this list: send the line "unsubscribe
> linux-kernel" in the body of a message to
> majordomo@xxxxxxxxxxxxxxx More majordomo info at
> Please read the FAQ at http://www.tux.org/lkml/
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/