Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 580f8398 authored by Ilija Hadzic's avatar Ilija Hadzic Committed by Alex Deucher
Browse files

drm/radeon: remove unecessary assignment 



length_dw field was assigned twice. While at it, move user_ptr
assignment together with all other assignments to p->chunks[i]
structure to make the code more readable.

Signed-off-by: default avatarIlija Hadzic <ihadzic@research.bell-labs.com>
Reviewed-by: default avatarMarek Olšák <maraeo@gmail.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 3646e420

drivers/gpu/drm/radeon/radeon_cs.c

+1 −4
Original line number Diff line number Diff line
@@ -203,7 +203,7 @@ int radeon_cs_parser_init(struct radeon_cs_parser *p, void *data) 
		p->chunks[i].length_dw = user_chunk.length_dw;

		p->chunks[i].kdata = NULL;

		p->chunks[i].chunk_id = user_chunk.chunk_id;



		p->chunks[i].user_ptr = (void __user *)(unsigned long)user_chunk.chunk_data;

		if (p->chunks[i].chunk_id == RADEON_CHUNK_ID_RELOCS) {

			p->chunk_relocs_idx = i;

		}
@@ -226,9 +226,6 @@ int radeon_cs_parser_init(struct radeon_cs_parser *p, void *data) 
				return -EINVAL;

		}



		p->chunks[i].length_dw = user_chunk.length_dw;

		p->chunks[i].user_ptr = (void __user *)(unsigned long)user_chunk.chunk_data;



		cdata = (uint32_t *)(unsigned long)user_chunk.chunk_data;

		if ((p->chunks[i].chunk_id == RADEON_CHUNK_ID_RELOCS) ||

		    (p->chunks[i].chunk_id == RADEON_CHUNK_ID_FLAGS)) {