[xserver, 2/2] xf86RandR12: Fix XF86VidModeSetGamma triggering a BadImplementation error

[Hans de Goede]

Hi,

On 25-09-16 09:51, Michel Dänzer wrote:
> On 23/09/16 07:48 PM, Hans de Goede wrote:
>> On 09/23/2016 11:13 AM, Michel Dänzer wrote:
>>> On 23/09/16 03:46 PM, Hans de Goede wrote:
>>>>
>>>> Since this is a fallback which ideally should never get used (all
>>>> drivers should call xf86HandleColormaps) I believe it is not worth
>>>> the time to implement your (admittedly better) fallback suggestion.
>>>
>>> Fine with me, but please at least change the comment to be more accurate
>>> and clearer.
>>
>> OK, the comment now reads:
>> /*
>>  * Compatibility pScrn->ChangeGamma provider for ddx drivers which do
>> not call
>>  * xf86HandleColormaps(). Note such drivers really should be fixed to call
>>  * xf86HandleColormaps() as this clobbers any per-crtc setup.
>>  */
>
> As I mentioned before, "clobbers any per-CRTC setup" isn't accurate.
> Only the gamma ramp of the CRTC assigned to the RandR compatibility
> output is clobbered.

Ah I missed you had provided an improved comment and rolled my own.

I've fixed this in my personal tree now, and the fixed version
will be part of the updated pull-req for 1.19 I'm preparing.

Regards,

Hans