[Intel-gfx] [PATCH] drm/i915: apply phase pointer override on SNB+ too

Keith Packard keithp at keithp.com
Fri Jul 29 19:28:51 CEST 2011


On Thu, 28 Jul 2011 17:07:12 -0700, Jesse Barnes <jbarnes at virtuousgeek.org> wrote:

> +#define _TRANSA_CHICKEN2	 0xf0064
> +#define _TRANSB_CHICKEN2	 0xf1064
> +#define TRANS_CHICKEN2(pipe) _PIPE(pipe, _TRANSA_CHICKEN2, _TRANSB_CHICKEN2)
> +#define   TRANS_AUTOTRAIN_GEN_STALL_DIS	(1<<31)
> +
> +#define SOUTH_CHICKEN1		0xc2000
> +#define  FDIA_PHASE_SYNC_SHIFT	18
> +#define  FDI_PHASE_SYNC_OVR_EN	(3)

How about

#define FDIA_PHASE_SYNC_OVERRIDE_SHIFT  19
#define FDIA_PHASE_SYNC_ENABLE_SHIFT    18
#define FDI_PHASE_SYNC_OVERRIDE(pipe) (1 << (FDIA_PHASE_SYNC_OVERRIDE_SHIFT - (pipe) * 2))
#define FDI_PHASE_SYNC_ENABLE(pipe)   (1 << (FDIA_PHASE_SYNC_ENABLE_SHIFT - (pipe) * 2))

defines instead?

Then use FDI_PHASE_SYNC_OVERRIDE(pipe) | FDI_PHASE_SYNC_ENABLE(pipe)

in the code.

-- 
keith.packard at intel.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/intel-gfx/attachments/20110729/0049c003/attachment.sig>


More information about the Intel-gfx mailing list