[PATCH] intel_agp: fix G965 GTT size detect

[PATCH] intel_agp: fix G965 GTT size detect

[Wang Zhenyu]

Dave, 

On G965, I810_PGETBL_CTL is a mmio offset, but we wrongly take it
as pci config space offset in detecting GTT size. This one line patch
fixs this. 

Signed-off-by: Wang Zhenyu <zhenyu.z.wang@intel.com>

---
diff --git a/drivers/char/agp/intel-agp.c b/drivers/char/agp/intel-agp.c
index a9fdbf9..4c05c71 100644
--- a/drivers/char/agp/intel-agp.c
+++ b/drivers/char/agp/intel-agp.c
@@ -431,9 +431,8 @@ static void intel_i830_init_gtt_entries(
 
 	if (IS_I965) {
 		u32 pgetbl_ctl;
-
-		pci_read_config_dword(agp_bridge->dev, I810_PGETBL_CTL,
-				      &pgetbl_ctl);
+		pgetbl_ctl = readl(intel_i830_private.registers+I810_PGETBL_CTL);
+		    	
 		/* The 965 has a field telling us the size of the GTT,
 		 * which may be larger than what is necessary to map the
 		 * aperture.
---

Re: [PATCH] intel_agp: fix G965 GTT size detect

[Dave Jones]

On Tue, Apr 10, 2007 at 09:42:48AM +0800, Wang Zhenyu wrote:
 > 
 > Dave, 
 > 
 > On G965, I810_PGETBL_CTL is a mmio offset, but we wrongly take it
 > as pci config space offset in detecting GTT size. This one line patch
 > fixs this. 

Thanks, applied, and pushed on to Linus for .21

Be careful with trailing whitespace btw, git-applymbox complains loudly
when merging such patches. (I fixed it up by hand this time)

	Dave

-- 
http://www.codemonkey.org.uk