Re: [PATCH 05/11] drm/i915: ppgtt register definitions

[Ben Widawsky <ben@bwidawsk.net>]

On Mon, Nov 28, 2011 at 09:35:32PM +0100, Daniel Vetter wrote:
> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
> ---
>  drivers/gpu/drm/i915/i915_reg.h |   18 ++++++++++++++++++
>  1 files changed, 18 insertions(+), 0 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/i915_reg.h b/drivers/gpu/drm/i915/i915_reg.h
> index e9f5dc8..7227446 100644
> --- a/drivers/gpu/drm/i915/i915_reg.h
> +++ b/drivers/gpu/drm/i915/i915_reg.h
> @@ -86,6 +86,13 @@
>  #define   GEN6_MBC_SNPCR_LOW	(2<<21)
>  #define   GEN6_MBC_SNPCR_MIN	(3<<21) /* only 1/16th of the cache is shared */
>  
> +#define GEN6_MBCTL		0x0907c
> +#define   GEN6_MBCTL_ENABLE_BOOT_FETCH	(1 << 4)
> +#define   GEN6_MBCTL_CTX_FETCH_NEEDED	(1 << 3)
> +#define   GEN6_MBCTL_BME_UPDATE_ENABLE	(1 << 2)
> +#define   GEN6_MBCTL_MAE_UPDATE_ENABLE	(1 << 1)
> +#define   GEN6_MBCTL_BOOT_FETCH_MECH	(1 << 0)
> +

Maybe grep fail, but I don't see these registers used later in the
series.

>  #define GEN6_GDRST	0x941c
>  #define  GEN6_GRDOM_FULL		(1 << 0)
>  #define  GEN6_GRDOM_RENDER		(1 << 1)
> @@ -110,6 +117,16 @@
>  
>  #define GEN6_PTES_PER_PD		1024
>  
> +#define RING_PP_DIR_BASE(ring)		((ring)->mmio_base+0x228)
> +#define RING_PP_DIR_BASE_READ(ring)	((ring)->mmio_base+0x518)
> +#define RING_PP_DIR_DCLV(ring)		((ring)->mmio_base+0x220)
> +#define   PP_DIR_DCLV_2G		0xffffffff
> +
> +#define GAM_ECOCHK			0x4090
> +#define   ECOCHK_SNB_BIT		(1<<10)
> +#define   ECOCHK_PPGTT_CACHE64B		(0x3<<3)
> +#define   ECOCHK_PPGTT_CACHE4B		(0x0<<3)
> +
>  /* VGA stuff */
>  
>  #define VGA_ST01_MDA 0x3ba
> @@ -422,6 +439,7 @@
>  
>  #define GFX_MODE	0x02520
>  #define GFX_MODE_GEN7	0x0229c
> +#define RING_MODE_GEN7(ring)	((ring)->mmio_base+0x29c)
>  #define   GFX_RUN_LIST_ENABLE		(1<<15)
>  #define   GFX_TLB_INVALIDATE_ALWAYS	(1<<13)
>  #define   GFX_SURFACE_FAULT_ENABLE	(1<<12)

Ben