[PATCH modesetting] Implement ->driverFunc
Aaron Plattner
aplattner at nvidia.com
Fri Jul 20 15:37:36 PDT 2012
On 07/20/2012 03:28 PM, Adam Jackson wrote:
> On 7/20/12 3:44 PM, Aaron Plattner wrote:
>> On 07/19/2012 03:15 PM, Adam Jackson wrote:
>>> +static Bool
>>> +ms_driver_func(ScrnInfoPtr scrn, xorgDriverFuncOp op, void *data)
>>> +{
>>> + xorgHWFlags *flag;
>>> +
>>> + switch (op) {
>>> + case GET_REQUIRED_HW_INTERFACES:
>>> + flag = (CARD32 *)data;
>>> + (*flag) = 0;
>>
>> Should this be = 0, or &= ~HW_IO?
>
> It should not be ~HW_IO, since that would set HW_SKIP_CONSOLE, which we
> don't want.
&=, not =. I.e.,
*flag = *flag & ~HW_IO.
I.e. just clear the HW_IO bit and leave the rest alone, rather than clear all of
them.
> It would also set HW_MMIO; somewhat bizarrely that flag has never been
> interpreted by the server to mean anything, even in the initial commit.
Just setting it to 0 will also clear any future on-by-default flags that we
might want to add.
-- Aaron
More information about the xorg-devel
mailing list