Re: [PATCH] uxa/glamor: Refine CloseScreen and InitScreen process.

[Chris Wilson <chris@chris-wilson.co.uk>]

On Wed,  1 Feb 2012 19:47:28 +0800, zhigang.gong@linux.intel.com wrote:
> From: Zhigang Gong <zhigang.gong@linux.intel.com>
> 
> The previous version calls glamor_egl_close_screen and
> glamor_egl_free_screen manually which is not align with
> standard process. Now glamor change the way to follow
> standard method:
> 
> glamor layer and glamor egl layer both have their internal
> CloseScreens. The correct sequence is after the I830CloseScreen
> is registered, then register glamor_egl_close_screen and
> the last one is glamor_close_screen. So we move out the
> intel_glamor_init from the intel_uxa_init to I830ScreenInit
> and just after the registration of I830CloseScreen.
> 
> As the glamor interfaces changed, we need to check the
> glamor version when load the glamor egl module to make
> sure we are loading the right glamor module. If
> failed, it will switch back to UXA path.
> 
> Signed-off-by: Zhigang Gong <zhigang.gong@linux.intel.com>

I added the note about the glamor commit to this and the following patch
and pushed. Thanks,
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre