intel driver suspend/resume failure

Wang Zhenyu zhen78 at gmail.com
Mon Jun 11 08:49:20 PDT 2007


On 2007.06.11 08:05:47 +0000, Keith Packard wrote:
> On Mon, 2007-06-11 at 16:41 +0800, Wang Zhenyu wrote:
> 
> new agp_intel_suspend:
> 
> > +	pci_save_state(pdev);
> > +	pci_save_state(intel_private.pcidev);
> 
> old agp_intel_resume:
> 
> >  	pci_restore_state(pdev);
> 
> > +	if (intel_private.pcidev) {
> 
> Why is there a check for intel_private.pcidev in the resume path but not
> the suspend path?

yep, I should also check it in suspend path, in case like my dell 875p
desktop, which has no gfx chip on MCH. Thanks.




More information about the xorg mailing list