Nouveau - Jul 2017 - [PATCH xf86-video-amdgpu] Adapt to PixmapDirtyUpdateRec::src being a DrawablePtr

On 18/04/17 07:07 PM, Michel Dänzer wrote:
> From: Michel Dänzer <michel.daenzer at amd.com>
> 
> Signed-off-by: Michel Dänzer <michel.daenzer at amd.com>
> ---
> 
> Chris / Ilia / Ben, this should be manageable for the intel/nouveau
> drivers, right?
Any feedback, guys?

I want to push the xserver change soon-ish. I could probably come up
with a corresponding patch for nouveau in the worst case, but I'm afraid
not for intel.


-- 
Earthling Michel Dänzer               |               http://www.amd.com
Libre software enthusiast             |             Mesa and X developer

On Thu, Jul 13, 2017 at 4:27 AM, Michel Dänzer <michel at daenzer.net>
wrote:
> On 18/04/17 07:07 PM, Michel Dänzer wrote:
>> From: Michel Dänzer <michel.daenzer at amd.com>
>>
>> Signed-off-by: Michel Dänzer <michel.daenzer at amd.com>
>> ---
>>
>> Chris / Ilia / Ben, this should be manageable for the intel/nouveau
>> drivers, right?
>
> Any feedback, guys?
>
> I want to push the xserver change soon-ish. I could probably come up
> with a corresponding patch for nouveau in the worst case, but I'm
afraid
> not for intel.
Sorry, I'm not 1000% clear on what this patch is doing. Is it
literally just a type change from A to B and so code has to be fixed
up for the new API? Or are there further implications like having to
do/support new things?

Thanks,

  -ilia

On 13/07/17 09:31 PM, Ilia Mirkin wrote:
> On Thu, Jul 13, 2017 at 4:27 AM, Michel Dänzer <michel at
daenzer.net> wrote:
>> On 18/04/17 07:07 PM, Michel Dänzer wrote:
>>> From: Michel Dänzer <michel.daenzer at amd.com>
>>>
>>> Signed-off-by: Michel Dänzer <michel.daenzer at amd.com>
>>> ---
>>>
>>> Chris / Ilia / Ben, this should be manageable for the intel/nouveau
>>> drivers, right?
>>
>> Any feedback, guys?
>>
>> I want to push the xserver change soon-ish. I could probably come up
>> with a corresponding patch for nouveau in the worst case, but I'm
afraid
>> not for intel.
> 
> Sorry, I'm not 1000% clear on what this patch is doing.
100% should suffice. ;)
> Is it literally just a type change from A to B and so code has to be fixed
> up for the new API?
That's basically it, corresponding to
https://patchwork.freedesktop.org/patch/150938/ .


-- 
Earthling Michel Dänzer               |               http://www.amd.com
Libre software enthusiast             |             Mesa and X developer