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