diff options
author | Daniel Vetter <daniel.vetter@ffwll.ch> | 2010-09-24 15:14:22 -0400 |
---|---|---|
committer | Chris Wilson <chris@chris-wilson.co.uk> | 2010-09-25 07:23:15 -0400 |
commit | 3d281d8cca1acb2483444e0d1519c8ab6dda3a47 (patch) | |
tree | 9d40394e09361254aa1c8208f5b56dfbd1a31cf1 /drivers/gpu/drm/i915/intel_ringbuffer.c | |
parent | bf7e0e1268f72ea1687140603a910eeaca031fa1 (diff) |
drm/i915: kill per-ring macros
Two macros that use a base address for HWS_PGA were missing, add them.
Also switch the remaining users of *_ACTHD to the ring-base one.
Kill the other ring-specific macros because they're now unused.
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
[ickle: And silence checkpatch whilst in the vicinity]
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Diffstat (limited to 'drivers/gpu/drm/i915/intel_ringbuffer.c')
-rw-r--r-- | drivers/gpu/drm/i915/intel_ringbuffer.c | 25 |
1 files changed, 14 insertions, 11 deletions
diff --git a/drivers/gpu/drm/i915/intel_ringbuffer.c b/drivers/gpu/drm/i915/intel_ringbuffer.c index ede436ba22d2..487575f2340d 100644 --- a/drivers/gpu/drm/i915/intel_ringbuffer.c +++ b/drivers/gpu/drm/i915/intel_ringbuffer.c | |||
@@ -131,7 +131,8 @@ static unsigned int render_ring_get_active_head(struct drm_device *dev, | |||
131 | struct intel_ring_buffer *ring) | 131 | struct intel_ring_buffer *ring) |
132 | { | 132 | { |
133 | drm_i915_private_t *dev_priv = dev->dev_private; | 133 | drm_i915_private_t *dev_priv = dev->dev_private; |
134 | u32 acthd_reg = INTEL_INFO(dev)->gen >= 4 ? ACTHD_I965 : ACTHD; | 134 | u32 acthd_reg = INTEL_INFO(dev)->gen >= 4 ? |
135 | RING_ACTHD(ring->mmio_base) : ACTHD; | ||
135 | 136 | ||
136 | return I915_READ(acthd_reg); | 137 | return I915_READ(acthd_reg); |
137 | } | 138 | } |
@@ -352,11 +353,13 @@ static void render_setup_status_page(struct drm_device *dev, | |||
352 | { | 353 | { |
353 | drm_i915_private_t *dev_priv = dev->dev_private; | 354 | drm_i915_private_t *dev_priv = dev->dev_private; |
354 | if (IS_GEN6(dev)) { | 355 | if (IS_GEN6(dev)) { |
355 | I915_WRITE(HWS_PGA_GEN6, ring->status_page.gfx_addr); | 356 | I915_WRITE(RING_HWS_PGA_GEN6(ring->mmio_base), |
356 | I915_READ(HWS_PGA_GEN6); /* posting read */ | 357 | ring->status_page.gfx_addr); |
358 | I915_READ(RING_HWS_PGA_GEN6(ring->mmio_base)); /* posting read */ | ||
357 | } else { | 359 | } else { |
358 | I915_WRITE(HWS_PGA, ring->status_page.gfx_addr); | 360 | I915_WRITE(RING_HWS_PGA(ring->mmio_base), |
359 | I915_READ(HWS_PGA); /* posting read */ | 361 | ring->status_page.gfx_addr); |
362 | I915_READ(RING_HWS_PGA(ring->mmio_base)); /* posting read */ | ||
360 | } | 363 | } |
361 | 364 | ||
362 | } | 365 | } |
@@ -377,7 +380,7 @@ static unsigned int bsd_ring_get_active_head(struct drm_device *dev, | |||
377 | struct intel_ring_buffer *ring) | 380 | struct intel_ring_buffer *ring) |
378 | { | 381 | { |
379 | drm_i915_private_t *dev_priv = dev->dev_private; | 382 | drm_i915_private_t *dev_priv = dev->dev_private; |
380 | return I915_READ(BSD_RING_ACTHD); | 383 | return I915_READ(RING_ACTHD(ring->mmio_base)); |
381 | } | 384 | } |
382 | 385 | ||
383 | static int init_bsd_ring(struct drm_device *dev, | 386 | static int init_bsd_ring(struct drm_device *dev, |
@@ -412,8 +415,8 @@ static void bsd_setup_status_page(struct drm_device *dev, | |||
412 | struct intel_ring_buffer *ring) | 415 | struct intel_ring_buffer *ring) |
413 | { | 416 | { |
414 | drm_i915_private_t *dev_priv = dev->dev_private; | 417 | drm_i915_private_t *dev_priv = dev->dev_private; |
415 | I915_WRITE(BSD_HWS_PGA, ring->status_page.gfx_addr); | 418 | I915_WRITE(RING_HWS_PGA(ring->mmio_base), ring->status_page.gfx_addr); |
416 | I915_READ(BSD_HWS_PGA); | 419 | I915_READ(RING_HWS_PGA(ring->mmio_base)); |
417 | } | 420 | } |
418 | 421 | ||
419 | static void | 422 | static void |
@@ -801,8 +804,8 @@ static void gen6_bsd_setup_status_page(struct drm_device *dev, | |||
801 | struct intel_ring_buffer *ring) | 804 | struct intel_ring_buffer *ring) |
802 | { | 805 | { |
803 | drm_i915_private_t *dev_priv = dev->dev_private; | 806 | drm_i915_private_t *dev_priv = dev->dev_private; |
804 | I915_WRITE(GEN6_BSD_HWS_PGA, ring->status_page.gfx_addr); | 807 | I915_WRITE(RING_HWS_PGA_GEN6(ring->mmio_base), ring->status_page.gfx_addr); |
805 | I915_READ(GEN6_BSD_HWS_PGA); | 808 | I915_READ(RING_HWS_PGA_GEN6(ring->mmio_base)); |
806 | } | 809 | } |
807 | 810 | ||
808 | static void gen6_bsd_ring_set_tail(struct drm_device *dev, | 811 | static void gen6_bsd_ring_set_tail(struct drm_device *dev, |
@@ -832,7 +835,7 @@ static unsigned int gen6_bsd_ring_get_active_head(struct drm_device *dev, | |||
832 | struct intel_ring_buffer *ring) | 835 | struct intel_ring_buffer *ring) |
833 | { | 836 | { |
834 | drm_i915_private_t *dev_priv = dev->dev_private; | 837 | drm_i915_private_t *dev_priv = dev->dev_private; |
835 | return I915_READ(GEN6_BSD_RING_ACTHD); | 838 | return I915_READ(RING_ACTHD(ring->mmio_base)); |
836 | } | 839 | } |
837 | 840 | ||
838 | static void gen6_bsd_ring_flush(struct drm_device *dev, | 841 | static void gen6_bsd_ring_flush(struct drm_device *dev, |