Re: [PATCH libdrm v8] amdgpu: move asic id table to a separate file

2017-06-15 Thread Michel Dänzer
On 15/06/17 04:42 PM, Emil Velikov wrote: > On 15 June 2017 at 04:16, Michel Dänzer wrote: >> On 14/06/17 08:34 PM, Emil Velikov wrote: >> >>> Personally I would not have bothered with the table_max_size thing >> >> It seemed silly to reallocate the memory in the default case where the >> amdgpu.i

Re: [PATCH libdrm v8] amdgpu: move asic id table to a separate file

2017-06-15 Thread Emil Velikov
On 15 June 2017 at 04:16, Michel Dänzer wrote: > On 14/06/17 08:34 PM, Emil Velikov wrote: >> On 13 June 2017 at 10:45, Michel Dänzer wrote: >>> From: Xiaojie Yuan >>> >>> v2: fix an off by one error and leading white spaces >>> v3: use thread safe strtok_r(); initialize len before calling getli

Re: [PATCH libdrm v8] amdgpu: move asic id table to a separate file

2017-06-14 Thread Michel Dänzer
On 14/06/17 08:34 PM, Emil Velikov wrote: > On 13 June 2017 at 10:45, Michel Dänzer wrote: >> From: Xiaojie Yuan >> >> v2: fix an off by one error and leading white spaces >> v3: use thread safe strtok_r(); initialize len before calling getline(); >> change printf() to drmMsg(); add initial a

Re: [PATCH libdrm v8] amdgpu: move asic id table to a separate file

2017-06-14 Thread Emil Velikov
On 13 June 2017 at 10:45, Michel Dänzer wrote: > From: Xiaojie Yuan > > v2: fix an off by one error and leading white spaces > v3: use thread safe strtok_r(); initialize len before calling getline(); > change printf() to drmMsg(); add initial amdgpu.ids > v4: integrate some recent internal ch

Re: [PATCH libdrm v8] amdgpu: move asic id table to a separate file

2017-06-13 Thread Alex Deucher
On Tue, Jun 13, 2017 at 5:45 AM, Michel Dänzer wrote: > From: Xiaojie Yuan > > v2: fix an off by one error and leading white spaces > v3: use thread safe strtok_r(); initialize len before calling getline(); > change printf() to drmMsg(); add initial amdgpu.ids > v4: integrate some recent inte

[PATCH libdrm v8] amdgpu: move asic id table to a separate file

2017-06-13 Thread Michel Dänzer
From: Xiaojie Yuan v2: fix an off by one error and leading white spaces v3: use thread safe strtok_r(); initialize len before calling getline(); change printf() to drmMsg(); add initial amdgpu.ids v4: integrate some recent internal changes, including format changes v5: fix line number for emp