On Tue, 06 Dec 2011 11:56:07 -0800
Eric Anholt <[email protected]> wrote:
> > It's one of those "been there forever" bits.  A comment wouldn't hurt,
> > but we should also put in a doc change request (in fact I think I did
> > that but it went into a black hole).
> 
> Well, by "been there forever", you mean "introduced in:
> 
> commit a69ffdbfcba8eabf2ca9d384b578e6f28b339c61
> Author: Zhenyu Wang <[email protected]>
> Date:   Mon Aug 30 16:12:42 2010 +0800
> 
>     drm/i915: Enable MI_FLUSH on Sandybridge
>     
>     MI_FLUSH is being deprecated, but still available on Sandybridge.
>     Make sure it's enabled as userspace still uses MI_FLUSH.
>     
>     Signed-off-by: Zhenyu Wang <[email protected]>
>     Cc: [email protected]
>     Signed-off-by: Chris Wilson <[email protected]>"
> 
> It should be removed on Ironlake, and fixed to be 12 on SNB or just
> removed.

No, "fixed to be 12" would just encourage people to still try it and
break their machines.  We can safely remove it now that we don't use it
anywhere.  And we can fix the docs so that if people do want to mess
with it they won't try bit 12.

-- 
Jesse Barnes, Intel Open Source Technology Center

Attachment: signature.asc
Description: PGP signature

_______________________________________________
Intel-gfx mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to