On Fri, 2009-06-05 at 13:15 +0000, Eric Anholt wrote:

> s-o-b?  also, a weird cherry-pick message here.

operator error.

> Seems like #10 was "Import DisplayPort" code and this is something
else?

Yeah, I've gone ahead and just merged all of the 'add display port' into
one giant patch.

> No // comments in kernel code.

Removed.

> gratuitous move of code in this commit?

Un-moved.

> The commmit doesn't seem to match the commit message here.

Actually, it does, in that the patch doesn't actually work, and that a
'real' fix would involve re-setting the current mode instead of
attempting to save/restore simple register contents. I've just removed
this patch from the series; it doesn't work, and it isn't even moving in
the right direction.

> Looks like this should be squashed into #14 that uses it?

Squashed.

(new patch series coming shortly)

-- 
keith.pack...@intel.com

Attachment: signature.asc
Description: This is a digitally signed message part

------------------------------------------------------------------------------
OpenSolaris 2009.06 is a cutting edge operating system for enterprises 
looking to deploy the next generation of Solaris that includes the latest 
innovations from Sun and the OpenSource community. Download a copy and 
enjoy capabilities such as Networking, Storage and Virtualization. 
Go to: http://p.sf.net/sfu/opensolaris-get
--
_______________________________________________
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel

Reply via email to