On Fri,  3 Feb 2012 17:47:15 -0200
Paulo Zanoni <[email protected]> wrote:

> From: Paulo Zanoni <[email protected]>
> 
> I'm not sure why they are needed (I didn't notice any difference in my
> tests), but these bits are in our documentation and they are also set by
> the Windows driver.
> 
> Signed-off-by: Paulo Zanoni <[email protected]>
> ---

This makes me think we should really just split the PCH related stuff
into separate functions and headers with their own register and bit
definitions.  Duplicate code is better than fragility and having to
scour through multiple docs when shared code is modified.

But obviously that's a much bigger cleanup that can come after this
patch.

Ideally, anytime we had a register block with any different offsets or
bits than a previous generation, we'd fork it into its own reg
definition header and add function hooks for it.

-- 
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