[PATCH] xf86: add rgb ordering helper

Dave Airlie airlied at gmail.com
Thu Jan 26 23:06:29 PST 2012


On Thu, Jan 26, 2012 at 10:32 PM, Daniel Stone <daniel at fooishbar.org> wrote:
> Hi,
>
> On 27 January 2012 09:28, Aaron Plattner <aplattner at nvidia.com> wrote:
>> On 01/26/2012 01:46 PM, Daniel Stone wrote:
>>> Can you not drop the ScreenPtr argument and just say that the function
>>> relies on ScrnInfoPtr::pScreen being valid?
>>
>> It would presumably be typically called from ScreenInit, where
>> pScrn->pScreen is NULL.
>>
>> If you want to have the server wire that up earlier, that'd be another
>> matter.
>
> Rather.  A few drivers (at least radeon, nouveau, and vmware, as well
> as the one I just wrote) wire it up during ScreenInit, which I'd
> thought was more widespread.  I don't see any downside to doing it in
> the server before ScreenInit though.

Yeah thats not the shed I was painting right now.

So we can either change it to passing pScreen, and doing the
xf86Screens[pScreen->myNum] dance, or we can leave as it is.

Dave.
>
> Cheers,
> Daniel


More information about the xorg-devel mailing list