Commit 7ea1db28 authored by Srinivasan Shanmugam's avatar Srinivasan Shanmugam Committed by Alex Deucher
Browse files

drm/radeon: Prefer strscpy over strlcpy calls in radeon_atombios.c 



Use the strscpy method instead of strlcpy method.

Fixes the below:

WARNING: Prefer strscpy over strlcpy

Cc: Christian König <christian.koenig@amd.com>
Cc: Alex Deucher <alexander.deucher@amd.com>
Signed-off-by: default avatarSrinivasan Shanmugam <srinivasan.shanmugam@amd.com>
Reviewed-by: default avatarAlex Deucher <alexander.deucher@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 6f38bdb8

drivers/gpu/drm/radeon/radeon_atombios.c

+2 −2
Original line number Diff line number Diff line
@@ -2105,7 +2105,7 @@ static int radeon_atombios_parse_power_table_1_3(struct radeon_device *rdev) 
			const char *name = thermal_controller_names[power_info->info.

								    ucOverdriveThermalController];

			info.addr = power_info->info.ucOverdriveControllerAddress >> 1;

			strlcpy(info.type, name, sizeof(info.type));

			strscpy(info.type, name, sizeof(info.type));

			i2c_new_client_device(&rdev->pm.i2c_bus->adapter, &info);

		}

	}
@@ -2355,7 +2355,7 @@ static void radeon_atombios_add_pplib_thermal_controller(struct radeon_device *r 
				struct i2c_board_info info = { };

				const char *name = pp_lib_thermal_controller_names[controller->ucType];

				info.addr = controller->ucI2cAddress >> 1;

				strlcpy(info.type, name, sizeof(info.type));

				strscpy(info.type, name, sizeof(info.type));

				i2c_new_client_device(&rdev->pm.i2c_bus->adapter, &info);

			}

		} else {