On 3 May 2017 at 15:15, Emil Velikov <[email protected]> wrote:
> On 2 May 2017 at 19:49, Daniel Stone <[email protected]> wrote:
>> -   ret |= mod;
>> +   ret |= (uint64_t)(mod & 0xffffffff);
>>
> I was itching to suggest the following from the start, but your patch
> also works ;-)
> Fixes: 8378c576abd ("gbm: Export a get modifiers")
> Reviewed-by: Emil Velikov <[email protected]>
>
>     return (uint64_t)mod_hi << 32 | (uint64_t)mod_lo & 0xffffffff;

Thanks Emil! I already pushed with Ben's R-b last night, as it was a
fairly critical fix for anyone using the new ABI. I'll try to be more
careful about using the 'Fixes' tags in future. :(

Cheers,
Daniel
_______________________________________________
mesa-dev mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to