Messages in this thread | | | Date | Mon, 9 Aug 2004 12:13:37 +0200 (CEST) | From | Sven-Haegar Koch <> | Subject | Re: Suspend/Resume support for ati-agp |
| |
On Sun, 8 Aug 2004, Pavel Machek wrote:
> > while trying to debug a strange resume problem with 2.6.8-rc3-mm1 and > > software suspend 2 I suspeced ati-agp, and created the following attached > > patch to add powermanagement support for it. > > > > I don't know if it's the completely right thing to do, I just copied the > > way via-agp and intel-agp do it - but perhaps you like it and want to send > > it upstream. > > Looks good to me. > > > ps: > > this did not fix the strange "weird vertical bars instead of movie in > > mplayer after resume" I have, but does not do any bad things either > > ;) > > Hmm, and does it fix anything?
Nothing I could find.
suspend to disk with swsusp in 2.6.8-rc3-mm1 does not work at all for me, I just get a oops longer than screen height after resume, which I can't capture because my thinkpad r40e does not have a serial port. sorry, but I didnt test with it after creating this patch.
suspend to disk with softwaresuspend 2 works mostly, I just need to remove and reload the snd_ali5154 soundmodule after resume, and mplayer output with -vo xv is broken.
pmdisk in 2.6.7-mm5 worked with the same problems as softwaresuspend 2 in the newer kernel.
> > Pavel > > > > --- linux/drivers/char/agp/ati-agp.c.orig 2004-08-08 02:53:10.000000000 +0200 > > +++ linux/drivers/char/agp/ati-agp.c 2004-08-08 03:06:03.000000000 +0200 > > @@ -505,6 +505,33 @@ > > agp_put_bridge(bridge); > > } > > > > +#ifdef CONFIG_PM > > + > > +static int agp_ati_suspend(struct pci_dev *pdev, u32 state) > > +{ > > + pci_save_state (pdev, pdev->saved_config_space); > > + pci_set_power_state (pdev, 3); > > + > > + return 0; > > +} > > + > > +static int agp_ati_resume(struct pci_dev *pdev) > > +{ > > + struct agp_bridge_data *bridge = pci_get_drvdata(pdev); > > + > > + /* set power state 0 and restore PCI space */ > > + pci_set_power_state (pdev, 0); > > + pci_restore_state(pdev, pdev->saved_config_space); > > + > > + /* reconfigure AGP hardware again */ > > + if (bridge->driver == &ati_generic_bridge) > > + return ati_configure(); > > + > > + return 0; > > +} > > + > > +#endif /* CONFIG_PM */ > > + > > static struct pci_device_id agp_ati_pci_table[] = { > > { > > .class = (PCI_CLASS_BRIDGE_HOST << 8), > > @@ -524,6 +551,10 @@ > > .id_table = agp_ati_pci_table, > > .probe = agp_ati_probe, > > .remove = agp_ati_remove, > > +#ifdef CONFIG_PM > > + .suspend = agp_ati_suspend, > > + .resume = agp_ati_resume, > > +#endif > > }; > > > > static int __init agp_ati_init(void) >
c'ya sven
--
The Internet treats censorship as a routing problem, and routes around it. (John Gilmore on http://www.cygnus.com/~gnu/) - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |