[PATCH] Re: Mullins support in xf86-video-amdgpu

[Michel Dänzer]

On 23/08/16 11:57 AM, Reid Hekman wrote:
> On 08/22/2016 08:27 PM, Michel Dänzer wrote:
>> On 23/08/16 10:18 AM, Reid Hekman wrote:
>>>
>>> I was encouraged by a commit I saw today to add Sea Islands PCI ids to
>>> xf86-video-amdgpu.  However I did not see MULLINS included. Are there
>>> other showstoppers preventing inclusion?
>>
>> I assume the commit you're referring to was the one adding the
>> *Southern* Islands (SI) PCI IDs. Since Mullins is Sea Islands (CIK), it
>> should already be supported by the amdgpu kernel driver; its omission
>> from xf86-video-amdgpu is probably just an oversight. Can you send a
>> patch to the amd-gfx mailing list?
>>
> 
> Michael, oops, yes, sorry, apparently my head was on the wrong AMD
> islands. Admittedly I'm a total newb at this, but David's script makes
> this seem fairly simple.  Does the following look correct?

It looks good, but it fails to apply for me for some reason. How did you
generate the patch?

Ideally, generate a patch (with a proper commit log) with git
format-patch and send it with git send-email.


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