Commit 69b54d7c authored by Dheeraj Reddy Jonnalagadda's avatar Dheeraj Reddy Jonnalagadda Committed by Alex Deucher
Browse files

drm/amdgpu: simplify return statement in amdgpu_ras_eeprom_init 

Remove the logically dead code in the last return statement of
amdgpu_ras_eeprom_init. The condition res < 0 is redundant since
res is already checked for a negative value earlier. Replace
return res < 0 ? res : 0; with return 0 to improve clarity.

Fixes: 63d4c081 ("drm/amdgpu: Optimize EEPROM RAS table I/O")
Closes: https://scan7.scan.coverity.com/#/project-view/52337/11354?selectedIssue=1602413


Signed-off-by: default avatarDheeraj Reddy Jonnalagadda <dheeraj.linuxdev@gmail.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 7e50642d

drivers/gpu/drm/amd/amdgpu/amdgpu_ras_eeprom.c

+1 −1
Original line number Diff line number Diff line
@@ -1390,7 +1390,7 @@ int amdgpu_ras_eeprom_init(struct amdgpu_ras_eeprom_control *control) 
	}

	control->ras_fri = RAS_OFFSET_TO_INDEX(control, hdr->first_rec_offset);



	return res < 0 ? res : 0;

	return 0;

}



int amdgpu_ras_eeprom_check(struct amdgpu_ras_eeprom_control *control)