Re: [PATCH libdrm] xf86drm: fix compile error for declare i in for loop

2017-05-17 Thread Yu, Qiang
OK, but my build script uses "make CC=gcc". If you think current state is OK, I can change my build script for that. Regards, Qiang From: Emil Velikov Sent: Wednesday, May 17, 2017 9:17:53 PM To: Yu, Qiang Cc: ML dri-devel Subject: Re: [PATCH lib

Re: [PATCH libdrm] amdgpu: Remove dummy CU_ASSERT_EQUAL.

2017-12-12 Thread Yu, Qiang
Reviewed-by: Qiang Yu Regards, Qiang From: Andrey Grodzovsky Sent: Tuesday, December 12, 2017 9:36:53 PM To: dri-devel@lists.freedesktop.org Cc: amd-...@lists.freedesktop.org; Yu, Qiang; Ma, Le; Huan, Alvin; Jin, Anna; Grodzovsky, Andrey Subject

[PATCH][libdrm] drm: Fix multi GPU drmGetDevice return wrong device

2016-07-13 Thread Yu, Qiang
Hi Emil, Nice to hear from you. On 11 July 2016 at 06:17, Qiang Yu wrote: > drmGetDevice will always return the first device it find > under /dev/dri/. This is not true for multi GPU situation. > How does the following alternative solution sound: - keep drmFoldDuplicatedDevices as is - after

[PATCH][libdrm] drm: Fix multi GPU drmGetDevice return wrong device

2016-07-14 Thread Yu, Qiang
Thanks Emil, I'll submit v2 to address your comments. I'm using office365, not sure this mail is OK for formatting, otherwise I'll switch to a mail client. Regards, Qiang From: Emil Velikov Sent: Wednesday, July 13, 2016 7:15:43 PM To: Yu, Qi

[libdrm][PATCH v2 1/2] drm: Fix multi GPU drmGetDevice return wrong device

2016-07-15 Thread Yu, Qiang
On 14 July 2016 at 10:10, Qiang Yu wrote: > drmGetDevice will always return the first device it find > under /dev/dri/. This is not true for multi GPU situation. > > Change-Id: I2a85a8a4feba8a5cc517ad75c6afb532fa07c53d Hope you don't mind if I drop the change-id lines before committing ? [yuq] O

[PATCH][libdrm] drm: Fix multi GPU drmGetDevice return wrong device

2016-07-15 Thread Yu, Qiang
On 14 July 2016 at 04:02, Yu, Qiang wrote: > Thanks Emil, I'll submit v2 to address your comments. > I believed you covered them all. Thanks ! Small suggestion for the future - I many devs are appreciate when patches have a brief shortlog before or after the --- line. [yuq] You mea