[PATCH 1/1] amdgpu: move asic id table to a separate file

Michel Dänzer michel at daenzer.net
Tue May 16 01:28:13 UTC 2017


On 16/05/17 05:43 AM, Li, Samuel wrote:
>> Please provide specific reasons why this file cannot be in the libdrm repository upstream.
> A separate repo is preferred here,
> 1)  Keep a clean separation between code and data.

Why do they need to be separate?


> 2) Ideally, the commit of this data file should contain a Signed-Off-By:
> from someone authoritative on the licensing of Marketing names, i.e.
> from someone within the vendor (AMD).

Does it matter for that which repository the file is located in? How so?

> It is very similar to linux-firmware repo in that it can be treated as
> a black box from the vendor (It could be a binary if requested, as it
> is on Catalyst Windows driver).

Right now it's just a simple text file.


> I am open to other suggestions, if you would like to bring up. 

Let's put it in the libdrm repository, unless there's an actual
technical reason why it can't be there. Otherwise we're just creating
busywork for distro packagers and early community testers.


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


More information about the amd-gfx mailing list