Am 16.09.2016 um 04:17 schrieb Michel Dänzer:
On 15/09/16 10:10 PM, Christian König wrote:
From: Christian König <christian.koe...@amd.com>
No intented functional change.
Signed-off-by: Christian König <christian.koe...@amd.com>
---
drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c | 65 +++++++++++++++++++++------------
1 file changed, 42 insertions(+), 23 deletions(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c
b/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c
index 428ffb6..1965209 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c
@@ -214,9 +214,11 @@ static void amdgpu_evict_flags(struct ttm_buffer_object
*bo,
switch (bo->mem.mem_type) {
case TTM_PL_VRAM:
if (rbo->adev->mman.buffer_funcs_ring->ready == false) {
- amdgpu_ttm_placement_from_domain(rbo,
AMDGPU_GEM_DOMAIN_CPU);
+ amdgpu_ttm_placement_from_domain(rbo,
+ AMDGPU_GEM_DOMAIN_CPU);
} else {
- amdgpu_ttm_placement_from_domain(rbo,
AMDGPU_GEM_DOMAIN_GTT);
+ amdgpu_ttm_placement_from_domain(rbo,
+ AMDGPU_GEM_DOMAIN_GTT);
I don't see the point of this kind of change. We save all of 5 columns
of horizontal space, but for that we reduce readability by breaking up a
single line statement to multiple lines.
It makes more sense when saving a more significant amount of horizontal
space or when the statement already spans multiple lines though.
I was mostly just following what the auto formatter suggested, but now
that you say it it's probably not such a good idea in some cases.
Should I just drop those two patches or just fix what Alex and you have
noted and send them again? Some of the changes still seem to make sense
to me.
Regards,
Christain.
_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx
