Commit efdf7a93 authored by Edward O'Callaghan's avatar Edward O'Callaghan Committed by Alex Deucher
Browse files

amd/powerplay: use kmemdup instead of kmalloc + memcpy



Save a few clocks by replacing calls to kmalloc followed by memcpy
with a single call to kmemdup.

Reviewed-by: default avatarChristian König <christian.koenig@amd.com>
Signed-off-by: default avatarEdward O'Callaghan <funfunctor@folklore1984.net>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 41698abb
Loading
Loading
Loading
Loading
+3 −6
Original line number Diff line number Diff line
@@ -764,15 +764,12 @@ static int pp_dpm_set_pp_table(void *handle, const char *buf, size_t size)
	PP_CHECK_HW(hwmgr);

	if (!hwmgr->hardcode_pp_table) {
		hwmgr->hardcode_pp_table =
				kzalloc(hwmgr->soft_pp_table_size, GFP_KERNEL);
		hwmgr->hardcode_pp_table = kmemdup(hwmgr->soft_pp_table,
						   hwmgr->soft_pp_table_size,
						   GFP_KERNEL);

		if (!hwmgr->hardcode_pp_table)
			return -ENOMEM;

		/* to avoid powerplay crash when hardcode pptable is empty */
		memcpy(hwmgr->hardcode_pp_table, hwmgr->soft_pp_table,
				hwmgr->soft_pp_table_size);
	}

	memcpy(hwmgr->hardcode_pp_table, buf, size);