On Thu, 28 Jul 2011 17:07:12 -0700, Jesse Barnes <jbar...@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.pack...@intel.com

Attachment: pgp6nFQPoNZ42.pgp
Description: PGP signature

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to