Hi!
> Based on the patch at
> http://unixhead.org/docs/thinkpad/ati-agp/ati-agp.diff, add support for
> suspend/resume in the ati-agp module.
>
> Signed-of-by: Jaco Kroon <[email protected]>
>
> --- linux-2.6.15-rc6/drivers/char/agp/ati-agp.c.orig 2005-12-25
> 22:21:32.000000000 +0200
> +++ linux-2.6.15-rc6/drivers/char/agp/ati-agp.c 2005-12-25
> 22:23:33.000000000 +0200
> @@ -243,6 +243,15 @@
> return 0;
> }
>
> +static int ati_resume(struct pci_dev *dev)
> +{
> + return ati_configure();
> +}
> +
> +static int ati_suspend(struct pci_dev *dev, pm_message_t state)
> +{
> + return 0;
> +}
I think you can just leave out empty function; that should work,
too. Otherwise it looks good, thanks... ...
> /*
> *Since we don't need contigious memory we just try
> @@ -525,6 +534,8 @@
> .id_table = agp_ati_pci_table,
> .probe = agp_ati_probe,
> .remove = agp_ati_remove,
> + .resume = ati_resume,
> + .suspend = ati_suspend,
> };
...aha, plus you may want to keep naming convention. It should
probably be agp_ati_resume.
Pavel
--
Thanks, Sharp!
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
