Page MenuHomeFreeBSD

agp: Do not attach to Intel GEN6+
ClosedPublic

Authored by dumbbell on Mar 8 2016, 10:25 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Oct 19, 11:28 AM
Unknown Object (File)
Sep 28 2024, 2:30 PM
Unknown Object (File)
Sep 26 2024, 4:09 AM
Unknown Object (File)
Sep 22 2024, 7:38 PM
Unknown Object (File)
Sep 16 2024, 6:30 PM
Unknown Object (File)
Sep 8 2024, 6:01 PM
Unknown Object (File)
Sep 7 2024, 2:51 PM
Unknown Object (File)
Sep 7 2024, 7:30 AM
Subscribers

Details

Summary

The i915 video driver doesn't depend on agp(4) anymore for Sandybridge and later GPUs. Therefore, there is no need to attach agp(4) to those devices.

kib@, I noticed that the agp_g4x_res_spec variable was used by all GEN6+, but not by G4X. Is is expected?

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 2857
Build 2881: arc lint + arc unit

Event Timeline

dumbbell retitled this revision from to agp: Do not attach to Intel GEN6+.
dumbbell updated this object.
dumbbell edited the test plan for this revision. (Show Details)
dumbbell added reviewers: jhb, kib, kwm, zeising.
kib edited edge metadata.

About agp_g4x_res_spec, I think that it actually does not matter. I re-read GMCH datasheets for 965, G45 and SandyBridge. On all three chipsets, device 0:2:0 BAR at 0x10 is combined mmio and GTT, and BAR at 0x18 is the aperture. So the correct thing to do is to add a name for 965 BAR at 0x18 and add a second resource line to agp_i965_res_spec, which should be used by Gen4+.

I think that i965 and g4x res_specs where split due to my mistake.

This revision is now accepted and ready to land.Mar 9 2016, 12:21 PM

This patch works fine on my sandybridge system.

dumbbell edited edge metadata.

Fix agp_i965_res_spec

This revision now requires review to proceed.Mar 11 2016, 5:52 PM
kib edited edge metadata.

Can we leave the AGP_SNB registers definitions in the header ? I find it easier to consult our reg.h than to search datasheets.

This revision is now accepted and ready to land.Mar 12 2016, 4:21 AM
This revision was automatically updated to reflect the committed changes.