[PATCH] xf86: add rgb ordering helper

Daniel Stone daniel at fooishbar.org
Thu Jan 26 13:46:00 PST 2012


On 26 January 2012 22:36, Dave Airlie <airlied at gmail.com> wrote:
> From: Dave Airlie <airlied at redhat.com>
>
> At least the intel, nouveau and radeon driver contain this chunk of code.
>
> We should move it into the server just for the sake of stopping further cut-n-paste.
>
> Drivers can be adapted to use it later.
>
> Signed-off-by: Dave Airlie <airlied at redhat.com>

Can you not drop the ScreenPtr argument and just say that the function
relies on ScrnInfoPtr::pScreen being valid?

With that:
Reviewed-by: Daniel Stone <daniel at fooishbar.org>

Cheers,
Daniel


More information about the xorg-devel mailing list