Re: [PATCH libdrm 1/7] remove unnecessary empty statements

2018-01-26 Thread Emil Velikov
I think you meant s/empty statements/double semi-colon/. - please keep the nouveau debug patch out for the time being - regardless if you keep as-is or fold the Wundef patch with the Werror one - with the shadowing issue pointed by Chrisian Both 7 patch series are Reviewed-by: Emil Velikov -

Re: [PATCH libdrm 1/7] remove unnecessary empty statements

2018-01-26 Thread Christian König
Am 26.01.2018 um 12:32 schrieb Eric Engestrom: Signed-off-by: Eric Engestrom Reviewed-by: Christian König --- amdgpu/amdgpu_cs.c| 2 +- tests/exynos/exynos_fimg2d_test.c | 2 +- tests/modeprint/modeprint.c | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-)

[PATCH libdrm 1/7] remove unnecessary empty statements

2018-01-26 Thread Eric Engestrom
Signed-off-by: Eric Engestrom --- amdgpu/amdgpu_cs.c| 2 +- tests/exynos/exynos_fimg2d_test.c | 2 +- tests/modeprint/modeprint.c | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/amdgpu/amdgpu_cs.c b/amdgpu/amdgpu_cs.c index 987daa4036f2d99ff15f..46dffe